Почему в условии «o» есть true в Javascript?

Question

[DarthJS]

Ребята помогите понять почему так происходит.

Вот участок кода, результат можно посмотреть по ссылке ниже.
Я задаю услови, что если ячейка1 и ячейка2 и ячейка3 == "Х" должно говорить ты победил, но если в середину попадает буква "о", то оно тоже говорит ты победил, но "Х" == "о" false. Почему же так происходит?

this.rezult = function(){

this.arr2.forEach(function(arr, i, innerArr){

arr.forEach(function(val, j){


var wincomb = innerArr[i][j] && innerArr[i][j+1] && innerArr[i][j+2];
var wincomb2 = innerArr[i][j] && innerArr[i+1][j] && innerArr[i+2][j];
var wincomb3 = innerArr[i][j] && innerArr[i+1][j+1] && innerArr[i+2][j+2];

console.log(wincomb == "X" && innerArr[i][j] !== "o");

if(wincomb == "X"){

alert(' X wins!');

}
})

})

}

ссылка JSFiddle

Answer 1

[Interface]

Судя по этой конструкции:

var wincomb = innerArr[i][j] && innerArr[i][j+1] && innerArr[i][j+2];

я так понимаю, вы расчитываете, что в переменной wincomb окажется "X". Однако, там окажется булева переменная, т.е. true/false. При этом false там будет, если каждый из этих:

innerArr[i][j];
innerArr[i][j+1];
innerArr[i][j+2];

элементов будет пустой строкой.
Затем вы сравниваете if(wincomb == "X"){
результат (булева переменная) со строкой, которая приведется также к булевой, а так как строка это "Х", то всегда будет приводиться к true.

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

Comments

[DarthJS]

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

[Interface]

DarthJS: вы можете изложить вашу задачу более развернуто - возможно найдется компактное элегантное решение

[Interface]

DarthJS: вот вам про приведение типов - будет полезно, думаю :) (https://learn.javascript.ru/types-conversion)

[Евгений Петров]

Interface: А в каком месте вы видите преобразование в boolean?
DarthJS: Код ваш и правда запутан на ровном месте...

[DarthJS]

Interface: задачу я решил, проблема была в том, что Javascript как-то особенно понимает условие &&. То есть, если (0 && "X" && "X") === 0 оно почему-то возвращает true. Так и получилось у меня, я хотел чтоб 3 Х в ряд давали true, а при попадании "о" оно все равно авало тру. Я это переписал как innerArr[i][j] === "X" && innerArr[i][j+1] === "X" && innerArr[i][j+2] === "X"; Работает! Ура!. Но когда много условий, получается много кода. За компактное и особенно "элегантное" :) решение буду премного благодарен.
Вот обновленный вариант jsfiddle.net/ufaj6kcm/13 последняя функция rezult. Очень некрасивые условия, если я решу делать 5 в ряд и еще условия для опонента, то получится очень много кода

[DarthJS]

Евгений Петров: я новичек и учусь, поэтому даже особо и не расчитывал, что мне все дружно воскликнут "Ах какой код!" :)

[Евгений Петров]

DarthJS: Вот этот оператор 0 && 'X' вернёт X. А вот этот 'X' == 'X' вернёт true.

[Interface]

Евгений Петров: каюсь, в первом случае, насколько я понимаю, будет возвращен последний элемент выражения

[DarthJS]

Евгений Петров: да, но если это у вас 0 - zero, а не буква, то вернет 0

[Евгений Петров]

DarthJS: Не, ну это — совсем начальное:)
Конечное же, !!'O' вернёт true. Пустая строка, 0 — это при проверке преобразуется в false. Остальное — в true. Даже NaN.
Не нужно заполнять пустыми строками массив. При обращении к несуществующему свойству объекта вы получите undefined. Что для проверки && монопенисуально.

Answer 2

[Андрей Мохов]

логику не проверял, но на сколько я знаю оператор !== в JavaScript отсутствует, один символ "=" уберите

Comments

[Евгений Петров]

Конечно же, присутствует https://people.mozilla.org/~jorendorff/es6-draft.h...

[DevMan]

Андрей Мохов
console.log( 0 != '0' )
console.log( 0 !== '0' )

[Андрей Мохов]

ок ок, накинулись... видимо мои сведения устарели

[Евгений Петров]

Андрей Мохов: Да не накинулись... Просто в языках со слабой типизацией такая проверка есть изначально.