[C]関数で構造体配列の中身を表示させる際に、コンソールが停止

Question

閲覧ありがとうございます。

• 言語 ： C
• コンパイラ ： gcc
• コンソール ： コマンドプロンプト

下記ソースをコンパイルし、実行すると、マークした行でコンソールが動作停止してしまいます。

#include<stdio.h>

/* 構造体 */
typedef struct{
	char	name[60];
	short	age;
	char	sex;
}People;

/* プロトタイプ宣言 */
void writeData(People data[]);	/* 入力用の関数 */
void printData(People data[]);	/* 出力用の関数 */

/* main関数 */
int main(void)
{
	People data[3];	/* 構造体変数 */
	People *pdata;	/* 構造体ポインタ変数 */
	
	pdata = data;	/* ポインタにセット */
	
	writeData(pdata);	/* 入力 */
	printData(pdata);	/* 出力 */
	
	return 0;
}

/* 入力用の関数 */
void writeData(People data[])
{
	short cnt;
	
	printf("3人の氏名、年齢、性別(0:男性 1:女性)を入力してください。 \n");
	for(cnt = 0; cnt < 3; cnt++)
	{
		printf("[%d人目] \n", cnt + 1);
		printf("\t氏名 :", cnt);scanf("%s", data[cnt].name);
		printf("\t年齢 :", cnt);scanf("%d", &data[cnt].age);
		printf("\t性別 :", cnt);scanf("%d", &data[cnt].sex);
	}
	printf("◆入力完了 \n\n");
}

/* 出力用の関数 */
void printData(People data[])
{
	short cnt;
	
	printf("3人分のデータを出力します。 \n");
	for(cnt = 0; cnt < 3; cnt++)
	{
		printf("[%d人目] \n", cnt + 1);
		printf("\t氏名 : %s \n", data[cnt].name); /* ◆この行を実行しようとすると動作停止 */
		printf("\t年齢 : %d \n", data[cnt].age);
		if(data[cnt].sex == 0)
			printf("\t性別 : 男性 \n");
		else
			printf("\t性別 : 女性 \n");
	}
}

コンソールが停止するだけでエラー表示も何もないので、対処方法が分かりません。
このバグの原因は何なのでしょうか？ご教示いただけると幸いです。

Answer 1

Chironian さんのおっしゃる通り、sex は char 型なのでscanf("%d"で入力するには領域が足らず、関係ないところにも上書きしてしまいます。 writeData() で使っている data の実体は、以下のように main() で定義されている data ですので、すぐ下に定義してある pdata を上書きして壊しているのだと思います。

    People data[3];    /* 構造体変数 */
    People *pdata;    /* 構造体ポインタ変数 */

つまり、writeData() で pdata を壊してしまい、printData() に渡すときには pdata の中身はデタラメなものになっているのでしょう。

Comments

[wAsabi1107]

なるほど、メモリ領域の破壊がコンソールの停止する理由なのですね。
分かりやすい解説ありがとうございます。

Answer 2

こんにちは。

取り敢えず危険な部分が２箇所あります。
scanf("%d"で読み出先はint型変数です。しかし、ageとsecはそれぞれshortとcharですので領域が足りません。
しかし、ここで落ちず、更に先で落ちているのは不可解です。

取り敢えず、ageとsecをint型とし、一部のprintf文の余分なcntを削除して下記を入力してみました。

abc 21 0

def 22 1
ghi 23 0

特に問題なく動作しました。wandbox

入力データの相違だろうと思います。どんなデータを入力されたでしょうか？

Comments

[wAsabi1107]

回答ありがとうございます！
手元の環境でもご指摘いただいた箇所を修正し(sexをshort型に、scanf関数の変換指定子を%dから%hdに)、無事動作しました。

こちら修正前ソースでの実行結果と、入力データですhttps://wandbox.org/permlink/PSiIkbzLizJWWQpo
なぜか動作していますが・・・笑（変換指定子の警告が出ているので、そこを上手く調整してくれた？）
hideki1234さんのご指摘にあるように、メモリ領域を破壊された構造体を呼び出そうとしたので動作停止というバグとなったのではないか、と愚考します。