初心者です。C言語のポインタ配列を勉強しているのですが現在のソースコードで間違えている個所を指摘してください。

Question

実現したいこと

scanfで書き込んだ文字を3回表示させたいです。
現状のソースコードではpstr=%が1回しか表示されないです。

前提

ポインタ配列がよくわからないため勉強している最中です。
現状のコードで間違えている個所を指摘してほしいです。

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

Warningのみ表示されています。

(16,23): warning C4047: '関数': 間接参照のレベルが 'rsize_t' と 'char [100]' で異なっています。
(16,19): warning C4024: 'strcpy_s': の型が 2 の仮引数および実引数と異なります。
(16,37): warning C4047: '関数': 間接参照のレベルが 'const char *' と 'size_t' で異なっています。
(16,25): warning C4024: 'strcpy_s': の型が 3 の仮引数および実引数と異なります。
(27,16): warning C4047: '関数': 間接参照のレベルが 'char **' と 'char *(*)[3]' で異なっています。
(27,12): warning C4024: 'write_str': の型が 1 の仮引数および実引数と異なります。

該当のソースコード

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

int write_str(char **pstr) {
	int i;
	char mozi[100];
	
	for (i = 0; i < 3; i++) {
		printf("-----\n");
		scanf_s("%s", mozi, 100);
		*(pstr + i) = (char*)malloc(sizeof(char) * (strlen(mozi) + 1));
		strcpy_s(*pstr, mozi, sizeof(mozi));
		printf("pstr=%s\n", *pstr);
	}
	return 0;
}


int main()
{
	char* str[3];

	write_str(&str);

	return 0;
}

試したこと

ループされているか確認したがループされていなかった。

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

ここにより詳細な情報を記載してください。

Answer 1

(※chatGPTによる回答です）
問題の原因は、write_str 関数内でポインタ pstr にアドレスを格納する際、配列要素を指定していないことにあります。配列要素を指定せずにポインタ pstr をインクリメントしてしまうと、異なるアドレスにアクセスすることになり、想定外の動作が発生します。以下は、修正されたコードです。

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

int write_str(char **pstr) {
    int i;
    char mozi[100];

    for (i = 0; i < 3; i++) {
        printf("-----\n");
        scanf_s("%s", mozi, 100);
        *(pstr + i) = (char*)malloc(sizeof(char) * (strlen(mozi) + 1));
        strcpy_s(*(pstr + i), strlen(mozi) + 1, mozi);
        printf("pstr[%d]=%s\n", i, *(pstr + i));
    }
    return 0;
}

int main() {
    char* str[3];
    write_str(str);
    return 0;
}

この修正されたコードでは、write_str 関数内で配列要素を指定して、正しいアドレスを pstr に格納するように修正されています。また、strcpy_s 関数の引数には、ポインタの指し示す領域のサイズを指定する必要があるため、strlen(mozi) + 1 となっています。さらに、printf文の表示も配列要素を指定するように修正されています。

Comments

[zero1132]

解決し理解できました。
迅速な回答ありがとうございました。