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

Question

Привет! У меня есть такой код 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кажите, пожалуйста, как лучше в итоге?
На мой взгляд, во втором варианте читаемость ухудшается.

#javascript #programming #russian

0

23.11.2023

18 ответов

38 просмотров

nnn. Hunter Автор вопроса

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

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

0

23.11.2023

Лёша Рубцов

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

0

23.11.2023

nnn. Hunter Автор вопроса

Лёша Рубцов
твой лучше, но ещё лучше, используя return в услов...

Не понимаю немного, что ты имеешь в виду

0

23.11.2023

Лёша Рубцов

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

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

0

23.11.2023

/ /

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

пойми

0

23.11.2023

/ /

Лёша Рубцов
if (a) return b; if (c) return d; if (e) return f;...

вот так норм

0

23.11.2023

nnn. Hunter Автор вопроса

Лёша Рубцов
if (a) return b; if (c) return d; if (e) return f;...

А, понял, спасибо!

0

23.11.2023

Лёша Рубцов

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

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

0

23.11.2023

/ /

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

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

0

23.11.2023

Bohdan Bida 🇺🇦

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

0

23.11.2023

nnn. Hunter Автор вопроса

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

Ну, пулл реквест он отметил

0

23.11.2023

/ /

nnn. Hunter
Ну, пулл реквест он отметил

шиз

0

23.11.2023

nnn. Hunter Автор вопроса

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

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

0

23.11.2023

/ /

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

да

0

23.11.2023

Bohdan Bida 🇺🇦

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

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

0

23.11.2023

nnn. Hunter Автор вопроса

Bohdan Bida 🇺🇦
Дело не сколько в них, сколько в том что это читае...

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

0

23.11.2023

Bohdan Bida 🇺🇦

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

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

0

23.11.2023

Евгений 🎃 · Accepted Answer

Евгений 🎃

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

0

23.11.2023

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

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

18 ответов

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