Schließen Sie die Rückgabewertprüfung

Vom comp.lang.c:

Die fclose () - Aufruf kann fehlschlagen, und sollte
werden Fehler überprüft, genauso gewissenhaft
wie alle anderen Datei-Operationen.
Klingt pedantisch, richtig? Falsch. In einem
ehemalige Leben, mein Unternehmen, Produkt
es geschafft, zu zerstören Kunden-Daten
durch das auslassen von ein überprüfen Sie für Fehler, wenn
schließen einer Datei. Die Sequenz ging
so etwas wie (paraphrasiert):

stream = fopen(tempfile, "w"); 
if (stream == NULL) ... 
    while (more_to_write) 
        if (fwrite(buffer, 1, buflen, stream) != buflen) ... 
fclose (stream); 

/* The new version has been written successfully.  Delete 
 * the old one and rename. 
 */ 
remove (realfile); 
rename (tempfile, realfile); 

Natürlich, was passiert war, dass
fclose() ran out of disk space versuchen
zu schreiben die letzten paar blocks von
Daten, also die `tempfile' wurde abgeschnitten
und unbrauchbar. Und da die fclose()
Fehler nicht erkannt, das Programm
ging Sie nach rechts weiter und zerstört die
beste erhaltene version der Daten in
zugunsten der beschädigten version. Und, wie
Murphy wollte es, dass die Opfer in
dieser bestimmte Vorfall wurde der
Verantwortlichen des Kunden
Abteilung, die person, mit der Befugnis
kaufen Sie mehr von unserem Produkt oder zu ersetzen
es mit einem Konkurrenzprodukt --
und, ' türlich, eine person, die bereits
unzufrieden mit uns aus anderen Gründen.

Es wäre eine Strecke zu sehen, alle
die daraus resultierenden Elend dieser einzigen
Unterlassung, aber es ist vielleicht einen Hinweis Wert
heraus, dass sowohl der Kunde als auch mein
ehemalige Unternehmen haben seit dem verschwunden
von der corporate ökologie.

ÜBERPRÜFEN SIE DIE FEHLER-CODES!

InformationsquelleAutor der Antwort

1. fclose() wird flush ungeschriebenen Ausgang (über fflush()) vor der Rückkehr, so dass der Fehler resultiert aus der zugrunde liegenden write() wird nicht berichtet fwrite() oder fprintf() Zeit, aber wenn Sie tun, die fclose(). Als Ergebnis werden alle Fehler, die write() oder fflush() erzeugen kann, generiert werden kann fclose().

2. fclose() auch nennen close() erzeugt werden können Fehler, die sich auf NFS-clients, wo die geänderte Datei gar nicht hochladen auf den remote-server bis close() Zeit. Wenn der NFS-server ist abgestürzt, dann werden die close() fehl, und so fclose() wird scheitern, wie gut. Dies wahr sein könnte, von anderen vernetzten Dateisystemen.

InformationsquelleAutor der Antwort geocar

Sagen wir, Sie sind die Erzeugung von Daten. Sie haben alte Daten, die Sie fread() aus einer Datei, und führen Sie dann einige Verarbeitung auf die Daten, generieren Sie mehr Daten, und dann schreiben Sie in eine neue Datei. Sie sind vorsichtig, um nicht die alte Datei überschreiben, weil Sie wissen, dass der Versuch, neues zu schaffen-Datei kann fehlschlagen, und Sie möchten, dass Ihre alten Daten, in diesem Fall (einige Daten sind besser als gar keine Daten). Nach Abschluss aller die fwrite()s, die alle erfolgreich sind (weil Sie akribisch überprüft den Rückgabewert von fwrite()), Sie fclose() Sie die Datei. Dann, Sie rename() Ihre just-Datei geschrieben und die alte Datei überschreiben.

Wenn fclose() ist fehlgeschlagen, da der write-Fehler (Datenträger voll?), Sie einfach überschrieben haben, Ihre Letzte gute Datei mit etwas, das könnte sein, junk. Oops.

So, wenn es wichtig ist, Sie sollten überprüfen Sie den Rückgabewert von fclose().

In Bezug auf code:

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

int main(void)
{
    FILE *ifp = fopen("in.dat", "rb");
    FILE *ofp = fopen("out.dat", "wb");
    char buf[BUFSIZ];
    size_t n;
    int success = 1;

    if (ifp == NULL) {
        fprintf(stderr, "error opening in.dat\n");
        perror("in.dat");
        return EXIT_FAILURE;
    }

    if (ofp == NULL) {
        fclose(ifp);
        fprintf(stderr, "error opening out.dat\n");
        perror("out.dat");
        return EXIT_FAILURE;
    }

    while ((n = fread(buf, 1, sizeof buf, ifp)) > 0) {
        size_t nw;
        if ((nw = fwrite(buf, 1, n, ofp)) != n) {
            fprintf(stderr, "error writing, wrote %lu bytes instead of %lu\n",
                            (unsigned long)n,
                            (unsigned long)nw);
            fclose(ifp);
            fclose(ofp);
            return EXIT_FAILURE;
        }
    }
    if (ferror(ifp)) {
        fprintf(stderr, "ferror on ifp\n");
        fclose(ofp);
        fclose(ifp);
        return EXIT_FAILURE;
    }

#ifdef MAYLOSE_DATA
    fclose(ofp);
    fclose(ifp);
    rename("out.dat", "in.dat"); /* Oops, may lose data */
#else
    if (fclose(ofp) == EOF) {
        perror("out.dat");
        success = 0;
    }
    if (fclose(ifp) == EOF) {
        perror("in.dat");
        success = 0;
    }
    if (success) {
        rename("out.dat", "in.dat"); /* Good */
    }
#endif
    return EXIT_SUCCESS;
}

In dem obigen code, haben wir vorsichtig über fopen()fwrite()und fread()aber selbst dann nicht die Kontrolle fclose() zu Datenverlust führen kann (bei der Kompilierung mit MAYLOSE_DATA definiert).

InformationsquelleAutor der Antwort Alok Singhal

Habe ich viele Male gesehen fclose() die Rückgabe null.

Und bei sorgfältiger Untersuchung fand heraus, dass das eigentliche problem war mit dem schreiben und nicht mit fclose.

Da die Sachen, die geschrieben werden gepuffert werden, bevor die eigentliche schreiben passiert, und wenn ein fclose() wird aufgerufen, alle Puffer geleert wird. Also, ein problem in schreiben der gepufferten suff, sagen, wie Festplatte voll, wird angezeigt, während der fclose(). Als David Yellen sagt, zu schreiben, ein Kugel-Beweis-Anwendung, die Sie benötigen, zu prüfen, den return-Wert von fclose().

InformationsquelleAutor der Antwort codaddict

Wenn Sie in den Kontext Ihrer Anwendung, können Sie denken, etwas sinnvolles zu tun, wenn fclose() fehlschlägt, testen Sie dann die return-Wert. Wenn Sie nicht können, nicht.

InformationsquelleAutor der Antwort