А тут и не производиться работа с глобальными переменными, это уже сделал modRequest, мы лишь спрашиваем у него переменную по ключу и имени массива, и то только в случае если в параметрах процессора не был найден параметр «code». Ведь он уже инстанцирован и содержит все эти данные и без нас. Почему не использовать?
Ответ «не правильно» без аргументации не ответ имхо.