Czy muszę ręcznie zamknąć ifstream?

Question

Czy muszę ręcznie dzwonić pod numer close(), gdy używam numeru std::ifstream?

Na przykład, w kodzie:

std::string readContentsOfFile(std::string fileName) { 

    std::ifstream file(fileName.c_str()); 

    if (file.good()) { 
     std::stringstream buffer; 
     buffer << file.rdbuf(); 
     file.close(); 

     return buffer.str(); 
    } 
    throw std::runtime_exception("file not found"); 
} 

Czy muszę zadzwonić file.close() ręcznie? Czy nie powinno być ifstream używać do zamykania plików RAII?

Answer 1

NO

To, co jest dla RAII, niech destructor wykonywać swoje zadania. Nie ma nic złego w ręcznym zamykaniu go, ale nie jest to metoda C++, to programowanie w C z klasami.

Jeśli chcesz zamknąć plik przed zakończeniem funkcji, zawsze możesz użyć zagnieżdżonego zakresu.

W standardzie (27.8.1.5 Szablon klasy basic_ifstream), ifstream ma zostać zaimplementowany z elementem basic_filebuf z rzeczywistym uchwytem pliku. Jest przechowywany jako element, więc gdy obiekt typu ifstream ulega zniszczeniu, wywołuje on również destruktor pod numerem basic_filebuf. I od normy (27.8.1.2), to destruktor zamyka plik:

virtual ˜basic_filebuf();

efekty: niszczy obiekt klasy basic_filebuf<charT,traits>. Połączenia close().

Answer 2

Czy chcesz zamknąć plik?
NIE

Czy należy zamknąć plik?
Zależy.

Czy interesują Cię możliwe warunki błędu, które mogą wystąpić, jeśli plik nie zostanie poprawnie zamknięty? Pamiętaj, że bliskie połączenia setstate (failbit), jeśli się nie powiedzie. Destruktor wywoła automatycznie funkcję close() z powodu: RAII, ale nie pozostawi ci możliwości testowania bitu błędu, ponieważ obiekt już nie istnieje.

Answer 3

Nie, jest to wykonywane automatycznie przez destruktor ifstream. Jedynym powodem, dla którego powinieneś wywołać to ręcznie, jest to, że instancja fstream ma duży zakres, na przykład jeśli jest to zmienna składowa długiej instancji klasy żywej.

Answer 4

Zgadzam się z @Martin. Jeśli napiszesz do pliku, dane mogą nadal znajdować się w buforze i mogą nie zostać zapisane w pliku do czasu wywołania close(). Nie robiąc tego ręcznie, nie masz pojęcia, czy wystąpił błąd, czy nie. Brak zgłaszania błędów użytkownikom jest bardzo złą praktyką.