Question

我有一个系统，用户可以根据需要输入尽可能多的输入并进行一些计算。

以下是完成此任务的代码：

int main() {
    char op = 's';
    float time=0, fuel_rate=0, start=0, end=0, pace=0;


    while(op != 'x'){
        cout << "Please select: " << endl;
        cout << "1 ---> A" << endl;
        cout << "2 ---> B" << endl;
        cout << "3 ---> Calculate" << endl;
        cout << "x ---> Exit" << endl;

        op = std::getchar();

        //remove the rest of the line from input stream
        int temp;
        while ( (temp = std::getchar()) != '\n' && temp != EOF );

        switch(op){
        case '1':
            cout << "Enter time: ";
            cin >> time;
            cout << "Enter fuel rate: ";
            cin >> fuel_rate;
            break;
        case '2':
            cout << "Enter start: ";
            cin >> start;
            cout << "Enter end: ";
            cin >> end;
            cout << "Enter pace: ";
            cin >> pace;
            cout << "Enter fuel rate: ";
            cin >> fuel_rate;
            break;
        case '3':
            cout << "Total value";
            break;
        case 'x':
            return 0;
        default:
            continue;
        }
    }
    return 0;
}

系统适用于第一个输入。示例控制台日志如下所示：

 Please select: 
1 ---> A
2 ---> B
3 ---> Calculate
x ---> Exit
1
Enter time: 2
Enter fuel rate: 3
Please select: 
1 ---> A
2 ---> B
3 ---> Calculate
x ---> Exit
2
Please select: 
1 ---> A
2 ---> B
3 ---> Calculate
x ---> Exit

第一个用户进入操作1，系统询问时间和燃油率。当用户进入操作2时，系统不询问开始，结束或步速。

关于如何解决这个问题的任何想法？

Answer 1

我相当肯定std::getchar()是导致大多数问题的原因。如果我更改您的代码使用：

cin >> op;
switch (op) { 
//...

而不是

op = std::getchar();

//remove the rest of the line from input stream
int temp;
while ( (temp = std::getchar()) != '\n' && temp != EOF );

switch(op){
//...

程序运行得很好。

Answer 2

您混合使用std::cin和stdin。你应该坚持使用其中一个。而不是

    op = std::getchar();

使用

    op = cin.get();

你应该移动线条：

    int temp;
    while ( (temp = std::getchar()) != '\n' && temp != EOF );

在switch块结束后，确保您使用temp = cin.get()

由于您使用operator>>()来读取数据，因此未使用的换行符会保留在输入流中，例如：

    cin >> fuel_rate;

Answer 3

Adding debugging print code到交换机的默认情况下会清楚地显示正在发生的事情：

// ...
default:
  cout << "unexpected: " << int(op) << endl;
  continue;
// ...

意外：10

十进制10是使用\n上的operator>>格式化输入后仍在输入缓冲区中的换行符std::cin。

为了纠正这个问题，您可以ignore剩余字符（想想当用户在您请求时输入数字时会发生什么情况）从流到包括下一个换行符在内格式化输入：

cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');

或使用格式化输入（自行跳过）：

cin >> op;

虽然您还需要处理文件结束条件，当前代码无法执行此操作，如上面的示例运行所示。

使用Switch C ++从用户那里获得输入

3 个答案: