Was unterscheidet dieses gespeicherte Array von dem geladenen Array?

Question

Ich habe diese Funktion, die das gesamte Array ia in eine Datei ‚Dateinamen‘ in einem JSON Textdatei Array Dateiformat namens speichert:

int intarr_save_json(intarr_t* ia, const char* filename) 
{ 
    if(ia==NULL) 
    { 
     return 1; 
    } 
    int *arr=ia->data; 
    int n=ia->len; 
    int i; 
    FILE *p; 
    p=fopen(filename,"w"); 
    if(p!=NULL) 
    { 
     fprintf(p,"[\n"); 
     for(i=0;i<n;i++) 
     { 
      if(i!=n-1) 
      { 
       fprintf(p," %d,\n",arr[i]); 
      } 
      else 
      { 
       fprintf(p," %d\n",arr[i]); 
      } 
     } 
     fprintf(p,"]"); 
     fclose(p); 
     return 0; 
    } 
    return 1; 
} 

Und diese Funktion, lädt ein neues Array aus der Datei mit dem Namen ‚Dateinamen‘ , das war zuvor mit intarr_save() gespeichert.

intarr_t* intarr_load_json(const char* filename) 
{ 
    FILE* f = fopen(filename, "r"); 
    if (f == NULL) return NULL; 

    intarr_t* loaded = intarr_create(0); 

    int value; 
    //Get rid of [ 
    fscanf(f, "%c ", &value); 

    while (fscanf(f, "%d, ", &value)) { 
     if (value == ']') break; 
     intarr_push(loaded, value); 
    } 


    fclose(f); 
    return loaded; 
} 

Es scheint alles schön, aber aus irgendeinem Grunde, die Länge des geladenen Array ist nicht das gleiche wie die Länge des gespeicherten Array. Was könnte das verursachen?

EDIT: Jetzt ist meine Funktion ist wie dieser

int intarr_save_json(intarr_t* ia, const char* filename) 
{ 
    if(ia == NULL) 
    { 
     return 1; 
    } 
    int *arr=ia->data; 
    int n=ia->len; 
    int i; 
    FILE *p; 
    p=fopen(filename,"w"); 
    if(p!=NULL) 
    { 
     fprintf(p,"[\n"); 
     for(i=0;i<n;i++) 
     { 
      if(i!=n-1) 
      { 
       fprintf(p," %d,\n",arr[i]); 
      } 
      else 
      { 
       fprintf(p," %d\n",arr[i]); 
      } 
     } 
     fprintf(p,"]"); 
     fclose(p); 
     return 0; 
    } 
    return 1; 
} 

and 
intarr_t* intarr_load_json(const char* filename) 
{ 
    FILE* f = fopen(filename, "r"); 
    if (f == NULL) 
    { 
     return NULL; 
    } 

    intarr_t* loaded = intarr_create(0); 

    int value; 


    char line[100]; 
    while (fgets(line, 100, f)) 
    { 
     if (line[0] == ']') 
      { 
      break; 
      } 

     if (sscanf(line, "%d", &value) != 1) 
      { 
       break; 
      } 

    // Use the number 
    intarr_push(loaded, value); 
    } 

    fclose(f); 
    return loaded; 
} 

Aber jetzt ist die Last immer einen Null-Zeiger zurück. Was könnte das verursachen?

Answer 1

Die in

verwendete Logik

while (fscanf(f, "%d, ", &value)) { 
    if (value == ']') break; 

ist fehlerhaft.

Die letzten beiden Zeilen der Datei wird

<some number> 
] 

Der Aufruf von fscanf an dieser Linie scheitern. Daher werden Sie am Ende eine Nummer weniger lesen, als Sie in die Datei geschrieben haben.

Ich schlage vor, die Strategie ein wenig zu ändern.

// Read the contents of the file line by line 
// Process each line. 
char line[100]; 
while (fgets(line, 100, f)) 
{ 
    if (line[0] == ']') 
    { 
     break; 
    } 

    if (sscanf(line, "%d", &value) != 1) 
    { 
     break; 
    } 

    // Use the number 
    intarr_push(loaded, value); 
} 

Answer 2

Sie müssen einen nicht Leerzeichen nach jeder integer, lesen und aufhören zu lesen, wenn es sich um eine schließenden Klammer ']' statt Komma ',' ist. So sollte Ihre Leseschleife sein:

do { 
    char sep[2]; 
    int cr = fscanf(f, "%d%1s", &value, sep)) { /* read one value and the separator */ 
    if (cr != 2) { 
     /* signal the error, exit, ... */ 
    } 
    intarr_push(loaded, value); /* always push the value */ 
} while (sep[0]== ']') break; /* loop until the separator if a ] */