Kann jemand den Segmentierungsfehler in diesem erklären?

Question

Ich habe ein Stück Code geschrieben, die eine Textdatei foo.txt liest, die von Format ist:

1 
2 
3 
4 

Es speichert die zuletzt gewählte Nummer und erhöht sie und schreibt sie zurück. Dies ist der Teil, wo ich die letzte Nummer speichern und der nächste Teil des Codes wird die Nummer in die Datei schreiben. Die Datei wird von ./readfile 100 foo.txt aufgerufen, wobei 100 die Anzahl angibt, mit der die letzte Nummer erhöht werden muss.

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

int main(int argc, char *argv[]) { 
    FILE *fp; 
    char ch[100]; 
    int i = 0; 
    int j = 0; 
    int count = atoi(argv[1]); 

    while (count != 0) { 
     fp = fopen(argv[2], "r"); 
     while (fp) { 
      ch[i] = fgetc(fp); 
      i++; 
      j = i; 
      printf("%d", j);  
     } 
     fclose(fp); 
     count--; 
    } 
    return 0; 
} 

Answer 1

Ihr Problem ist:

while (fp) 

A FILE* ist, in einen Booleschen Test, nur ein weiterer Zeiger. So lange es erfolgreich geöffnet wurde, wird dieser Test immer bestanden. Sie lesen Zeichen immer wieder und speichern sie in aufeinanderfolgenden Elementen des Arrays, bis Sie überlaufen und schließlich ganz in den nicht zugewiesenen Speicher schreiben (oder zufälligerweise etwas Wichtiges stampfen und auf andere Weise einen Absturz verursachen).

Sie wollen wirklich so etwas wie:

// Must read into int; EOF itself is not a char value 
    int inchar; 
    // Read and test for EOF 
    while((inchar = fgetc(fp)) != EOF) 
    { 
     ch[i] = inchar; // Store 
     i++; 
     printf("%d", inchar); // Print raw value returned by fgetc 
    } 

Answer 2

Nachdem Sie "während (fp)" Problem zu beheben. Du hast immer noch Probleme mit ch [i]. Mit Ihrem Code wird "i" auf 400 erhöht. Daher tritt ein Segmentierungsfehler auf.

while(fp) 
    { 
     ch[i] = fgetc(fp); 
     i++; 
     j = i; 
     printf("%d", j);  
    }