$resz_h) > MAX_PIC
&& 0 < $koef = $resz_w / $resz_h
) {
$say->width = koef > 1 ? RSZ_PIC : round(RSZ_PIC * $koef);
}
что? я вижу в условии присваивание, ого
Гавно)
например, если такую херню писал))
Присвоение в условии зло?)
https://starecat.com/content/wp-content/uploads/the-robots-are-killing-us-but-why-we-never-programmed-them-to-do-this-if-crazymurderingrobot-true-then-kill-humans-programming-code.jpg
Ну в таких случах помогает йода стайл
Как правило человек, читающий код, не ожидает присваивания в условиях, кроме того, есть случаи, когда возникают ошибки, поэтому лучше этого избегать + настроить статические анализаторы на проверку этого момента.
Блин, а у нас в проекте это бест практис, а где присвоение делать, до условий ? Сторожевые пункты, все дела
может ты перепутал бест и верст?)
Практис без которой таску не примут)
смысл убирать шорткод, если он встроен разрабами?
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 {...} напишите это в нормальном виде без дублирования присваиваний и коротко, а потом уже скажите, что присваивание в условиях - зло
это другое уже. присвоение не в ифе)
а что насчет PSR'ов, которые регламентируют стиль кода?
судя по всему там унылое говнолегаси
ага и require_once вместо автолоадера
Ебаный рот, вот тут присваивание точно не катит
автолоадер не будет работать, если конфиги, которые тебе нужны используются на разных серверах в разных директориях
если автолоадер не работает - надо выбросить это говно, взять бизнес код, и засунуть в нормальный проект.
самописный движок далеко не самого умного разраба. тут того автолоадера только подключение нескольких классов, которые и то, надо переписывать
я понимаю, что у тебя там какое-то жуткое легаси, но можно и свой автозагрузчик написать, который будет всё это учитывать
а зачем ты рабоатешь на таких проектах? почему не найдешь нормальный?
как стану сеньором, обязательно напишу)
свободного другого разраба нет, чтобы сплавить проект, да и я его уже полностью знаю. частично переписал его и всё равно бреда много ещё
Обсуждают сегодня