ファイルからの読み込みに失敗しました

Question

ファイルから文字列を読み込みました。関数が2回目または3回目のように実行された後、バッファー文字列に1つ以上のランダムな文字が追加され、なぜそれが起こるのかわかりません。ここで

は、コードの一部です：

scorefile = fopen("highscore.dat", "rb"); 

if (scorefile) 
{ 
    fseek(scorefile, 0, SEEK_END); 
    length = ftell(scorefile); 
    fseek(scorefile, 0, SEEK_SET); 
    buffer = malloc(length); 
    if (buffer) 
    { 
     fread(buffer, 1, length, scorefile); 
    } 
    fclose(scorefile); 
} 

私はここで何か間違ったことをやっていますか？

Answer 1

buffer = malloc(length);を使用してlengthバイトを読み取ると、1バイトが短すぎます。 Cの文字配列はゼロ終端なので、余分なバイトが必要ですが、そのゼロは必要です。 buffer = malloc(length+1);これを修正します。

Answer 2

さんはそれをすべて綴ると少しより堅牢に行こう：

#include <stdio.h> 
#include <stdlib.h> 
#include <assert.h> 

char *loadScoreFile(const char *filename) 
{ 
    char *buffer = NULL; 

    FILE *scorefile = fopen(filename, "r"); 

    if (scorefile != NULL) 
    { 
     (void) fseek(scorefile, 0, SEEK_END); 
     int length = ftell(scorefile); 

     (void) fseek(scorefile, 0, SEEK_SET); 

     buffer = malloc(length + 1); 

     if (buffer != NULL) 
     { 
      assert(length == fread(buffer, 1, length, scorefile)); 

      buffer[length] = '\0'; 
     } 
     (void) fclose(scorefile); 
    } 

    return buffer; 
} 

int main() 
{ 
    for (int i = 0; i < 10; i++) 
    { 
     char *pointer = loadScoreFile("highscore.dat"); 

     if (pointer != NULL) 
     { 
      printf("%s", pointer); 
      free(pointer); 
     } 
    } 

    return 0; 
}