布尔值表现为真，但被列为假？

Question

我正在创建一个简单的井字游戏，我有一个名为winAlert的布尔值，如果它为真，它应该提醒玩家他们赢了。这在大多数情况下都能正常工作，但有一个实例不能。如果游戏获胜并且所有单元格都被填满，控制台记录winAlert的值为假，但它仍然提醒玩家他们已经获胜，就好像它是真的一样。有人能看一下这段代码，看看为什么它是这样运行的吗?http://jsfiddle.net/Z5c9P/3/这个函数是我认为问题所在，但我不确定。

var determineWin = function (pMoves) {
    for (var i = 0; i < winConditions.length; i++) {
        if (winConditions[i].length > pMoves.length) {
            continue;
        }
        for (var j = 0; j < winConditions[i].length; j++) {
            winAlert = false;
            for (var k = 0; k < pMoves.length; k++) {
                if (pMoves[k] === winConditions[i][j]) {
                    winAlert = true;
                    break;
                }
            }
            if (!winAlert) break;
        }
        if (winAlert) {
            alert(currentPlayer + " wins!");
            break;
        }
    }
};

下面是调用这个函数的代码:

$('td').one('click', function () {
    turnCount += 1;
    setCurrentPlayer();
    $(this).text(currentPlayer);
    cellTracker = $(this).attr('id');
    storeMoves();
    determineWin(xMoves);
    determineWin(oMoves);
    if(turnCount === 9 && winAlert === false) {
        alert("Tie game!");
    }
    console.log(turnCount, xMoves, oMoves, winAlert);
});

Answer 1

这是因为您的代码执行了以下操作:

storeMoves();
determineWin(xMoves);
determineWin(oMoves);
if(turnCount === 9 && winAlert === false) {
    alert("Tie game!");
}
console.log(turnCount, xMoves, oMoves, winAlert);

所以如果X赢了，determineWin(xMoves)将把变量设为true, determinWin(oMoves)将把它设为false，所有这些都在console.log()

之前

解决这个问题的一种方法是只检查当前玩家的移动是否获胜:

storeMoves();
determineWin(currentPlayer == 'X' ? xMoves : yMoves);
if(turnCount === 9 && winAlert === false) {
    alert("Tie game!");
}
console.log(turnCount, xMoves, oMoves, winAlert);

Answer 2

您已经对每个球员调用了determineWin。因此，如果x获胜，determineWin(oMoves);将使winAlert为假。这就是问题所在吗?也许你应该创建一个新的determineWin，它只调用一次来确定谁是赢家。

这段代码将跳过另一个用户(所以winAlert仍然是正确的)，当他的单元格小于3时，所以这个问题不需要填充所有的单元格，而只是每个玩家有超过3个单元格。

    if (winConditions[i].length > pMoves.length) {
        continue;
    }

Answer 3

我稍微改变一下你的代码

 var determineWin = function (pMoves) {
    for (var i = 0; i < winConditions.length; i++) {
        if (winConditions[i].length > pMoves.length) {
            continue;
        }
        winAlert = false;
        matches = 0;
        for (var j = 0; j < winConditions[i].length; j++) {                       
            for (var k = 0; k < pMoves.length; k++) {
                if (pMoves[k] === winConditions[i][j]) {                        
                    matches++;
                }            
            }               
        }
        if (matches == 3) return true;

    }
   return false;
};

然后

  $('td').one('click', function () {
    turnCount += 1;
    setCurrentPlayer();
    $(this).text(currentPlayer);
    cellTracker = $(this).attr('id');
    storeMoves();
    if (determineWin(xMoves)){ // this is changed
        alert("X Win")
        return;
    };
    if (determineWin(oMoves)){
        alert("O Win")
        return;
    };
    if(turnCount === 9 && winAlert === false) {
        alert("Tie game!");
    }
    console.log(turnCount, xMoves, oMoves, winAlert);
});

**更新以澄清