Question

我被赋予了修改8皇后程序以使用一维阵列和使用暴力（已经做回溯）的任务。我想出了以下代码：

#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;
}

它永远循环，我不知道为什么。有人会介意帮助我吗？

Answer 1

有一些事情可以改进：

如果您将对八个字符的常量数组的引用传递给ok()而不是仅指向非常量字符的指针，编译器可能已经告诉您其中一个问题。
女王有多少个不同的职位？我会说64，尽管你的代码暗示了8。我首先要记录整个代码中变量和常量的实际含义，因为你自己似乎很困惑。
你检查棋盘[x]是否为棋盘[y]，但x和y相等，并且你声称有重复的数字。
你在不同的皇后之间做出了贡献。换句话说，你的程序将找到皇后如何定位在相同的八个位置的所有排列。这不是错误的，但效率低下。如果您确定了头寸数量，那将会产生显着的差异。

如何用蛮力解决8皇后1D阵列？

1 个答案: