C言語 関数が文字列を含む構造体を返すとき

質問

生徒名、テストの点数
を要素とする構造体

struct test{char name[10];int score;};

を定義し、
点数の大きいほうを返す関数

struct test Top(struct test test1,struct test test2)
{
if(test1.score > test2.score)
return test1;
else
return test2;
}

を定義しました。（この状態では同点のとき後者を返してしまいますが）

main関数内で

struct test test1;
struct test test2;
strcpy(test1.name,"yamada");
test1.score=70;
strcpy(test2.name,"tanaka");
test2.score=90;

と宣言・代入したあと、Top関数が返した点数と名前を表示したいのですが、
名前の表示がうまくいってくれません。
文字配列の操作が誤っているのでしょうか？

実行したのは次の操作です。
printf("Top score = %d\n",Top(test1,test2).score);
printf("Top man = %c\n",Top(test1,test2).name[0]);
printf("Top man = %s\n",Top(test1,test2).name);

Top score = 90
Top man = t
Top man = tanaka
と表示されるのを期待していたのですが、
前二つの表示が終わるとダンプを起こしてしまいます

回答No.1

wormhole · ベストアンサー

＞文字配列の操作が誤っているのでしょうか？

文字配列の操作が間違ってるのではなくて
構造体の戻り値の扱い方が間違ってます。

＞printf("Top man = %s\n",Top(test1,test2).name);

printf()が処理しようとするときにはTop()の返す構造体の寿命は尽きてる。

お礼

ありがとうございます。
なぜできていないのかよく分かりました。

回答No.2

noname#208507

例えば最後の部分をこうします。

struct test test3 = Top(test1,test2);
printf("Top score = %d\n",test3.score);
printf("Top man = %c\n",test3.name[0]);
printf("Top man = %s\n",test3.name);

理由はANo.1さんの言われる通り。

関数Topは構造体を（確保されたメモリへの参照ではなく）値で返しています。値をどこかに代入しておかないと、破棄されてアクセスできなくなってしまいます。

お礼

解説ありがとうございます。
おかげさまで解決しました。