Question

我得到了这个síngleton缓存对象，它暴露了一个IEnumerable属性，它只返回一个私有的IEnumerable变量。

我的singleton对象上有一个静态方法，用于更新此成员变量（存在于此缓存对象的单个“Instance”实例上）。

假设我的缓存正在更新时，某个线程正在迭代此IEnumerable变量/属性。我做了它，所以缓存正在更新一个新的局部变量，最后设置暴露的私有变量指向这个新的局部变量。

我知道我只是更新一个引用，将其他（旧）对象留在内存中等待GC接收但我的问题是 - 我不是100％肯定一旦我设置了新引用会发生什么？另一个线程是否突然迭代新对象或它通过IEnumerable接口传递的旧对象？如果这是一个正常的参考我会说'不'。调用线程将在旧对象上运行，但我不确定IEnumerable是否也是如此？

这是剥离的类：

internal sealed class SektionCache : CacheBase
{
    public static readonly SektionCache Instance = new SektionCache();
    private static readonly object lockObject = new object();
    private static bool isUpdating;

    private IEnumerable<Sektion> sektioner;

    static SektionCache()
    {
        UpdateCache();
    }

    public IEnumerable<Sektion> Sektioner
    {
        get { return sektioner; }
    }

    public static void UpdateCache()
    {
    // SNIP - getting data, locking etc.
    Instance.sektioner = newSektioner;
    // SNIP
    }
}

Answer 1

由于在将新值放入字段之前调用getter { return sektioner; }，因此返回旧值。然后，循环foreach (Sektion s in cache.Sektioner)使用调用getter时收到的值，即旧值。该值将在整个foreach循环中使用。

Answer 2

当你在单例中更新引用时，当前枚举sektioner的线程将继续枚举它。实现IEnumerable的对象没什么特别的。

您可能应该将volatile关键字添加到sektioner字段，因为您没有提供读锁定，并且多个线程正在读/写它。

Answer 3

首先我看不到对象锁定，未使用的lockObject变量让我很伤心。 IEnumerable并不特别。每个线程都有自己的sektioner对象实例的引用副本。你不能以这种方式影响其他线程。 sektioner字段指向的旧版数据会发生什么，主要取决于主叫方。

Answer 4

我认为，如果你想要线程安全，你应该这样使用：

internal sealed class SektionCache : CacheBase
{
    //public static readonly SektionCache Instance = new SektionCache();

    // this template is better ( safer ) than the previous one, for thread-safe singleton patter >>>
    private static SektionCache defaultInstance;
    private static object readonly lockObject = new object();
    public static SektionCach Default {
        get {
            SektionCach result = defaultInstance;
            if ( null == result ) {
                lock( lockObject ) {
                    if ( null == result ) {
                        defaultInstance = result = new SektionCache();
                    }
                }
            }

            return result;
        }
    }
    // <<< this template is better ( safer ) than the previous one

    //private static readonly object lockObject = new object();
    //private static bool isUpdating;
    //private IEnumerable<Sektion> sektioner;

    // this declaration is enough
    private volatile IEnumerable<Sektion> sektioner;

    // no static constructor is required >>>
    //static SektionCache()
    //{
    //    UpdateCache();
    //}
    // <<< no static constructor is required

    // I think, you can use getter and setter for reading & changing a collection
    public IEnumerable<Sektion> Sektioner {
        get {
            IEnumerable<Sektion> result = this.sektioner;
            // i don't know, if you need this functionality >>>
            // if ( null == result ) { result = new Sektion[0]; }
            // <<< i don't know, if you need this functionality
            return result;
        }
        set { this.sektion = value; }
    }

    //public static void UpdateCache()
    //{
    //// SNIP - getting data, locking etc.
    //Instance.sektioner = newSektioner;
    //// SNIP
    //}
}

更新对正在使用的成员变量的引用

4 个答案: