持久内存缓存中的单例模式

Question

使用我在Implementing the Singleton Pattern in C# 惊人的文章中判断为世界上最好的东西，我一直成功地使用以下类将用户定义的数据持久保存在内存中（对于很少修改的数据）：

public class Params
{
  static readonly Params Instance = new Params();

  Params()
  {
  }

  public static Params InMemory
  {
    get
    {
      return Instance;
    }
  }

  private IEnumerable<Localization> _localizations;

  public IEnumerable<Localization> Localizations
  {
    get
    {
      return _localizations ?? (_localizations = new Repository<Localization>().Get());
    }
  }

  public int ChunkSize
  {
    get
    {
      // Loc uses the Localizations impl
      LC.Loc("params.chunksize").To<int>();
    }
  }

  public void RebuildLocalizations()
  {
    _localizations = null;
  }

  // other similar values coming from the DB and staying in-memory,
  // and their refresh methods

}

我的用法如下所示：

var allLocs = Params.InMemory.Localizations; //etc

每当我更新数据库时，都会调用 RefreshLocalizations，因此只有部分内存存储被重建。我有一个单个 生产环境（大约 10 个），当调用 RefreshLocalizations 时，似乎表现不正常，根本不刷新，但这似乎也是间歇性的，非常完全奇怪。

我目前的怀疑是单例，我认为它做得很好，所有的单元测试都证明单例机制、刷新机制和 RAM 性能都按预期工作。

也就是说，我认为有这些可能性：

1. 这位客户说他们的环境没有使用负载平衡是在撒谎，这是我不希望内存中的东西正常工作的设置（对吗？）
2. 我正在测试他们的 IIS 中的一些非标准池配置（可能在 Web Garden 设置中？）
3. 单例以某种方式失败，但不确定如何。

有什么建议吗？

.NET 3.5 所以没有太多可用的并行果汁，目前还没有准备好使用反应式扩展

Edit1：根据建议，吸气剂看起来像：

public IEnumerable<Localization> Localizations
{
  get
  {
    lock(_localizations) {
      return _localizations ?? (_localizations = new Repository<Localization>().Get());
    }
  }
}

Answer 1

为了扩展我的评论，您可以通过以下方式使 Localizations 属性线程安全：

public class Params
{
  private object _lock = new object();

  private IEnumerable<Localization> _localizations;    
  public IEnumerable<Localization> Localizations
  {
    get
    {
      lock (_lock) {
         if ( _localizations == null ) {
            _localizations = new Repository<Localization>().Get();
         }

         return _localizations;
      }
    }
  }

  public void RebuildLocalizations()
  {
     lock(_lock) {
        _localizations = null;
     }
  }

  // other similar values coming from the DB and staying in-memory,
  // and their refresh methods

}

Answer 2

如果你的属性不是线程安全的，那么创建线程安全的单例是没有意义的。

您应该锁定_localization 字段的分配，或者在您的单例构造函数中实例化（首选）。任何适用于单例实例化的建议都适用于这个惰性实例化的属性。

同样的事情还适用于Localization 的所有属性（及其属性）。如果这是一个 Singleton，则意味着任何线程都可以随时访问它，而简单地锁定 getter 将再次无济于事。

例如，考虑这种情况：

线程 1 线程 2 // 两个线程都访问单例，但是你是“安全的”，因为你锁定了 1. var loc1 = Params.Localizations; var loc2 = 参数。本地化； // 做事 // 线程 2 调用相同的属性... 2. 变量值 = loc1.ChunkSize; var chunk = LC.Loc("params.chunksize"); // 无效 // ...这里有一个轻微的停顿... 3. loc1.RebuildLocalizations(); // ...并得到错误的值 4. var value = chunk.To();

如果您只是阅读这些值，那可能并不重要，但您可以看到使用这种方法很容易遇到麻烦。

请记住，使用线程，您永远不知道不同的线程是否会在两条指令之间执行某些操作。 只有简单的 32 位赋值是原子的，没有别的。

这意味着，在此行中：

return LC.Loc("params.chunksize").To<int>();

就线程而言，相当于：

var loc = LC.Loc("params.chunksize");
Thread.Sleep(1); // anything can happen here :-(
return loc.To<int>();

任何线程都可以在Loc 和To 之间跳转。