scrutinizer

Последняя активность
Файлы топика
558
.
Koenig
21 июн 2017 г., 14:53
(\/)____o_O____(\/)
L!MP, с одной стороны так удобнее рефакторить и понимать, но тоже не понял чем ему switch не устроил
.
L!MP
21 июн 2017 г., 14:58
# Koenig (21.06.2017 / 14:53)
L!MP, с одной стороны так удобнее рефакторить и понимать, но тоже не понял чем ему switch не устроил
Удобнее прыгать по методам держа в голове контекст?
Не, я понимаю что код нужно разбивать на операции, но это нужно не всегда и не везде.
То что метод имеет тело в 20 строк кода автоматически не означает что он плох и этот код нужно непременно разбить на три/четыре метода.
.
Koenig
21 июн 2017 г., 14:59
(\/)____o_O____(\/)
вместо switch сделал длинный тернарник, и он от меня отстал
.
L!MP
21 июн 2017 г., 15:01
# Koenig (21.06.2017 / 14:59)
вместо switch сделал длинный тернарник, и он от меня отстал
Не делай длинных тернарников никогда, их сложно правильно прочитать при беглом просмотре и к тому же там могут быть сюрпризы в порядке его исполнения.
.
Koenig
21 июн 2017 г., 15:01
(\/)____o_O____(\/)
L!MP, я думаю это от того что есть варианты ветви, if else
у Олега каптча, там гд операции куча подряд, и 10 стоит
.
Koenig
21 июн 2017 г., 15:02
(\/)____o_O____(\/)
L!MP, не сильно длинный, if elseif else всего ли ж
.
L!MP
21 июн 2017 г., 15:05
# Koenig (21.06.2017 / 15:02)
L!MP, не сильно длинный, if elseif else всего ли ж
Это уже длинный. Лучше просто if elseif else использовать.
.
Koenig
21 июн 2017 г., 16:14
(\/)____o_O____(\/)
Добавлено: 21.06.2017 / 15:08
L!MP, https://github.com/mobicms/mob ... ptcha и пачка бейджиков

Добавлено: 21.06.2017 / 16:14
надо будет конфиг сделать, а то каждый коммит анализ делать напрягает
.
Koenig
21 июн 2017 г., 19:53
(\/)____o_O____(\/)
L!MP, у тебя там было два варианта решения проблемы, либо добавить пробел слева, либо убрать справа
там у тебя было 
$propertySetterName = 'set'. $ucPropertyName;
$propertyGetterName = 'get'. $ucPropertyName;
.
Koenig
26 июн 2017 г., 11:01
(\/)____o_O____(\/)
Добавлено: 21.06.2017 / 20:01
L!MP, если в шторме пишешь код, то настрой psr
потом жми ctrl + alt + l

Добавлено: 26.06.2017 / 11:01
хочу ещё другие бейджики, но один совсем упоротый, там как то все сложно. и сам сервис не держит нагрузку
https://connect.sensiolabs.com/
с этим все более менее понятно, но он в основном только мастер ветку смотрит, хотя написано смотрит дефолтную ветку репа
https://codeclimate.com/
Прикрепленные файлы:
149806450322.png
240
(167.22 KB)
149806450322.png
Всего: 20
© 2024, JohnCMS