構造体配列の動的なメモリ確保の仕方を教えて下さい

Question

先ほどは失礼いたしました
質問のプログラムを変えました
構造体型配列への動的メモリ確保について教えてください

前提条件：mallocかcallocの使用、構造体配列でデータを持つ事

以上の条件下で動的にメモリを確保しようと
下記プログラムを実行した所
初期値が0以外になってしまい
プログラムが誤作動を起こします。

わからない事：mallocをどのように構造体配列のデータに参照させるか
（動的メモリ確保は190行目付近に有ります）
以上ですよろしくお願い致します。

#define _CRT_SECURE_NO_WARNINGS
#define MAX_LINE 5
#define MAX_DATA 50
#define MAX_NAME_LEN 24
#define MAX_ANOTHER_NAME_LEN 24
#define MAX_TEL_LEN 18
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>///////////数字判定用

/* 顧客住所録構造体 */
typedef struct tag_CustomerData {
	long lId;										// 顧客番号
	char szName[MAX_NAME_LEN + 1];					// 顧客名
	char szAnotherName[MAX_NAME_LEN + 1];			// ふりがな
	char szTel[MAX_TEL_LEN + 1];					// 電話番号

} CUSTOMER_DATA;

void openMenu(int nInputnum);
void addData(CUSTOMER_DATA head[]);

int main()
{
	char str[128];
	int len;
	int nInputnum;//////////登録の天井処理で使う
	int i;

	while (1) {
		system("cls");
		printf("顧客情報がありません。何件分確保しますか？(1 - 50)");
		fgets(str, sizeof(str), stdin);

		/*	改行文字が含まれているかの確認		*/
		/*	改行文字を終端記号に置換する		*/
		/*	else時バッファはクリアする			*/
		if (strchr(str, '\n') != NULL) {
			str[strlen(str) - 1] = '\0';
			
		}else{
			while (getchar() != '\n');
		}
		len = strlen(str);
		
		str[0] = toupper(str[0]);
		if (len == 1 && str[0] == 'Q') {
			return 0;
		}
		/* 不正文字混入時はループを抜けるので	*/
		/* lenとiの差異で分岐させる				*/
		for (i = 0; i < len; i++) {
			if (!(isdigit(str[i]))) {
				break;
			}
		}
		if (len == 0 || i != len) {
			printf("＜＜入力範囲にありません＞＞");
			getchar();
			continue;
		}
		
		nInputnum = atoi(str);
		if (!(nInputnum >= 1 && nInputnum <= 50)) {
			printf("＜＜入力範囲にありません＞＞");
			getchar();
			continue;
		}
		openMenu(nInputnum);
		return 0;
	}
}

void openMenu(int nInputnum)
{
	long lId = 0;
	char str[128];
	int len;
	int num;
	int i;
	int nCountup;
	CUSTOMER_DATA *head;
	
	while (1) {
		system("cls");
		printf("***** 顧客情報管理 *****\n");
		printf("1:登録\n");
		printf("2:一覧表\示\n");												/*	'表'文字エスケープ処理で\挿入	*/
		printf("3:削除\n");
		printf("4:ファイル出力\n");
		printf("5:終了\n");
		printf("************************\n");
		printf("(処理区分)＞");
		fgets(str, sizeof(str), stdin);

		/*	改行文字が含まれているかの確認		*/
		/*	改行文字を終端記号に置換する		*/
		/*	else時バッファはクリアする			*/
		if (strchr(str, '\n') != NULL) {
			str[strlen(str) - 1] = '\0';
		}else{
			while (getchar() != '\n');
		}
		len = strlen(str);
		
		/* 不正文字混入時はループを抜けるので	*/
		/* lenとiの差異で分岐させる				*/
		for (i = 0; i < len; i++) {
			if (!(isdigit(str[i]))) {
				break;
			}
		}
		if (len == 0 || i != len) {
			printf("＜＜入力した文字が不正です＞＞");
			getchar();
			continue;
		}
		num = atoi(str);
		if (!(num >= 1 && num <= 5)) {
			printf("＜＜処理区分エラー＞＞\n");
			getchar();
			continue;
		}
		switch (num) {
			case 1:		addData(head);
						continue;
						
			case 2:		for (i = 0; i < 128; i++) {	
							if (head[i].lId != 0 && head[i].lId != -1) {
								nCountup++;
							};
						}
						if(nCountup == 0){
							printf("表\示するデータがありません\n");
							getchar();
							continue;
						}
						showData(head, nInputnum);
						continue;
						
			case 3:		for (i = 0; i < 128; i++) {	
							if (head[i].lId != 0 && head[i].lId != -1) {
								nCountup++;
							};
						}
						if(nCountup == 0){
							printf("表\示するデータがありません\n");
							getchar();
							continue;
						}
						showData(head, nInputnum);
						continue;
						
			case 4:		for (i = 0; i < 128; i++) {	
							if (head[i].lId != 0 && head[i].lId != -1) {
								nCountup++;
							};
						}
						if(nCountup == 0){
							printf("表\示するデータがありません\n");
							getchar();
							continue;
						}
						//////////
						continue;
						
			case 5:		return;
		}
	}
}

