如何更好地初始化不可创建的 COM 对象的引用计数器？

Question

我有一个 COM 接口，其中有一个返回对象的方法：

interface ICreatorInterface {
    HRESULT CreateObject( IObjectToCreate** );
};

关键是调用 ICreatorInterface::CreateObject() 是检索实现 IObjectToCreate 接口的对象的唯一方法。

在 C++ 中，我可以这样做：

 HRESULT CCreatorInterfaceImpl::CreateObject( IObjectToCreate** result )
 {
     //CObjectToCreateImpl constructor sets reference count to 0
     CObjectToCreateImpl* newObject = new CObjectToCreateImpl();
     HRESULT hr = newObject->QueryInterface( __uuidof(IObjectToCreate), (void**)result );
     if( FAILED(hr) ) {
         delete newObject;
     }
     return hr;
 }

或 这种方式

 HRESULT CCreatorInterfaceImpl::CreateObject( IObjectToCreate** result )
 {
     //CObjectToCreateImpl constructor sets reference count to 1
     CObjectToCreateImpl* newObject = new CObjectToCreateImpl();
     HRESULT hr = newObject->QueryInterface( __uuidof(IObjectToCreate), (void**)result );
     // if QI() failed reference count is still 1 so this will delete the object
     newObject->Release();
     return hr;
 }

区别在于引用计数器的初始化方式以及在 QueryInterface() 失败时如何实现对象删除。由于我完全控制了 CCreatorInterfaceImpl 和 CObjectToCreateImpl，所以我可以选择任何一种方式。

IMO 第一个变体更清晰 - 所有引用计数的东西都在一段代码中。我有监督什么吗？为什么第二种方法会更好？以上哪一个更好，为什么？

Answer 1

这两种变体都违反了 COM 的一个非常基本的原则

• 。除了 AddRef 之外，切勿对引用计数为零的 COM 对象调用任何方法。

否则会导致各种错误。简单地说，因为它阻止人们对对象进行完全合法的操作。就像把它们放入智能指针一样。智能指针将调用 AddRef，将计数设置为 1，然后 Release 将计数设置为 0 并导致对象自毁。

是的，我意识到 90% 的 QueryInterface 实现都没有这样做。但我也向您保证，有一些可以做到:)

我认为最简单的方法是在创建对象后立即调用 AddRef。这允许对象尽早表现得像普通 COM 对象一样。

我过去遇到过这个问题，并且编写了一个很好的小帮助器方法（假设该对象是在 ATL 中实现的）。

template <class T>
static 
HRESULT CreateWithRef(T** ppObject)
{
    CComObject<T> *pObject;
    HRESULT hr = CComObject<T>::CreateInstance(&pObject);
    if ( SUCCEEDED(hr) )
    {
        pObject->AddRef();
        *ppObject = pObject;
    }

    return hr; 
}

Answer 2

Raymond Chen 在他的博客上写了一篇相关文章：
引用计数为零的对象

Answer 3

我总是使用以下代码场景来创建返回的 com 对象，以避免内存问题。当然，这是可行的，因为我的对象在创建时引用计数 = 0。对我来说，这总是比尝试使用删除运算符处理错误条件更清晰。

 HRESULT CCreatorInterfaceImpl::CreateObject( IObjectToCreate** result )
 {
     //CObjectToCreateImpl constructor sets reference count to 0
     CObjectToCreateImpl* newObject = new CObjectToCreateImpl();

     newObject->AddRef();

     HRESULT hr = newObject->QueryInterface( __uuidof(IObjectToCreate), (void**)result);

     newObject->Release(); // release my addref, if QI succeeded it AddRef'd, if not the object is destroyed

     return hr; // return result from QI
 }