класс File_pointer - это хороший C++?

Question

В моей программе на C++ много указателей на файлы и есть много операторов возврата.
Чтобы избежать выполнения инструкции return, не закрывая все открытые указатели файлов, я написал этот класс, исходя из предположения, что мне гарантировано, что все деструкторы объектов автоматически вызываются при достижении оператора return.

Является ли мое предположение истинным?
И хороший код?

Конструктор и деструктор в File_pointer.cpp

File_pointer::File_pointer(string filename) 
{ 
    fptr.open(filename.c_str()); 
}; 

File_pointer::~File_pointer() 
{ 
    fptr.close(); 
}; 

Пример реализации

int main() 
{ 
    File_pointer myfile("myfile.txt"); 
    int x = 2; 
    switch(x) 
    { 
     case 1:  return x; 
     case 2:  return x; 
     default: 
      break; 
    }; 

return 0; 

} 

Answer 1

Т его правда?

Да; все объекты с automatic storage duration имеют свои деструкторы, когда они выходят за рамки.

Для уточнения, автоматический объект - нестатический локальный объект, который не был динамически распределен.

И хороший код?

Это субъективно. Но то, что я скажу, заключается в том, что полагаться на определение области для автоматического вызова деструкторов - очень распространенный идиом C++, известный странным именем resource acquisition is initialisation (RAII).

Классы, подобные std::ofstream, уже используют RAII; их деструктор автоматически закрывает файл. Например, если ваш fptr - это std::ofstream, то ваш класс-оболочка совершенно лишний.

Answer 2

RAII, обязательно.

Или используйте специальные интеллектуальные указатели, если можете, до wrap resources. Всегда лучше тестироваться, чем все, что вы делаете сами, но просты.

Answer 3

я полагаю, что fptr является FILE указателем в вашем File_pointer классе

Хотя ваш подход близок к тому, что RAII парадигма, я должен указать, что вы не обрабатываете ошибки надлежащим образом.

например. Что делать, если fopen не удается?