3

参照カウントの目的で、以下のサンプルコードのようにstd :: tr1 :: shared_ptrを使用するのは安全で正しいですか？（これは単なる特定のサンプルであり、クラスにはFILE *の代わりに他のもの（void *）を含めることができます）

class File
{
public:
    File(const char* path, const char* mode) :
        _refcount(new int(0))
    {
        this->_file = fopen(path, mode);
    }

    ~File()
    {
        if (this->_refcount.unique())
        {
            if (this->_file != NULL)
            {
                fclose(this->_file);
            }
        }
    }

    int write(void* buff, size_t size)
    {
        fwrite(buff, size, 1, this->_file);
    }

private:
    FILE* _file;
    std::tr1::shared_ptr<int> _refcount;
};
4

1 に答える 1

9

代わりにshared_ptr<FILE>、カスタム削除機能でを使用することを検討してください。

struct fclose_deleter
{
    void operator()(FILE* f)
    {
        if (f)
        {
            std::fclose(f);
        }
    }
};

そうすれば、Fileクラスははるかに単純になります（そして修正されます）。

class File
{
public:
    File(const char* path, const char* mode)
        : _file(std::fopen(path, mode), fclose_deleter())
    {
    }

    int write(void const* buff, size_t size)
    {
        // You'll want to verify that _file.get() is valid, or you'll want to
        // throw in the constructor if the call to 'std::fopen()' fails.
        std::fwrite(buff, size, 1, _file.get());
    }

private:
    std::tr1::shared_ptr<FILE> _file;
};
于 2012-01-10T07:13:06.693 に答える