synchronized块使用string遇到的问题

上文中，将脚本代码生成md5指纹作为锁的时候，使用的是synchronized (key.intern())，而不是synchronized (key)。

String key = fingerKey(scriptText);
        Class<Script> script = zlassMaps.get(key);
        if (script == null) {
            synchronized (key.intern()) {
                // Double Check
                script = zlassMaps.get(key);
                if (script == null) {
                    GroovyClassLoader classLoader = new GroovyClassLoader();
                    script = classLoader.parseClass(scriptText);
                    zlassMaps.put(key, script);
                }
            }
        }

之所以这样处理，是因为synchronized (key)并不会生效。

我们先实现一个测试方法

private static final ConcurrentMap<String, String> KEYMAP = new ConcurrentHashMap<String, String>();

    public static String getKey(String keyId) throws Exception {
        String content = KEYMAP.get(keyId);
        if (null == content) {
            content = loadKey(keyId);
        } else {
            System.out.println("时间+--" + System.currentTimeMillis() + "，" + Thread.currentThread().getName() + "Map已获取，直接返回");
        }
        return content;
    }

    public static String loadKey(String keyId) throws Exception {
        synchronized (keyId) {
            String content = KEYMAP.get(keyId);

            if (null != content) {
                System.out.println("时间：" + System.currentTimeMillis() + "，" + Thread.currentThread().getName() + "【进入同步块直接返回】。。。。");
                return content;
            }
            System.out.println("时间：" + System.currentTimeMillis() + "，" + Thread.currentThread().getName() + "【开始执行】。。。。");
            Thread.sleep(1000);
            System.out.println("时间：" + System.currentTimeMillis() + "，" + Thread.currentThread().getName() + "id" + keyId + "获取缓存为" + "1");
            KEYMAP.put(keyId, "1");
            return content;
        }
    }

    public static void main(String aa[]) {
        for (int i = 0; i < 10; i++) {
            new Thread("线程-" + i + "-") {
                @Override
                public void run() {
                    try {
                        Test.getKey(new String("1"));
                    } catch (Exception e) {
                        e.printStackTrace();
                    }
                }
            }.start();
        }
    }

根据输出结果来看，多个线程同时进入了同步块，synchronized (key)并没有起到作用。

原因

synchronized在作用在同一个对象上才会生效，从该方法来看，每次String都是”1”，因此认为属于同一对象，synchronized会生效。

但是，实际上测试就可以发现并不是，因为String类重写了equals和hashcode方法。如果使用System.identityHashCode(key)，可以发现这10个‘keyid’真正的hashcode却是不同，所以在根本上来说并不是同一个对象。 因此synchronized (key)并不会起到作用。

解决方法

synchronized (key.intern())

String#intern是一个 native 的方法，如果常量池中存在当前字符串, 就会直接返回当前字符串. 如果常量池中没有此字符串, 会将此字符串放入常量池中后,再返回，因此String#intern每次返回同一个对象。