多次元配列についてです。

Question

１、ポインタのポインタを２次元配列で初期化したいです。

	char buf[3][100] = {
		"you will make me happy",
		"I love you!",
		"Thanks"
	};

	char **p;
	char *q[3] = {buf[0], buf[1], buf[2]};
	p = q;
}

これは、うまくいきました。
char **pを上記のように初期化すると面倒です。

qを用意しないで、

char **p = buf;
とやりたいのですが・・・・うまくいきません。(warningが出てきます)
どうすれば、よろしいでしょうか？

２、多次元配列の動的確保で落ちます・・・・

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#define line 100

int main() {
	int i = 0;
	char **p;
	p = (char **)calloc(10, sizeof(char *));

	char (*q)[line];  // char q[][line]
	int n = 4;
	q = (char(*)[line])malloc(n * (sizeof(char) * line));  // char q[n][line]

	char *r[line];  // char r[line][]
	for (i = 0; i < line; i++)
		r[i] = calloc(100, sizeof(char));

	if (p) {
		p[0] = "DESTINY"; 
		p[1] = "I love you!!";

		printf("p : %s %s\n", p[0], p[1]);
		for(i = 0; i < 10; i++)
			free(p[n]);
	} else
		printf("failed!!\n");

	if (q) {
		strcpy(q[0], "you will make me sad");
		strcpy(q[1], "basketball");

		printf("q : %s %s\n", q[0], q[1]);
		free(q);
	} else
		printf("failed!!\n");

	if (r) {
		r[0] = "you will make me happy";
		r[1] = "but I don't like you";

		printf("r : %s %s\n", r[0], r[1]);
		for(i = 0; i < line; i++)
			free(r[i]);
	}
	else
		printf("failed!!\n");

	return 0;
}

これ、コンパイルはおｋでしたが、実行すると落ちます。
一体どこがおかしいのでしょう・・・

3番目のrを追加したら、おかしくなりました。
ダメな箇所の指摘をお願いします。

Windows
visual stdio
でやってみました。

Answer 1

こんにちは。

pはchar型へのポインタへのポインタです。
従ってp[0]は「char型へのポインタ」を指してます。
ということは、pが指せるメモリ領域の構造は下記です。

struct p_array_image
{
    char* p0;
    char* p1;
    char* p2;
        ：
};

要素数が3の時、sizeof(p_array_image)は32ビット・ビルドなら12でしょう。

逆にchar buf[3][100]は下記のような構造です。

struct buff_image
{
    char buff0[100];
    char buff1[100];
    char buff2[100];
};

sizeof(buff_image)は300ですね。

このようにpが指す構造とbuffの構造が全く異なるため、p経由でbuffの要素にアクセス出来るはずありません。従って、p=buff;はやるべきでない操作です。

動的配列の方は単なるバグと思います。

for(i = 0; i < 10; i++)
   free(p[n]);

当然、p[4]を10回も解放してはだめです。
（p[4]はNULLなので落ちているのはここが原因ではないと思いますが。）

3番目のrを追加したら、おかしくなりました。

ダメな箇所の指摘をお願いします。

r[0], r[1]はcallocで確保したメモリではなく、文字列定数へのポインタですね。メモリ・マネージャが管理していない領域を解放すると異常が発生します。

そもそも、free関数に渡すポインタは、mallocやcallocで受け取ったポインタです。
それ以外のものを渡してはいけません。たとえそれがmallocやcallocで割り当てられた領域であったとしても途中のアドレスを渡してはダメです。必ず先頭アドレスを渡して下さい。
qは適切に解放されていると思います。pとrはNGです。

Comments

[strike1217]

ありがとうございます
char buffer[100];
char* re = buffer; これができるので２次元の時もできるものだと思ってしまいました。

p[0] = "DESTINY";
こういうことはしてはダメなんですね！！
修正してみます。

[strike1217]

char **p;
p = (char **)calloc(10, sizeof(char *));
この子の解放の仕方は・・・・
for(i = 0; i < 10; i++)
free(p[i]);
こうですか？？
free(p);
こうですか？？

[Chironian]

後者です。callocから返ってきたメモリの先頭アドレスをfreeに渡して解放します。

また、free(p[i])で実際にfree()に渡っている値はなんでしょうか？
printf("%p\n", p[i]);してみると良いと思います。

[strike1217]

どうも２次元配列が苦手なので、もう少し勉強いたします。
ありがとうございました。

[strike1217]

ちょっと、質問の仕方が悪すぎたので、再度質問します！

Answer 2

アドレスとして配列に入れる方法ではどうでしょうか。

#include<stdio.h>
void main(void){

    char buf[3][100] = {
        "you will make me happy",
        "I love you!",
        "Thanks"
    };

    void *q[] = {&buf[0], &buf[1], &buf[2]};

    printf("%s %s %s\n",*q, *(q+1), *(q+2));
}

Comments

[strike1217]

ふむふむ・・・
ありがとうございます！