Size: a a a

2020 January 22

AS

Alexey Shatunov in phpGeeks
🤔
источник

AS

Alexey Shatunov in phpGeeks
я бы оставил такой код - опкеш закеширует все равно
источник

AM

Anatoly Medvedev in phpGeeks
Alexander Berezin
Нужно мнение экспертов.
Руководитель сказал, что использовать тернарные операторы нельзя, потому что это плохо читается и такое можно писать только в шаблонах view при выводе
$idEmployee = empty($currentUser) ? 0 : $currentUser->getState('id_employee');

а вот так писать можно и нужно
$idEmployee = 0;
if (empty($currentUser)) {
   $idEmployee = 0;
} else {
   $idEmployee = $currentUser->getState('id_employee');
}

что вы думаете по этому поводу?
Поддерживаю руководителя если он не отменил :? и ??
источник

AS

Alexey Shatunov in phpGeeks
на гошечке это вообще стандартная практика с if, только там есть объявление переенной без присваивания
источник

AM

Anatoly Medvedev in phpGeeks
Alexander Berezin
Нужно мнение экспертов.
Руководитель сказал, что использовать тернарные операторы нельзя, потому что это плохо читается и такое можно писать только в шаблонах view при выводе
$idEmployee = empty($currentUser) ? 0 : $currentUser->getState('id_employee');

а вот так писать можно и нужно
$idEmployee = 0;
if (empty($currentUser)) {
   $idEmployee = 0;
} else {
   $idEmployee = $currentUser->getState('id_employee');
}

что вы думаете по этому поводу?
Но код говно т.к. true ветку можно убрать
источник

AS

Alexey Shatunov in phpGeeks
ой, извините я по диагонали читаю как обычно...
источник

AS

Alexey Shatunov in phpGeeks
щас прочитал только что "Руководитель сказал, что использовать тернарные операторы нельзя"

ну я бы тогда не использовал
источник

AS

Alexey Shatunov in phpGeeks
какой смысл с руководителем спорить? Вы готовы его свергнуть и занять его место?
источник

AB

Alexander Berezin in phpGeeks
Alexey Shatunov
какой смысл с руководителем спорить? Вы готовы его свергнуть и занять его место?
ну это верно 😅
источник

AB

Alexander Berezin in phpGeeks
вообще это норм практика?
источник

AS

Alexey Shatunov in phpGeeks
Alexander Berezin
вообще это норм практика?
нет конечно
источник

AM

Anatoly Medvedev in phpGeeks
Alexander Berezin
вообще это норм практика?
Норм
источник

AS

Alexey Shatunov in phpGeeks
🤔наши мнения кажутся полярно противоречивыми
источник

VM

Vladimir Makarov in phpGeeks
Alexander Berezin
вообще это норм практика?
ну только если ты не пришел в отдел, где есть устоявшиеся правила/нормы кода и сразу пытаешься с ними спорить
источник

AS

Alexey Shatunov in phpGeeks
Vladimir Makarov
ну только если ты не пришел в отдел, где есть устоявшиеся правила/нормы кода и сразу пытаешься с ними спорить
ааа негибкость умов руководства вот это все
источник

AM

Anatoly Medvedev in phpGeeks
Alexey Shatunov
🤔наши мнения кажутся полярно противоречивыми
Наши мнения отличаются и это нормально. Я считаю что такой код конвеншн это норм, ты нет, все просто.
источник

AS

Alexey Shatunov in phpGeeks
спорить - надо
источник

П

Павел in phpGeeks
Блондин
я хочу - го меняться)
Где ты откопал моё сообщение за 11 ноября😂😂
источник

VM

Vladimir Makarov in phpGeeks
а если руководитель полупрограммист и по фану хочет вставить свою ляпту в код, то можно и поспорить
источник

AB

Alexander Berezin in phpGeeks
Vladimir Makarov
ну только если ты не пришел в отдел, где есть устоявшиеся правила/нормы кода и сразу пытаешься с ними спорить
почти сразу, полтора месяца прошло.
Сколько времени можно ещё выждать, чтобы можно было пытаться объяснять полезность тернарных операторов?
источник