NO
Скорее наследие jsx
Size: a a a
2021 September 08
M
M
В
Мне 2 больше нравится
M
но
return на одной строке с if для меня вообще не читаемIE
А someString - всегда строка?
РО
Просто есть такой аргумент, что в первом методе по сути 3 if’а. И когда ты читаешь код, опускаешься глубже и читаешь вложенные условия, тебе нужно запоминать, что там сверху есть еще одно условие. В то время как во втором случае if’ов только 2 и свои условия они содержат в себе полностью.
РО
Да, всегда строка.
M
я за меньшее кол-во уровней абстракций
IE
function fn () {
const trimmed = someString.trim()
if (trimmed.length >= 15) {
finalMethod()
return
}
if (trimmed) {
handleString()
}
}VK
Тут один уровень абстракции во всех вариантах)
РО
Выглядит как компромис, спасибо :) Но тут скорее просто интересно услышать мнение других людей. Относительно того, что лучше использовать вложенные условия, или вынести все на первый уровень.
DI
IE
Если можно сделать быстрый выход из функции, то лучше его сделать, поэтому мне оба не нравятся. В целом, длинные условия сложнее воспринимать отдельно, но и цепочки ифов тоже не облегчают задачу.
IE
Если там пробелы, то это не сработает
DI
Сработает во втором условии. Первое условие проверяет имеется ли вообще значение.
D
Если длина при обрезке будет 0 то твой вариант неправильно отработает же
D
А нет, my bad. Все верно )
РО
Там прикол в том, что когда строка пустая, то нам нужно вызвать finalMethod. Для контекста, это обработка инпута. Мы должны вызвать finalMethod если длина строки боольше 15 символов и мы должны так же его вызвать, когда длина строки равна 0 (пользователь очистил инпут).
M
Могу ли я сделать dropdown input чтоб он с верху появлялся а не снизу:?