是否有任何理由反对在 QueryInterface() 实现中直接调用 AddRef() ？

发布于 2024-09-01 07:14:50 字数 790 浏览 6 评论 0原文

在 C++ 中实现 IUnknown::QueryInterface() 时，有一些关于指针操作的注意事项。例如，当类实现多个接口（多重继承）显式向上转换是必要的：

class CMyClass : public IInterface1, public IInterface2 { 
}; 

//inside CMyClass::QueryInterface():
if( iid == __uuidof( IUnknown ) ) { 
     *ppv = static_cast<IInterface1*>( this ); // upcast in order to properly adjust the pointer
     //call Addref(), return S_OK 
}

在多重继承场景中，向上转换的原因非常清楚。然而，我也时不时地看到以下内容：

static_cast<IUnknown*>( *ppv )->AddRef();

而不是简单地从 QueryInterface() 实现内部调用 AddRef() 。

我是否应该对之前复制到 ppv 中的值进行转换，而不是仅仅在当前对象上调用 AddRef() ？

原文

When implementing IUnknown::QueryInterface() in C++ there're several caveats with pointers manipulation. For example, when the class implements several interfaces (multiple inheritance) explicit upcasts are necessary:

class CMyClass : public IInterface1, public IInterface2 { 
}; 

//inside CMyClass::QueryInterface():
if( iid == __uuidof( IUnknown ) ) { 
     *ppv = static_cast<IInterface1*>( this ); // upcast in order to properly adjust the pointer
     //call Addref(), return S_OK 
}

The reason for upcast is quite clear in multiple inheritance scenarios. However every here and there I also see the following:

static_cast<IUnknown*>( *ppv )->AddRef();

instead of simply invoking AddRef() from inside QueryInterface() implementation.

Is there any reason I should do the cast of the value previously copied into ppv instead of just calling AddRef() on the current object?

分享到QQ

分享到微博

如果你对这篇内容有疑问，欢迎到本站社区发帖提问参与讨论，获取更多帮助，或者扫码二维码加入 Web 技术交流群。

发布评论

需要登录才能够评论，你可以免费注册一个本站的账号。

守不住的情 2024-09-08 07:14:50

AddRef 在 IUnknown 中是纯虚拟的，其他接口都没有实现它，因此程序中唯一的实现就是您在 CMyClass 中编写的实现>。该方法会覆盖 IInterface1::AddRef 和 IInterface2::AddRef 两者。 IUnknown 没有任何数据成员（例如引用计数），因此菱形问题不会导致您的类容易受到诸如对 AddRef 作用于同一类中的不同数据。

对 this->AddRef() 的调用将被路由到与 static_cast(*ppv)->AddRef() 相同的位置。我认为没有理由采用更冗长的风格。

回复收藏 0 原文

旧伤慢歌 2024-09-08 07:14:50

语法冗长的原因可能在于 ppv 中不仅可能返回 this，这在较长的 if-else- 中很容易被忽视 -如果。来自 Don Box 的 Essential COM 的示例：

STDMETHODIMP Car::IternalQueryInterface(REFIID riid, void **ppv) {
    if (riid == IID_IUnknown)
        *ppv = static_cast<IUnknown*>(&m_innerUnknown);
    else if (riid == IID_IVehicle)
        *ppv = static_cast<IVehicle*>(this);
    else if (riid == IID_ICar)
        *ppv = static_cast<ICar*>(this);
    else
        return (*ppv = 0), E_NOINTERFACE;
    ((IUnknown*)*ppv)->AddRef();
    return S_OK;
}

The reason for the verbose syntax probably lies in the fact that not only this might be returned in ppv, which is easy to overlook in a longer if-else-if. An example from Don Box's Essential COM:

STDMETHODIMP Car::IternalQueryInterface(REFIID riid, void **ppv) {
    if (riid == IID_IUnknown)
        *ppv = static_cast<IUnknown*>(&m_innerUnknown);
    else if (riid == IID_IVehicle)
        *ppv = static_cast<IVehicle*>(this);
    else if (riid == IID_ICar)
        *ppv = static_cast<ICar*>(this);
    else
        return (*ppv = 0), E_NOINTERFACE;
    ((IUnknown*)*ppv)->AddRef();
    return S_OK;
}

回复收藏 0 原文

~没有更多了~