void addData(CUSTOMER_DATA acList[])
{
	int nCountup = 0;
	char achName[128];
	char achAnotherName[128];
	char achTel[128];
	int len;
	int i;
	
	CUSTOMER_DATA *pnL;/////////////////////////////////////////////////////////////////////////////////////////////////////////////////
	
	if ((pnL = (CUSTOMER_DATA *)malloc(sizeof(CUSTOMER_DATA))) == NULL) {			/* 記憶領域の確保 */
		exit(EXIT_FAILURE);
	}
	
	acList = pnL;///////////////////////////////////////////////////////////////////////////////////////////////////////////////////////

	for (i = 0; i < 128; i++) {													/*0の値以外の登録済み及び			*/
		if (acList[i].lId != 0) {												/*削除済みの値はカウントアップする	*/
			nCountup++;
		};
	}
	
	while (1) {
		system("cls");
		printf("***** 顧客登録画面 *****\n");
		printf("氏名  ：");

		fgets(achName, sizeof(achName), stdin);
		if (strchr(achName, '\n') != NULL) {
			achName[strlen(achName) - 1] = '\0';
			
		}else{
			while (getchar() != '\n');
		}
		len = strlen(achName);
		
		if (len == 0) {
			printf("＜＜入力した文字が不正です＞＞\n");
			getchar();
			continue;
		}
		if (len > MAX_NAME_LEN) {
			printf("＜＜入力した文字の長さが大きすぎます＞＞\n");
			getchar();
			continue;
		}
		break;
	}

	while (1) {
		system("cls");
		printf("***** 顧客登録画面 *****\n");
		printf("氏名  ：%s\n", achName);
		printf("ふりがな：");

		fgets(achAnotherName, sizeof(achAnotherName), stdin);
		if (strchr(achAnotherName, '\n') != NULL) {
			achAnotherName[strlen(achAnotherName) - 1] = '\0';
		}else{
			while (getchar() != '\n');
		}
		len = strlen(achAnotherName);

		if (len == 0) {
			printf("＜＜入力した文字が不正です＞＞\n");
			getchar();
			continue;
		}
		if (len > MAX_ANOTHER_NAME_LEN) {
			printf("＜＜入力した文字の長さが大きすぎます＞＞\n");
			getchar();
			continue;
		}
		break;
	}

	while (1) {
		system("cls");
		printf("***** 顧客登録画面 *****\n");
		printf("氏名  ：%s\n", achName);
		printf("ふりがな：%s\n", achAnotherName);
		printf("電話番号：");

		fgets(achTel, sizeof(achTel), stdin);
		if (strchr(achTel, '\n') != NULL) {
			achTel[strlen(achTel) - 1] = '\0';
		}else{
			while (getchar() != '\n');
		}
		len = strlen(achTel);
		
		if (len > MAX_TEL_LEN) {
			printf("＜＜入力した文字の長さが大きすぎます＞＞\n");
			getchar();
			continue;
		}
		
		for (i = 0; i < len; i++) {
			if (achTel[i] == '-' || achTel[i] == '(' || achTel[i] == ')' || achTel[i] == '0'
				|| achTel[i] == '1' || achTel[i] == '2' || achTel[i] == '3'
				|| achTel[i] == '4' || achTel[i] == '5' || achTel[i] == '6'
				|| achTel[i] == '7' || achTel[i] == '8' || achTel[i] == '9') {
				continue;
			}
			break;
		}
		/* 不正文字混入時はループを抜けるので	*/
		/* lenとiの差異で分岐させる				*/
		if (len == 0 || i != len) {
			printf("＜＜入力した文字が不正です＞＞\n");
			getchar();
			continue;
		}
																					/*------------------------------*/
		acList[nCountup].lId = nCountup + 1;										/* 顧客番号を構造体に登録する	*/
																					/*------------------------------*/
		for (i = 0; achName[i] != '\0'; i++) {										/*------------------------------*/
			acList[nCountup].szName[i] = achName[i];								/* 氏名を構造体に登録する		*/
		}																			/*------------------------------*/
		acList[nCountup].szName[i] = '\0';

		for (i = 0; achAnotherName[i] != '\0'; i++) {								/*------------------------------*/
			acList[nCountup].szAnotherName[i] = achAnotherName[i];					/* ふりがなを構造体に登録する	*/
		}																			/*------------------------------*/
		acList[nCountup].szAnotherName[i] = '\0';

		for (i = 0; achTel[i] != '\0'; i++) {										/*------------------------------*/
			acList[nCountup].szTel[i] = achTel[i];									/* 電話番号を構造体に登録する	*/
		}																			/*------------------------------*/
		acList[nCountup].szTel[i] = '\0';

//		printf("\n%s%s%s\n", acList[nCountup].szName, acList[nCountup].szAnotherName, acList[nCountup].szTel);	////////////////デバッグ
//		printf("\n%ld%s%s%s\n", acList[0].lId, acList[0].szName, acList[0].szAnotherName, acList[0].szTel);	////////////////デバッグ
//		printf("\n%ld%s%s%s\n", acList[1].lId, acList[1].szName, acList[1].szAnotherName, acList[1].szTel);	////////////////デバッグ
//		printf("\n%ld%s%s%s\n", acList[2].lId, acList[2].szName, acList[2].szAnotherName, acList[2].szTel);	////////////////デバッグ
		printf("\n登録しました。\n");
		getchar();
		return;
	}
}

