如何减少冗余InterlockedCompareExchange调用及共享资源安全访问

Q1 答案 #

该写法完全正确。
InterlockedCompareExchangePointer()的核心语义是原子完成三个操作：读取目标地址的原始值、比较原始值与传入的比较值、相等则将目标地址的值替换为交换值，最终始终返回目标地址的原始值。当你传入交换值和比较值均为nullptr时，整个操作不会修改目标指针的值，本质就是对目标指针做一次原子读取，完全符合联锁变量的读取要求。
C28112警告并非误报：直接读取共享的多线程变量属于非原子访问，在弱内存序平台、编译器开启优化的场景下，可能出现读取值撕裂、访问指令重排等问题，确实存在安全隐患，使用联锁接口读取是合规的解决方案。

Q2 答案 #

你遇到的安全问题和可读性问题的核心原因是重复多次读取共享变量，解决思路非常简单：仅做一次原子读取，将结果存到线程私有局部变量中，后续所有校验、操作都基于这个局部变量执行即可。
局部变量不会被其他线程修改，既不会出现两次读取结果不一致导致的空指针异常，也能彻底消除重复的类型转换和联锁调用代码。

优化方案

1. 先封装通用的联锁读取指针的内联函数，消除重复样板代码：

template<typename T>
inline T* InterlockedReadPointer(T* volatile* pTargetAddr)
{
    return static_cast<T*>(
        InterlockedCompareExchangePointer(
            reinterpret_cast<void* volatile*>(pTargetAddr),
            nullptr,
            nullptr
        )
    );
}

2. 业务代码中仅调用一次读取接口，缓存结果到局部变量：

// 仅原子读取一次，后续操作全部使用局部变量pQueue
auto pQueue = InterlockedReadPointer(&m_pCurrentKeyEventQueue);
if (pQueue == nullptr)
{
    break;
}
if (pQueue->Enqueue(EKeyEvent::MouseLeft_Down) != Container::EQueueResult::Success)
{
    break;
}

额外注意

以上方案仅解决指针读取的原子性问题，如果你的业务逻辑中存在「其他线程可能在你读取到非空指针后，直接释放队列对象」的场景，还需要额外搭配引用计数、读写锁等机制保护队列对象的生命周期，仅靠原子读取指针无法避免对象被释放后访问的野指针问题。

内容的提问来源于stack exchange，提问作者YoonSeok OH