HashMap缓存中的同步

Question

我有一个Web应用程序，人们需要资源。使用同步哈希映射来缓存此资源以提高效率。这里的问题是，当两个不同的请求同时到达同一未缓存资源时：检索资源的操作占用大量内存，因此我想避免为同一资源多次调用它。

有人可以告诉我以下代码段是否存在任何潜在问题吗？提前致谢。

private Map<String, Resource> resources = Collections.synchronizedMap(new HashMap<String, Resource>());

public void request(String name) {

  Resource resource = resources.get(name);

  if (resource == null) {
    synchronized(this) {
      if (resources.get(name) == null) {
        resource = veryCostlyOperation(name); // This should only be invoked once per resource...
        resources.put(resource);
      } else {
        resource = resources.get(name);
      }
    }
  }

  ...

}

Answer 1

一个可能的问题是，您通过veryCostlyOperation()在一个synchronized块内执行来创建不必要的竞争，因此许多线程无法同时检索其（独立）资源。这可以通过使用Future<Resource>map的值来解决：

Map<String, Future<Resource>> map = new ConcurrentHashMap<String, Future<Resource>>();    
...
Future<Resource> r = map.get(name);
if (r == null) {
    FutureTask task = null;
    synchronized (lock) {
        r = map.get(name);
        if (r == null) {
            task = new FutureTask(new Callable<Resource>() {
                public Resource call() {
                    return veryCostlyOperation(name);
                }
            });
            r = task;
            map.put(name, r);
        }
    }
    if (task != null) task.run(); // Retrieve the resource
}

return r.get(); // Wait while other thread is retrieving the resource if necessary