1

关于我的电脑的基本信息：Windows 7 x64 OS
8gb RAM
Using MingW compiler

因此，每次我在编译后通过终端运行程序时，它都会立即停止响应。我自己似乎无法找到问题所在，所以我要求多一双眼睛来为我指出问题。目前我实际上是 C++ 的初学者，所以请耐心等待。

    #include <iostream>
    #include <cstdlib>
    #include <string>
    #include <vector>
    #include <ctime>

    using namespace std;

int main()
{   
    bool correct = false;
    srand(time(NULL));  
    vector<string> guess;
    vector<string> pallete;
    vector<string> secret;
    pallete.push_back("red");
    pallete.push_back("green");
    pallete.push_back("blue");
    pallete.push_back("yellow");
    for(int i = 0; i < 4 ; i++)
    {
        secret.push_back(pallete.at(rand()%secret.size()));
    }
    for(int j = 0; j < 4 ; j++)
    {
        guess.push_back(pallete.at(rand()%guess.size()));
    }
    vector<string> secretMem = secret;
    cout << "the guess was:";
    for(int x = 0; x<4; x++)
    {
        cout << guess[x] << endl;
    }

    while(correct == false)
    {
        int blackP = 0; 
        int whiteP = 0;
        secret = secretMem;
        for (int idxB = 0; idxB < secret.size(); idxB++)
        {
            if (guess[idxB] == secret[idxB])
            {
                secret[idxB] = "0";
                guess[idxB] = "1";
                blackP++;
            }
        }   
        for (int idxW = 0; idxW < secret.size(); idxW++)
            for (int idyW = 0; idyW < guess.size(); idyW++)
            {
                if (secret[idxW] == guess[idyW])
                {
                secret[idxW] = "0";
                guess[idxW] = "1";
                whiteP++;
            }
        }
        if (blackP == 4)
        {
            cout << "Congratulations you win." << endl << "The secret code was:"<< endl;
            for(int y = 0; y<4; y++)
            {
                cout << secretMem[y] << endl;
            }
            correct = true;
        }
        else
            cout << "you scored: " << blackP << " Black pegs" << "and" << whiteP << " White pegs" << endl;
    }
    return 0;
}

另外....这种可怕的代码格式是怎么回事....我只是无法在预览中正确排列，就像我在编写它时一样....

4

3 回答 3

5

除以零？secret.size() 最初为 0。

secret.push_back(pallete.at(rand()%secret.size()));
于 2013-08-06T15:31:34.143 回答
1

你除以零。

在您的行secret.push_back(pallete.at(rand()%secret.size()));中，secret.size() 为零。根据this previous question's answer，Mod 0是“未定义，并且可能会抛出'除以零'异常。”

secret.push_back(pallete.at(rand() * pallete.size()-1 ))如果您尝试选择介于 0 和要猜测的最大潜在颜色数之间的数字，我可能会将该行更改为 。

于 2013-08-06T15:33:55.803 回答
0

我认为您的意思是两个位置的palette.size（secret.size，guess.size）。然后你有一个无限循环，因为 blackP 永远不会是 4，并且每次循环运行都会做同样的事情，因为 secretMem 永远不会更新。也许在 Windows 上它能够处理溢出（不知何故？）但陷入了那个循环。虽然在那种情况下，你会得到很多“你得分：0 Black pegsand1 White pegs”行。

于 2013-08-06T15:38:34.727 回答