C#:volatile读写HashSet

Question

我有一节课:

public class Checker
{
    private HashSet<int> _hs = new HashSet<int>();

    public bool Check(int a)
    {
        return Volatile.Read(ref _hs).Contains(a);
    }

    public void Update(IEnumerable<int> items)
    {
        Volatile.Write(ref _hs, new HashSet<int>(items));
    }
}

Check经常从多个线程调用方法.Update从单个线程调用方法,该线程监视某些源(数据库,http服务等).这种Volatile.Read / Volatile.Write用法是否正确？

Answer 1

如果你的意思是"将Check始终使用最新版本的字段",那么是的,作为波动的副作用就是这种情况 - 并且交换整个引用要比不断同步便宜得多(.NET确保你不能有一个撕裂的引用,所以引用交换保证是原子的).

注意:此场景中的线程安全性严格依赖于哈希集在创建和交换引用后未发生变异的事实,这就是问题代码中发生的情况.

但是,通过将该字段声明为volatile以下内容,您可以更方便地获得相同的结果:

public class Checker
{
    private volatile HashSet<int> _hs = new HashSet<int>();

    public bool Check(int a) => _hs.Contains(a);

    public void Update(IEnumerable<int> items) => _hs = new HashSet<int>(items);
}