Как можно упростить код?

Question

[vetsmen]

Как можно упростить этот код?

if(!parseInt(data.teg)) {
            data.teg = 'Нет';
          } else {
            data.teg = 'Да';
          }

Comments

[Negwereth]

что в data.teg вообще может находиться?

Answer 1

[Alexander Pushkarev]

data.teg = ( !parseInt(data.teg) ? 'Нет' : 'Да' )

Пример работы:

Comments

[Eugene Mosyukov]

Чтобы было совсем просто, лучше инвертировать условие и вынести его отдельно, чтобы по названию переменной было понятно, на что у нас идет проверка:

let isNumericTag = parseInt(data.teg, 10);
data.teg = isNumericTag ? 'Да' : 'Нет';

Answer 2

[Сергей delphinpro]

Не забывайте указывать систему счисления

parseInt(tag, 10);

А то ненароком можно и шестнадцатиричное и восьмеричное (кроме вебкитов) число получить. =)

Comments

[vetsmen]

По умолчанию разве не 10-ая система счисления стоит?

[Сергей delphinpro]

vetsmen: в javascript всегда всё не просто. по дефолту десятичная, но он пытается опознать систему счисления. Если сможет - интерпретирует как 16-ричную. Если в строке будет первым ноль - в 8-ричную (это кроме вебкитов)

[vetsmen]

Сергей:
parseInt(tag*10 + 5, 10);
Вот так делать тоже можно? То есть указание системы счисления при стоящем выражении ругаться не будет?

[Сергей delphinpro]

vetsmen: можно, не будет