170 похожих чатов

Это же нормально писать без скобок? if ($resz_w && max($resz_w,

$resz_h) > MAX_PIC
&& 0 < $koef = $resz_w / $resz_h
) {
$say->width = koef > 1 ? RSZ_PIC : round(RSZ_PIC * $koef);
}

24 ответов

14 просмотров

что? я вижу в условии присваивание, ого

Гавно)

Piter- Автор вопроса

например, если такую херню писал))

Sergey
Присвоение в условии зло?)

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

Vladimir
https://starecat.com/content/wp-content/uploads/th...

Ну в таких случах помогает йода стайл

Sergey
Ну в таких случах помогает йода стайл

Как правило человек, читающий код, не ожидает присваивания в условиях, кроме того, есть случаи, когда возникают ошибки, поэтому лучше этого избегать + настроить статические анализаторы на проверку этого момента.

Vladimir
Как правило человек, читающий код, не ожидает прис...

Блин, а у нас в проекте это бест практис, а где присвоение делать, до условий ? Сторожевые пункты, все дела

Alexey Gevondyan
может ты перепутал бест и верст?)

Практис без которой таску не примут)

Vladimir
Как правило человек, читающий код, не ожидает прис...

смысл убирать шорткод, если он встроен разрабами?

Ратмир Диронин
смысл убирать шорткод, если он встроен разрабами?

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")) requ...

это другое уже. присвоение не в ифе)

Ратмир Диронин
if (file_exists($file = ROOT . "/file1.php")) requ...

а что насчет PSR'ов, которые регламентируют стиль кода?

Alexey Gevondyan
судя по всему там унылое говнолегаси

ага и require_once вместо автолоадера

Ратмир Диронин
if (file_exists($file = ROOT . "/file1.php")) requ...

Ебаный рот, вот тут присваивание точно не катит

Vladimir
ага и require_once вместо автолоадера

автолоадер не будет работать, если конфиги, которые тебе нужны используются на разных серверах в разных директориях

Ратмир Диронин
автолоадер не будет работать, если конфиги, которы...

если автолоадер не работает - надо выбросить это говно, взять бизнес код, и засунуть в нормальный проект.

Alexey Gevondyan
если автолоадер не работает - надо выбросить это г...

самописный движок далеко не самого умного разраба. тут того автолоадера только подключение нескольких классов, которые и то, надо переписывать

Ратмир Диронин
автолоадер не будет работать, если конфиги, которы...

я понимаю, что у тебя там какое-то жуткое легаси, но можно и свой автозагрузчик написать, который будет всё это учитывать

Ратмир Диронин
самописный движок далеко не самого умного разраба....

а зачем ты рабоатешь на таких проектах? почему не найдешь нормальный?

Vladimir
а зачем ты рабоатешь на таких проектах? почему не ...

свободного другого разраба нет, чтобы сплавить проект, да и я его уже полностью знаю. частично переписал его и всё равно бреда много ещё

Похожие вопросы

Обсуждают сегодня

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
Добрый день! Скажите пожалуйста, а какие программы вы бы рекомендовали написать для того, чтобы научиться управлять памятью? Можно написать динамический массив, можно связный ...
Филипп
7
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
14
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Всем привет! Имеется функция: function IsValidChar(ch: UTF8Char): Boolean; var i: Integer; ValidChars: AnsiString; begin ValidChars := 'abcdefghijklmnopqrstuvwxyzABCDE...
Евгений
44
Карта сайта