C言語 ファイルを読み込んで最大値をを持つ名前を出力したい

Question

という問題を解いているのですが、最後の個人の名前を出力という部分だけが分かりません。調べても、配列を使った最大値の求め方しか出てこなくて困っています。

#include <stdio.h>
int main(void){
  FILE *fp;
  char name[20], max_name[20];
  double height, weight, bmi, max = 0;

  fp = fopen("score.txt","r");

  if(fp == NULL)
    printf("ファイルをオープンできませんでした\n");
  else{
    while(fscanf(fp, "%s%lf%lf",name,&height,&weight) == 3){
       bmi = weight / ((height * 1/100)  * (height * 1/100));
       if(bmi > max)
          max = bmi;
          max_name = name;
    }
    printf("BMIが最も高いのは%sさんです\n",max_name);
  }
fclose(fp);
return 0;
}

Answer 1

max_name = name;
ではなく
strcpy(max_name, name);
せよ。

※ #include <string.h> をお忘れなく

Comments

[dodox86]

もうひとつ、if(bmi > max)の{}カッコが無いために、max_name = name; 相当の処理が常に実行されてしまいます。

[keeen]

できました！ありがとうございます！

[kazuma-s]

さらにもうひとつ。fclose(fp); は else の中に入れましょう。fclose(NULL); はダメです。

[episteme]

おまけ。できました！はいいとして、なぜ max_name = name ではダメか / なぜ strcpy(max_name, name) なのか考えよ。

[keeen]

char文字列を比較するとポインタの比較になるからでしょうか？

[episteme]

え？　どこでchar文字列を比較してます？

[keeen]

char配列は宣言時以外の=による代入ができないからstrcpyを使う必要がある、ということですか？

[episteme]

charであろうがなかろうが、配列のナカミを = で丸ごとコピーできない。
なので(このケースでは)文字列専用コピー関数:strccpyでコピーします。