Question

我是否需要使用锁（createPaletteLocker）？我正在使用它，因为我认为这样做可以避免写入ColorPaletteHandle.palettesList [type]而其他线程正在读取它。

编辑1：字典仅在静态构造函数中设置。因此我不会再改变（添加/删除），但它保持的姿势将从null更改为相应的创建调色板。

public static class ColorPaletteHandle
{
    private static readonly object createPaletteLocker = new object();
    private static Dictionary<ColorPaletteType, ColorPalette> palettesList = null;

    static ColorPaletteHandle()
    {
        palettesList = new Dictionary<ColorPaletteType, ColorPalette>();
        palettesList.Add(ColorPaletteType.Default, ColorPaletteHandle.defaultPalette);
        palettesList.Add(ColorPaletteType.EdgesHighlight, ColorPaletteHandle.edgesHighlight);
        palettesList.Add(ColorPaletteType.GrayScale, ColorPaletteHandle.grayScale);
        palettesList.Add(ColorPaletteType.HeatMap, ColorPaletteHandle.heatMap);
    }

    /// <summary>
    /// Gets a palette.
    /// </summary>
    /// <param name="type"></param>
    /// <returns></returns>
    public static ColorPalette GetPalette(ColorPaletteType type)
    {
        ColorPalette pal = null;
        lock (ColorPaletteHandle.createPaletteLocker)
        {
            pal = ColorPaletteHandle.palettesList[type];
            if (pal == null)
            {
                ColorPaletteHandle.palettesList[type] = ColorPaletteHandle.CreatePalette(type);
                pal = ColorPaletteHandle.palettesList[type];
            }
        }
        return pal;
    }

   //stuff...
   }

Answer 1

Dictionary类不是为了从多个线程访问而设计的。除非您同步访问，否则这样做确实会导致问题。使用lock语句，方法很好，没有它就不行。

使用lock甚至比使用ConcurrentDictionary更好。它是一个专门设计用于同时从多个线程使用的字典。您可以将整个方法重构为对GetOrAdd的一次调用。

Answer 2

是的，这个方法是线程安全的，因为一次只有一个线程可以修改palettesList，是的，如果你打算使用Dictionary

，你确实需要锁定

但是，如果您使用的是.NET 4或更高版本，ConcurrentDictionary将更适合此

静态类上的静态方法，这个方法线程安全吗？

2 个答案: