Как сократить запись в «или»?

Question

[danilr]

export const isBs = (lead: IData): boolean =>
  lead.processingWay === ProcessingWay.Remote &&
  (lead.identityCheck === IdentityCheck.ESIA || lead.identityCheck === IdentityCheck.SMEV); // тут как-то можно сократить?чтобы не писать два раза lead.identityCheck ===

ProcessingWay и IdentityCheck енумы

Comments

[G F]

Установи вебшторм себе, он все такие моменты показывает и предлагает варианты сокращений

[xmoonlight]

Bavashi, вообще по-барабану..

Answer 1

[twobomb]

может типа такого

[IdentityCheck.SMEV,IdentityCheck.ESIA].includes(lead.identityCheck )

Answer 2

[xmoonlight]

Вот ОНО!

[IdentityCheck.SMEV,IdentityCheck.ESIA].some((a)=>lead.identityCheck===a)

Все варианты можно тут глянуть.

Comments

[twobomb]

Это какая-то дичь, оно так не работает.
P.S. Тут сначала вычислится условие в скобках, а потом будет сравниваться

[xmoonlight]

twobomb, она не может не работать или пруф сюда.

[xmoonlight]

twobomb,

Тут сначала вычислится условие в скобках, а потом будет сравниваться

Так. И почему оно не сработает?

[twobomb]

xmoonlight, ну она как-то работает, естественно синтаксической ошибки не будет, тут логическая ошибка. Это не аналог условия

(lead.identityCheck === IdentityCheck.ESIA || lead.identityCheck === IdentityCheck.SMEV)

[twobomb]

Bavashi, это еще хуже предыдущего зачем тут оператор присвоения, и кто вообще в енумах хранит булевы значения? и вообще разве в js есть енумы?

[danilr]

twobomb, да, в енумах не хранят були) это ts, я js добавил так как можно тут разницы нет, ну енум и енум, представьте что это строки

[xmoonlight]

twobomb, danilr, поправил на точно рабочий вариант.

[twobomb]

danilr, я показал как можно, а если нет там includes, то оставьте как есть это самый лучший и самый быстрый и правильный вариант, зачем что-то сокращать

[twobomb]

Bavashi, Зачем тогда так все усложнять, зачем енумы, если их всего 2 и условие

(lead.identityCheck === IdentityCheck.ESIA || lead.identityCheck === IdentityCheck.SMEV);

А это значит оно всегда тру, если в нем лежит один из енумов. А это значит его можно вообще выкинуть.

[twobomb]

поле енума может быть вычисляемым

wat?

[xmoonlight]

danilr, twobomb, Bavashi, добил! ))
Надо было запятую вместо || ставить в скобки!

[WbICHA]

xmoonlight, кхм... Прочитайте как работают круглые скобки в жс.

[twobomb]

xmoonlight, Чё ты там добил? Какой-то свой синтаксис изобретаешь, не работает он нихрена, он просто возвращает последнее значение из скобок, а у тебя оно совпало с проверяемым и тут тебя осенило.... не мучайся

[xmoonlight]

twobomb, да... совпало) Точно) Ну, ладно...
Отправил вариант, который проверил на буквах, а не на цифрах.

[twobomb]

Bavashi, that's bullshit
Твой вариант не содержит никаких вычисляемых енумов, там будут просто константные значения которые вернем getStatus в момент инициализации переменных.
Чтобы сделать подобие "вычисляемых енумов" нужно делать нечто типа

let i  = 1;
enum SomeEnum {
    No = ()=>{return i++},
    Yes =()=>{return No()*2}
}

и обращаться к этим енумам как SomeEnum.No()
Ну это такой бред такого никто и никогда не делает, ты походу никогда енумы не использовал. енумы это набор логически связанных констант и никак иначе

[WbICHA]

twobomb, Bavashi, вы оба правы, но явно не поняли друг друга. Значение ключа энума действительно будет вычислено в рантайме, только вот функция будет вызвана 1 раз при инициализации энема и после этого оно не изменится, то есть, по сути, становится константой.
Или я не так понял Bavashi и он ошибся.

[WbICHA]

Bavashi, помимо чтения документации, ещё хорошо бы проверять на практике. Тем более, что про слово "computed" вы только сейчас вспомнили. Или наконец-то залезли в доки.)
Вот к чему этот пример? Что он демонстрирует? Абсолютно ничего.
По вашей логике тут постоянно должны быть разные значения, но, внезапно, это не так:

console.log(SomeUsers.UserSaysNo);
console.log(SomeUsers.UserSaysNo);
console.log(SomeUsers.UserSaysNo);
console.log(SomeUsers.UserSaysNo);
console.log(SomeUsers.UsersSaysYes);
console.log(SomeUsers.UsersSaysYes);
console.log(SomeUsers.UsersSaysYes);
console.log(SomeUsers.UsersSaysYes);

Везде будет одно значение, полученное при инициализации энума.

[WbICHA]

Bavashi, боюсь, как раз наоборот, это вы не понимаете. Выше я уже расписал, что это значит (естественно не прибегая к этому слову), а вы что ответили?

нужно читать доки и не писать домыслы

Так вот, жду от вас разъяснения в текстовом виде что означает "computed".

[WbICHA]

Bavashi,

выше я уже написал, что это значит

Превосходно! В таком случае, думаю, вам будет нетрудно процитировать самого себя ниже, а то у меня проблемы с поиском.

если вам мой непонятен

Пока что мне понятно одно, вы поняли, что моё описание полностью верное, однако признавать это по какой-то причине отказываетесь. Как и написать своё определение значения этого термина.

[WbICHA]

Bavashi, я не понимаю, вы троллите или что? Я вас конкретно попросил:

процитировать самого себя ниже

В ответ вы про демагогию пишите...
Вам трудно сделать копи-паст вашего описания термина "computed"?

[WbICHA]

Bavashi, вы так и будете уходить от ответа? На данный момент меня интересует сам факт наличия описания от вас термина "computed".
Просто. Сделайте. Копи-паст.

[WbICHA]

Bavashi, удивительно, как же трудно некоторым людям признать, что они в чём-то ошиблись, даже в таких мелочах.
Казалось бы, ну не написал ты определение термина выше, ответь, что да, ошибся, но нет, надо постоянно переводить тему, посылать в доки, наверх, и всячески избегать своей явной неправоты.
При всём при этом ещё и говорить, что моё определение неверное.

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

[danilr]

WЫCHA был прав в этом споре, буду судьёй)
а то что я в енаме всего два значения указал так это от того, чтобы не захламлять лишним и в енаме ещё могут появиться новые записи