Question

我正在做一个简单的程序，我接受电话号码的前四位数字，并且必须为其余数字生成所有可能性。最后6位必须加到33位。如果第四位是奇数，则第五位必须是偶数，如果第四位是偶数，则第五位必须是奇数。

我已经确定它没有将数字加在一起，而函数plotNumbers（）中的for循环实际上并没有循环。我完全迷失了。有人有什么想法吗？

#include "stdafx.h"
#include <iostream>
#include <vector>

using namespace std;

void plotNumbers(vector<int> number1, int digit1, int digit2, int digit3, int digit4);
bool findTrueFalse(vector<int> vec);

int _tmain(int argc, _TCHAR* argv[])
{
int digit1, digit2, digit3, digit4;
vector<int> number;

cout << "Enter the first four digits:\n";
cin >> digit1;
cin >> digit2;
cin >> digit3;
cin >> digit4;

if (digit4 == 0 || digit4 == 2 || digit4 == 4 || digit4 == 6 || digit4 == 8)
{
    number.push_back(1);        
    plotNumbers(number, digit1, digit2, digit3, digit4);
}
else if (digit4 == 1 || digit4 == 3 || digit4 == 5 || digit4 == 7 || digit4 == 9)
{
    number.push_back(0);
    plotNumbers(number, digit1, digit2, digit3, digit4);
}
else
{
    cout << "\nYou entered an illegal value.  Good-bye.\n\n";
}

system("pause");
return 0;
}

void plotNumbers(vector<int> number1, int digit1, int digit2, int digit3, int digit4)
{

for ( ; number1[0] < 10; number1[0] += 2)
{
    number1.push_back(0);


    for ( ; number1[1] < 10; ++number1[1])
    {
        number1.push_back(0);


        for ( ; number1[2] < 10; ++number1[2])
        {
            number1.push_back(0);


            for ( ; number1[3] < 10; ++number1[3])
            {
                number1.push_back(0);


                for ( ; number1[4] < 10; ++number1[4])
                {
                    number1.push_back(0);


                    for ( ; number1[5] < 10; ++number1[5])
                    {

                        bool total33 = findTrueFalse(number1);

                        if (total33)
                        {
                            cout << "(" << digit1 << digit2 << digit3 << ") ";
                            cout << digit4 << number1[0] << number1[1] << " - ";
                            cout << number1[2] << number1[3] << number1[4] << number1[5] << endl;
                        }
                    }
                }
            }
        }
    }
}
}

bool findTrueFalse(vector<int> vec)
{
int total = 0;
vector<int>::iterator postIt;

for (postIt = vec.begin(); postIt != vec.end(); ++postIt)
{
    total += *postIt;
}

if (total == 33)
{
    return true;
}
else
{
    return false;
}

}

Answer 1

首先，尝试使用短代码来避免错误，例如：

if (digit4 % 2 == 0)

而不是

if (digit4 == 0 || digit4 == 2 || digit4 == 4 || digit4 == 6 || digit4 == 8)

其次，你只需要做number1.pushback(0)一次，但你这样做了好几次。你知道当你运行number1.pushback(0)时会发生什么吗？我猜不是：

它会在您的向量0

number1

我建议使用以下代码（您进行缩进）：

#include "stdafx.h"
#include <iostream>
#include <vector>

using namespace std;

void plotNumbers(vector<int> number1, int digit1, int digit2, int digit3, int digit4);
bool findTrueFalse(vector<int> vec);

int _tmain(int argc, _TCHAR* argv[])
{
int digit1, digit2, digit3, digit4;
vector<int> number;

cout << "Enter the first four digits:\n";
cin >> digit1;
cin >> digit2;
cin >> digit3;
cin >> digit4;

if (digit4 % 2 == 0)
{
    number.push_back(1);        
    for (int i=0; i<5; i++)
        number.push_back(0);
    plotNumbers(number, digit1, digit2, digit3, digit4);
}
else if (digit4 % 2 == 1)
{
    number.push_back(0);
    for (int i=0; i<5; i++)
        number.push_back(0);
    plotNumbers(number, digit1, digit2, digit3, digit4);
}
else
{
    cout << "\nYou entered an illegal value.  Good-bye.\n\n";
}

system("pause");
return 0;
}

void plotNumbers(vector<int> number1, int digit1, int digit2, int digit3, int digit4)
{

for ( ; number1[0] < 10; number1[0] += 2)
{
    number1[1] = 0;


    for ( ; number1[1] < 10; ++number1[1])
    {
        number1[2] = 0;


        for ( ; number1[2] < 10; ++number1[2])
        {
            number1[3] = 0;


            for ( ; number1[3] < 10; ++number1[3])
            {
                number1[4] = 0;


                for ( ; number1[4] < 10; ++number1[4])
                {
                    number1[5] = 0;


                    for ( ; number1[5] < 10; ++number1[5])
                    {

                        bool total33 = findTrueFalse(number1);

                        if (total33)
                        {
                            cout << "(" << digit1 << digit2 << digit3 << ") ";
                            cout << digit4 << number1[0] << number1[1] << " - ";
                            cout << number1[2] << number1[3] << number1[4] << number1[5] << endl;
                        }
                    }
                }
            }
        }
    }
}
}

bool findTrueFalse(vector<int> vec)
{
int total = 0;
vector<int>::iterator postIt;

for (postIt = vec.begin(); postIt != vec.end(); ++postIt)
{
    total += *postIt;
}

if (total == 33)
{
    return true;
}
else
{
    return false;
}

}

Answer 2

你是如何确切地检查它们是否在循环？因为在我的计划中他们是，但我认为你不明白他们的实际工作方式（没有冒犯意味着）。

请考虑以下代码段：

int i = 0;
int j = 0;

for (; i < 10; i += 1) { //OUTER for
    cout << "i == " << i << endl;

    for (; i < 10; j += 1;) { //INNER for
        cout << "j == " << j << endl;
    }
//THIS POINT
}

此程序将进入OUTER循环，打印出“i == 0”，然后输入INNER循环打印10行“j == ...”。现在在这个点j == 10，所以当代码为'i'做另一个循环并检查INNER的条件时，它看到j == 10所以10不低于10，所以它跳过INNER for。

这正是您的计划中发生的事情。当您使用这样指定的控制变量退出INNER循环时，如果要在下一轮OUTER循环中输入INNER循环，则必须记住将其值更改为通过条件的值。

我已经改变了你的代码来做到这一点，看起来它工作正常。您需要做的就是在代码中的INNER循环之后添加4行代码，以使其控制变量归零。

编辑：另外，考虑一下Emabs的答案，因为你的矢量维护很差。

我没有任何编译错误，但我的代码不起作用。怎么了？

2 个答案: