【C】動的配列と構造体の連携がうまくできません

問題

「3人分の、名前、年齢、性別、を入力して表示するプログラムを作りなさい。ただし、データは構造体で記憶することとし、また、データの入力と表示はそれぞれ専用の関数を作って行うこととする。」という問題を元に、何人分でも入力できるように改造しなさい。なお、年齢に-1が入力されれば入力終了とする。

発生している問題・エラーメッセージ

エラー自体は発生していないですがうまくwhile文が機能しないでループを抜け出せません。

該当のソースコード

C
1#include <stdio.h>
2#include <string.h>
3#include <stdlib.h>
4
5typedef struct
6{
7  char name[64], sex[10];
8  int age;
9} profile;
10
11void input_profile(profile data);
12void output_profile(profile data);
13
14int main(void)
15{
16  int i, count = 0, datasize = 10;
17  profile* data = malloc(sizeof(profile) * datasize);
18
19  while (1)
20  {
21    input_profile(data[count]);
22    if (data[count].age == -1) break;
23
24    count++;
25
26    if (count >= datasize)
27    {
28      datasize += 10;
29      data = (profile *)realloc(data, sizeof(profile) * datasize);
30    }
31  }
32
33  for (i = 0; i < count; i++)
34  {
35    output_profile(data[i]);
36  }
37
38  free(data);
39  return 0;
40  }
41
42void input_profile(profile data)
43{
44  printf("名前:");
45  scanf("%s", data.name);
46  printf("年齢:");
47  scanf("%d", &data.age);
48  printf("性別:");
49  scanf("%s", data.sex);
50}
51
52void output_profile(profile data)
53{
54  printf("%s\n", data.name);
55  printf("%d\n", data.age);
56  printf("%s\n", data.sex);
57}
58

###実行結果
ざっとこんな感じで永遠と入力が促されます

名前:as
年齢:20
性別:male
名前:tanaka
年齢:90
性別:female
名前:o
年齢:41
性別:male
名前:

###備考
一度正常に機能するかループの部分を一旦消してプログラムを実行してみなのですがoutput_profile関数がうまく機能しませんでした。output_profileの引数に問題があるのではないかと考えていますがどう間違っているかまだわかっておりません。さらに、if (data[count].age == -1)break;と記述してるのにbreakが機能しない理由もわかりせん。

y_waiwai

2021/12/17 12:04

うまく行かない、機能しない、って言ってるだけではどうにもなりません。どういうふうに動いているのか、それのどこがまずいのか、を詳しく説明しよう

退会済みユーザー

2021/12/17 12:11

アドバイスありがとうございます。実行結果を追加してみました。

行動規範の内容に同意します

回答3件

ひんと:

C
1#include <stdio.h>
2void set0(int x) { x = 0; }
3
4int main() {
5  int a = 123;
6  set0(a);
7  printf("%d\n", a); // ここで 0 が出力...されません。
8  return 0;
9}

input_profileで↑と同じことやってます。

投稿2021/12/17 14:41

episteme

総合スコア16612

ベストアンサー

diff
1-void input_profile(profile data);
2+void input_profile(profile *data);
3
4-    input_profile(data[count]);
5+    input_profile(&data[count]);
6 
7-void input_profile(profile data)
8+void input_profile(profile *data)
9
10-  scanf("%s", data.name);
11+  scanf("%s", data->name);
12
13-  scanf("%d", &data.age);
14+  scanf("%d", &data->age);
15
16-  scanf("%s", data.sex);
17+  scanf("%s", data->sex);

なぜ、この修正が必要なのか説明できますか？

投稿2021/12/17 13:34

kazuma-s

総合スコア8224

退会済みユーザー

2021/12/18 07:20

ありがとうございます。一回持ち帰って検討しました。勘違いしていたこととしてvoid input_profileのdataは配列だからdata[] or *dataと宣言しなくてもアドレスが渡されるのではないかということでした。しかしあくまでもアドレスを渡せるのはdata[] or *data or data[~]であったことに気づきました。 scanfのところは構造体のポインタ変数を通常変数モードに切り替える必要がありその記述方法がアロー演算子ということですね。ようやく自分の中で噛み砕くことができました。ありがとうございます。