肥満度の小さい人 for文

Question

肥満度を知るのに BMI (Body Mass Index) がよく使われる。BMI は、 体重 w Kg, 身長 h m に対してつぎの式で定義される値をいう。

w / h^2

何人かの人について、名前・身長・体重が与えられているとき、 その中で一番 BMI の小さい人を見つけるプログラムを作れ。

入力は、何行かにわたる。各行には、名前（ローマ字姓）、身長（単位 cm）、体重（単位 Kg）が この順に1個以上の空白で区切られて書かれている。標準入力がキーボードから行われるときは、入力の最後に C-d を入力する。出力は、 BMI が最も小さかった人の名前を１行として書き出す。

が問題です。下のソースコードを実行した結果答えが一致しませんでした。数人いる中、どの場合でも一番上に来ている人が答えとなっているように感じます。多分ですが、for文の中身が上手くできていないと思うのですがわかりません。
どなたかお答えいただけると嬉しいです。

入力を以下のようにすると
kawamoto 170 65
shimano 167 60
okano 180 90
出力結果がkawamotoとなりました。本来ならshimanoとなるのが正解です。

コード#include <stdio.h>

struct number{
    char name[1000];
    double h;
    double w;
    double x;
};

int main(void){
    struct number b[1000];
    int i;
    while(1){
        if(scanf("%s %d %d", b[i].name, &b[i].h, &b[i].w)==EOF){
            b[i].x=b[i].w/(b[i].h*b[i].h);
            break;
        }
        i++;
    }
    double min=100;
    int num;
    int j=0;
    for(j=0;j<i;j++){
        if(min>b[j].x){
            min=b[j].x;
            num=j;
        }
    }
    printf("%s\n", b[num].name);
    return 0;
}

Answer 1

BMI は、 体重 w Kg, 身長 h m

入力は、何行かにわたる。各行には、名前（ローマ字姓）、身長（単位 cm）、

身長のセンチメートルからメートルへの変換はどこでやっているのですか？

質問には、「うまくいかない」ではなく、「入力をこれこれにしたら、出力がこれになった。
実際はこうなってほしい」のように具体的に書きましょう。

Comments

[YuhiKUROIWA]

単位は共通なので変換がいらないと思い行っていません。
実行結果乗せました。よろしくおねがいします。

[YuhiKUROIWA]

解決しました。ご指摘ありがとうございます。

Answer 2

scanf("%s %d %d", b[i].name, &b[i].h, &b[i].w)
doubleを読み取るための書式指定は%lfです。

numとiが初期化されていません。

scanfの戻り値がEOFになった時しかBMI計算していません。
つまり、入力された身長体重に対してはBMI計算せずに不定値が入っています。

        if (scanf("%s %lf %lf", b[i].name, &b[i].h, &b[i].w) == EOF) {
            b[i].x = b[i].w / (b[i].h * b[i].h);
            break;
        }

Comments

[YuhiKUROIWA]

ありがとうございます。

[YuhiKUROIWA]

変更して実行してみたのですがうまくいきません。どうしたらよいですか。

[SHOMI]

うまくいきませんとだけ書かれてもわかりません。
どのように書き換えて、どううまくいかないのですか？

[YuhiKUROIWA]

scanf分をlfにして、numとiを0で初期化しました。

[SHOMI]

>scanf分をlfにして、numとiを0で初期化しました。
質問文に変更後のコードを追記してください。
「どううまくいかない」のかも書いてください。

[YuhiKUROIWA]

実行結果が最初と変わらず表示されてしまいます。

[YuhiKUROIWA]

出力結果は質問のところに追記しました。

[SHOMI]

コードも貼ってください

[SHOMI]

>scanf分をlfにして、numとiを0で初期化しました
コードの指摘はそれだけではありませんが…

[YuhiKUROIWA]

#include <stdio.h>

struct number{
char name[1000];
double h;
double w;
double x;
};

int main(void){
struct number b[1000];
int i=0;
while(1){
if(scanf("%s %lf %lf", b[i].name, &b[i].h, &b[i].w)==EOF){
b[i].x=b[i].w/(b[i].h*b[i].h);
break;
}
i++;
}
double min=100;
int num=0;
int j=0;
for(j=0;j<i;j++){
if(min>b[j].x){
min=b[j].x;
num=j;
}
}
printf("%s\n", b[num].name);
return 0;
}

コード貼りました。よろしくお願いします。

[SHOMI]

他の方たちにも気づかれるようコメント欄ではなく、質問文に追記してください。
b[i].x=b[i].w/(b[i].h*b[i].h);の指摘が直っていません。
ifの外に出してください。

[YuhiKUROIWA]

すみません。よく読んでいませんでした。無事実行できました。ありがとうございました。