Я написал игру судоку, включающую решатель на C, и хотел попробовать ее на Java, чтобы люди могли использовать ее немного проще (переносимость). Я полагал, что порт будет довольно простым из-за огромного сходства между языками, но, похоже, это немного затруднительно.
Мой решатель бесконечно рекурсирует, чего никогда не было в C. Вот моя исходная функция C для решения головоломки:
int sudoku_solve(struct sudoku* sudoku)
{
if(!sudoku) return 0;
int mask = 0x1ff;
int best_x = 0, best_y = 0;
int best_mask = 0x2ff;
for(int y = 0; y < 9; ++y){
for(int x = 0; x < 9; ++x){
if( sudoku->grid[y][x] != 0 ) continue;
mask = sudoku_get_mask(sudoku, x, y);
if( mask < best_mask ){
best_mask = mask;
best_x = x;
best_y = y;
}
}
}
if( best_mask == 0x2ff ) return 1; // this puzzle is already solved!
if( best_mask == 0x000 ) return 0; // this puzzle can't be solved!
int start_c = rand() % 9;
int c = start_c;
do{
if( (best_mask & (1<<c)) ){
sudoku->grid[best_y][best_x] = c+1;
if( sudoku_solve(sudoku) ) return 1;
}
c = (c+1) % 9;
} while( c != start_c );
sudoku->grid[best_y][best_x] = 0;
return 0;
}
Я знаю, что это не обязательно самый быстрый или лучший написанный решатель, но он работал. Он просто находит плитку с наименьшими возможными значениями, а затем начинает со случайных значений и пробует все возможные значения, пока не получится решаемая головоломка (с использованием рекурсии). sudoku_get_mask возвращает целое число с первыми 9 битами, установленными для соответствующего значения. Он проверяет горизонтальные, вертикальные и подквадраты на наличие значений, которые уже используются, и удаляет их из маски.
Теперь вот порт Java:
public int Solve()
{
int mask = 0x2FF;
int bmask = 0x2FF, bx = 0, by = 0;
for(int y = 0; y < 9; ++y){
for(int x = 0; x < 9; ++x){
if( grid[y][x] != 0 ) continue; // ignore spaces with values already set
mask = GetMask(x, y);
if( mask < bmask ) // less bits set == less possible choices
{
bmask = mask;
bx = x;
by = y;
}
}
}
if( bmask == 0x2FF ) // the puzzle had no good slots, it must be solved
return 1;
if( bmask == 0 ) // the puzzle is unsolvable
return -1;
int start_c = rand() % 9;
int c = start_c;
do{
if( (bmask & (1<<c)) != 0 ){
grid[by][bx] = (char) (c+1);
if( Solve() == 1 ) return 1;
}
c = (c+1)%9;
}while( c != start_c );
grid[by][bx] = 0; // restore old value
return 0;
}
Они почти идентичны, поэтому я не могу понять, почему порт Java бесконечно повторяется! Решатель всегда должен либо 1. найти решение, либо 2. обнаружить, что решения нет. По моей логике я не вижу, как это должно повторяться бесконечно.
Вот Java-код GetMask:
protected int GetMask(int x, int y)
{
int mask = 0x1FF;
for(int cx = 0; cx < 9; ++cx){
mask &= (grid[y][cx] == 0 ? mask : ~(1 << (grid[y][cx]-1)));
}
for(int cy = 0; cy < 9; ++cy){
mask &= (grid[cy][x] == 0 ? mask : ~(1 << (grid[cy][x]-1)));
}
int idx = squareIndex[y][x];
int[] pt = null;
for(int c = 0; c < 9; ++c){
pt = squarePoint[idx][c];
mask &= (grid[pt[1]][pt[0]] == 0 ? mask : ~(1 << (grid[pt[1]][pt[0]]-1)));
}
return mask;
}
Вот SquareIndex и SquarePoint (просто поисковые таблицы для подквадратов):
static int squareIndex[][] = {
{0,0,0,1,1,1,2,2,2},
{0,0,0,1,1,1,2,2,2},
{0,0,0,1,1,1,2,2,2},
{3,3,3,4,4,4,5,5,5},
{3,3,3,4,4,4,5,5,5},
{3,3,3,4,4,4,5,5,5},
{6,6,6,7,7,7,8,8,8},
{6,6,6,7,7,7,8,8,8},
{6,6,6,7,7,7,8,8,8}
};
static int[] squarePoint[][] = {
{ {0,0}, {1,0}, {2,0}, {0,1}, {1,1}, {2,1}, {0,2}, {1,2}, {2,2} },
{ {3,0}, {4,0}, {5,0}, {3,1}, {4,1}, {5,1}, {3,2}, {4,2}, {5,2} },
{ {6,0}, {7,0}, {8,0}, {6,1}, {7,1}, {8,1}, {6,2}, {7,2}, {8,2} },
{ {0,3}, {1,3}, {2,3}, {0,4}, {1,4}, {2,4}, {0,5}, {1,5}, {2,5} },
{ {3,3}, {4,3}, {5,3}, {3,4}, {4,4}, {5,4}, {3,5}, {4,5}, {5,5} },
{ {6,3}, {7,3}, {8,3}, {6,4}, {7,4}, {8,4}, {6,5}, {7,5}, {8,5} },
{ {0,6}, {1,6}, {2,6}, {0,7}, {1,7}, {2,7}, {0,8}, {1,8}, {2,8} },
{ {3,6}, {4,6}, {5,6}, {3,7}, {4,7}, {5,7}, {3,8}, {4,8}, {5,8} },
{ {6,6}, {7,6}, {8,6}, {6,7}, {7,7}, {8,7}, {6,8}, {7,8}, {8,8} }
};
generator
? - person Mister Smith   schedule 07.02.2012Random.nextInt
возвращает значения в диапазоне [Integer.MIN_VALUE,Integer.MAX_VALUE], поэтому также создаются отрицательные случайные числа. Я предлагаю вам изменить его наRandom.nextInt(9)
и избавиться от%9
. - person Mister Smith   schedule 07.02.2012