malloc関数、文字化け

Question

Windows で実験コードを作成していたら、表示がおかしくなりました。

#include<stdio.h>
#include<stdlib.h>
#include <string.h>
#define str "you will make me happy"

int main() {
	int* p = NULL;
	long i = 0;
	p = (int *)calloc(1, sizeof(int));  // sizeof(int)の大きさを1個作る.

	char* q = (char *)malloc(sizeof(char) * (strlen(str)));

	if (p && q) {
		*p = 100;
		for (i = 0; i < strlen(str); i++)
			q[i] = str[i];
		printf("Succeed!!%d : %s : %d", *p, q, strlen(str));
		free(p);
		free(q);
	}
	else
		printf("failed");
	return 0;
}

Succeed!!100 : you will make me happy86･ﾓT・ : 22

という風に文字化けしてしまいます。
mallocのところをcalloc()でやるとうまくいきました。
このコードどこがおかしいのでしょうか？？
ダメな部分の指摘をお願いします。

ちなみに、for文を以下のようにしたら、化けなくなりました。

for (i = 0; i <= strlen(str); i++)
			q[i] = str[i];

それから、文字列を短いとうまくいきます。
長くするダメになります。

#define str "HAPPY"

なぜですか？？
配列イメージだと一番最初のコードでよさそうですが・・・・

Answer 1

strxxx系のファンクションは、長さを指定しない代わりに\0をストッパーとしています
ですので\0が見つかるまでを対象としています。
また、printfの%sも同様な動きをします。callocは\0で埋めてくれますが、mallocで、
同様とするには自身で\0埋める必要があります。定義された領域もたまたま後ろが\0で
有る場合が多く正常っぽく動く事があります。

#include<stdio.h>
#include<stdlib.h>
#include <string.h>
#define str "you will make me happy"

int main() {
    int* p = NULL;
    long i = 0;
    p = (int *)calloc(1, sizeof(int));  // sizeof(int)の大きさを1個作る.

    char* q = (char *)malloc(sizeof(char) * (sizeof(str))); //sizeofを使う23

    if (p && q) {
        *p = 100;
        for (i = 0; i <= strlen(str); i++) //最後に\0を入れる
            q[i] = str[i];
        printf("Succeed!!%d : %s : %d", *p, q, strlen(str));
        free(p);
        free(q);
    }
    else
        printf("failed");
    return 0;
}

Comments

[strike1217]

今まで、配列のコピーとかは以下のようにやっていました。
for (i = 0; i < strlen(str); i++)

ただの配列の時は問題なくできるのですが、これではダメということですね！

for (i = 0; i < strlen(str) + 1; i++)
こうでもよいのでしょうか？？

[A.Ichi]

焦点は、q[i] = str[i];の部分たと思います。#defineは、プリコンパイラがstr文字列を　"you will make me happy"　に変えてCコンパイラに渡すと思います。ですのでstr[22]は、領域外となります。
ですので「q」のエリアを１バイト長くmallocしてそこに\0を入れる様にしています。for文は22で抜けて
来ますのでそれを使っています。

[strike1217]

ふむふむ・・・

#define str "you will make me happy"でも終端にヌル文字は付きますよね？？

[A.Ichi]

訂正します。
すみませんstrは23バイトの領域を取り、NULLが付いています。ご指摘ありがとうございます。
ですので　for (i = 0; i < strlen(str) + 1; i++)　for (i = 0; i <= strlen(str); i++)　でもOKです。

[strike1217]

ありがとうございます！

[otn]

文字列のコピーはforでループを回さず、strncpy関数を使いましょう。

[strike1217]

やはり標準関数を使用した方が良いんですね！
ありがとうございます。

Answer 2

終端に 0 が必要だった気がします。

Comments

[strike1217]

ありがとうございます。

Answer 3

メモリの確保量が1バイト足りません。

char* q = (char *)malloc(sizeof(char) * (strlen(str)+1));

あと malloc したあとのメモリはゼロクリアされていないので自分で 0x00 を最後に入れる必要があります。マクロの場合は最後に 0x00 が入っているのでそれがコピーされます。

Comments

[strike1217]

char* q = (char *)malloc(sizeof(char) * (strlen(str)+1));
これだけでは、うまくいきませんでした。
for (i = 0; i < strlen(str) + 1; i++)
こうしないとダメみたいです。

[strike1217]

#define str "you will make me happy"でも終端にヌル文字は付きますよね？？

[mattn]

> こうしないと

0x00 を最後に入れる必要があります。

がその意味になります。

[mattn]

マクロで、と言った意味は strcpy を使えばそこまではコピーしてくれるという意味です。

[strike1217]

ありがとうございます。