回答率: 85.48%

質問するログイン新規登録

トップ Cに関する質問 C言語でのメモリの解放が出来ません

編集履歴

回答編集履歴

2

サンプルのミスを修正

2018/07/18 05:56

投稿

スコア429

test CHANGED Viewed

@@ -196,7 +196,7 @@
 void memory(void){
-    pimage = (image_type *)malloc(sizeof(image_type) * MAX_IMAGESIZE);
+    pimage = (image_type *)malloc(sizeof(image_type) * MAX_NUM_OF_IMAGES);
 }

1

おまけを追加

2018/07/18 05:56

投稿

スコア429

test CHANGED Viewed

@@ -212,4 +212,20 @@
-本題とは逸れますが、 `printf()` や `FILE` 構造体などは `stdio.h` というヘッダファイルに入っています。できれば `#include <stdio.h>` しておいたほうがよいと思います。
+本題とは逸れますが、
+1. `printf()` や `FILE` 構造体などは `stdio.h` というヘッダファイルに入っています。できれば `#include <stdio.h>` しておいたほうがよいと思います。
+1. `ReadImage()` 関数の定義のところで、その引数のところが
+        void ReadImage(set) {
+          // ...
+        }
+    となっていますが、きちんと `set` の型を明示するべきです。コンパイルエラーにはならないのですが、C言語では何も書かないと勝手に int と思われるという恐ろしいルールがあります。今回は `set` の型は使われ方から見ても int なので結果的に問題はなかったのですが、思わぬバグの温床になります。見た目にもわかりづらいので、例え int を受け取るのであろうと、きちんと書くべきです。