Комбинированный if?

Question

[Nwton]

Уверен, вопрос задавался много раз, но так и не смог ничего найти. Меня инетерсует, как можно построить if, используя название переменной лишь один раз, например:
if(var == (52 || 62))
вместо
if(var == 52 || var == 62)

Answer 1

[Олег]

if( [52, 62].indexOf( var ) !== -1 )

Comments

[Дмитрий Лузанов]

Кажется это ещё более запутано :D

[Nwton]

Не плохой вариант, тогда лучше так: if(~[52, 62].indexOf(var))

[Олег]

Nwton: Менее читаемо, но да, можно и так.

[Nwton]

Олег: https://jsfiddle.net/fzo4L4bu/

[Stepanya]

Nwton: что-то Ваш вариант сильно уступает, как в читаемости, так и в скорости

Answer 2

[lega]

if({52:1, 62:1}[var])

if('52,62'.match(52))

Comments

[Nwton]

Как называется первая конструкция, что это?

[Nwton]

Подозреваю, что если во втором случае я буду проверять 52, а в строке будет '62,552,22) то выдаст true

[Дмитрий Лузанов]

Nwton: в первом случае для наглядности:
var matchObj = {
52:true,
62: true,
100600: true
}; //объект со всеми возможными значениями
var match = 52;
if (matchObj[match])

[Денис Загаевский]

Какая жесть. И это чтобы сэкономить... что?

[abberati]

О БОЖЕ МОЙ

не делайте так никогда!

[Nwton]

abberati: да никто и не делает)

Answer 3

[dmz9]

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

var X = 52;
    console.time('someFunction1');
    for (var i=0,x=0; i < 10000; i++) {
        if ([52, 62, 72, 82, 92].indexOf(X) !== -1) {
            x++;
        }
    }
    console.log(x);
    console.timeEnd('someFunction1');
    console.log('--------------');
    console.time('someFunction2');
    for (var i=0,y=0; i < 10000; i++) {
        switch (X) {
            case 52:
            case 62:
            case 72:
            case 82:
            case 92:
                y++;
                break;
        }
    }
    console.log(y);
    console.timeEnd('someFunction2');
    console.log('--------------');
    console.time('someFunction3');

    for (var i=0,z=0; i < 10000; i++) {
        if({52:1, 62:1, 72:1, 82:1, 92:1}[X]) { z++; }
    }
    console.log(z);
    console.timeEnd('someFunction3');

10000
someFunction1: 2.541ms
--------------
10000
someFunction2: 1.192ms
--------------
10000
someFunction3: 3.676ms

Comments

[Nwton]

Тут вопрос удобства написания (а писать несколько раз case как-то не комильфо). А сам код выполняется у клиента и мс не имеют значения, так как native js без мусора и все летает.

[dmz9]

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

Answer 4

[Александр]

Еще можно так(поиск ключа в массиве):
if(34 in {34:"",55:""}){}

Comments

[abberati]

НЕТ!

Answer 5

[sasha]

оператор || это бинарный оператор который возвращает булевый тип. На вход этот оператор принимает два булевых типа. Тоесть если ты используешь (52 || 62) то из скобки выйдет true.

Comments

[Nwton]

Я понимаю, что || тут не сработает, я привел в качестве примера

[Ivanq]

[зануда mode]Не true, а 52[/зануда mode]