Answer 1

まず、
}```c /構造体配列の動的なメモリ確保/

の部分改行をいれてください、レイアウト崩れていて見づらいです。

内容ですが、何をされたいのかかなり分からないソースになっています。

SEI []seito;
？？？
SEI型の配列を宣言したい？
のであれば
SEI seito[16];
のようにサイズも必要、でもそうなるとmalloc等での動的確保と矛盾する部分があるのでこの行じたいいらないのでは？

(pns = (SEI *)malloc(sizeof(SEI)))
の部分、これではSEI構造体１個分のメモリしか確保できていませんが、配列でとかかれているので違うのでは？

sizeof(SEI) * 8 のように要素分確保するべき？

seito = pns;
seitoは宣言しないのでいらない、もし宣言していてもメモリ確保すみのseitoのポイントを別に置き換えているというよくわからない処理になっている。

seito.[0] = {100,"あああ",47}
書き方おかしい。。。

ざっとみてもこれだけ？？の部分がありそうです。

配列の中身（int型）が０で初期化されないので

問題が発生する。

という問題ではないような気がします。

##追記

質問のタイトルに戻って動的なメモリの確保といことで

typedef struct seiseki {     
    int no;        /* 学生番号 */     
    char name[128];    /* 氏名 */     
    int kokugo;        /* 国語点数 */

}SEI,*lpSEI;

としておいて

lpSEI pns;

pns = (lpSEI)malloc(sizeof(SEI) * 10);

で10個分確保です

質問文修正後への追記

全文は読んでません、mallocが使われている周辺だけ読んでのコメントですが

void addData(CUSTOMER_DATA acList[]) 

〜略〜
if ((pnL = (CUSTOMER_DATA *)malloc(sizeof(CUSTOMER_DATA))) == NULL) { 
〜略〜


acList = pnL;

void addData(CUSTOMER_DATA acList[])
多分私なら
void addData(CUSTOMER_DATA *acList)
かな

そして

if ((pnL = (CUSTOMER_DATA *)malloc(sizeof(CUSTOMER_DATA))) == NULL)
ではなく

if ((acList = (CUSTOMER_DATA *)malloc(sizeof(CUSTOMER_DATA) * 128)) == NULL) {

この部分で1要素分しか確保してないので

for (i = 0; i < 128; i++) { /*0の値以外の登録済み及び */
if (acList[i].lId != 0) {

こんな風にfor分で回すと未確保のメモリにいってします気がします。

###追記

メモリの静的確保、と動的確保、またポインタと配列あたりの理解が曖昧になっている気がします。

静的確保と動的確保

// 静的確保
char buff[12];

と

char *buff;

// 動的確保
buff = (char *)malloc(sizeof(char)*12);

の二つは等価です。どちらか片方行えばOKです。

そして動的に確保したbuffも配列同様
buff[0],buff[1]などの使い方もできます。

しかし

char *buff;

buff = (char *)malloc(sizeof(char));// char１要素分を確保(char buff[1];と等価)

にたいして

buff[0] = 'a';
//　ここまではOK
// ここから下はできません、メモリ破壊です
buff[1] = 'b';
buff[2] = 'c';
buff[3] = 'd';

Comments

[dec5798]

御指摘の通り
動的なメモリと静的なメモリで
大きな勘違いをしていました。
もう一度調べ直して曖昧な部分を解消します。
アドバイス大変ありがとうございました。