并发情况下synchronized死锁

存在缺陷的代码：

public class DataPropertyIdAndNameRepositoryImpl{
    /** 发布标志 */
    private volatile boolean                      publishFlag;

    public Integer getStandardId(int dataId, String propertyName) {
        if (!publishFlag) {
            loadToCache();
        }
        Integer standardId = 0;
        Map<String, Integer> propertyIdMap = propertyIdLocalCache.get(dataId);
        if (propertyIdMap != null) {
            standardId = propertyIdMap.get(propertyName);
        }
        return standardId;
    }

    public synchronized boolean loadToCache() {
        try {
            DataPropertyIdAndName dataPropertyIdAndName = dataPropertyIdAndNameService
                .queryDataPropertyIdAndName();
            publishFlag = true;
        } catch (Exception e) {
            publishFlag = false;
        }
        return publishFlag;
    }
}

存在缺陷的流程：

a. 绿色表示第一个线程，蓝色表示第二个线程。

b. 黄色模块的代码为synchronized标记的代码，并发情况下只会有一个线程执行此方法。

c. 绿色线程执行到紫色模块时，蓝色线程等待进入黄色模块。

d. 蓝色线程执行抛异常，导致publishFlag被置为false。

e. 此时再次有线程进入，判断publishFlag仍为false，因此导致重复不断加载loadToCache.

修复后的代码：

public class DataPropertyIdAndNameRepositoryImpl{
    /** 发布标志 */
    private volatile boolean                      publishFlag;

    public Integer getStandardId(int dataId, String propertyName) {
        if (!publishFlag) {
            loadToCache();
        }
        Integer standardId = 0;
        Map<String, Integer> propertyIdMap = propertyIdLocalCache.get(dataId);
        if (propertyIdMap != null) {
            standardId = propertyIdMap.get(propertyName);
        }

        return standardId;
    }

    public synchronized boolean loadToCache() {
        try {

            // 双检锁
            if (publishFlag) {
                return publishFlag;
            }

            DataPropertyIdAndName dataPropertyIdAndName = dataPropertyIdAndNameService.queryDataPropertyIdAndName();
            publishFlag = true;

        } catch (Exception e) {
            publishFlag = false;
        }

        return publishFlag;
    }
}