Как упростить условие?

Question

[XenK]

Есть такой такое условие, для воспроизведения звука при смене картинки:

if  ((id == 100) && (type == 'off') && (arrSound['v1'] != 1)) {
    ion.sound.play('off');
    arrSound['v1'] = 1;
} else if ((id == 100) && (type == 'on') && (arrSound['v1'] == 1)) {
    ion.sound.play('on');
    arrSound['v1'] = 0;
}

Как можно упростить это условие, или сделать короче?

Answer 1

[Дмитрий Ковальский]

if  (id == 100) {
ion.sound.play(type);
arrSound['v1'] = type == "on" ? 0 : 1;
}

Comments

[werevolff]

if (id === 100) {
    ion.sound.play(type);
    arrSound['v1'] = type === "on" ? 0 : 1;
}

Answer 2

[werevolff]

if (id === 100) {
    (type === 'off') ? ion.sound.play('off') | ion.sound.play('on');
    arrSound['v1'] = (arrSound['v1'] !== 1) ? 1 : 0;
}

Comments

[Therapyx]

А почему вы ставите "|", а не ":" ?

[werevolff]

Therapyx: Очепятался.

[Therapyx]

werevolff: а.. а то я думаю, что опять мой пробел хД Хорошо, а 3 раза равно? "==="

[werevolff]

Therapyx:
var val1 = 1;
var val2 = true;
console.log(val1 == val2);
>> true

[Therapyx]

werevolff: я об этом) if (id === 100) -> тут 3 знака равно)) - и тут тоже (arrSound['v1'] !== 1) и тут (type === 'off')

[werevolff]

Therapyx: var val2 = true;
console.log(val1 == val2);
>> true
или
var val1 = 1;
var val2 = true;
console.log(val1 === val2);
>> false

Разница есть?

[werevolff]

Therapyx: Для строк это не обязательно, но желательно. В JS == означает эквивалентно. Т.е. 1 эквивалентен true, например. Строка '15' эквивалентна целому числу 15. А знак === означает идентичность, и проверяет не только значение, но и тип данных. Поэтому, логично использовать при таких проверках идентичность, а не эквивалент. Но это ИМХО.

[werevolff]

Therapyx: Вообще, у Дмитрия наверху более правильный вариант. Только arrSound надо из условия убрать. И я бы настаивал на идентичности, а не эквивалентности. Впрочем, если айдишник может передаваться как строкой, так и числом, то лучше использовать эквивалент.

[Therapyx]

werevolff: просто переворачивается ответ с тру на фолс? )

[Therapyx]

werevolff: ааа)) ясно, спасибо. Я просто с JS работал когда-то давным давно на уровне "копипейста". В Жаве и плюсах как-то ниразу не видел такого