メモリ

質問

メモリについてご質問します。
例えば
char *p;
p = (char *)malloc(100);

とすればpに100バイトのメモリが割り当てられる・・
pに文字列を代入する場合100バイトまで格納可能である。

次にファイルからの一行のある部分の文字列を入れる場合
char *s;
while(fgets(buf,1000,fp) !=NULL){
s = GetWord(・・)//何らかの文字列が返ってくる関数です

とする場合このときのsについてもメモリを取得しなければなりません。
その際
s = (char *)malloc(100);
while(/*省略*/){

とwhileの外でとるのか、それとも
while(/*省略*/){
s = (char *)malloc(100);
と中でとるべきなのかどっちでしょう？

というのもwhile文の中でメモリを作成すると
１回ごとに新たなメモリをとっているのか
更新されていっているのかがわかりません。

回答No.3

麻野 なぎ（@AsanoNagi） · ベストアンサー

ちょっと話がずれますが、
s = (char *)malloc(100);
と、
s = GetWord(・・)//何らかの文字列が返っ
てくるは（もしもこの通りなら）連続して
使ってはいけません。

Ｃ／Ｃ＋＋には、文法的には、文字列とい
う型はなくて、多くの場合、char * という、
ポインタです。（string の場合もある）

ですから、GetWord() は、char *GetWord();
とchar * を返すと思いますが、そうすると、

s = (char *)malloc(100);
で、確保した領域（の先頭）をセットしても、
s = GetWord();
とした瞬間、malloc() で確保した領域は永
久にアクセスできなくなります。
ついでにいえば、この状態で、
free(s);
とすると、良くないことが起こるかもしれま
せん。（GetWord() の中で、malloc() してい
たらＯＫ、GetWord の static な領域を使っ
ていたらＮＧ）

もしくは、
s = (char *)malloc(100);
した領域に、strcpy(); などで、コピーする
必要があります。

さて、
s = (char *)malloc(100);
がループの中か外かは、
「ひとつ前のループの時、s に代入されたポ
インタは、どうなるか？」で決まります。

１回のループ毎に、どこかに、ポインタその
ものが保存されたりして、ひとつ前のループ
の時のポインタがを保持しておく必要がある
なら、ループの中で毎回確保する必要があり
ます。

ひとつ前の s に入っていた内容が、上書き
されてもいいなら、ループの外に置いておけ
ばいいでしょう。

もしかして、

最初、
char s[100];
で作っていたら、
s = GetWord();
でコンパイラに怒られたので、
char *s;
に変えてみた。
ということなら、上述の通り

・事前に s の領域を確保する必要はない
　（GetWord() 内部で確保された領域を
　　流用すればいい）
・strcpy で GetWord() の返却値を
　コピーする必要がある

のどちらかでしょう。

回答No.4

asuncion

＞このときのsについてもメモリを取得しなければなりません。

必ずしもそうとはいえません。
GetWordがchar型へのポインタを返すのであれば、
sのためにmallocする必要はありません。

回答No.2

nda23

何か認識が違う気がします。

>pに100バイトのメモリが割り当てられる
プログラム領域以外のメモリはシステム管理領域（ヒープと言う）で、
そこから100バイトのメモリが割り当てられます。mallocの戻り値は
その領域のアドレスであって、ポインタ型（極端に言えばlong型)なら
どんな変数に入れて使用してもかまいません。
あくまで、「ｐに」ではありません。

>s = GetWord(・・)//何らかの文字列が返ってくる
返ってくる文字列は誰が用意したメモリ上にあるものですか？
呼び出し元が用意した領域を返すなら、ループの外側で定義して、
使い回しをする方が効率的です。関数自体が領域を確保しているなら
ループの終わりで、毎回freeしなければなりません。

>while文の中でメモリを作成すると
mallocの度に新しいメモリが確保されます。解放しないで、mallocを
繰り返すと、ヒープをどんどん消費してしまいます。
更新されるのではありません。

>すると残りは90バイトでしょうか
何処に何バイト読み込むか指令するのはプログラマの責任です。
使用した部分だけ有効なのであって、「残り」を気にするのは
何か重大な勘違いをしていると思います。
ファイルの１行が最大99バイトまでなので100バイトの領域を確保した
というのが正しい考えで、１行読み込む度に有効長を調べながら処理
するのが正しい態度です。

回答No.1

rinkun

一回取った領域を使い回すならループの外で、ループのたびに新しい領域を使うならループの中で。
なお、どちらの場合も確保した領域は必要なくなったらfreeで開放しなければいけません。

補足

使いまわすとありますが例えば100バイトとって
最初の一行で10バイトを使用した
すると残りは90バイトでしょうか？