Question

我想在地图初始化时只创建一次锁。这是我要使用的代码。

public static Map<String, String> getOrderStatusInstance() {
    if (orderStatusMap == null) {
        synchronized (Utility.class) {
            orderStatusMap = new HashMap<String, String>();

            orderStatusMap.put("Key1", "Value1");
            orderStatusMap.put("Key2", "Value2");
            orderStatusMap.put("Key3", "Value3");
            orderStatusMap.put("Key4", "Value4");
        }
    }

    return orderStatusMap;
}

Answer 1

不,这是一个坏主意 - 您正在返回一个可变的、非铁面安全地图。您也正在< em> 尝试 < / em> 来执行双重检查的锁定, 但没有正确完成它 - 并且要纠正它比使用静态初始化要困难得多。

我会创建一个不可改变的地图(使用“http://guava-librarys.googlecode.com>>Guava ,以换取偏好),

private static final ImmutableMap<String, String> STATUS_MAP = ImmutableMap.of(
    "Key1", "Value1",
    "Key2", "Value2",
    "Key3", "Value3",
    "Key4", "Value4");

public static ImmutableMap<String, String> getOrderStatusInstance() {
    return STATUS_MAP;
}

(关于5个以上的钥匙/价值配对,请使用immobleMap.Builder 。)

您是否真的需要 需要来做任何懒惰的吗?

Answer 2

几乎正确... 想想一个线索检查命令 StatusMap =Null and gets true. 然后排程器切换到另一个线条做相同的检查。没有两个线条会执行同步块。

使用同步区块中的 null 来防止此检查 :

if (orderStatusMap == null) {
    synchronized (Utility.class) {
        if (orderStatusMap == null) {
            Map<String, String> tmp = new HashMap<String, String>();

            tmp.put("Key1", "Value1");
            tmp.put("Key2", "Value2");
            tmp.put("Key3", "Value3");
            tmp.put("Key4", "Value4");

            orderStatusMap = tmp;
        }
    }
}
return orderStatusMap;

是的,这样做两次是正常的。外部检查仍然有助于业绩,因为地图创建后,进入同步区块的昂贵步骤就不需要再做了。

记住这是一个创建 hashmap 的线索安全方法。它不能使地图的线索安全。

P.S.:如果你喜欢这个问题,你也可以想:

Answer 3

您应该对 同步 区块内的 null 进行另一次检查。

当两个线索呼唤您的方法时, 他们都会发现 < code> order StatusMap 是空的, 其中一个会进入 < code> 同步区块, 而另一个会屏蔽。但最终会通过同步区块, 并重新初始化地图。

Answer 4

否,不正确

此处的懒惰初始化也无需 < em> / em (我认为这已被过度使用) 。

private static final Map<String, String> orderStatusMap;
static{
    orderStatusMap = Collections.synchronizedMap(new HashMap<String, String>());//making it thread safe
    //or use a ConcurrentHashMap

    orderStatusMap.put("Key1", "Value1");
    orderStatusMap.put("Key2", "Value2");
    orderStatusMap.put("Key3", "Value3");
    orderStatusMap.put("Key4", "Value4");
}

public static Map<String, String> getOrderStatusInstance() {
    return orderStatusMap;
}

友情链接