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

質問内容

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

該当のソースコード

c
1#include <stdio.h>
2#include <stdlib.h>
3#include <string.h>
4int main()
5{
6    char *x, *y, *tmp;
7    char a[100] = "hello";
8    char b[100] = "abc";
9    x = (char *)malloc(strlen(a) + 1);
10    strcpy(x, a);
11    y = (char *)malloc(strlen(b) + 1);
12    strcpy(y, b);
13    tmp = x;
14    x = y;
15    y = tmp;
16    free(x);
17    free(y);
18}

行動規範の内容に同意します

回答2件

ベストアンサー

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

投稿2021/07/31 21:32

episteme

総合スコア16612

pikahyogo

2021/08/01 00:49

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

行動規範の内容に同意します

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);という記述をしたことは整合するでしょうか?

投稿2021/07/31 23:00