0

この質問が以前に尋ねられたことは知っていますが、答えがわかりません。私は再帰的な迷路の解決策を書くことになっていますが、これまでに行ったことは次のとおりです。

import java.awt.Color;
import java.util.ArrayList;

public class RecursiveMazeSolution implements MazeSolver {
boolean[][] marked;
ArrayList<Maze.Door> solutionPath = new ArrayList<>();

public ArrayList<Maze.Door> solveMaze(int startRow, int finishRow, int startCol, int finishCol, Maze maze) {
    marked = new boolean[maze.getRows()][maze.getColumns()];
    return solveMaze(startRow, finishRow, startCol, finishCol, maze, marked);
}

public ArrayList<Maze.Door> solveMaze(int startRow, int finishRow, int startCol, int finishCol, Maze maze, boolean[][] marked) {
    System.out.println(startRow + " " + startCol + " " + finishRow + " " + finishCol);
    if(startRow < 0 || startCol < 0 || startRow > maze.getRows() - 1|| startCol > maze.getColumns() - 1) return null;
    if(marked[startRow][startCol]) {
        System.out.println("I'm inside marked if");
        return null;
    }

    marked[startRow][startCol] = true;
    if(startRow == finishRow && startCol == finishCol) {
        solutionPath.add(new Maze.Door(startRow, startCol, Maze.NO_DIRECTION, Color.RED));
        return solutionPath;
    }

    if(solveMaze(startRow - 1, finishRow, startCol, finishCol,maze, marked) != null && !maze.isClosed(startRow, startCol, Maze.NORTH)) {
        solutionPath.add(new Maze.Door(startRow, startCol, Maze.NORTH, Color.RED));
        return solutionPath;
    }
    if(solveMaze(startRow + 1, finishRow, startCol, finishCol,maze, marked) != null && !maze.isClosed(startRow, startCol, Maze.SOUTH)){
        solutionPath.add(new Maze.Door(startRow, startCol, Maze.SOUTH, Color.RED));
        return solutionPath;
    }
    if(solveMaze(startRow, finishRow, startCol - 1, finishCol,maze, marked) != null && !maze.isClosed(startRow, startCol, Maze.WEST)){
        solutionPath.add(new Maze.Door(startRow, startCol, Maze.WEST, Color.RED));
        return solutionPath;
    }
    if(solveMaze(startRow, finishRow, startCol + 1, finishCol,maze, marked) != null && !maze.isClosed(startRow, startCol, Maze.EAST)){
        solutionPath.add(new Maze.Door(startRow, startCol, Maze.EAST, Color.RED));
        return solutionPath;
    }

    return null;
}


}

これが私に提供された迷路クラスです：

import java.io.Serializable;
import java.awt.Color;

public class Maze implements Serializable {

/**
 * 
 */
private static final long serialVersionUID = -787488019846627488L;
/**
 * the north wall of a room
 */
public static final int NORTH = 0;
/**
 * the east wall of a room
 */
public static final int EAST = 1;
/**
 * the south wall of a room
 */
public static final int SOUTH = 2;
/**
 * the west wall of a room
 */
public static final int WEST = 3;
/**
 * No direction from a room
 */
public static final int NO_DIRECTION = 4;
private static String[] walls = {"North", "East", "South", "West"};
private Room[][] rooms;

/**
 * Constructor
 * @param rows is the number of rows in the maze
 * @param columns is the number of columns in the maze
 */
public Maze(int rows, int columns) {
    rooms = new Room[rows][columns];
    for (int i = 0; i < rows; i++) {
        for (int j = 0; j < columns; j++) {
            rooms[i][j] = new Room();
        } // end for
    } // end for
}

/**
 * rows accessor
 * @return the number of rows in the maze
 */
public int getRows() {
    return rooms.length;
}

/**
 * columns accessor
 * @return the number of columns in the maze
 */
public int getColumns() {
    return rooms[0].length;
}

/**
 * Checks to see if a wall is closed
 * @param row the row number
 * @param column the column number
 * @param wall the wall number
 * @return true if wall is closed; false if it is open
 */
public boolean isClosed(int row, int column, int wall) {
    return rooms[row][column].closed[wall];
}

/**
 * Opens the wall
 * @param row the row number
 * @param column the column number
 * @param wall the wall number
 */
public void setOpen(int row, int column, int wall) {
    rooms[row][column].closed[wall] = false;
}

/**
 * Closes the wall
 * @param row the row number
 * @param column the column number
 * @param wall the wall number
 */
public void setClosed(int row, int column, int wall) {
    rooms[row][column].closed[wall] = true;
}

public static class Door {

    int row;
    int column;
    int wall;
    Color color;

    public Door(int row, int column, int wall, Color color) {
        this.row = row;
        this.column = column;
        this.wall = wall;
        this.color = color;
    } // end constructor

    public boolean equals(Object x) {
        if ( x == null ) return false;
        if (!(x.getClass().equals(this.getClass()))) {
            return false;
        }
        Door door = (Door) x;
        return row == door.row && column == door.column && wall == door.wall && color.equals(door.color);
    } // end equal

    public int hashCode() {
        return row + column + wall + color.hashCode();
    }

    public String toString() {
        return row + " " + column + " " + walls[wall] + "\n";
    } // end toString()
} // end Door

private class Room implements Serializable {

    boolean[] closed;

    Room() {
        closed = new boolean[4];
        for (int i = 0; i < 4; i++) {
            closed[i] = true;
        } // end for
    }
} // end Room
} // end Maze

私は解決策への正しい道を進んでいると思いますが、私のプログラムはしばらく再帰的に実行され、迷路の解決策が見つかりません。また、プログラムに「壁」を無視させると、(多くの再帰呼び出しの後) 解決策を見つけることができますが、壁を無視することは想定されていません。誰が私が間違っているのか教えてください。

4

1 に答える 1

1

あなたのコードをざっと見てみると、問題は、訪問した正方形のマークを外しsolveMazeていないことだと思います 関数に入ると、四角をマークして、二度とアクセスできないことを示します。ただし、戻る前にもう一度無料としてマークする必要があります。

これについてもう少し考えてみると、そのマスを通る解パスがないと判断した後でそのマスのマークを外すので、通常は問題にならないはずであると思われました。

次に、再帰呼び出しの後に壁テストを行っていることに気付きました。つまり、解決策を見つけるために壁越しに探していて、途中に壁があるために解決策を放棄していることを意味します。その間、すべての正方形を訪問済みとしてマークすると、有効な解決策を探す場所がなくなります。

再帰の前に壁をテストする必要があり、壁がある場合は再帰を実行しないでください。ここでは、短絡評価で十分です (ifステートメント内の用語を並べ替えることにより)。

if( !maze.isClosed(startRow, startCol, Maze.NORTH) &&
    solveMaze(startRow-1, finishRow, startCol, finishCol,maze, marked) != null )

ちなみに、markedパラメータとして渡す必要はありません。クラスメンバーです。

于 2013-03-06T09:12:21.190 に答える