C言語 malloc関数について

Question

C言語のmalloc関数の問題を解いています。
構造体の配列をmalloc関数で確保して、足らなくなったらrealloc関数で追加するということをしたいのですが
この確保した配列の要素数を変数に代入したら0になってしまっているのはなぜなのでしょうか？

コード
#include<stdio.h>
#include<stdlib.h>
#include<string.h>

typedef struct {

	char name[64];
	int age;
	int sex;

}people;

void inputpeople(people[]);
void showpeople(people);

int main(void) {

	people *data;
	int i,size;

	data = (people*)malloc(sizeof(people) * 10);
	if (data == NULL) exit(0);
	
	size = sizeof(data) / sizeof(data[0]);/*ここのsizeに0が入ってしまう*/

	printf("情報を入力して下さい。(入力終了なら年齢に-1)\n");

	for (i = 0; i < size; i++) {

		inputpeople(&data[i]);
		
		if (data[i].age == -1) {
			break;
		}

		if (i == size-1) {
			data = (people*)realloc(data,sizeof(people) * 10);
			size = sizeof(data) / sizeof(data[0]);
		}
	}

	for (i = 0; i < size; i++) {
		showpeople(data[i]);
	}

	free(data);

	return 0;


}

void inputpeople(people *data) {

	printf("名前を入力:");
	scanf("%s", data->name);
	printf("年齢を入力:");;
	scanf("%d", &data->age);
	printf("性別を入力(男なら1、女なら2):");
	scanf("%d", &data->sex);

}

void showpeople(people data) {

	char sex[64];

	printf("名前:%s\n", data.name);
	printf("年齢:%d歳\n", data.age);
	
	if (data.age == 1) {
		strcpy(sex, "男\n");
	}
	else if (data.age == 2) {
		strcpy(sex, "女\n");
	}
	else {
		strcpy(sex, "入力が間違っています。\n");
	}
	printf("性別:%s", sex);
}

Answer 1

sizeof(data) は people *data なのですから ポインタのサイズ(4 とか 8 でしょうか)が返ります。
sizeof(data[0]) は people のサイズ(72 とか 80 とか) ですので、整数同士の割り算で切り捨てられて 0 となります。
おかしいと思われたなら、まず size(data) や size(data[0]) を printf して確認するのが良いと思います。

---

ご質問のコードでは、 size 変数の使い方が逆です。
malloc/realloc した結果から size を求めるのではなく、 size の値から malloc/realloc する大きさを求めるべきです。
また、きちんとするなら、 realloc が返すアドレスを直接 data に入れてはいけません。
realloc は指定したサイズのメモリが確保出来たら元の領域を解放して新しい領域を返しますが、確保出来なかった場合は元の領域は解放せず NULL を返します。(ちなみに指定したサイズ分増やすわけではありません。また、サイズが同じ/縮小の場合は元の領域を流用して元のアドレスを返す場合もあります。)
返した値を data に入れてしまうと、 NULL だった場合に元の領域を解放することが出来ません。
→ 【C言語】realloc関数｜正しい使い方と注意点 メモリ断片化など
さらに、確保したメモリのサイズは入力したデータの数と一致しませんので、表示に size 分ループすると、 20 個分のメモリを確保しても 12 件しか入力しなかった場合、 13 件目以降は無駄にデタラメなデータが表示されてしまいます。

int main(void) {
  int size = 10; //初期サイズ
  int last = 0;

  people *data = malloc(sizeof(people) * size);
  if(data == NULL) exit(0);

  printf("情報を入力して下さい。(入力終了なら年齢に-1)\n");

  while(1) {
    inputpeople(&data[last]);
    if(data[last].age == -1) {
      break;
    }

    if(++last == size) {
      size += 10;
      people *p = realloc(data, sizeof(people) * size);
      if(p == NULL) { //realloc エラー
        free(data);
        exit(0);
      }
      data = p;
    }
  }

  for(int i = 0; i < last; i++) {
    showpeople(data[i]);
  }

  free(data);

  return 0;
}

Comments

[yusuke116462]

rellocは指定した分元の配列から増やすと思っていました。分かりやすい説明ありがとうございます。

Answer 2

sex
下ネタはよくないと思います。

Answer 3

size = sizeof(data) / sizeof(data[0]);

そもそもdataは配列ではないのでこれで配列の要素数にはなりません。
dataはポインタ変数ですのでsizeof(data)は4(32bit)とか8(64bit)などです。