MM
Не, впервые слышу. Но руки бы оторвал
Size: a a a
2020 January 29
SM
За switch (true) { $shit instanceof ShitClass: case
Ну как.. оупен-клоуз же.
КГ
Ну как.. оупен-клоуз же.
Ну речь больше про условия в каждом кейсе и передачи true
SP
Ребят, а почему switch (true) и instanceof в case это плохо?
По той же причине по которой просто instance of "не оч хорошо". В целом все упирается в расширяемость решения. В большинстве случаев проблемы подправить свитч нет
КГ
По той же причине по которой просто instance of "не оч хорошо". В целом все упирается в расширяемость решения. В большинстве случаев проблемы подправить свитч нет
Мне просто казалось саму конструкцию с условиями в кейсах хейтили. Но причин не запомнил
SP
У свитча обычно проблема с выходом из ветки
SP
Легко проебаться
SP
А так switch (true) сам по себе норм
SP
Не норм может быть ток как его юзать)
КГ
🤔 понял принял
AS
В бизнес логике, где чаще всего плевать на перформанс и важнее расширяемость и читаемость switch(true) может быть признаком подгнивания кода. А в инфраструктуре и ниже он сплощь и рядом.
KN
У меня все фектори из детерменированного количества конечных результатов на этом switch true держатся)))
KN
ну и вообще все редьюсеры для ast билдеров вообще напичканы этим, ибо там все потомки что описаны в ветвлении определены на уровне грамматики
KN
ну и вот в догонку (не про swith/true, а вообще), прост: https://derickrethans.nl/php7.2-switch.html
KN
но эта оптимизация работает только когда case одного типа
AM
А сколько подарков можно словить, если переменную переименовать захочется...
шторм распарсивает такие ситуации
AM
(хотя тут 50/50, зависит от опции во время ринейма)
AS
У меня все фектори из детерменированного количества конечных результатов на этом switch true держатся)))
Ну так у тебя вполне себе инфраструктурный код и есть, где это норма
AS
У меня в парсинге бинарных данных каких их тоже полно