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 ответов

10 просмотров

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

Гавно)

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
а зачем ты рабоатешь на таких проектах? почему не ...

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

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

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

Какой-то там пердун в 90-х решил, что есть какая-то разная типизация. Кого вообще это волнует?
КТ315
49
void terminal_scroll() { memmove(terminal_buffer, terminal_buffer + VGA_WIDTH, buffer_size - VGA_WIDTH); memset(terminal_buffer + buffer_size - VGA_WIDTH, 0, VGA_WIDTH); ...
Егор
47
Всем привет! Подскажите, пожалуйста, в чем ошибка? Настраиваю подключение к MySQL. Либы лежат рядом с exe. Все как по "учебнику"
Евгений
16
А можете как-то проверить меня по знаниям по ассемблеру?
A A
132
Здравствуйте! У меня появилась возможность купить книгу "Изучай Haskell во имя добра!". Но я где-то слышал, что эта книга устарела. Насколько это правда??
E
22
Здравствуйте! Я вот на stepic решаю задачи на хаскеле https://stepik.org/lesson/8443/step/8?unit=1578 мой код import Data.List (isInfixOf) removing :: String -> [String] ->...
E
10
Камрады, кто тесно работал с vtv, хотел уточнить. Ширина column задаётся жёстко на этапе создания дерева или можно в рантайме ее менять программно (не мышкой)?
Ed Doc
10
да ладно ... что там неочевидного ? глянуть в исх-ки датасета и/или кверика чтобы понять в каком месте и как выполняется обращения к св-вам blablaSQL - минутное дело, даже е...
Сергей
7
Здесь для arm кто-нибудь кодит ?
Nothing
52
Всем привет, у меня есть сервер принимающий входящие HTTP подключения, как проверить, что подключение было через прокси или нет, есть какие то поля в заголовках по которым мо...
DS
8
Карта сайта