C-Funktion, die Zeilen in Datei zählt

Question

Wenn ich versuche, mein Programm auszuführen, bekomme ich die falsche Anzahl von Zeilen gedruckt.

LINES: 0 

Dies ist die Ausgabe obwohl ich in meiner .txt-Datei

Hier mein Programm fünf Linien haben, ist:

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

int countlines(char *filename); 

void main(int argc, char *argv[]) 
{ 
    printf("LINES: %d\n",countlines(argv[1]));   
} 


int countlines(char *filename) 
{ 
    // count the number of lines in the file called filename          
    FILE *fp = fopen(filename,"r"); 
    int ch=0; 
    int lines=0; 

    if (fp == NULL); 
    return 0; 

    lines++; 
    while ((ch = fgetc(fp)) != EOF) 
    { 
     if (ch == '\n') 
    lines++; 
    } 
    fclose(fp); 
    return lines; 
} 

Ich bin sicher, dass es ein einfacher Fehler ist, aber ich bin neu auf Programmierung. Jede Hilfe würde sehr geschätzt werden.

Answer 1

while(!feof(fp)) 
{ 
    ch = fgetc(fp); 
    if(ch == '\n') 
    { 
    lines++; 
    } 
} 

Aber bitte beachten Sie: Why is “while (!feof (file))” always wrong?.

Answer 2

Sie erklären

int countlines(char *filename) 

ein char * Argument zu nehmen.

Sie nennen es wie dieses

countlines(fp) 

* in einer Datei übergeben.

Deshalb erhalten Sie diesen Kompilierungsfehler.

Sie sollten wahrscheinlich, dass die zweite Zeile zu

ändern

countlines("Test.txt") 

, da Sie die Datei in öffnen countlines

Ihre aktuellen Code die Datei an zwei verschiedenen Orten zu öffnen versucht.

Answer 3

Sie öffnen eine Datei und übergeben den Dateizeiger dann an eine Funktion, die nur einen Dateinamen zum Öffnen der Datei benötigt. Sie können Ihren Anruf vereinfachen;

void main(void) 
{ 
    printf("LINES: %d\n",countlines("Test.txt")); 
} 

EDIT: Sie ändern die Frage um, so ist es sehr schwer zu beantworten; Zuerst hast du deine Änderung an main() falsch gemacht, du hast vergessen, dass der erste Parameter argc ist, also ist er abgestürzt. Jetzt hast du das Problem von;

die immer 0 zurückgeben. Entfernen Sie das zusätzliche Semikolon, und Sie sollten eine angemessene Anzahl erhalten.

Answer 4

Ich sehe nichts sofort offensichtlich, was einen Segmentierungsfehler verursachen würde. Mein einziger Verdacht ist, dass Ihr Code erwartet, dass Sie einen Dateinamen als Parameter erhalten, wenn Sie ihn ausführen, aber wenn Sie ihn nicht übergeben, wird er trotzdem versuchen, einen zu referenzieren.

Zugriff auf argv[1], wenn es nicht existiert würde einen Segmentierungsfehler verursachen. Es empfiehlt sich in der Regel, die Anzahl der Argumente zu überprüfen, bevor Sie versuchen, sie zu referenzieren. Sie können dies tun, indem Sie den folgenden Funktionsprototyp für main() verwenden und überprüfen, dass argc größer als 1 ist (es wird einfach die Anzahl der Einträge in argv anzeigen).

int main(int argc, char** argv) 

Der beste Weg, um herauszufinden, was eine segfault im Allgemeinen verursacht, ist ein Debugger zu verwenden. Wenn Sie in Visual Studio sind, legen Sie einen Haltepunkt oben in der Hauptfunktion und wählen Sie Führen Sie mit dem Debuggen anstelle von "Ausführen ohne Debuggen" aus, wenn Sie das Programm starten. Es stoppt die Ausführung oben und lässt Sie Zeile für Zeile weiterlaufen, bis Sie ein Problem sehen.

Wenn Sie in Linux sind, können Sie einfach die Kerndatei (es wird "Kern" im Namen haben) und laden Sie diese mit gdb (GNU Debugger). Es kann Ihnen einen Stapelspeicherabzug geben, der Sie direkt zu der Zeile führt, die den Segmentierungsfehler verursacht hat.

EDIT: Ich sehe, Sie haben Ihre Frage und Code geändert. Also diese Antwort ist wahrscheinlich nicht mehr nützlich, aber ich werde es verlassen, da es sowieso ein guter Rat ist, und sehen, ob ich die modifizierte Frage (kurz) ansprechen kann.

Answer 5

Sie haben eine; am Ende der if. Wechsel:

if (fp == NULL); 
    return 0; 

zu

if (fp == NULL) 
    return 0; 

Answer 6

Hier ist meine Funktion

char *fileName = "input-1.txt"; 
countOfLinesFromFile(fileName); 

void countOfLinesFromFile(char *filename){ 
FILE* myfile = fopen(filename, "r"); 
int ch, number_of_lines = 0; 
do 
{ 
    ch = fgetc(myfile); 
    if(ch == '\n') 
     number_of_lines++; 
} 
while (ch != EOF); 
if(ch != '\n' && number_of_lines != 0) 
    number_of_lines++; 
fclose(myfile); 
printf("number of lines in %s = %d",filename, number_of_lines); 

}