Как исправить: Решатель судоку проблема переполнения стека - PullRequest
Новая
       38

Как исправить: Решатель судоку проблема переполнения стека

0 голосов
/ 08 февраля 2019

Я пытаюсь создать решатель судоку на Java, в общем, я новичок в программировании и в Java.Я действительно не знаю, как справиться с такого рода ошибками.Я продолжаю получать ошибку переполнения стека.

Я пробовал разные коды для него, ни один из них не работал, но в любом случае вот мой последний: 

public class Sudoku {

    private int[][] values;
    private boolean [][] writable;
    private static final int ZERO = 0;
    private static final int SIZE = 9;

    //just a normal constructor that sets which values are changeable and which aren't. only values equal to zero are changeable.
    public Sudoku(int[][] values) {
        this.values = new int[SIZE][SIZE];
        for(int row = 0; row< SIZE ; row++)
        {
            for(int col = 0; col< SIZE; col++)
            {
                this.values[row][col] = values[row][col];
            }
        }

         writable = new boolean[values.length][values[1].length];
         for(int i = 0;i < writable.length;i++)
         {
             for(int j = 0; j<writable[1].length;j++)
             {
                 if(values[i][j] == ZERO)
                 {
                     writable[i][j] = true;
                 }
             }
         }
    }

    public void setValues(int row,int col ,int value) //changes the value if the value was changeable.
    {
        if(writable[row][col])
        {
            values[row][col]= value;
        }

    }

    public int getValue(int row,int col) {
        return values[row][col];
    }

    public boolean isWritable(int row,int col)
    {
        return writable[row][col];
    }
    private boolean ConflictAtRow(int row , int num)
    {
        for(int i = 0;i < SIZE;i++)
            if(getValue(row,i) == num)
                return true;
            return false;
    }
    private boolean ConflictAtCol(int col, int num)
    {
        for(int i = 0;i<SIZE;i++)
            if(getValue(i,col) == num)
                return true;
            return false;
    }
    private boolean ConflictAtBox(int row, int col, int num)
    {
        int r = row - row %3;
        int c = col - col %3;

        for(int i = r;i<r+3;i++)
        {
            for(int j = c;j<c+3;j++)
            {
                if(getValue(i, j) == num && row != i && col != j)
                    return true;
            }
        }
        return false;
    }
    private boolean ConflictAt(int row, int col, int num)
    {
        return ConflictAtBox(row, col, num) && ConflictAtCol(col,num) && ConflictAtRow(row, num);  //line 108
    }

    public boolean solve(int row,int col) 
    {
        int nextRow = (col < 8) ? row:row+1;
        int nextCol = (col +1)%9;
        for (row = nextRow; row < SIZE; row++) {
            for (col = NextCol; col < SIZE; col++) {
                if(isWritable(row,col))
                {
                    for (int num = 1; num <= 9; num++) {
                        if(!ConflictAt(row,col,num)) //line 118
                        {
                            setValues(row,col,num);

                         if(solve(nextRow,nextCol)) //line 122
                             return true;
                         }
                             setValues(row,col,ZERO);

                    }
                }return !ConflictAt(row,col,getValue(row,col)) && 
                  solve(nextRow,nextCol);;

            }

        }return true;
    }

, и когда я запускаю решение ()метод я получаю ошибку переполнения стека 

Exception in thread "main" java.lang.StackOverflowError
    at Sudoku.Sudoku.ConflictAt(Sudoku.java:108) 
    at Sudoku.Sudoku.solve(Sudoku.java:118) 
    at Sudoku.Sudoku.solve(Sudoku.java:122) 
    at Sudoku.Sudoku.solve(Sudoku.java:122) 
    at Sudoku.Sudoku.solve(Sudoku.java:122) 
    at Sudoku.Sudoku.solve(Sudoku.java:122)

и так далее ......

1 Ответ

0 голосов
/ 08 февраля 2019

Как только элемент управления впервые вводит метод solve() и если все условия if оцениваются как true до строки 122, вы снова вызываете метод solve().

Проблема в том, что каждый раз, когда элемент управления запускает этот метод, он как будто выполняет его впервые.Поскольку условия не меняются (циклы for всегда начинаются с 0).

Это означает, что метод solve() неоднократно вызывается до тех пор, пока стеку не хватит памяти.

...