1

我一周前刚开始编程，我的第一个任务是编写一个 BMI 计算器。

启动时应该是这样的：

    BMI Calculator
    Your weight in kg: x 
    Your height in cm: x
    Gender (m/f): x

-> You are underweight/normal/overweight

到目前为止，这是我的代码：

            Console.WriteLine("BMI Calculator");
            Console.WriteLine("===========");
            Console.WriteLine();

            Console.Write("Weight in kg: ");
            int kg;
            kg = Convert.ToInt32(Console.ReadLine());

            Console.Write("Height in cm: ");
            int m;
            m = Convert.ToInt32(Console.ReadLine());

            Console.Write("Gender (m/f):");
            string Geschlecht = Console.ReadLine();

            int BMI;
            BMI = kg / (m / 100) * (m / 100);

            if (BMI < 19 & Gender == "f")
            { Console.WriteLine("-> Underweight"); }
            if (BMI >= 19 & BMI <= 24 & Gender == "f") 
            { Console.WriteLine("-> Normal"); }
            if (BMI > 24 & Geschlecht == "f")
            { Console.WriteLine("-> Overweight"); }

            if (BMI < 20 & Gender == "m")
            { Console.WriteLine("-> Underweight"); }
            if (BMI >= 20 & BMI <= 25 & Gender == "m")
            { Console.WriteLine("-> Normal"); }
            if (BMI > 25 & Gendert == "m")
            { Console.WriteLine("-> Overweight"); }

            Console.ReadLine();

我不确定我的代码有什么问题，但每当我输入 60 公斤、170 厘米和男性时，我就会超重，即使我应该恢复正常。实际上，10公斤以上的东西也是如此。

PS：我真的是编程的初学者，所以我为我对编程术语的掌握表示歉意。

为了您的方便：

http://i.stack.imgur.com/admqr.png

提前致谢！

4

5 回答 5

5

当你这样做时：

BMI = kg / (m / 100) * (m / 100);

并且m是int，您将进行整数除法，在这种情况下170 / 100 = 1. 正如 User1551892 指出的那样，您需要更具体地了解计算顺序。

尝试：

double BMI = kg / ( ( m / 100.0 ) * ( m / 100.0 ) );

这将迫使它进行浮点除法，并应该为您带来更好的结果。

此外，您可以使用Math.Pow以避免m / 100.0两次：

double BMI = kg / Math.Pow( m / 100.0, 2 );
于 2013-10-20T15:15:50.390 回答
2

仅供参考，新堆栈的交换门户已打开：codereview.stackexchange.com

应该是更好的地方要求审查

于 2013-10-20T15:32:03.263 回答
2

请检查这一行：

BMI = kg / (m / 100) * (m / 100);

它应该是这样的：

BMI = kg / ((m / 100) * (m / 100)) ;
于 2013-10-20T15:14:43.687 回答
1

虽然 BMI 的计算是关闭的。它是（体重/身高）/身高所以 int bmi = (kg/(m/100))/(m/100)

于 2020-01-12T22:36:43.780 回答
0

在每个 IF 语句中检查您的性别变量...

于 2020-10-23T05:58:26.830 回答