c++ - 在析构函数中调用 remove() 删除文件是否安全？

Question

我有一个类在调用某些成员函数时创建一些临时文件。我希望在类超出范围时(通常或由于异常)删除这些文件，所以我想在析构函数中删除它们:

#include <cstdio>
#include <string>
class MyClass
{
    //implementation details

    //Names of temp files
    std::string tempFile1, tempFile2,tempFile3;

    ~MyClass()
    {
         remove(tempFile1.c_str());
         remove(tempFile2.c_str());
         remove(tempFile3.c_str());
    }
};

问题是，如果由于异常调用析构函数，那么很可能不是所有的 3 个临时文件都已创建。根据cpluscplus.com ，在这种情况下，remove() 函数将返回一个非零值并向 stderr 写入一些内容。但因为是 C 函数，所以不会有异常。

我知道析构函数不应该抛出。像这样的错误怎么办？是否推荐编写这样的析构函数？

Answer 1

您所展示的将正常工作。但我通常更喜欢更 RAII 的方法，例如:

#include <cstdio>
#include <string>

struct RemovableFile
{
    std::string fileName;
    bool canRemove;

    RemovableFile() : canRemove(false) {}
    ~RemovableFile(){ if (canRemove) remove(fileName.c_str()); }
};

class MyClass
{
    ...
    //Names of temp files
    RemovableFile tempFile1, tempFile2, tempFile3;
    ...
};

void MyClass::doSomething()
{
    ...
    tempFile1.fileName = ...;
    ...
    if (file was created)
        tempFile1.canRemove = true;
    ...
};

或者更像这样:

#include <cstdio>
#include <string>
#include <fstream>

struct RemovableFile
{
    std::string  fileName;
    std::fstream file;

    ~RemovableFile() { if (file.is_open()) { file.close(); remove(fileName.c_str()); } }

    void createFile(const std::string &aFileName)
    {
        file.open(aFileName.c_str(), ...);
        fileName = aFileName;
    }
};

class MyClass
{
    ...
    //Names of temp files
    RemovableFile tempFile1, tempFile2, tempFile3;
    ...
};

void MyClass::doSomething()
{
    ...
    tempFile1.createFile(...);
    ...
};