Question

我正在使用Memcached存储数据以便快速访问。我已经读过创建MemcachedClient的成本很高，并且将MemcachedClient用作静态（参见：link）

所以我在我的客户端使用Singleton模式：

public class CommonObjectsCache
{
    private static CommonObjectsCache _cache;
    private static MemcachedClient _client;

    public static MemcachedClient Client
    {
        get
        {
            if (_client == null)
                _client = new MemcachedClient();

            return _client;
        }
        private set
        {
            _client = value;
        }
    }

    private CommonObjectsCache()
    {
        _client = new MemcachedClient();
    }

    public static CommonObjectsCache Cache
    {
        get
        {
            if (_cache == null)
                _cache = new CommonObjectsCache();

            return _cache;
        }
    }
}

在我的DAL中，我使用它们如下：

    public static List<Item1> AllItem1s
    {
        get
        {
            if (CommonObjectsCache.Client.Get<List<Item1>>("AllItem1s") == null)
                RefreshItem1Cache();

            return CommonObjectsCache.Client.Get<List<Item1>>("AllItem1s");
        }
        private set
        {
            CommonObjectsCache.Client.Store(StoreMode.Set, "AllItem1s", value);
        }
    }
    public static List<Item2> AllItem2s
    {
        get {  // Same as above }
        private set { // Same as above }
    }
    public static List<Item3> AllItem3s
    {
        get {  // Same as above }
        private set { // Same as above }
    }
    public static List<Item4> AllItem4s
    {
        get {  // Same as above }
        private set { // Same as above }

    }

并填写为：

public static void RefreshItem1Cache()
{
    List<Item1> items = (from i ctx.Item1
                        select i).ToList();
    AllItem1s = items;
}

在我的DAL代码中，我有一个方法：

public static MyModel GetMyModel(int? id)
{
    // I use AllItem1s here.
}

当我运行代码时，它有时会说AllItem1s.Count == 0，但是当我在AllItem1s中放置断点并诊断该值时，我发现它已被填充。所以，我更新了代码如下，以检查我是否做错了：

public static MyModel GetMyModel(int? id)
{
    if (AllItem1s == null || AllItem1s.Count == 0 || AllItem2s == null || AllItem2s.Count == 0 || AllItem3s == null || AllItem3s.Count == 0 || AllItem4s == null || AllItem4s.Count == 0)
    {
        string msg = "Error!!!!!";
    }
    // I use AllItem1s here.
}

令人惊讶的是，代码落到string msg = "Error!!!!!";阻止!!!

但是当我在if块内放置一个断点并注意每个集合的Count属性时，我发现它有数字。

所以我得出结论，在获取AllItemXs财产时存在竞争条件。当它检查条件时，其中至少有一个没有正确设置（没有意义，因为它们在同一个线程上，并且属性的getter不能返回空集合）。 / p>

任何人都可以解释为什么会发生这种情况以及如何克服这个问题吗？

Answer 1

您的单例实现不是线程安全的。想象一下，两个（或更多）线程同时命中空检查：两个（所有）线程将初始化它们自己的CommonObjectsCache实例。

您可以使用lock语句对实例进行空检查和初始化，也可以使用双重检查锁定模式。

只需谷歌在C＃中实现线程安全的单例实现。

使用静态MemcachedClient的问题

1 个答案: