私のifステートメントのデバッグ

Question

私は友人が体重を減らすのを助けることに決めました。私はコーディングを知っていて、あなたのBMI（体質指数）を計算するプログラムを開発しようとしています。 ifステートメント。彼らは私がそれらを期待する方法を働いていない。ここに私がこれまでに持っていることがあり、それを見て、それがなぜ機能していないのか理解できません。

#include <iostream> 
#include <cmath> 
using namespace std; 

int main() { 
    double weight; 
    double height; 
    double bmi; 
    cout << "Enter weight in kilograms:\n"; 
    cin >> weight; 
    cout << "Enter height in metres:\n"; 
    cin >> height; 

    bmi = weight/(pow(height,2)); 

    if (bmi < 18.5) { 
      cout <<"Your BMI is " << bmi <<", which means you are in the Underweight range" << endl; 
    } 
    else { 
    (bmi > 18.5) && (bmi < 24.9) 
      cout <<"Your BMI is " << bmi <<", which means you are in the Normal range" << endl; 
    } 

    else { 
     (bmi > 25.0) && (bmi < 29.9) 
      cout <<"Your BMI is " << bmi <<", which means you are in the Normal range" << endl; 
    } 

    else (bmi > 30) { 
      cout <<"Your BMI is " << bmi <<", which means you are in the Obese range" << endl; 
    } 
    return 0; 
} 

私は他に何度も使用できると言われましたが、それがエラーの原因かもしれないと私は考えています。

ありがとうございます。

Answer 1

正しい構文は次のとおりです。

if (bmi < 18.5) { 
     cout <<"Your BMI is " << bmi <<", which means you are in the Underweight range" << endl; 
} 
// else bmi >= 18.5, no need to test it again 
else if(bmi < 24.9) { 
     cout <<"Your BMI is " << bmi <<", which means you are in the Normal range" << endl; 
} 
else if(...) { 
    // ... 
} 
else { 
    // ... 
} 

Answer 2

あなたのif-else構文が間違っています。それは、（条件） - （もし他の条件） - そうすれば、このようになります：

bmi = weight/(pow(height,2)); 

if (bmi < 18.5) { 
     cout <<"Your BMI is " << bmi <<", which means you are in the Underweight range" << endl; 
} 
else if ((bmi > 18.5) && (bmi < 24.9)) 
     cout <<"Your BMI is " << bmi <<", which means you are in the Normal range" << endl; 
} 

else if ((bmi > 25.0) && (bmi < 29.9)) 
     cout <<"Your BMI is " << bmi <<", which means you are in the Normal range" << endl; 
} 

else /*(bmi > 30)*/ { 
     cout <<"Your BMI is " << bmi <<", which means you are in the Obese range" << endl; 
} 
return 0;