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; 
}