深度搜索第一个bug?

时间:2012-02-24 09:07:50

标签: java algorithm depth-first-search

所以我有一个N×M的矩阵。在给定位置,我有一个代表颜色的值。如果此时没有任何内容,则值为-1。我需要做的是在我添加一个新点之后,检查所有具有相同颜色值的邻居,如果有多于2个,则将它们全部设置为-1

如果我所说的没有意义我正在尝试做的是一种算法,我用它来消除我屏幕上所有相同颜色的气泡,其中气泡被记忆在一个矩阵中-1表示没有气泡,{0,1,2,...}表示存在具有特定颜色的气泡。

另外,如果您有任何建议,我将不胜感激。谢谢。 这就是我尝试过但失败的原因:

public class Testing {

    static private int[][] gameMatrix=
        {{3, 3, 4, 1, 1, 2, 2, 2, 0, 0},
        {1, 4, 1, 4, 2, 2, 1, 3, 0, 0},
        {2, 2, 4, 4, 3, 1, 2, 4, 0, 0},
        {0, 4, 2, 3, 4, 1, 0, 0, 0, 0},
        {0, 0, 0, 0, 0, 0, 0, 0, 0, 0},
        {0, 0, 0, 0, 0, 0, 0, 0, 0, 0},
        };

    static int Rows=6;
    static int Cols=10;
    static int count;
    static boolean[][] visited=new boolean[15][15];
    static int NOCOLOR = -1;
    static int color = 1;

    public static void dfs(int r, int c, int color, boolean set)
    {
        for(int dr = -1; dr <= 1; dr++) 
            for(int dc = -1; dc <= 1; dc++)
                if(!(dr == 0 && dc == 0) && ok(r+dr, c+dc))
                {
                    int nr = r+dr;
                    int nc = c+dc;

                    // if it is the same color and we haven't visited this location before
                    if(gameMatrix[nr][nc] == color && !visited[nr][nc]) 
                    {
                        visited[nr][nc] = true;
                        count++;

                        dfs(nr, nc, color, set);

                        if(set)
                        {
                            gameMatrix[nr][nc] = NOCOLOR;
                        }
                    }
                }
    }

    static boolean ok(int r, int c)
    {
        return r >= 0 && r < Rows && c >= 0 && c < Cols;
    }

    static void showMatrix(){
        for(int i = 0; i < gameMatrix.length; i++) {
            System.out.print("[");
            for(int j = 0; j < gameMatrix[0].length; j++) {
                System.out.print(" " + gameMatrix[i][j]);
            }
            System.out.println(" ]");
        }
        System.out.println();

    }

    static void putValue(int value,int row,int col){
        gameMatrix[row][col]=value;
    }

    public static void main(String[] args){
        System.out.println("Initial Matrix:"); 
        putValue(1, 4, 1);
        putValue(1, 5, 1);
        putValue(1, 4, 2);
        showMatrix();

        for(int n = 0; n < Rows; n++)
            for(int m = 0; m < Cols; m++)
                visited[n][m] = false;

        //reset count
        count = 0;
        dfs(5,1,color,false);
        //if there are more than 2 set the color to NOCOLOR
        for(int n = 0; n < Rows; n++)
            for(int m = 0; m < Cols; m++)
                visited[n][m] = false;
        if(count > 2)
        {
            dfs(5,1,color,true);
        }
        System.out.println("Matrix after dfs:");
        showMatrix();
    }

}

3 个答案:

答案 0 :(得分:1)

您的代码也将对角线单元计为邻居。如果您只需要左/右/上/下单元格,则可以检查

if(!(dr == 0 && dc == 0) && ok(r+dr, c+dc) && dr * dc == 0)

您还需要计算第一个细胞。你不计算你开始的细胞。

答案 1 :(得分:1)

我认为你是在flood-fill algorithm之后(可能有一个奇怪的约束,即必须至少有两个相同颜色的邻居)?我不确定为什么深度优先搜索在这里是合适的。

答案 2 :(得分:1)

您遇到的一个问题是您没有检查上排和leftest col:

static boolean ok(int r, int c)
{
    return r > 0 && r < Rows && c > 0 && c < Cols;
}

您应该检查r >= 0c>= 0

第二个问题是您使用dfs()两次,但 visited字段是静态的 - 在true的第二次运行之前,它都设置为dfs() 你需要在第二次运行之前在所有字段中将其初始化为false,否则算法将立即终止而不做任何事情[因为所有节点都已在visited中 - 并且算法将决定不再重新探索这些节点。]。