エラーはでないが数値がおかしい（BMI値計算プログラム）

今以下のようなプログラムを製作中なのですがどうしてもうまくいきません；

(1)ユーザに身長と体重を入力してもらう
(2)(1)の値からＢＭＩ値を計算し、その結果から「太りすぎ」などの判定を表示し
(3)身長・体重から計算した標準体重と現在の体重との差を表示する

問題は以下です。
・身長と体重の数値を変えても結果(BMI値)は毎回同じで、
100万台の桁違いな数値がでる；（しかし「太りすぎ」などの判定は変化する）

どこがおかしいのかチェックするために(1)の部分のあとに改めて身長と体重を
表示させてみたところ、そこからすでに数値が変なことになっていたので、
おかしいとしたら(1)のscanfなどの部分か型宣言だと思うのですが…

回答よろしくお願いします；；

プログラム
------------------------------------------------
/* bmi.c */
#include <stdio.h>

main()
{
int sintyo, taizyu;
double sintyo2, bmi, hyozyun, sa;

//入力
printf("身長(cm)を入力してください\n");
scanf("%d", &sintyo);
printf("体重(kg)を入力してください\n");
scanf("%d", &taizyu);

//※チェック（身長・体重）
printf("%d\n", &sintyo);
printf("%d\n", &taizyu);

//計算
sintyo2 = sintyo / 100; //ｃｍ→ｍ
bmi= taizyu / (sintyo2 *sintyo2); //BMI値計算
hyozyun = (sintyo2 *sintyo2) * 22;//標準体重
sa = hyozyun - taizyu;

//※チェック（BMI値）
printf("%d\n", &bmi);

//出力
if(bmi>=25)
printf("BMI値：%f\nあなたは太りすぎです\n", &bmi);
else if(bmi>=23 && bmi<25)
printf("BMI値：%f\nあなたは太りぎみです\n", &bmi);
else if(bmi>=21 && bmi<23)
printf("BMI値：%f\nあなたは標準です\n", &bmi);
else if(bmi>=18.5 && bmi<21)
printf("BMI値：%f\nあなたは痩せぎみです\n", &bmi);
else
printf("BMI値：%f\nあなたは痩せすぎです\n", &bmi);

if(sa>=0)
printf("標準体重：%fkg\n標準体重を＋%fkgオーバーしています\n",&hyozyun, &sa);
else
printf("標準体重：%fkg\n標準体重より－%fkgです\n",&hyozyun, &sa);

return 0;
}

結果（身長160、体重50で入力）
------------------------------------------------
身長(cm)を入力してください
160
体重(kg）を入力してください
50

1310600　（←身長チェック　以下３つの値は不動です；）
1310596　（←体重チェック）

1310580　（←BMI値チェック）

BMI値：0.000000
あなたは太りすぎです
標準体重：0.000000kg
標準体重より－+NANkgです

No.3 ベストアンサー 回答者： asuncion 回答日時： 2008/07/26 11:26

ザッと見ただけですが…。

> //※チェック（身長・体重）
> printf("%d\n", &sintyo);
> printf("%d\n", &taizyu);

sintyoのアドレス値とtaizyuのアドレス値を出力しています。
& は、不要です。

> //計算
> sintyo2 = sintyo / 100; //ｃｍ→ｍ

int型のsintyoをint型の100で割った結果、小数点以下を切り捨ててsintyo2に代入してしまっています。
これでは、正確なBMI値を求められません。

身長と体重を入力する際、整数値でよいのですか？
身長172.5cmとか、体重62.4kgなんていう入力があってもよいのではないでしょうか。

この回答へのお礼

＞int型のsintyoをint型の100で割った結果、小数点以下を切り捨ててsintyo2に代入してしまっています。
なるほど！とても分かりやすい説明ありがとうございます。
たしかに身長や体重は小数点以下も入力できたほうがいいですね。アドバイスまでありがとうございます！

お礼日時：2008/07/26 12:01

No.5 回答者： asuncion 回答日時： 2008/07/26 11:32

ついでにもう一個。

> //※チェック（BMI値）
> printf("%d\n", &bmi);

& が不要なことは、前に書きました。
bmi は double 型なので、%d じゃなくて %f にしてください。

この回答へのお礼

細かいところまで気づいて下さってありがとうございます！

お礼日時：2008/07/26 12:06

No.4 回答者： asuncion 回答日時： 2008/07/26 11:29

あ～。

printf()の第2引数以降に、全部 & が付いてしまっているのですね。
scanf()との違いを今一度確認なさってください。

この回答へのお礼

ほんとですね；；
みなさんに何度も同じご指摘をさせてしまって申し訳ないです；
ご指摘ありがとうございました☆

お礼日時：2008/07/26 12:03

No.2 回答者： kagerou777 回答日時： 2008/07/26 11:24

printf("%d\n", &bmi);

ですが、これでは、変数bmiのアドレスを参照してしまうので
printf("%d\n", bmi);

かと思います。

この回答へのお礼

おなじくとっても初歩的なミスにご指摘ありがとうございます；
おかげさまで無事に解決することができました☆

お礼日時：2008/07/26 11:57

No.1 回答者： bluecampus 回答日時： 2008/07/26 11:23

scanfでは値を格納するのでポインタを指定しますが、

printfでは値を格納する必要がないため、ポインタではなく値そのものを指定します。

int n;
scanf("%d", &n); // 値を格納するから &n
printf("%d\n", n); // 値を格納する必要がないから n


ちなみに
printf("%d\n", &n);
で表示されるものは変数nのポインタの値になります。

この回答へのお礼

とっても基礎的なところでミスしてたんですね；
全然気づきませんでした；ありがとうございます！！
おかげ様で無事に解決することができました☆

お礼日時：2008/07/26 11:56