ポインタのswap処理の添削をお願いしたいです

Question

質問内容

動的メモリ確保解放の練習として以下のコードを書きました。確保したメモリを自分で全て解放したい場合、freeの仕方は合っているでしょうか？他にも間違った点があったら指摘お願いします。

該当のソースコード

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main()
{
    char *x, *y, *tmp;
    char a[100] = "hello";
    char b[100] = "abc";
    x = (char *)malloc(strlen(a) + 1);
    strcpy(x, a);
    y = (char *)malloc(strlen(b) + 1);
    strcpy(y, b);
    tmp = x;
    x = y;
    y = tmp;
    free(x);
    free(y);
}

Answer 1

printf("%p", ポインタ) で任意のポインタ値を出力できます。
mallocの直後/freeの直前にポインタ値を出力し、それぞれがひとつずつ現れていればOK.

Comments

[pikahyogo]

なるほど。では、合ってそうですね。ありがとうございました。

Answer 2

malloc/freeについては既に良回答があるのでそちらにおまかせするとして。

他にも間違った点があったら

とのことなので、重箱の隅をつつきます。

• C++ではなくC言語であれば

int main()は
int main(void)のほうがよい気がします。
C言語では引数リストが空なのはvoidと等価ではありません。そしてCの規格ではプログラムの開始点の標準的なものとして決められているものの一つがint main(void)であってint main()ではないのです。
一方、プログラムの開始点の規格には「処理系が認めるもの」というのもあるので、コンパイラが文句をいわないなら「間違ってはいない」ということにもなるのですが。

• mallocでメモリ取得操作をしたら、結果としてちゃんと取得できたかどうか(NULLが返ってきていないか)チェックするのがお作法でしょう。

• char a[] = "hello";ではなくchar a[100] = "hello";にした意図があるはずです。その意図に対してmalloc(100)ではなくmalloc(strlen(a) + 1);という記述をしたことは整合するでしょうか?

Comments

[pikahyogo]

回答ありがとうございます。今度からvoid使いますね。

char = a[100]にした理由は、いつもループで違う大きさの文字列を格納するという処理を書いていたのでその名残で書いていました。確かにこのプログラムだとa[]でいいですね。ありがとうございました。