2016-08-10 14 views
5

私は3日間、数独バックトラッキングソルバーで私の間違いを把握しようとしています。問題はleetcode Sudoku Solverです。数独バックトラックソルバー(Java)

私のソルバーは添付画像のdeductionに基づいています。問題は、ルートからリーフへのパスが無効であっても、私のボードが変更されることです。

つまり、無効なパスを通過した後、試行した値は元のボードに固定されます。しかし、私は元のボードが子どもが真を返すときにのみ更新します(ヘルパーメソッドの部分を参照してください://数値を入れて子供を生成する)。

基本的に、各 '。'に対して、1から9までのすべての可能性を生成し、現在の '。'を塗りつぶす一時ボードを構築します。 1つの可能性を持って、次のヘルパーに電話してください。臨時ボード付き。スペースコストのために、それぞれの子供のために同じサイズのboardTempを保存するのは良いことではありませんが、私の主な関心事は、まずコストを最適化する前に問題を解決することです。

すべての子供が正当でない場合でも私のボードはなぜ変わるのですか?例えば

、最初のボード

..9748のベース...。 7........; .2.1.9 ...;

.. 7 ... 24。 ; 64.1.59。 .98 ... 3 ..;

... 8.3.2。 ........ 6; ... 2759 ..;

私が実行した後、私は最終的な結果を得た:

139748652。 745326189; 826159437;; 64.1.59。 .98 ... 3 ..;

... 8.3.2。 ........ 6; ... 2759 ..;

public void sudokuSolver(char[][] board) { 
    for (int i = 0 ; i < board.length ; i++) { 
     for (int j = 0 ; j < board.length ; j++) { 
      if (board[i][j] == '.') { 
       // find the first '.' as root 
       helper(i, j, board); 
       return; 
      } 
     } 
    } 
} 

private boolean helper(int row, int col, char[][] board) { 
    // case 2. check if it has following '.' and store its position 
    boolean hasNext = false; 
    boolean nextSearching = false; 
    int nextRow = row; 
    int nextCol = col; 
    for (int i = 0 ; i < board.length ; i++) { 
     for (int j = 0; j < board.length ; j++) { 
      if (nextSearching && !hasNext && board[i][j] == '.') { 
       hasNext = true; // there is next! 
       nextRow = i; 
       nextCol = j; 
      } 
      if (i == row && j == col) { 
       nextSearching = true; 
      } 
     } 
    } 

    // exit condition: last '.' 
    if (!hasNext) { 
     for (char put = '1' ; put <= '9' ; put ++) { 
      if (isValid(row, col, board, put)) { 
       return true; 
      } 
     } 
     return false; 
    } 

    // put a number and generate children 
    for (char put = '1' ; put <= '9' ; put ++) { 
     if (isValid(row, col, board, put)) { 
      char[][] boardTemp = board; 
      boardTemp[row][col] = put; 
      boolean valid = helper(nextRow, nextCol, boardTemp); 
      if (valid) { 
       // board is supposed to change only when valid is true. 
       board[row][col] = put; 
       return true; 
      } 
     } 
    } 
    return false; 
} 

private boolean isValid(int row, int col, char[][] board, char c) { 
    // go through each row, column, and subblock to determine if c is a valid choice based on current board. 
    for (int jCol = 0 ; jCol < 9 ; jCol ++) { 
     if (board[row][jCol] == c) { 
      return false; 
     } 
    } 
    for (int iRow = 0 ; iRow < 9 ; iRow ++) { 
     if (board[iRow][col] == c) { 
      return false; 
     } 
    } 
    for (int i = row/3*3 ; i < row/3*3 + 3 ; i++) { 
     for (int j = col/3*3 ; j < col/3*3 + 3 ; j++) { 
      if (board[i][j] == c) { 
       return false; 
      } 
     } 
    } 
    return true; 
} 

My tree for backtracking

+3

サイドノート:良いオブジェクト指向プログラミングは便利な抽象化を作成する方法についてですchar[][] boardTemp = boardは、彼らが同じメモリを指す意味...あなたがあなたの元のコードで逃したことは、あなたが無効な番号を入れた後でelse一部です。同様に、あなたは "本当の"クラスであなたのボードをモデル化することができます。 2 dim char配列に頼るのではなく、 "低レベル"の抽象化を使用する結果、コード自体はかなり抽象的です。あなたの命名は役に立たない。名前はそれの後ろに何があるかを言うべきです。だから、*ヘルパー*と呼ばれる方法は何ですか?なぜそれがブール値を返すのですか?そして、その方法は少なくとも3つのリターンを持っています。 WTF?長い話が短い:このコードは読みにくく、理解しにくいです。見つけにくいバグがあることは驚くことではありません! – GhostCat

+3

言い換えれば、例えば、 "抽象概念の単層"の原理について読んでください。あなたはあなたのコードをあなたのヘルパーメソッドで**すべて**持っています。代わりに、あなたはそれぞれ**一つ**のことをする小さなメソッドがたくさんあるはずです。また、小さなメソッドがたくさんある場合は、それらのそれぞれに対して小さな単位テストを記述することもできます(間違いなく、TDDと単体テストの使用には**完全**です)。本質的には、問題を小さなものに分解しようとする。それらを独立して解決し、それらをテストしてください。次に、「大きな写真」をまとめてください。 – GhostCat

+0

コメントありがとうございました。あなたが言及した一般的な方法は、大きな問題を多くの小さなサブ問題に分解するためには本当に良いはずです。それから、どこが間違っているのか分かります。私はそれをいくつかの難しい問題に使用するかもしれません。 –

答えて

4

boardTempboardを使用しての間に違いはありません。

for (char put = '1' ; put <= '9' ; put ++) { 
    if (isValid(row, col, board, put)) { 
     char[][] boardTemp = board; // board and boardTemp are essentially the same thing 
     boardTemp[row][col] = put; 
     boolean valid = helper(nextRow, nextCol, boardTemp); 
     if (valid) { 
      // board is supposed to change only when valid is true. 
      board[row][col] = put; 
      return true; 
     } 
     // THIS IS WHAT YOU MISSED!! 
     // if you don't reset the '.' back, your later backtrack will not work 
     // because you put an invalid number on your board and you will never find a solution 
     boardTemp[row][col] = '.'; 
    } 
} 
+0

:DDDD。どうもありがとうございます。 Stackoverflowは私の前回の返信を削除しました。誰かがそのS ...の言葉に不平を言うと思う。 –