Как запретить невалидные сравнения в if?

Question

[Даниил Сугоняев]

Столкнулся со следующей ситуацией, упрощенный пример

const isItLanding = 1;

    if (isItLanding === undefined) {
      return next();
    }

Ни у TS, ни у линтера нет проблем с подобным кодом, хотя мы сравниваем разные типы данных и код внутри if никогда не исполнится. Подскажите, можно ли дополнить конфигурацию TS или линтера правилом от таких случаев?

Comments

[WbICHA]

Но зачем?

[Вадим]

А чем это сравнение не валидное?
Тем что сравнивает разные типы данных? Ну так это валидно для языка.
Тем что Вам это не нравится? Ну тогда напишите предварительно сравнение типов данных.

[Даниил Сугоняев]

WbICHA, потому что код выше никогда не сработает?

[Даниил Сугоняев]

Вадим, Однако при этом при попытке сравнить строку и число TS выдаст ошибку. Это же кейс покрыли, не понимаю почему сравнение числа с undefined более валидное?

[Вадим]

Даниил Сугоняев, ну вот, уже более понятно что Вас не устраивает. Стоило с этого начинать.

[Даниил Сугоняев]

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

[Вадим]

Даниил Сугоняев, Возможно, ответ тут:
https://www.typescriptlang.org/tsconfig#strictNull...

[Даниил Сугоняев]

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

[Aetae]

Ну вот не учитывает, там моного всяких особенностей работы с undefined в ts.
Однако возможно это потому, что undefined - это переменная, и кто-то может сделать так:

{
  const undefined = 1; // за кадром
  const isItLanding = 1;

  if (isItLanding === undefined) {
      alert('оппа');
  }
}

Хотя врядли конечно.)

[Даниил Сугоняев]

Aetae, Удивительно, что это работает) Я думал undefined зарезервировано
Это, конечно, тот еще бред) Подумываю написать свое правило для ts-lint, если не удастся найти готовое решение

Answer 1

[Даниил Сугоняев]

Нашел нужное правило в линтере https://typescript-eslint.io/rules/no-unnecessary-...
По недоразумению его не было в проекте, зато теперь я знаю больше про работу ts)