Результат работы функции в качестве аргумента?

Question

[JastaFly]

Доброго всем времени суток! Где то слышал что в PHP не рекомендуется в качестве аргумента передавать результат работы функции и вот такой код:
array_keys(getAllUsers());
Лучше заменить на такой:

$users = getAllUsers();
array_keys($users);

Так ли это?!???

Comments

[Максим Федоров]

[JastaFly]

Максим Федоров, от куда этот кусок!??

[Максим Федоров]

JastaFly, в одной из либ:
на самом деле ничего такого в нем нет — собираются ключи из 3х источников и выбираются уникальные (без дублей)
просто и понятно

Answer 1

[LionG]

Нет. Это рофл.

Хотя зависит от того принимает ли принимающая функция все варианты ответов (типов) вызываемой. Например если вызываемая getAllUsers вернет false (boolean) может случиться фаталка тк array_keys принимает только (array) массив. Но в таком случае можно сделать такую проверку:

array_keys(getAllUsers() ?: []);//если положительный ответ то его иначе (по умолчанию) пустой массив

Либо если функция ваша можно внутри getAllUsers всегда возвращать массив и в случае если пользователей нет то пустой.

Comments

[Lander]

JastaFly кроме того есть функции, которые принимают аргументы по ссылке. К таким, например относятся функции sort, array_walk... в этом случае они так же выдадут ошибку при попытке подсунуть им результат функции вместо переменной. Но а если версия PHP совсем древняя у вас < 5.6, то там могут быть приколы вроде таких что в функцию empty так же нужно передавать только переменную (так что вы вполне могли такое слышать, но очень давно).