Блог

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

 s === "test" ? MyFunc() : MyFunc2();
  

Сгенерированная ошибка была:

«Ожидал присваивания или вызова функции, а вместо этого увидел выражение».

Очевидно, что JSLint ожидает присвоения здесь, что-то больше похожее:

 var y = (s === "test") ? MyFunc() : MyFunc2();
  

Но я действительно не вижу проблемы с первым примером. Действительно ли так, что тернарные операторы должны использоваться только для присваиваний?

Я ничего не смог увидеть на JSLint.com также не было ничего очевидного в книге JavaScript: The Good Parts. И о той же ошибке также сообщается в форке сообщества JSHint.

Кто-нибудь?

Это выражение. Это эквивалентно написанию

0 === 1;

Вы пишете выражение, которое имеет немедленные побочные эффекты и считается плохим.

Обычно выражения представляют собой бесполезные инструкции, которые не имеют побочного эффекта. Считается лучшей формой просто сделать

 if (s === "test") {
  MyFunc();
} else {
  MyFunc2();
}
  

Помимо этого, это совершенно надежный синтаксис. Лично я согласен с тем, что писать краткий тернарный код в качестве альтернативы if — это плохо, и вам лучше использовать его только для присваивания.

Другие краткие выражения, которые были (ab) использованы для краткости

 someCondition amp;amp; doMagic(magic);
someCondition || doMagic(magic);
  

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

У JSHint есть опция expr для этого. Смотрите тикет

Выполняется:

 /*jshint
  expr: true
*/

var s, MyFunc, MyFunc2;
s === "test" ? MyFunc() : MyFunc2();
0 === 1;
  

Пройдет