C++ if ... then语句

Question

所以我试图做一个简单的应用程序，测验用户。它会提出一个问题，用户回答1,2,3或4.然后，应用程序将采用该答案，如果正确，则会在最后显示的total_score中添加+1。一切看起来都对我有用，但是当我运行它并且到达if (q1_valid == false)部分时，它跳过cout并运行goto，无论q1_valid是true还是false。

#include <iostream> 

using namespace std; 

int main() 
{ 
    int q1_answer; 
    int total_score; 
    bool q1_correct; 
    bool q1_valid; 

    Question_1: 
    cout << "Question 1 (#3 is correct)" << endl; 
    cout << "1.) Answer 1" <<endl; 
    cout << "2.) Answer 2" <<endl; 
    cout << "3.) Answer 3" <<endl; 
    cout << "4.) Answer 4" <<endl; 
    cin >> q1_answer; 

    if (q1_answer == 1) 
    q1_correct = false; 
    q1_valid = true; 

    if (q1_answer == 2) 
    q1_correct = false; 
    q1_valid = true; 

    if (q1_answer == 3) 
    q1_correct = true; 
    q1_valid = true; 

    if (q1_answer == 4) 
    q1_correct = false; 
    q1_valid = true; 

    if (q1_valid == false) 
    cout << "Invalid answer." <<endl; 
    goto Question_1; 

    if (q1_correct == true) 
    cout << "Correct!" <<endl; 
    (total_score + 1); 
    goto Question_2; 

    if (q1_correct == false) 
    cout << "Incorrect." <<endl; 
    goto Question_2; 

    if (q1_valid == false) 
    goto Question_1; 

    Question_2: 
    cout<< "Q2" <<endl; 
cin.ignore(); 
cin.ignore(); 
} 

Answer 1

我这里有几个技巧：

1. 如果...则是有条件的，不是循环。对不起，这只是我有点挑剔。 ;）
2. 永远不会使用goto。高级用法：只有在有充分的理由时才能使用goto。
3. 测试布尔值时，不需要“== true”或“== false”。
4. 看起来你还没有学会如何使用else语句。这将使您的程序更易于阅读，调试和理解。如上所述，括号也是必需的。

Answer 2

你不必绕语句括号中的如果，所以才有了第一条语句是有条件后。在这种情况下，这意味着“q1_valid = true;”无论答案是什么，它都会运行。

if (q1_answer == 4) { 
q1_correct = false; 
q1_valid = true; 
} 

您需要一个编辑器，通过缩进显示您。

Emacs会，例如

Answer 3

您需要使用括号：

if (q1_valid == false) { 
    cout << "Invalid answer." <<endl; 
    goto Question_1; 
} 

如果不使用斗拱，if只执行第一条语句直接跟随它，如果if条件评估为真。

Answer 4

不要忘记使用括号。

Answer 5

if语句的括号是必需的。

你有没有考虑使用switch语句：

switch (q1_answer){ 
    case 1: 
    q1_correct = false; 
    q1_valid = true; 
    break; 
    case 2: 
    q1_correct = false; 
    q1_valid = true; 
    break; 
    case n: 
    //... 
    break; 
}