更新对使用中的成员变量的引用

我得到了这个síngleton缓存对象，它公开了一个IEnumerable属性，它只返回一个私有的IEnumerable变量。更新对使用中的成员变量的引用

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

假设一些线程正在迭代此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 
    } 
}

来源

2008-09-22 Per Hornshøj-Schierbeck

当前正在枚举sektioner的线程将继续枚举它，即使您更新单例内的引用。实现IEnumerable的对象没有什么特别之处。

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

来源

2008-09-22 10:47:14

它正在枚举的实例不需要是新的（和易失性关键字）？我的意思是我让它IEnumerable所以它不能改变任何东西，或者我在这里错过了什么？ – 2008-09-22 10:49:30

由于吸气{ return sektioner; }被称为放在前场的新值，返回旧的值。然后，循环foreach (Sektion s in cache.Sektioner)使用调用getter时收到的值，即旧值。该值将在整个foreach循环中使用。

来源

2008-09-22 10:42:16 configurator

是啊，这就是我想太多，因为它会在任何其他情况下，不过这种特殊情况下提出了一个问题在审查我的对象期间，所以我想确定我是否没有错过任何东西。 – 2008-09-22 10:46:39

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

来源

2008-09-22 10:45:23 aku

锁定位于我的UpdateCache的'SNIP'部分:) – 2008-09-22 10:48:00

我想，如果你想有一个线程安全的，你应该用这样的方式：

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 
    //} 
}

来源

2008-09-22 12:05:54 TcKs

更新对使用中的成员变量的引用

回答

相关问题