Lesen von Datei fehlschlägt

Question

Ich lese eine Zeichenfolge aus einer Datei. Nach dem zweiten oder dritten Mal, wenn die Funktion ausgeführt wird, werden ein oder mehrere zufällige Zeichen an die Pufferzeichenfolge angehängt und ich habe keine Ahnung, warum dies geschieht.

Hier ist das Stück Code:

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); 
} 

Bin ich hier etwas falsch zu machen?

Answer 1

Wenn Sie buffer = malloc(length); verwenden und dann length Bytes darin lesen, wird es ein Byte zu kurz sein. Char-Arrays in C sind Null-terminiert, so dass sie ein zusätzliches Byte bis zu dieser Null benötigen. buffer = malloc(length+1); wird das beheben.

Answer 2

wir buchstabieren sie alle aus und gehen etwas robuster:

#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; 
}