Есть ли смысл передавать $_POST в качестве параметра?

Question

[S-a-n-d-r-0]

Имеет ли смысл делать так:

function process($request_data)
{

}

process($_POST);

Или лучше использовать $_POST сразу в функции process, если он все равно суперглобальный?

Answer 1

[ThunderCat]

Использование глобальных переменных в функциях вообще плохая практика, надо стараться сохранять принципы чистоты функций, что исключает использование внутри них глобальных переменных.

Answer 2

[kafkiansky]

Все данные лучше передавать снаружи, а $_POST, $_GET, $_FILES не использовать вообще. Вернее, использовать правильно. У вас одна точка входа - в ней и инициализируйте специальный класс, который работает с этими массивами, и передавайте его дальше в код. Это избавит вас от обращения к глобальным массивам и точно ограничит контекст данными, которые вы можете использовать в текущем запросе.

Answer 3

[Александр]

Имеет. Вдруг вы потом захотите использовать другой метод запроса? Если будете использовать $_POST в теле функции, то при изменении метода запроса код придётся переписать

Answer 4

[SagePtr]

Лучше вообще не использовать $_POST, для безопасного получения пользовательских данных есть функция filter_input

Comments

[S-a-n-d-r-0]

почитал, интересная штука. Настораживает только, что никогда раньше не видел, чтобы кто-то ей пользовался. Все почему-то только $_POST и юзают. Есть какой-то подвох?

[SagePtr]

S-a-n-d-r-0, к примеру, лень, ибо $_POST["name"] пишется короче, чем filter_input(INPUT_POST, "name", FILTER_SANITIZE_STRING)

[Северное Сияние]

Лучше вообще не использовать $_POST,

лол што?

для безопасного получения пользовательских данных есть функция filter_input

безопасность пользовательских данных должна проверяться там, где нужно, а именно - при работе с субд.

filter_input(INPUT_POST, "name", FILTER_SANITIZE_STRING)

Удаляет теги, при необходимости удаляет или кодирует специальные символы. ничего не должно удаляться просто так ради мнимой безопасности. Теги вообще опасности не представляют.

бессмысленное расширение filter_input, вредное

[SagePtr]

Северное Сияние, используй, если хочешь, но не забудь тогда про множество проверок. Например, проверку на скаляр, которую многие школяры поголовно забывают и сыпятся на запросах вроде ?param[]=value.
А так-то конечно, если хорошо понимаете, что именно делаете, то можете использовать суперглобальные массивы или вообще вручную парсить входной поток, но зачем велосипедировать, если разработчики добавили в язык удобные функции для санитизации входных данных.