質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

多次元配列

1次元配列内にさらに配列を格納している配列を、多次元配列と呼びます。

ポインタ

ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

メモリリーク

メモリリークは、プログラムファイルがメモリの解放に失敗した時に起こります。

Q&A

解決済

2回答

1994閲覧

mallocのエラー処理(ダブルポインタを用いた2次元配列)

TripleTriad

総合スコア5

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

多次元配列

1次元配列内にさらに配列を格納している配列を、多次元配列と呼びます。

ポインタ

ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

メモリリーク

メモリリークは、プログラムファイルがメモリの解放に失敗した時に起こります。

0グッド

0クリップ

投稿2020/02/01 05:32

前提・実現したいこと

C言語でmallocを用いて2次元配列を動的に確保したいと思っています.

あるサイトでダブルポインタを用いて2次元配列を実現するソースコードがあったのですが,
mallocでメモリ確保に失敗した場合のエラー処理で,メモリリークしているのではないかと思われる個所がありました.

発生している問題・エラーメッセージ

そこで,以下の2点を確認したく質問いたします.
・メモリリークの有無(該当箇所にコメント付記)
・メモリリークがあれば,その修正方法が正しいか

以上,よろしくお願いいたします.

該当のソースコード

#include <stdio.h> #include <stdlib.h> int main(void) { int n = 10; int m = 10; double **A = (double **) malloc( sizeof(double*) * n ); if ( A == NULL ) { return 1; } for (int i = 0; i < n; i++) { A[i] = (double *) malloc( sizeof(double) * m ); if ( A[i] == NULL ) { for (int j = i-1; j >= 0; j--) { free(A[j]); } // free(A); // ここでもメモリ解放が必要なのではないかと思うのですが..いかかでしょうか? return 1; } for (int j = 0; j < m; j++ ) { A[i][j] = i*j; } } for (int i = 0; i < n; i++) { for (int j = 0; j < m; j++) { printf("%6.2f ", A[i][j]); } printf("\n"); } for (int i = 0; i < n; i++) { free(A[i]); } free(A); return 0; }

試したこと

補足情報(FW/ツールのバージョンなど)

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

一度に全部確保するサンプル

C

1#include <stdio.h> 2#include <stdlib.h> 3 4int main(void) { 5 6 int n = 10; 7 int m = 10; 8 9 double **A = (double **) malloc( sizeof(double*)*n + 10 sizeof(double)*n*m); 11 if ( A == NULL ) { 12 return 1; 13 } 14 double* p = (double*)(A+n); 15 for (int i = 0; i < n; i++) { 16 A[i] = p; 17 p += m; 18 19 for (int j = 0; j < m; j++ ) { 20 A[i][j] = i*j; 21 } 22 } 23 24 for (int i = 0; i < n; i++) { 25 for (int j = 0; j < m; j++) { 26 printf("%6.2f ", A[i][j]); 27 } 28 printf("\n"); 29 } 30 31 free(A); 32 33 return 0; 34 35}

投稿2020/02/01 12:17

episteme

総合スコア16614

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

// free(A); // ここでもメモリ解放が必要なのではないかと思うのですが..いかかでしょうか?

「最後までカンペキに確保できなかったら、その時点で確保した領域を全部解放したい」
ならその通りです。

投稿2020/02/01 06:00

episteme

総合スコア16614

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

TripleTriad

2020/02/01 06:44

回答ありがとうございます. 自分の思っていた通りで安心しました.
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問