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

Привет! У меня есть такой код if (selectedOptions.length === 2 || selectedOptions.length

=== 0) {
return 2
} else if (selectedOptions.includes('Контент')) {
return 0
} else {
return 1
}


Мне предложили в команде при пулл-реквесте переписать это так
return selectedOptions.length === 2 || selectedOptions.length === 0 ? 2 : selectedOptions.includes('Контент') ? 0 : 1

Cкажите, пожалуйста, как лучше в итоге?
На мой взгляд, во втором варианте читаемость ухудшается.

18 ответов

32 просмотра

В пулреквесте предложили говно

nnn.-Hunter Автор вопроса
Евгений 🎃
В пулреквесте предложили говно

Как тактично об этом сказать?)

твой лучше, но ещё лучше, используя return в условии, избегать использования else

nnn.-Hunter Автор вопроса
nnn. Hunter
Не понимаю немного, что ты имеешь в виду

if (a) return b; if (c) return d; if (e) return f; return g;

nnn.-Hunter Автор вопроса
nnn. Hunter
Как тактично об этом сказать?)

спроси у него, Как писать, если там будет 50 условий 🗿

nnn. Hunter
Как тактично об этом сказать?)

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

скажи тому кто код ревью делает, что-бы пошел почитал "чистый код" или подобные книжки)

nnn.-Hunter Автор вопроса
nnn.-Hunter Автор вопроса
Bohdan Bida 🇺🇦
скажи тому кто код ревью делает, что-бы пошел почи...

Спасибо) Вложенные тернарники - всегда так себе затея, да?

nnn. Hunter
Спасибо) Вложенные тернарники - всегда так себе з...

Дело не сколько в них, сколько в том что это читается ужасно плохо и неочевидно Сколько требуется усилий при беглом взгляде на код в твоем коде и в том коде что предложили Как по мне в твоем коде сам уже понял излишне else, но ты хотя-бы бегло смотришь и понимаешь сразу же что там А с тернарниками поди еще эти ребусы разгляди и глаза поломай)) В любом коде только одно важно - что-бы он был предсказуем и хорошо читался, ну и что-бы он был логичен, но то уже вопросы логики а не правильности(чистоты) кода Исходя из этого можно уже решать что плохо а что нет, что так себе затея а то хорошая

nnn.-Hunter Автор вопроса
Bohdan Bida 🇺🇦
Дело не сколько в них, сколько в том что это читае...

Благодарю за такой полный ответ :) else в подобных ситуациях всегда лучше упускать?

nnn. Hunter
Благодарю за такой полный ответ :) else в подобн...

Ну в нем просто смысла нет У тебя же ретерн сработает и код ниже даже не пойдет, интерпретатор даже не узнает что там какой-то else есть Просто лишний груз в коде

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

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

Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
привет всем. Народ, кто-нибудь работал с мультипоточностью? Где посмотреть/ что почитать посоветуете? В идеале чтоб материал был максимально сжатый, времени читать например в...
Serg Vish
18
Всем привет! Скажите, никто не пытался уменьшить размер процесса ssl, которые ассоциируется с открытым соединением (не помню точное название этого процесса, но там была какая-...
Алексей
20
now Linux kernel development is becoming a dictatorship group: https://www.youtube.com/watch?v=xPF2NSFvNCc
Sharuzzaman Ahmat Raslan
23
This is a big issue. Just by being a citizen of a country, you are denied to contribute to Open Source software: https://youtu.be/L5Ec5jrpLVk?si=1iIuHnMPbCB4anV-
Sharuzzaman Ahmat Raslan
72
а проверьте, собирается ли у кого сейчас транк лазаря через делюкс? у меня вот: fpcupdeluxe: info: Lazarus Native Installer (BuildModuleCustom: UserIDE): LazBuild: building Us...
Iluha Companets
20
добрый день. возможно ли изменить цвет окон лазаруса? Как?
Budemposmotret
35
А какие существуют способы обработки ошибок выделения памяти в ядре? Т.е., допустим, есть функция, которая возвращает адрес свободной страницы в физической памяти и диапазон в...
disba1ancer
51
Мне тут приспичило встроить в программу форматировние текста SQL, расставить переносы строк и отступы так, чтобы лучше читалось. Я что-то свое изобразил, оно после ключевых сл...
Sergey Bodrov
11
Карта сайта