![](http://oshiete.xgoo.jp/images/v2/pc/qa/question_title.png?8acaa2e)
初めにお断りしておきたい事があります。
出先で緊急な事でしたので、今即興で書いたソースになります。
Cコンパイル環境が有りませんでしたので、動作確認がとれておりません。
その為、ケアレスミス等有るかもしれませんがご容赦願います。
例として標準入力より文字列を取得する処理を記述しました。
以下の処理でreallocの際の断片化対策となるでしょうか。
(最終的に*strにセットされている領域が断片化されていない事)
実際の処理では1行辺り0~30000文字程の可変長の文字列を読み込む事を想定しております。
C(windows)のみ可でC++は不可になります。
その他、冗長な記述等指摘が有りましたらよろしくお願い致します。
#define MAX_BUF_SIZE 128
int getText(char **str) {
char *buf[MAX_BUF_SIZE];
char *tmp = NULL;
int tmpsize = 0;
if ((tmp = (char*) malloc(1)) == NULL) {
return FALSE;
}
*tmp = '\0';
while (fgets(buf, MAX_BUF_SIZE, stdin) != NULL) {
tmpsize += MAX_BUF_SIZE;
if ((tmp = realloc(tmp, tmpsize)) == NULL) {
free(tmp);
return FALSE;
}
strcat(tmp, buf);
}
if (*str != NULL) {
free(*str);
}
if (*str = (char*) malloc(strlen(tmp) + 1) != NULL) {
return FALSE;
}
strcpy(*str, tmp);
free(tmp);
return TRUE;
}
A 回答 (5件)
- 最新から表示
- 回答順に表示
No.4
- 回答日時:
>はい、そうです。
>今までreallocで確保を繰り返していたのでメモリ上>でフラグメンテーションが起こっているとの指摘で>したので。
じゃあこのソースでもだめです。
WindowsならVirtualAllocとかで予め大きく仮想メモリを予約しておくとかで解決できるかもしれません。
それかreallocで大き目にメモリ確保してreallocの回数を減らすようにしてみるべきです。
> WindowsならVirtualAllocとかで予め大きく仮想メモリを予約しておくとかで解決できるかもしれません。
> それかreallocで大き目にメモリ確保してreallocの回数を減らすようにしてみるべきです。
ありがとうございます。
やはりreallocで実装する場合は回数を減らすしかないのですね。
No.3
- 回答日時:
while (fgets(buf, MAX_BUF_SIZE, stdin) != NULL) {
tmpsize += MAX_BUF_SIZE;
if ((tmp = realloc(tmp, tmpsize)) == NULL) {
free(tmp);
return FALSE;
}
strcat(tmp, buf);
}
それよりも、このstrcatは危険です。
確かreallocは新たに増加した分のメモリ初期化をしません。
この点には注意してください。
> 確かreallocは新たに増加した分のメモリ初期化をしません。
> この点には注意してください。
ご指摘ありがとうございます。
了解しました。
No.2
- 回答日時:
何を心配しているのか不明ですが, malloc で確保されたメモリは (少なくとも) 引数で指定された分だけは連続しているにきまっています. もちろん, realloc を繰り返したときに「それらが連続しているか」は全くわかりません.
むしろ
if ((tmp = realloc(tmp, tmpsize)) == NULL) {
free(tmp);
return FALSE;
}
の方が危険. realloc が NULL を返すと tmp == NULL となって free(NULL); してしまい, それまでに確保していた分を解放できなくなってしまいます.
> 何を心配しているのか不明ですが
今までreallocを繰り返していたので、mallocで再確保して文字列コピーする事によって断片化を解消しようと考えました。
> realloc が NULL を返すと tmp == NULL となって free(NULL); してしまい, それまでに確保していた分を解放できなくなってしまいます.
ご指摘ありがとうございます。
対処致します。
とりあえず*strに期待通りに確保出来ている事が判りましたので、締め切らせていただきます。
どうもありがとうございました。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
似たような質問が見つかりました
- C言語・C++・C# プログラミングの授業の課題です 1 2023/01/17 22:15
- C言語・C++・C# c言語 プログラムのエラー 1 2023/02/11 20:31
- C言語・C++・C# 宣言する関数の形が決まっている状態で、 str1とstr2の文字列をこの順に引っ付けてstrに保存し 2 2022/05/30 18:21
- C言語・C++・C# leetcode 155 minstack 1 2022/05/07 16:43
- C言語・C++・C# c言語の問題の説明、各所ごとに 5 2023/07/26 11:03
- Excel(エクセル) 2つのVBAを一緒にしたら機能しなくなりました(エクセル) 7 2022/06/02 12:41
- C言語・C++・C# C言語で再起関数とポインタを用いて文字列反転をする方法がわかりません。 4 2023/04/29 20:32
- Excel(エクセル) エクセルVBAでオブジェクトが必要です 2 2022/09/10 16:37
- C言語・C++・C# バイナリファイルをコピーするのにかかる時間を測りたいのですが実行するとFatel error:gli 2 2022/11/03 01:10
- UNIX・Linux cronの@rebootでのdateコマンドの実行につきまして 2 2023/06/11 16:23
関連するカテゴリからQ&Aを探す
おすすめ情報
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
c言語のポインタへの文字列入力...
-
allocってなんですか?
-
newしないオブジェクトについて
-
16進ダンプのプログラム
-
Accessで、メモリを開放するタ...
-
stringの最大サイズ
-
デストラクタを呼びたい
-
配列の添え字の最大数とは?
-
ヒープ領域の限界値設定
-
mallocで確保するメモリの領域...
-
DLLで同じメモリ領域を参照する...
-
free関数で動作が止まる
-
DLLのマルチスレッドの動作につ...
-
アセンブラでのメモリの動的確...
-
行列内の行の交換,列の交換を...
-
C++で、メンバもヒープに確保さ...
-
freeで開放される範囲
-
確保という言葉の使い方について
-
構造体でchar name[]と*nameの...
-
ビットをローテートするプログ...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
c言語のポインタへの文字列入力...
-
allocってなんですか?
-
newしないオブジェクトについて
-
mallocについて
-
配列の添え字の最大数とは?
-
ヒープメモリの解放について
-
ビットをローテートするプログ...
-
C++で、メンバもヒープに確保さ...
-
malloc呼び出し時のセグメンテ...
-
プログラムが途中で強制終了し...
-
スタック破壊の上手な見つけ方...
-
指定したメモリアドレスの値の...
-
void*型のデータサイズ
-
ポインタのポインタの初期化法
-
MSDNがgethostbynameではなくge...
-
callocの処理速度
-
free関数で動作が止まる
-
C++のnewで確保したメモリーの...
-
sprintf()の使い方について
-
Accessで、メモリを開放するタ...
おすすめ情報