需要一些帮助在我的while循环中发现逻辑错误

Question

我正在学习C ++，并且正在使用的书中做一些练习。 其中一个要求编写一个程序，询问用户要累加多少个数字。 然后提示用户要添加的数字或输入“ |” 一旦完成。 然后将数字压入向量。 该程序的一部分要求检查向量的大小是否等于输入项的原始数量，而这就是我不断收到错误的地方。

cout << "Please enter the numbers and | once you are done: ";

while(true)
{

    for(int num; cin >> num; )
    {
        if(num == '|')
        {
            break;
        }

        ints.push_back(num);
    }

    if(ints.size() != n)
    {
        cout << "There are more or less numbers in the vector than originally specified\n"
            << "Vector will be cleared; please re-enter the values: ";
        ints.clear();
        continue;
    }
    else
    {
        break;
    }
}

问题是，如果输入数量关闭，则消息将陷入无限循环，并且我不确定如何解决。

编辑：n是保存在用户想要输入的值数中的变量。

谢谢！

Answer 1

num是一个整数，并且cin >> num不会提取| 符号。 比较编号num == '|' 可能无法正常工作，因为num的数值可能是| ASCII符号，即使用户未输入任何符号| 符号。 您应该正确处理结束标记读取：

// loop will break when user enters `|` because it is not an integer
// setting failbit of cin
for(int num; cin >> num;)
{
    ints.push_back(num);
}
cin.clear(); // reset failbit making cin able to read again
// check the end marker entered by user
{
    string end_marker;
    cin >> end_marker;
    if("|" != end_marker)
    {
        // use of endl will flush the stream ensuring that
        // printed text won't stuck in the buffer
        cout << "Please use | as end marker" << endl;
        continue;
    }
}

Answer 2

这是我的实现方法。 我担心您的while循环中的逻辑。 我被教导要尽可能地避免while(true) 。 您知道代码应该如何工作的逻辑。 通过更多的练习，您将开始认识到需要使用的条件。 我相信有更好的方法可以做到这一点。 但这是我尝试的方式。

但是要回答您的问题，它失败的主要原因是因为整数无法将自己与字符进行比较。

if(num == '|') 

这是行不通的，因为num是整数而不是字符。

通常，我会在一个类中实现它，并且由于对全局变量的关注度不高，所以我创建了自己的命名空间。 但是，您必须自己完成其余的逻辑：

#include <iostream>
#include <vector>
#include <string>

namespace global
{
    std::vector<std::string> strings;
    std::vector<int> ints;
    std::string a = " ";
    int num = 0;
}   

void doWork()
{   

    std::cout << "Please enter the number of integers you would like to add up: ";
    std::cin >> global::num;

    std::cout << "Please enter the numbers and | once you are done: ";
    while (global::a != "|")
    {
        std::cin >> global::a;
        global::strings.push_back(global::a);
    }

    global::strings.pop_back();

    for(auto &e : global::strings)
    {
         global::ints.push_back(std::stoi(e));
    }
}   


int main()
{

    doWork();

    if(global::ints.size() != global::num)
    {
        std::cout << "Size of vector does not match the size specified. Clearing vector" << std::endl;
        global::ints.clear(); 
        global::strings.clear();
        global::num = 0;
        global::a = " ";
        doWork();
    }   

}

我制作了一个char的向量并将其转换为整数，以便可以将它们相加。 while循环应检查| 而不是始终如一。 然后它将最后检查向量的大小，如果不匹配则将其清除，然后要求您再次进行。 这是我想到的最好方法。

编辑：正如VTT所指出的， char一次只能做一个字符。 我已将其转换为字符串以处理转换。

编辑2：在故障结束时将global::num和global::a的值重置为其默认值，以防止崩溃。