Styjun

I am building a minesweeper game in JavaScript and I am running into a problem while trying to clear the adjacent empty cells when a cell without a mine is clicked.

I am using a function called clearEmptySpace() to check if the variable adjacentMines is greater than 0 on all cells adjacent to the one clicked.
If the adjacentMines variable is 0 on any of the checked cells I call the checkCell() function which uncovers the cell and then calls clearEmptySpace(), with the new cell location.

If clearEmptySpace() finds a cell with 1 or more adjacent mines it calls checkCell() with a flag to ensure clearEmptySpace() is not called again with that cell location.

The issue is clearEmptySpace() checks the cell directly above itself and when it finds the cell above it has an adjacentMines value of 0 it calls checkCell() with the cell position above itself as intended but does not continue checking the other directions after the function call is made.

The only time the next check (Upper left cell) is made is when the cell above has 1 or more adjacent mine and the following checkCell() does not re-call clearEmptySpace().

Is the second clearEmptySpace() call stopping the current clearEmptySpace() from further execution? if so how do I go about fixing this issue?

For simplicity's sake the clearEmptySpace() code provided only checks the cell directly above and the upper left cell.

function checkCell(cell,onlyOnce = false)
 var cellLocation = cell.id.toString();
 var clickedCell = cellLocation.split(',');
 if(cellArray[clickedCell[0]][clickedCell[1]].hasMine)
 cell.innerHTML = "M"
 gameOver();
 else
 cellArray[clickedCell[0]][clickedCell[1]].flagged = false;
 if(cellArray[clickedCell[0]][clickedCell[1]].adjacentMines > 0)
 getFontColor(cell);
 cell.innerHTML = cellArray[clickedCell[0]][clickedCell[1]].adjacentMines.toString();
 else
 cell.innerHTML = "0";
 
 cell.style.backgroundColor = "#EBEBEB";
 if(!onlyOnce && !cellArray[clickedCell[0]][clickedCell[1]].adjacentMines)
 clearEmptySpace(cell);
 
 
 cellArray[clickedCell[0]][clickedCell[1]].hidden = false;


_____________________________________________________________________________

function clearEmptySpace(inputCell)
 var cellLocation = inputCell.id.toString();
 var currentCell = cellLocation.split(',');

 var targetCellLocation = new Array();
 targetCellLocation = [currentCell[0]-1,currentCell[1]+1];
 var targetCell = document.getElementById(targetCellLocation[0].toString()+","+targetCellLocation[1].toString());

 if(currentCell[0] > 0)
 if(!cellArray[currentCell[0]-1][currentCell[1]].adjacentMines)
 alert("The cell directly above has no adjacent mines");
 targetCellLocation = [currentCell[0]-1,currentCell[1]];
 checkCell(document.getElementById(targetCellLocation[0].toString()+","+targetCellLocation[1].toString()));
 else
 alert("The cell directly above has at least one adjacent mine");
 targetCellLocation = [currentCell[0]-1,currentCell[1]];
 checkCell(document.getElementById(targetCellLocation[0].toString()+","+targetCellLocation[1].toString()),true);
 
 if(currentCell[1] > 0 )
 if(!cellArray[currentCell[0]-1][currentCell[1]-1].adjacentMines)
 alert("The cell on the upper left has no adjacent mines");
 targetCellLocation = [currentCell[0]-1,currentCell[1]-1];
 checkCell(document.getElementById(targetCellLocation[0].toString()+","+targetCellLocation[1].toString()));
 else
 alert("The cell on the upper left has at least one adjacent mine");
 targetCellLocation = [currentCell[0]-1,currentCell[1]-1];
 checkCell(document.getElementById(targetCellLocation[0].toString()+","+targetCellLocation[1].toString()),true);