4

我无法让这个递归算法工作。它应该解决数独网格（整数的二维数组）。我在 Java 中做了同样的事情（当然是 OOP），而且效果很好。但在 JS 中却没有。网格中的值不会改变，也不会达到基本情况。该算法是solveSudoku - 基本情况是，当它搜索网格但没有找到任何“空”单元格（值为'0'）时。

我已将所有代码放在一个文件中。你能发现错误吗？我已经尝试了一个星期，并且即将放弃这个小项目。

"use strict";

var grid = [
        [0,0,8,4,0,3,5,0,6],
        [0,0,3,1,0,2,0,0,4],
        [0,4,5,7,0,0,0,9,0],
        [6,9,0,0,0,5,0,0,7],
        [0,8,0,0,0,0,0,5,0],
        [4,0,0,3,0,0,0,1,8],
        [0,7,0,0,0,6,2,4,0],
        [1,0,0,5,0,7,8,0,0],
        [8,0,6,9,0,1,3,0,0]
    ];

// recursive algo
function solveSudoku(grid, row, col) {
    var cell = findUnassignedLocation(grid, row, col);
    row = cell[0];
    col = cell[1];

    // base case: if no empty cell  
    if (row == -1) {
        console.log("solved");
        return true;
    }

    for (var num = 1; num <= 9; num++) {

        if ( noConflicts(grid, row, col, num) ) {   
            grid[row][col] = num;

            if ( solveSudoku(grid, row, col) ) {                
                return true;
            }

                    // mark cell as empty (with 0)    
            grid[row][col] = 0;
        }
    }

    // trigger back tracking
    return false;
}


function findUnassignedLocation(grid, row, col) {
    var done = false;
    var res = [-1, -1];

    while (!done) {
        if (row == 9) {
            done = true;
        }
        else {
            if (grid[row][col] == 0) {
                res[0] = row;
                res[1] = col;
                done = true;
            }
            else {
                if (col < 8) {
                    col++;
                }
                else {
                    row++;
                    col = 0;
                }
            }
        }
    }

    return res;
}

function noConflicts(grid, row, col, num) {
    return isRowOk(grid, row, num) && isColOk(grid, col, num) && isBoxOk(grid, row, col, num);
}

function isRowOk(grid, row, num) {
    for (var col = 0; col < 9; col++)
        if (grid[row][col] == num)
            return false;

    return true;
}
function isColOk(grid, col, num) {
    for (var row = 0; row < 9; row++)
    if (grid[row][col] == num)
        return false;

    return true;    
}
function isBoxOk(grid, row, col, num) {
    row = (row / 3) * 3;
    col = (col / 3) * 3;

    for (var r = 0; r < 3; r++)
        for (var c = 0; c < 3; c++)
            if (grid[row + r][col + c] == num)
                return false;

    return true;
}

function printGrid(grid) {
    var res = "";

    for (var i = 0; i < 9; i++) {
        for (var j = 0; j < 9; j++) {
            res += grid[i][j];
        }
        res += "\n";
    }
    console.log(res);
}
4

2 回答 2

5

很难找到……但问题出在

row = (row / 3) * 3;
col = (col / 3) * 3;

Javascript 只使用浮点数，所以这不是你认为的那样。

解决方案是

row = Math.floor(row / 3) * 3;
col = Math.floor(col / 3) * 3;
于 2013-08-24T16:33:55.290 回答
3

在 javascript 中，数字不能被强制为整数，所以类似(a / 3) * 3的东西与a. 中isBoxOk，您需要更换

row = (row / 3) * 3;
col = (col / 3) * 3;

经过

row = Math.floor(row / 3) * 3;
col = Math.floor(col / 3) * 3;

此外，我认为您可以简化findUnassignedLocation功能：

function findUnassignedLocation(grid, row, col) {
    for (; row < 9 ; col = 0, row++)
        for (; col < 9 ; col++)
            if (grid[row][col] == 0)
                return [row, col];
    return [-1, -1];
}
于 2013-08-24T16:40:31.107 回答