1

我的代码中有很多接口，我想将重复Release代码封装在另一个方法中，而不是宏中，因为这是 C++，我讨厌使用宏。我最初的尝试是编写一个方法，例如

void SafeRelease(IUnknown **ppInterface) {
    if(*ppInterface) {
        (*ppInterface)->Release();
        (*ppInterface) = nullptr;
    }
}

但是，将此方法应用于IDirect3DSurface9 *例如类似会SafeRelease(&mySurface)产生错误IDirect3DSurface9 **与IUnknown **.

  1. 我在这里做错了什么？
  2. 有没有更好的方法（希望不使用宏）来实现这样的功能？
4

3 回答 3

1

这是我的方法：

template <typename T> void SafeRelease(T*& ptr)
{
    if(ptr)
    {
        ptr->Release();
        ptr = nullptr;
    }
}

示例用法：

IDirect3DDevice9 *pD3DDevice = NULL;
d3d->CreateDevice(..., &pD3DDevice);
SafeRelease(pD3DDevice);

如果需要，您可以inline使用此功能。

于 2013-02-04T22:20:14.743 回答
0

您可以使用模板：

template<class DXInterface>
void SafeRelease(DXInterface **ppInterface) {
    if(*ppInterface) {
        (*ppInterface)->Release();
        (*ppInterface) = nullptr;
    }
}

您还可以使用 std::unique_ptr 或 std::shared_ptr 自动清理：

#include <memory>
#include <iostream>

struct Releaser {
   template<class DXInterface>
   void operator()(DXInterface *pInterface) const {
       if(pInterface) {
           pInterface->Release();
       }
   }
};

// For illustrative purposes only (supplied in DX9 headers)
struct IDirect3DSurface9 { void Release() { std::cout << "Released surface\n";} };
struct IDirect3DTexture9 { void Release() { std::cout << "Released texture\n";} };

void DX9CreateSurface( IDirect3DSurface9** surface ) 
{ 
    *surface = new IDirect3DSurface9();
}

void DX9CreateTexture( IDirect3DTexture9** texture ) 
{ 
    *texture = new IDirect3DTexture9();
}

// Your factory functions
IDirect3DSurface9* createSurface( /*init params go here*/ )
{
    IDirect3DSurface9* surface;
    DX9CreateSurface( &surface );
    return surface;
}

IDirect3DTexture9* createTexture( /*init params go here*/ )
{
    IDirect3DTexture9* texture;
    DX9CreateTexture( &texture );
    return texture;
}

int main()
{
  typedef std::unique_ptr<IDirect3DSurface9, Releaser> SurfacePtr;
  typedef std::unique_ptr<IDirect3DTexture9, Releaser> TexturePtr;

  SurfacePtr surface( createSurface() );
  TexturePtr texture( createTexture() );
  // ... use surface and texture here
  // Automatically released here when their lifetimes ends.
}

请注意，它们使用相同的 Releaser，并注意对 surface.reset() 的调用也会释放接口并将 unique_ptr 内的指针设置为 null 以启动。这两个对象可能是您的类的成员，而不是 main() 中的对象。

于 2013-02-04T21:53:27.077 回答
0

我在这里做错了什么？

我也有同样的问题，也针对 COM SafeRelease。所以这里是：

void SafeRelease(IUnknown **ppInterface) 
...
IDirect3DSurface9 * mySurface = new ...
...
SafeRelease(&mySurface);

IDirect3DSurface9 *，凭借继承，可以转换为IUnknown *. 但是，违反直觉的是，IDirect3DSurface9 **不能强制转换为IUnknown **. 如果允许，那么在您的内部，SafeRelease(IUnknown**)您可以执行以下操作：

// obtain a pointer to an instance of some random subinterface of IUnknown
*ppInterface = pMyRamdomComInterfacePointer;

因此，我们将在指向 的指针中存储了指向某个随机IUnknown导数的指针IDirect3DSurface9。这将违反 C++ 类型系统。这就是为什么不允许强制转换除T**to以外的任何其他类型的原因。T**换句话说，一个类型的变量T**只能被分配一个ppT（一个类型的值T**）而不是一个ppSomeSubytpeOfT。

比较一下：为什么不能将派生类的指针传递给期望引用基类指针的函数？而这个：铸造基类的双指针。

对于 COM SafeRelease，模板（如这里建议的）或宏都可以。

于 2014-05-02T15:17:42.120 回答