Считается ли плохой практикой выражения в значениях массива?

Question

[serikd]

Как предпочтительней писать?

$value = Class::do($data);
$arr = ['key' => $value];
//vs
$arr = ['key' => Class::do($data)];
//vs
$arr = ['key' => function() {
    return Class::do($data);
}];

И что можно почитать, типа лучшие практики или что то в этом роде, чтобы не возникало подобных вопросов.
Или только github? )

Answer 1

[Muhammad]

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

Answer 2

[Сергей Зеленский]

$arr['key'] = Class::do($data);

Answer 3

[Антон Иванов]

Если значение $value нигде больше не будет использоваться, то нормален второй вариант. Третий вариант не работает, т.к. $data внутри Closure не объявлено. Вероятно нужно было написать так:

$arr = ['key' => function($data) {
    return Class::do($data);
}];

Comments

[Антон Иванов]

PSR что-то ничего по этому поводу не говорит, вроде... Может быть кто-то меня поправит.

Answer 4

[Marcuzy]

По-моему, все варианты приемлемы, но каждый по ситуации:

1. Если выражение длинное и сложное
   
2. Выражение короткое и простое
   
3. Если нужно произвести ленивое вычисление.
   

Также важно наличие других ключей.