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

66 阅读 0 评论 44 点赞

我是靠谱客的博主无私果汁，最近开发中收集的这篇文章主要介绍不要使用字符串常量作为对象锁去同步代码，觉得挺不错的，现在分享给大家，希望可以做个参考。

概述

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

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

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包中的工具。