问如何在Java中同步String对象？

EN

有一个webapp，我正在进行一些负载/性能测试，特别是在我们希望几百个用户访问同一页面并在此页面上每10秒钟点击刷新的功能上。我们发现我们可以使用此功能进行改进的一个方面是在一段时间内缓存来自Web服务的响应，因为数据没有变化。

在实现这个基本缓存之后，在一些进一步的测试中，我发现我没有考虑并发线程如何同时访问Cache。我发现在大约100ms的时间内，大约有50个线程试图从Cache中获取对象，发现它已经过期，命中Web服务以获取数据，然后将对象放回缓存中。

原始代码看起来像这样：

private SomeData[] getSomeDataByEmail(WebServiceInterface service, String email) {

  final String key = "Data-" + email;
  SomeData[] data = (SomeData[]) StaticCache.get(key);

  if (data == null) {
      data = service.getSomeDataForEmail(email);

      StaticCache.set(key, data, CACHE_TIME);
  }
  else {
      logger.debug("getSomeDataForEmail: using cached object");
  }

  return data;
}

因此，为了确保在对象key到期时只有一个线程正在调用Web服务，我认为我需要同步Cache get / set操作，并且似乎使用缓存键对于一个对象来说是一个很好的候选者。同步（这样，对电子邮件b@b.com的此方法的调用不会被方法调用a@a.com阻止）。

我将方法更新为如下所示：

private SomeData[] getSomeDataByEmail(WebServiceInterface service, String email) {


  SomeData[] data = null;
  final String key = "Data-" + email;

  synchronized(key) {      
    data =(SomeData[]) StaticCache.get(key);

    if (data == null) {
        data = service.getSomeDataForEmail(email);
        StaticCache.set(key, data, CACHE_TIME);
    }
    else {
      logger.debug("getSomeDataForEmail: using cached object");
    }
  }

  return data;
}

我还为“同步块之前”，“内部同步块”，“即将离开同步块”和“同步块之后”之类的内容添加了日志行，因此我可以确定是否有效地同步了get / set操作。

然而，这似乎并没有起作用。我的测试日志输出如下：

（日志输出是'threadname''记录器名''消息'） http-80-Processor253 jsp.view-page - getSomeDataForEmail：即将进入同步块 http-80-Processor253 jsp.view-page - getSomeDataForEmail：内部同步块 http -80-Processor253 cache.StaticCache - get：key at [SomeData-test@test.com]已过期 http-80-Processor253 cache.StaticCache - get：key [SomeData-test@test.com]返回值[null] http-80-Processor263 jsp.view-page - getSomeDataForEmail：即将进入同步块 http-80-Processor263 jsp.view-page - getSomeDataForEmail：内部同步块 http-80-Processor263 cache.StaticCache - get：object at key [SomeData -test@test.com]已过期 http-80-Processor263 cache.StaticCache - get：key [SomeData-test@test.com]返回值[null] http-80-Processor131 jsp.view-page - getSomeDataForEmail：即将进入同步块 http-80-Processor131 jsp .view-page - getSomeDataForEmail：内部同步块 http-80-Processor131 cache.StaticCache - get：object at key [SomeData-test@test.com]已过期 http-80-Processor131 cache.StaticCache - get：key [SomeData- test@test.com]返回值[null] http-80-Processor104 jsp.view-page - getSomeDataForEmail：内部同步块 http-80-Processor104 cache.StaticCache - get：object at key [SomeData-test@test.com]已过期 http-80-Processor104 cache.StaticCache - get：key [SomeData-test@test.com]返回值[null] http-80-Processor252 jsp.view-page - getSomeDataForEmail：即将进入同步块 http-80-Processor283 jsp .view-page - getSomeDataForEmail：即将进入同步块 http-80-Processor2 jsp.view-page - getSomeDataForEmail：即将进入同步块 http-80-Processor2 jsp.view-page - getSomeDataForEmail：内部同步块

我希望在get / set操作周围一次只能看到一个线程进入/退出同步块。

在String对象上同步是否存在问题？我认为缓存键是一个很好的选择，因为它对于操作是唯一的，即使final String key在方法中声明，我也认为每个线程都会获得对同一个对象的引用，因此会同步单个对象。

我在这做错了什么？

更新：在进一步查看日志之后，似乎具有相同同步逻辑的方法，其中密钥始终相同，例如

final String key = "blah";
...
synchronized(key) { ...

不会出现相同的并发问题 - 一次只有一个线程进入该块。

更新2：感谢大家的帮助！我接受了关于intern()字符串的第一个答案，它解决了我的初始问题 - 多个线程进入同步块，我认为它们不应该，因为它key具有相同的值。

正如其他人已经指出的那样，使用intern()这样的目的并同步这些字符串确实是一个坏主意 - 当针对webapp运行JMeter测试来模拟预期的负载时，我看到使用的堆大小增长到接近1GB不到20分钟。