Нужно ли оптимизировать вызов функции?

Question

[Jesse Pinkman]

Привет.
Такой вопрос возник совершенно случайно и возможно очевидный, но я все же решил спросить пережде чем переписывать везде где я так делал)

есть функция которая например достает что-то из базы.

function getUserRole() {
 .... //тут запросы из базы
  return $user_role;
}

далее я просто вызывал эту функцию, иногда по несколько раз

return [
 'some_key' => getUserRole();
 'data' => [
    'user_role' => getUserRole()
 ]
];

Если в функцию getUserRole() добавить var_dump('test')
То оно выведет 2 раза

Но если занести функцию в переменную, то оно выводит лишь 1 раз:

$user_role = getUserRole();
return [
 'some_key' => $user_role,
 'data' => [
    'user_role' =>$user_role
 ]
];

Это значит, что каждый раз когда я вызывал функцию (как в первом примере) я делал запрос в базу и дополнительно нагружал ее, а во втором примере я лишь один раз вызываю и далее просто ссылаюсь на переменную, так ли это ?
Стоит ли переписывать все ?

Comments

[Jesse Pinkman]

Лентюй, Ну получается да :)

Answer 1

[Ищю в поисковиках]

То, что ты поместил вызов функции в переменную - правильный подход к разработке. Так ты не насилуешь базу.

Почитай чистый код, там много хорошего.

Answer 2

[nokimaro]

Можно использовать кеширование внутри подобных функций

function getUserRole() {
    static $user_role = null;

    if(is_null($user_role) {
        //запросы к базе и присвоение нужного значения в $user_role
    }

    return $user_role;
}

Дальше можно сколько угодно раз вызывать getUserRole() и получать один и тот же результат без повторных запросов в БД в рамках работы скрипта

Comments

[nokimaro]

Лентюй, в рамках заданного вопроса считаю свой ответ вполне уместным, так как кеш сбросится вместе с окончанием работы скрипта.

Ведь автор спрашивает стоит ли делать так?

$user_role = getUserRole();
//используем $user_role вместо вызова getUserRole()
//используем $user_role вместо вызова getUserRole()
//используем $user_role вместо вызова getUserRole()

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

Или уважаемый Куратор тега PHP не в курсе как работает кеширование с использованием статической переменной внутри функции?

[galliard]

nokimaro, кэширование результата функции внутри самой функции - говнокод в любом случает. А если еще и с использованием статика - говнокод в квадрате.

Подход с использованием внешней переменной - правильный.

[nokimaro]

galliard, согласен по всем пунктам. Хуже только использовать $GLOBALS для подобных вещей.

[rPman]

galliard,

кэширование результата функции внутри самой функции - говнокод в любом случае

если функцию назвать getCachedUserRole() либо сделать параметр boolean useCache, то идеологически это будет верно

[galliard]

rPman, нет.

Это как перед говнокодом оставить комментарий "здесь говнокод", вроде бы и предупредил, а суть не поменялась.