C liest Datei Zeile für Zeile

Question 1

Ich habe diese Funktion geschrieben, um eine Zeile aus einer Datei zu lesen:

const char *readLine(FILE *file) {

    if (file == NULL) {
        printf("Error: file pointer is null.");
        exit(1);
    }

    int maximumLineLength = 128;
    char *lineBuffer = (char *)malloc(sizeof(char) * maximumLineLength);

    if (lineBuffer == NULL) {
        printf("Error allocating memory for line buffer.");
        exit(1);
    }

    char ch = getc(file);
    int count = 0;

    while ((ch != '\n') && (ch != EOF)) {
        if (count == maximumLineLength) {
            maximumLineLength += 128;
            lineBuffer = realloc(lineBuffer, maximumLineLength);
            if (lineBuffer == NULL) {
                printf("Error reallocating space for line buffer.");
                exit(1);
            }
        }
        lineBuffer[count] = ch;
        count++;

        ch = getc(file);
    }

    lineBuffer[count] = '\0';
    char line[count + 1];
    strncpy(line, lineBuffer, (count + 1));
    free(lineBuffer);
    const char *constLine = line;
    return constLine;
}

Die Funktion liest die Datei korrekt, und mit printf sehe ich, dass die constLine-Zeichenfolge auch korrekt gelesen wurde.

Wenn ich die Funktion jedoch zB so verwende:

while (!feof(myFile)) {
    const char *line = readLine(myFile);
    printf("%s\n", line);
}

printf gibt Kauderwelsch aus. Wieso den?

Question 2

Wenn Ihre Aufgabe nicht darin besteht, die zeilenweise Lesefunktion zu erfinden, sondern nur die Datei zeilenweise zu lesen, können Sie ein typisches Code-Snippet mit verwenden getline() Funktion (siehe Handbuchseite hier):

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

int main(void)
{
    FILE * fp;
    char * line = NULL;
    size_t len = 0;
    ssize_t read;

    fp = fopen("/etc/motd", "r");
    if (fp == NULL)
        exit(EXIT_FAILURE);

    while ((read = getline(&line, &len, fp)) != -1) {
        printf("Retrieved line of length %zu:\n", read);
        printf("%s", line);
    }

    fclose(fp);
    if (line)
        free(line);
    exit(EXIT_SUCCESS);
}

Question 3

FILE* filePointer;
int bufferLength = 255;
char buffer[bufferLength]; /* not ISO 90 compatible */

filePointer = fopen("file.txt", "r");

while(fgets(buffer, bufferLength, filePointer)) {
    printf("%s\n", buffer);
}

fclose(filePointer);

Question 4

In deiner readLine Funktion geben Sie einen Zeiger auf die zurück line array (Genau genommen ein Zeiger auf sein erstes Zeichen, aber der Unterschied ist hier irrelevant). Da es sich um eine automatische Variable handelt (dh „auf dem Stack“), wird der Speicher zurückgefordert, wenn die Funktion zurückkehrt. Sie sehen Kauderwelsch, weil printf hat seine eigenen Sachen auf den Stapel gelegt.

Sie müssen einen dynamisch zugewiesenen Puffer von der Funktion zurückgeben. Sie haben bereits eine, es ist lineBuffer; Alles, was Sie tun müssen, ist es auf die gewünschte Länge zu kürzen.

    lineBuffer[count] = '\0';
    realloc(lineBuffer, count + 1);
    return lineBuffer;
}

HINZUGEFÜGT (Antwort auf die Folgefrage im Kommentar): readLine gibt einen Zeiger auf die Zeichen zurück, aus denen die Zeile besteht. Diesen Zeiger benötigen Sie, um mit dem Inhalt der Zeile zu arbeiten. Es ist auch das, wozu Sie übergehen müssen free wenn Sie den von diesen Charakteren belegten Speicher verbraucht haben. So können Sie die verwenden readLine Funktion:

char *line = readLine(file);
printf("LOG: read a line: %s\n", line);
if (strchr(line, 'a')) { puts("The line contains an a"); }
/* etc. */
free(line);
/* After this point, the memory allocated for the line has been reclaimed.
   You can't use the value of `line` again (though you can assign a new value
   to the `line` variable if you want). */

Question 5

//open and get the file handle
FILE* fh;
fopen_s(&fh, filename, "r");

//check if file exists
if (fh == NULL){
    printf("file does not exists %s", filename);
    return 0;
}


//read line by line
const size_t line_size = 300;
char* line = malloc(line_size);
while (fgets(line, line_size, fh) != NULL)  {
    printf(line);
}
free(line);    // dont forget to free heap memory

Question 6

Vollständig, fgets() Lösung:

#include <stdio.h>
#include <string.h>

#define MAX_LEN 256

int main(void)
{
    FILE* fp;
    fp = fopen("file.txt", "r");
    if (fp == NULL) {
      perror("Failed: ");
      return 1;
    }

    char buffer[MAX_LEN];
    while (fgets(buffer, MAX_LEN, fp))
    {
        // Remove trailing newline
        buffer[strcspn(buffer, "\n")] = 0;
        printf("%s\n", buffer);
    }

    fclose(fp);
    return 0;
}

Ausgabe:

First line of file
Second line of file
Third (and also last) line of file

Denken Sie daran, wenn Sie aus der Standardeingabe lesen möchten (und nicht wie in diesem Fall aus einer Datei), müssen Sie nur übergeben stdin als dritter Parameter von fgets() Methode, wie folgt:

while(fgets(buffer, MAX_LEN, stdin))

_Anhang

Entfernen des nachgestellten Newline-Zeichens aus der fgets()-Eingabe

wie man erkennt, dass eine Datei geöffnet ist oder nicht in c

Question 7

readLine() gibt Zeiger auf lokale Variable zurück, was zu undefiniertem Verhalten führt.

Um herumzukommen, können Sie:

Erstellen Sie eine Variable in der Aufruferfunktion und übergeben Sie ihre Adresse an readLine()

Speicher zuweisen für line verwenden malloc() – in diesem Fall line wird hartnäckig sein
Verwenden Sie globale Variablen, obwohl dies im Allgemeinen eine schlechte Praxis ist

Question 8

Verwenden fgets() um eine Zeile aus einem Dateihandle zu lesen.