不要使用字符串常量作为对象锁去同步代码

不要使用字符串常量作为对象锁去同步代码(翻)

 使用字符串常量作为锁去同步代码是非常危险的事情。请看下面的一段代码：

class Foo  {
    static private final String LOCK = "LOCK";
    void someMethod() {
      synchronized(LOCK) {
      ...
      }
    }
  }

 为什么这样会有危险呢？私有的字符串常量是真正的私有吗？答案是否定的。
  重新阅读一下Java语言规范2.0的第3.10.5节，“不同包不同类中的值相同的字符串常量引用的是同一个字符串对象。”
  上面的那段代码意味着外部任何的Class都可以包含指向同一个字符串对象的字符串常量，因此就有可能出现死锁的情况！如果你的字符串对象调用了intern()方法也有可能会出现这种情况！
  这种情况在Jetty类库中真正的出现过，两个部分使用了相同的字符串常量去同步临界区代码，那两个代码片段产生了死锁现象，并且出现了莫名其妙的栈的跟踪信息。（Jetty-Bug已经记录了这个Bug，Jetty-352）
  如果你确实需要一个对象锁的话，最好使用new Object()来创建。也可以考虑使用java.util.concurrent包中的工具。