Надо ли расписывать всё по переменным?

Question

[Посторонним В.]

Насколько знаю, программисты - те, кто меряется не "у кого длиннее", а у кого короче.

В связи с этим для меня часто возникают непонятки. Вот, скажем, такой абстрактный пример:

public function doTheWork($string) 
{
    $arg1 = $this->getValue(1);
    $arg2 = $this->processor->process($string);
    $arg3 = $this->data->some_string ?? '';

    $result = $this->doSomething($arg1, $arg2, $arg3);

    $return result;
}

Вот видел, что некоторые пишут так. Но это же очень длинно - и ведь введенные переменные используются только один раз, а есть принцип - не плодить лишние сущности. Почему бы не написать так:

public function doTheWork($string) 
{
   return $this->doSomething(
        $this->getValue(1),
        $this->processor->process($string),
        $this->data->some_string ?? ''
   );
}

- по-моему идеально, ничего лишнего. Это нормальный вариант? Если нет, то почему? В чем его недостатки? И в чем преимущества прописывания значений по переменным?

Answer 1

[D']

В первом варианте лучше читаемость. Ты не в твиттер пишешь, чтобы ужимать код.
Чем выше читаемость - тем проще поддержка кода.

Comments

[Посторонним В.]

Читаемость всегда важнее краткости? Исключены случаи, когда второй вариант будет работать быстрее?

[D']

Посторонним В.: читаемость всегда важнее.
>Исключены случаи, когда второй вариант будет работать быстрее
Даже если такие случаи и существуют, то разницу заметишь только на ОЧЕНЬ большой нагрузке.

[Посторонним В.]

понял, спасибо. Тут вот и простоту дебаггинга упомянули. Тоже это заметил последнее время. В общем, первый вариант однозначно выигрывает.

Answer 2

[Сергей]

Можно, но читаемость кода ухудшилась.
Моожно вообще тогда все в строку писать, все вычисление какойнить астрономической сущности

Answer 3

[Daemon23RUS]

Это путь к:

public function dW($s){return $this->dS($this->g(1),$this->p->p($s),$this->d->s ?? '');}

Надо ли ?

Comments

[Посторонним В.]

наверное это только в js может быть оправдано)

[Daemon23RUS]

из этого можно "получить профит" в любом интерпретируемом языке программирования. Запутывает код, сокращает объем кода.
Зачастую дополнительные переменные вводятся для легкости в отладке кода, а дальше принцип: рабтает -не трож. :)

Answer 4

[krypt3r]

Вставлю своё ИМХО. С точки зрения читабельности между примерами нет разницы (для меня, по крайней мере). Если в своем коде я вижу нечто примеру 1, я почти всегда приведу код ко второму варианту. А зачем создавать лишние переменные? Если бы я их как-то обрабатывал внутри метода с многократным к ним обращением, тогда да, создание дополнительных переменных оправдано. А так я за вариант 2.

Comments

[Посторонним В.]

Когда делаешь отладку - надо прослеживать значения аргументов, не выходя из тела текущей функции. Без промежуточных переменных это проблематично.