Lesen von strings mit Leerzeichen aus einer Datei

#include <stdio.h>

int main(void){
    char username[16];
    char text[512];
    int ch, i;
    FILE *my_file = fopen("inbox.txt", "r");

    while(1==fscanf(my_file, "%15s%*c", username)){
        i=0;
        while (i < sizeof(text)-1 && EOF!=(ch=fgetc(my_file))){
            if(ch == '\n' && i && text[i-1] == '\n')
                break;
            text[i++] = ch;
        }
        text[i] = 0;
        printf("user:%s\n", username);
        printf("text:\n%s\n", text);
    }
    fclose(my_file);
    return 0;
}

Können Sie fgets() leicht genug, solange du vorsichtig bist. Dieser code scheint zu funktionieren:

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

enum { MAX_MESSAGES = 20 };

typedef struct Message
{
    char dest[16]; 
    char text[512];
} Message;

static int read_message(FILE *fp, Message *msg)
{
    char line[sizeof(msg->text) + 1];
    msg->dest[0] = '\0';
    msg->text[0] = '\0';
    while (fgets(line, sizeof(line), fp) != 0)
    {
        //printf("Data: %zu <<%s>>\n", strlen(line), line);
        if (line[0] == '\n')
            continue;
        size_t len = strlen(line);
        line[--len] = '\0';
        if (msg->dest[0] == '\0')
        {
            if (len < sizeof(msg->dest))
            {
                memmove(msg->dest, line, len + 1);
                //printf("Name: <<%s>>\n", msg->dest);
            }
            else
            {
                fprintf(stderr, "Error: name (%s) too long (%zu vs %zu)\n",
                        line, len, sizeof(msg->dest)-1);
                exit(EXIT_FAILURE);
            }
        }
        else
        {
            if (len < sizeof(msg->text))
            {
                memmove(msg->text, line, len + 1);
                //printf("Text: <<%s>>\n", msg->dest);
                return 0;
            }
            else
            {
                fprintf(stderr, "Error: text for %s too long (%zu vs %zu)\n",
                        msg->dest, len, sizeof(msg->dest)-1);
                exit(EXIT_FAILURE);
            }
        }
    }
    return EOF;
}

int main(void)
{
    Message mbox[MAX_MESSAGES];
    int n_msgs;

    for (n_msgs = 0; n_msgs < MAX_MESSAGES; n_msgs++)
    {
        if (read_message(stdin, &mbox[n_msgs]) == EOF)
            break;
    }

    printf("Inbox (%d messages):\n\n", n_msgs);
    for (int i = 0; i < n_msgs; i++)
        printf("%d: %s\n   %s\n\n", i + 1, mbox[i].dest, mbox[i].text);

    return 0;
}

Dem Lesen von code handle (mehrere) Leerzeilen vor der ersten Namen, die zwischen einem Namen und dem text, und nachdem der Letzte name. Es ist etwas ungewöhnlich in der Sie, wie es sich entscheidet, ob zum speichern der Zeile, Lesen Sie einfach in der dest oder text Teile der Nachricht. Es nutzt memmove() weil er genau weiß, wie viele Daten zu verschieben, und die Daten null ist beendet. Sie könnte ersetzen Sie es mit strcpy() wenn Sie es vorziehen, aber es sollte langsamer sein (die wohl nicht messbar langsamer), weil strcpy() zum testen, jedes byte als Kopien, aber memmove() nicht. Ich benutze memmove() denn es ist immer richtig; memcpy() könnte hier verwendet werden, aber es funktioniert nur, wenn Sie garantieren, dass keine überlappung. Better safe than sorry; es gibt zahlreiche software-Fehler, ohne zu riskieren extras. Sie können sich entscheiden, ob der Fehler exit angemessen ist — es ist gut für test-code, aber nicht unbedingt eine gute Idee in den Produktions-code. Sie können entscheiden, wie Sie zu behandeln '0 Nachrichten' vs '1 Nachricht' vs '2 Nachrichten' etc.

Können Sie leicht überarbeiten Sie den code für die dynamische Zuweisung von Speicher für das array von Nachrichten. Es würde leicht sein, um die Nachricht zu Lesen, in eine einfache Message variable in main(), arrangieren und kopieren Sie in das dynamische array, wenn Sie bekommen eine vollständige Nachricht. Die alternative ist die 'Gefahr' über die Zuweisung der array, aber das ist unwahrscheinlich, dass ein großes problem (Sie würden nicht wachsen das array nur einen Eintrag sowieso zu vermeiden quadratischen Verhalten, wenn der Speicher bewegt werden, während jede Zuweisung).

Wenn es mehrere Felder bearbeitet werden, die für jede Nachricht (sagen, Eingangsdatum und Datum zu Lesen), dann würden Sie brauchen, um die Reorganisation der code einige mehr, wahrscheinlich mit einer anderen Funktion.

Beachten Sie, dass der code verhindert den reservierten Namensraum. Ein name wie _message ist reserviert für die 'Umsetzung'. Code wie dieser ist nicht Teil der Umsetzung (von der C-compiler und Ihre support-system), so sollte man nicht Namen erstellen, die mit einem Unterstrich beginnen. (Das über-vereinfacht die Einschränkung, aber nur leicht, und ist viel einfacher zu verstehen als die differenziertere version.)

Code ist vorsichtig, nicht zu schreiben keine Magische Zahl mehr als einmal.

Beispiel-Ausgabe:

Inbox (2 messages):

1: Marco
   How are you?

2: Tarma
   Are you going to the gym today?