構造体のポインタ変数を引数に持つ関数について

Question

前提

C言語を学習している初学者です。
現在構造体の学習をしており、構造体のポインタ変数を引数に持つ関数を作成し、
デバックを行ったのですが、エラーが発生してしまい原因が分からず困っています。
原因がお分かりの方がいらっしゃいましたらご教授いただきたいです。
宜しくお願いします。

該当のソースコード

#include <stdio.h>
#include <string.h>

//	学生情報を入れる構造体
typedef struct {
	int id;			//	学生番号
	char name[256];	//	氏名
	int age;		//	年齢
} student_data;

//	学生情報を構造体に格納する関数
void setData(student_data*, int, char*, int);
//	学生情報を表示する関数
void showData(student_data*);

void main() {
	int i;
	student_data* data[4];
	int id[] = { 1, 2, 3, 4 };
	char name[][256] = { "山田", "佐藤", "太田", "中田" };
	int age[] = { 18, 19, 21, 18 };
	//	学生情報をセット
	for (i = 0; i < 4; i++) {
		setData(&data[i], id[i], name[i], age[i]);
	}
	//	学生情報を表示
	for (i = 0; i < 4; i++) {
		showData(&data[i]);
	}
}

void setData(student_data* data, int id, char* name, int age) {
	data->id = id;
	strcpy(data->name, name);
	data->age = age;
}

void showData(student_data*data) {
	printf("学生番号：%d　", data->id);
	printf("氏名：%s　", data->name);
	printf("年齢：%d\n", data->age);
}

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

デバックを実行すると、

例外がスローされました
Run-Time Check Failure #2 - Stack around the variable 'data' was corrupted.

と表示されます。

また、標準出力も以下のように学生番号１～３の氏名がおかしな表示になってしまいます。

学生番号：1　氏名：　年齢：18
学生番号：2　氏名：　年齢：19
学生番号：3　氏名：　年齢：21
学生番号：4　氏名：中田　年齢：18

補足情報（FW/ツールのバージョンなど）

開発環境はVisual studio 2019を使用しています。

Answer 1

student_data* data[4];

これはstudent_data構造体を指すポインタが4個の配列を定義しています。
しかし、その4つのポインタが指すべき、student_data構造体の「実体」は定義されていないようです。
各ポインタの値は初期化されておらず不定ですので、それらのポインタでデータを読み書きするということは、つまり不定のアドレスに対してアクセスしていることになります。当然例外が引き起こされます。

---

よく見ると、それらのポインタの値での読み書きはしておらず、それらのポインタ変数のアドレスを渡しているので、ポインタ変数が自体のアドレスをstudent_data構造体のアドレスとみなしてそのメンバーにアクセスしていることになりますね。いずれにしてもそれは誤りです。

Comments

[mayjex]

ご回答ありがとうございます。

"student_data*data[4]"という定義だと、構造体を指すポインタを４つ格納する配列という意味になるということですね。
やりたかったこととしては、構造体を４つ格納する配列という意味で定義したかったので"student_data data[4]"
とするのが正しいと理解しました。ありがとうございました。

Answer 2

student_data* data[4]; の*を外して

student_data data[4];

とするだけでOK.

さもなくば:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

//    学生情報を入れる構造体
typedef struct {
    int id;            //    学生番号
    char name[256];    //    氏名
    int age;        //    年齢
} student_data;

//    学生情報を構造体に格納する関数
void setData(student_data*, int, char*, int);
//    学生情報を表示する関数
void showData(student_data*);

void main() {
    int i;
    student_data* data[4];
    int id[] = { 1, 2, 3, 4 };
    char name[][256] = { "山田", "佐藤", "太田", "中田" };
    int age[] = { 18, 19, 21, 18 };
    //    学生情報をセット
    for (i = 0; i < 4; i++) {
        data[i] = (student_data*)malloc(sizeof(student_data)); // ※追加
        setData(data[i], id[i], name[i], age[i]); // ※ &data[i] → data[i]
    }
    //    学生情報を表示
    for (i = 0; i < 4; i++) {
        showData(data[i]); // ※ &data[i] → data[i]
        free(data[i]); // ※追加
    }
}

void setData(student_data* data, int id, char* name, int age) {
    data->id = id;
    strcpy(data->name, name);
    data->age = age;
}

void showData(student_data*data) {
    printf("学生番号：%d　", data->id);
    printf("氏名：%s　", data->name);
    printf("年齢：%d\n", data->age);
}

※ なぜだかわかるかな?

Comments

[mayjex]

ご回答ありがとうございます。
教えていただいた方法で正常に動作しました。

*を外したら良い理由ですが、
構造体の配列自体がポインタを表しているため、*data[4]としてしまうと、
ポインタのポインタになってしまうからという認識であってますでしょうか？
（説明がわかりにくくてすいません。）

[episteme]

てか、*付いた状態でコンパイルしたらwarning出たんじゃありません?

[mayjex]

warningは出ていませんでした。

[episteme]

あれー? 僕とこと違うなー...
> foo.c(26): warning C4047: '関数': 間接参照のレベルが 'student_data *' と 'student_data **' で異なっています。
> foo.c(26): warning C4024: 'setData': の型が 1 の仮引数および実引数と異なります。
...って叱られたんだけどなー