Question

我正在学习C ++，我正在我正在使用的书中做一些练习。其中一个要求编写一个程序，询问用户他们想要添加多少个数字。然后提示输入用户想要添加的号码或输入＆＃39; |＆＃39;一旦完成。然后将数字推入矢量。程序的一部分要求检查向量的大小是否等于输入项的原始数量，这是我不断收到错误的地方。

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循环应检查|而不是始终运行true。然后它会检查向量的大小，如果它不匹配则清除它，并要求你再次执行它。这是我能想到的最佳方式。

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

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

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

2 个答案: