Question

有关我的电脑的基本信息： Windows 7 x64操作系统
8GB RAM 使用MingW编译器

因此，每次我编译后通过终端运行我的程序，它立即停止响应。我似乎无法自己找到问题，所以我要求额外的一双眼睛为我指出。我现在几乎是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;
}

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

Answer 1

除以零？ secret.size（）最初为0。

secret.push_back(pallete.at(rand()%secret.size()));

Answer 2

你除以零。

在您的第secret.push_back(pallete.at(rand()%secret.size()));行中，secret.size（）为零。根据{{3}}，Mod 0是“Undefined，并且可能会抛出'除以零'异常。”

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

Answer 3

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

C ++程序在执行时停止响应

3 个答案: