Synchronized String

直接上代码

    private static Map<String, String> insMap = Maps.newHashMap();

    public static String test(String host, int port) {
        String key = String.format("%s:%s", host, port);
        synchronized (key) {
            if (insMap.containsKey(key)) {
                return insMap.get(key);
            }
            insMap.put(key, "test");
            System.out.println(Thread.currentThread().getName() + " IN");
            return "test";
        }
    }

    public static void main(String[] args) {
        for (int i = 0; i < 1000; i++) {
            new Thread(new Runnable() {
                @Override
                public void run() {
                    test("a", 123);
                }
            }).start();
        }
         }

test方法为了防止同一个key写入Map，加了一个锁，希望能达到同一个key不put两次的结果 ，但是能达到我们期望的结果么,运行多次会有如下的结果：


Thread-60 IN
Thread-3 IN

坏了，不对，怎么两个线程进去了，仔细想想，String.format是StringBuilder实现的，而StringBuilder.toString()实现如下


    @Override
    public String toString() {
        // Create a copy, don't share the array
        return new String(value, 0, count);
    }

每次都是new的，自然锁不住了，那怎么修改呢，这样会好么

        String key = host + ":" + port;

还是不行，一个样的，+号底层也是StringBuilder实现的，还是锁Map吧，或者锁key.intern()