触发 COM 事件时 Lock()/Unlock() 调用的目的是什么？

Question

ATL COM 服务器中的一段典型代码触发事件如下所示（从this question 复制并稍作修改）：

HRESULT Fire_MessageTrigger()
{
    HRESULT hr = S_OK;
    T * pThis = static_cast<T *>(this);
    int count = m_vec.GetSize();

    for (int i = 0; i < count; i++)
    {
        pThis->Lock(); // I'm asking about this...
        CComPtr<IUnknown> punkConnection = m_vec.GetAt(i);
        pThis->Unlock(); // and this

        IDispatch* pConnection = static_cast<IDispatch *>(punkConnection.p);
        if (pConnection == 0)
            continue;

        DISPPARAMS params = { NULL, NULL, 0, 0 };
        hr = pConnection->Invoke(2, IID_NULL, LOCALE_USER_DEFAULT,
            DISPATCH_METHOD, &params, 0, NULL, NULL);
    }
    return hr;
}

Lock() 和 Unlock() 调用的目的是什么？

Answer 1

这是围绕m_vec 成员变量（接收器列表）的线程安全措施。

您可以在并发线程上执行以下操作：

template <class T, const IID* piid, class CDV>
STDMETHODIMP IConnectionPointImpl<T, piid, CDV>::Advise(
    _Inout_ IUnknown* pUnkSink,
    _Out_ DWORD* pdwCookie)
{
// ...
        pT->Lock();
        *pdwCookie = m_vec.Add(p); // <<--- Modifying m_vec here
        hRes = (*pdwCookie != NULL) ? S_OK : CONNECT_E_ADVISELIMIT;
        pT->Unlock();

您的下一个问题是为什么他们没有在访问m_vec.GetSize(); 时锁定上面的几行代码。

他们就是这样做的：在并发向量访问的情况下，他们可以错过单个事件传递，只要它不会造成内存损坏或其他未定义的行为。毕竟，订阅电话可能会在稍后发生，并且无论如何都会错过这个事件。这同样适用于取消订阅事件 (Unadvise) 的情况。

这里的优先级是尽可能少地锁定，然后尽快解锁。请注意，sink 调用是在解锁时发生的，并且 sink list 可以在我们迭代调用它们的 sink 列表时进行修改。

UPD。 Visual Studio 2008 总是生成Lock/Unlock 对。新版本也是如此。从 "\VC\VCWizards\CodeWiz\ATL\ImplementInterface\HTML\1033\default.htm" 文件：

strProxyMethod +=
    "\t\tfor (int iConnection = 0; iConnection < cConnections; iConnection++)\r\n"+
    "\t\t{\r\n"+
    "\t\t\tpThis->Lock();\r\n"+
    "\t\t\tCComPtr<IUnknown> punkConnection = m_vec.GetAt(iConnection);\r\n"+
    "\t\t\tpThis->Unlock();\r\n\r\n";

较早的 VS 版本可以生成无锁代码，并且代码只有在您从 VS IDE 手动重新生成代理类时才会更新。