是否值得在queryinterface()实现中检查空指针?

你(呼叫者)不需要检查指针是否不存在 NULL .

但是,您应该检查返回的 HRESULT . 方法将返回 E_POINTER 如果输出指针是 无效的 和 E_NOINTERFACE 如果接口不受支持。

被叫方应检查指针是否 无效的 并返回 指针 如果是 无效的 :

MSDN 返回值:

此方法返回 S_OK 如果支持接口,并且 ENO接口 否则。如果 ppvObject 是 无效的 ,此方法返回 指针 .

根据 MSDN docs ,queryinterface返回s_ok,在这种情况下,out参数将被正确设置。或者返回e_nointerface,在这种情况下,不会设置out参数。

如果传入的void**为空,则返回e_指针。

我不需要检查空值,而是检查iunknown::queryinterface的返回值。

检查空值可能没有什么坏处,但是考虑到接口的保证,它看起来像是一个冗余检查。

这取决于你是什么样的COM对象(或者你是哪个应用程序的宿主)。大多数情况下,只要检查hresult就足够了。如果您正在处理第三方对象(资源管理器替换等),您可能还应该检查是否为空。资源管理器会这样做,因此,如果您想避免在错误扩展中崩溃,还需要这样做:

• The ways people mess up IUnknown::QueryInterface
• The real cost of compatibility is not in the hacks; the hacks are small potatoes