0

我被分配修改一个 8 Queens 程序以使用一维数组并使用蛮力（已经进行了回溯）。我想出了以下代码：

#include <cmath>
#include <iostream>
using namespace std;

bool ok(int board[8]){

for(int j = 0; j <= 7; j++){ //check for repeating digits
    cout << "ok loop 1"<<endl;
    for (int k = 0; k <= 7; k++)
    {
        cout << "ok loop 2"<<endl;
        if (board[k] = board[j]){ return false; }
    }
}

for(int c = 7; c >= 0; c--){ //check if position is safe
    cout << "ok loop 3"<<endl;
    //int r = 0;

    for(int i = 1; i <= c; i++){
    cout << "ok loop 4"<<endl;
        if(board[c-i] == c)
            return false;
        else if ((board[c]-i)>0 && board[c-i]-i == 1)

            return false;
        else if ((board[c]+i)<=7 && board[c-i]+i == 1)
            return false;
    } // for loop

} // for loop
    return true;
} // ok




void print(int board[8], int c){
cout << "Solution " << c << ": " << endl;
for(int i = 0; i < 8; i++){
{
    cout << board[i] <<" ";
} 
}

cout << endl;
} 




int main ()
{

int b[8]={0}; //initialize the array
int count = 0;

for(b[0]=0; b[0]<8; b[0]++)
for(b[1]=0; b[1]<8; b[1]++)
    for(b[2]=0; b[2]<8; b[2]++)
        for(b[3]=0 ; b[3]<8; b[3]++)
            for(b[4]=0; b[4]<8; b[4]++)
                for(b[5]=0; b[5]<8; b[5]++)
                    for(b[6]=0; b[6]<8; b[6]++)
                        for(b[7]=0; b[7]<8; b[7]++)
                            if(ok(b)) 
                            {
                                count++;
                                print(b, count);
                            }
system("PAUSE");
return 0;
}

它一直在循环，我不知道为什么。有人愿意帮助我吗？

4

1 回答 1

1

有几点可以改进：

  • 如果您传递了对一个由八个字符组成的常量数组的引用，ok()而不仅仅是一个指向非常量整数的指针，那么编译器可能会告诉您其中一个问题。
  • 女王可以有多少个不同的位置？我会说 64，尽管您的代码建议 8。我将从在整个代码中记录变量和常量的实际含义开始，因为您自己似乎对此感到困惑。
  • 您检查 board[x] 是否为 board[y]，但 x 和 y 相等，然后您声称存在重复数字。
  • 你在不同的皇后之间有所作为。换句话说，您的程序将找到皇后如何定位在相同的八个位置上的所有排列。这不是错误的，而是低效的。如果你固定位置的数量，那将产生明显的差异。
于 2013-03-02T08:22:45.590 回答