HashSet+ReadWriterLock并发测试失败，改用ConcurrentDictionary成功，求原因

核心问题：锁的使用错误导致竞态条件或集合损坏 #

1. 读锁转写锁的间隙存在竞态 #

这是最常见的错误：先加读锁判断元素是否存在，释放读锁后再加写锁执行修改。比如你的移除逻辑如果是这样：

public void Remove(Guid id)
{
    _rwLock.AcquireReaderLock(Timeout.Infinite);
    var exists = _hashSet.Contains(id);
    _rwLock.ReleaseReaderLock();

    if (exists)
    {
        _rwLock.AcquireWriterLock(Timeout.Infinite);
        _hashSet.Remove(id);
        _rwLock.ReleaseWriterLock();
    }
}

读锁释放到写锁获取的间隙中，其他线程可能已经移除该元素，或因并发修改HashSet内部结构导致数据异常，最终出现元素未被正确移除、集合状态混乱的情况。

2. 锁的范围未覆盖完整操作 #

如果任何对HashSet的操作（比如获取Count、遍历集合、后台线程修改）没有被锁包裹，就会触发HashSet的非线程安全问题。HashSet的内部哈希表在并发修改时会损坏，比如出现链表循环、元素丢失，直接导致测试结果不符合预期。

3. ReadWriterLock本身的局限性（或使用不当） #

ReadWriterLock是较老的锁实现，不支持可升级读锁，强行用读锁+写锁的组合极易出现竞态。另外，如果锁的释放逻辑不严谨（比如异常时忘记释放锁），会导致死锁或锁泄漏，进一步破坏并发执行流程。

ConcurrentDictionary为什么能稳定工作 #

ConcurrentDictionary是.NET原生线程安全集合，内部用分段锁实现细粒度并发控制，所有操作（TryAdd、TryRemove、ContainsKey）都是原子性的，无需手动加锁。比如移除操作可通过TryRemove原子完成“检查+移除”，完全避免读-写间隙的竞态问题。同时它内部处理了集合扩容、哈希冲突等场景的线程安全，不会出现内部结构损坏。

修复HashSet+锁实现的关键点 #

如果要保留HashSet方案，必须做到：

• 所有写操作（Add/Remove）全程持有写锁，包括元素存在性检查和修改
• 所有读操作（Contains/Count/遍历）全程持有读锁
• 推荐用ReaderWriterLockSlim替代ReadWriterLock，它支持可升级读锁，能安全从读锁升级到写锁，避免竞态

比如正确的移除实现：

// 使用ReaderWriterLockSlim的可升级读锁
public bool Remove(Guid clientId)
{
    _rwLockSlim.EnterUpgradeableReadLock();
    try
    {
        if (!_hashSet.Contains(clientId))
            return false;

        _rwLockSlim.EnterWriteLock();
        try
        {
            return _hashSet.Remove(clientId);
        }
        finally
        {
            _rwLockSlim.ExitWriteLock();
        }
    }
    finally
    {
        _rwLockSlim.ExitUpgradeableReadLock();
    }
}

内容的提问来源于stack exchange，提问作者Rayane Hindi