V
Size: a a a
2020 November 11
АГ
шторм подсвечивать должен такое
V
шторм подсвечивать должен такое
тем более)
S
Как правило человек, читающий код, не ожидает присваивания в условиях, кроме того, есть случаи, когда возникают ошибки, поэтому лучше этого избегать + настроить статические анализаторы на проверку этого момента.
Блин, а у нас в проекте это бест практис, а где присвоение делать, до условий ? Сторожевые пункты, все дела
АГ
+ да, всякие линтеры и стат анализ
АГ
Sergey
Блин, а у нас в проекте это бест практис, а где присвоение делать, до условий ? Сторожевые пункты, все дела
может ты перепутал бест и верст?)
S
может ты перепутал бест и верст?)
Практис без которой таску не примут)
РД
Как правило человек, читающий код, не ожидает присваивания в условиях, кроме того, есть случаи, когда возникают ошибки, поэтому лучше этого избегать + настроить статические анализаторы на проверку этого момента.
смысл убирать шорткод, если он встроен разрабами?
АГ
кто не примет? зови его сюда, я ему втащу
V
упс, но хрен его знает, он сборку то не показывал еще
АГ
hirak hirak, i v prod?
РД
смысл убирать шорткод, если он встроен разрабами?
if (file_exists($file = ROOT . "/file1.php")) require_once($file);напишите это в нормальном виде без дублирования присваиваний и коротко, а потом уже скажите, что присваивание в условиях - зло
else if (file_exists($file = ROOT . "/file.php")) require_once($file);
else if (file_exists($file = __DIR__."/file.php")) require_once($file);
else {...}
АГ
if (file_exists($file = ROOT . "/file1.php")) require_once($file);напишите это в нормальном виде без дублирования присваиваний и коротко, а потом уже скажите, что присваивание в условиях - зло
else if (file_exists($file = ROOT . "/file.php")) require_once($file);
else if (file_exists($file = __DIR__."/file.php")) require_once($file);
else {...}
это другое уже. присвоение не в ифе)
АГ
но вообще, я бы имена (строки)ссложил в массив, и прошелся бы форычем)
РД
но вообще, я бы имена (строки)ссложил в массив, и прошелся бы форычем)
как вариант, но если у меня больше кода и на каждый файл - свой?
АГ
сделай функцию, которая на вход принимает массив строк.
РД
любой анализатор кода дубли подсветит