使用字符串常量作为锁去同步代码是非常危险的事情。请看下面的一段代码:
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包中的工具。
英文原文:http://www.javalobby.org/java/forums/t96352.html
http://www.blogjava.net/qujinlong123/