C++ リファレンスの fread の例

StackOverflow https://stackoverflow.com/questions/493650

  •  20-08-2019
  •  | 
  •  

質問

私は C コードを記述するときに、Web サイト www.cplusplus.com を参照としてよく使用します。

のページで引用されている例を読んでいました。 恐れる と質問がありました。

例として、彼らは次のように投稿しています。

/* fread example: read a complete file */
#include <stdio.h>
#include <stdlib.h>

int main () {
  FILE * pFile;
  long lSize;
  char * buffer;
  size_t result;

  pFile = fopen ( "myfile.bin" , "rb" );
  if (pFile==NULL) {fputs ("File error",stderr); exit (1);}

  // obtain file size:
  fseek (pFile , 0 , SEEK_END);
  lSize = ftell (pFile);
  rewind (pFile);

  // allocate memory to contain the whole file:
  buffer = (char*) malloc (sizeof(char)*lSize);
  if (buffer == NULL) {fputs ("Memory error",stderr); exit (2);}

  // copy the file into the buffer:
  result = fread (buffer,1,lSize,pFile);
  if (result != lSize) {fputs ("Reading error",stderr); exit (3);}

  /* the whole file is now loaded in the memory buffer. */

  // terminate
  fclose (pFile);
  free (buffer);
  return 0;
}

result != lSize の場合、free(buffer) は決して呼び出されないように思えます。この例ではメモリ リークになるでしょうか?

彼らのサイトのサンプルは非常に質が高いといつも思っています。おそらく私は正しく理解していませんか？

役に立ちましたか？

解決

（exit()を呼び出すことによって）プログラムを終了すると、それに関連付けられたすべてのメモリを解放するため、

は、メモリリークこの例でのではない。

あなたがサブルーチンとして、コードのこの部分を使用し、return 1;の代わりにexit()のようなものを呼び出した場合しかし、それはメモリリークになります。

他のヒント

技術的には、はい、それはメモリリークです。そのプロセスが終了すると、この例では自由（とfcloseの）への呼び出しが本当に必要とされていないので、しかし、プロセスによって割り当てられたすべてのメモリは自動的に、解放されます。

より複雑なプログラムでは、これはおそらく本当の問題だろう。メモリリークを作成し無料行方不明と行方不明FCLOSEは、リソースリークが発生します。

そのプロセスを閉じたときに、オペレーティング・システムは、プロセスによって任意の未解放メモリをクリーンアップします。少なくとも、の現代のオペレーティング・システムが行う。

！それは保証メモリリークです。

-

プログラムは、小数点の結果で出ていなかった場合= LSIZEは、つまり、それは[はい、実行の他のいくつかのパスを継続しました。

2つの可能なパスがあります。

（1）RESULT = LSIZE！ - この場合には、出口（0）と呼ばれています。これは、プロセスを強制終了し、オペレーティングシステムがメモリをクリーンアップします。

（2）結果は== LSIZE - これは、プロセスとオペレーティングシステム意志、もう一度、クリーンを殺すため、この場合には、バッファを明示的に解放されますが、リターンが右後で呼び出されたので、自由はほとんどがちょうど良いスタイルですメモリ最大ます。

だから、この単純なケースでは、何のメモリリークはありません。しかし、おそらく、あなたが書く任意のアプリケーションに割り当てられたすべてのメモリを解放していることを確認することをお勧めします。この習慣を身に取得することは、将来的にあなたのための多くの頭痛を防ぐことができます。

メモリリークの可能性については、他の人がすでにその質問に答えています。しばらく前に、指定されたコードのバリエーションを投稿しました。 すべき 考えられるすべてのエラー状態を正しく処理します。

ライセンス： CC-BY-SA と 帰属
所属していません StackOverflow
scroll top