Какая запись выглядит более эстетично, если цель - красивый код?
Такая
$user = $this->userName . implode("-", $this->birthDate);
Такая
$user = $this->userName;
$user .= implode("-", $this->birthDate);
Или такая
$user = $this->userName;
$birth = implode("-", $this->birthDate);
$user .= $birth;
Пример написан от балды, так что строго не судите. По сути, мне интересно только одно - допустимо ли пихать в конкатенацию функции или понятнее когда "конкатенируются" только переменные. Сам склоняюсь к первому варианту.
Ответ
Субъективно мне нравится первый вариант, так как в нем одна строка и она укладывается в 80 символов (т.е. удовлетворяет стандарту PSR-2). Третий вариант неплох, вы выделяете $birth в отдельную переменную, однако, как мне кажется $this->birthDate в implode() и так дает достаточно информации, что речь идет о датах рождения. Отторжения или желания переделать ни один из вариантов не вызвал.
Комментариев нет:
Отправить комментарий