Поиск соседей в игре из пяти в ряд

Question

Я могу объявить победителя, если игрок получает пять своих жетонов подряд, и теперь я пытаюсь реализовать функцию захвата частей противника. Например, я игрок X, я могу отловить части O, подобные этому X00X, в любом направлении на доске. Чтобы реализовать это, правильная логика: Найти всех соседей вокруг O, проверить и посмотреть, является ли сосед X или O, если в каждой строке есть два нуля, окруженных X, я могу возьмите эти куски. Есть ли лучший способ подойти к этой проблеме? Я имею в виду что-то вроде этого:

function isOCaptured(token, row, col){ 
    if(gameBoard[row][col]==="O" && gameBoard[row][col+1] === "X" 
     && gameBoard[row][col-1] === "X"){ 
    return true;  
    } 
    return false; 
} 

Но это не кажется, возвращает истину, когда я помещаю один O между двумя крестиками.

Вот что мой server.js и app.js выглядеть прямо сейчас: https://jsfiddle.net/Amidi/s3gnx3rL/4/ HTML, просто 13 х 13 сетка кнопок с событием, прикрепленного к каждому, который посылает кнопки координат функции Add() в моем app.js

Answer 1

Теперь проблема заключается в том, что код работает, только если «О» был помещен последним. Это можно решить с помощью цикла for.

for(var i = -1; i <= 1; i++) { 
    if (gameBoard[row][col+i]==="O" && gameBoard[row][col+i+1] === "X" 
     && gameBoard[row][col+i-1] === "X") { 
     return true; 
    } 
} 
return false; 

Вы также можете использовать ломтик:

// The closest 5 cells to the left/right of the newly added piece, as a string 
var str = gameBoard[row].slice(col-2,col+3).join(""); 
// Look for the pattern "XOX" in those 5 cells. Double negation to return bool. 
return !!str.match("XOX"); 

---

Старый ответ

По вашему комментарию, код был назван так:

if(isOCaptured(x,y)===true){console.log("y is between 2 x\s"); 

Этот означает, что функция вызывается с два вместо трех аргументов. Это приведет к следующим значениям аргументов: token = x, row = y и col = undefined.

Выражение оценивается таким образом, будет:

gameBoard[y][undefined]==="O" && gameBoard[y][undefined+1] === "X" 
    && gameBoard[y][undefined-1] === "X" 

Что бы оценить следующим образом:

1. Оценка gameBoard[y] будет успешным, в результате чего в колонке (не тот, хотя).
2. Затем мы попытаемся индексировать столбец с undefined, и поскольку массив не содержит элемент с именем undefined, он не будет работать, в результате чего (снова) undefined.
3. Затем мы сравниваем это значение undefined с "0", что, очевидно, неверно.
4. Выражение false && ... вернет false, как и вся функция.

Если вторая часть была бы выполнена, было бы вычислить undefined+1, который (довольно точно) оценивается в NaN (не число). Массив также не содержит NaN, поэтому этот расчет также будет оценен как false.

---

Теперь несколько советов:

1. Заявление if(expr){return true} else {return false} может быть упрощена только return expr.
2. Эти ошибки довольно легко найти, если вы используете Chrome Debugger (или Firebug для firefox), где вы можете пройти через код и посмотреть, что происходит и где что-то не так.