java同步方法入口点线程是否足够安全？

Question

我有一个Singleton类处理一种Hashmap中具有不同对象的缓存。 （键的格式直接链接到地图中存储的对象类型 - 因此地图是）

地图上可以执行三种不同的操作：添加，获取，删除。

我使用公共入口点方法（没有密集访问）来保护对地图的访问：

public synchronized Object doAction(String actionType, String key, Object data){
  Object myObj = null;
  if (actionType.equalsIgnorecase("ADD"){
    addDataToMyMap(key,data);
  } else if (actionType.equalsIgnorecase("GET"){
    myObj = getDataFromMyMap(key);
  } else if (actionType.equalsIgnorecase("REM"){  
    removeDataFromMyMap(key);      
  }
  return myObj;
}

笔记：

地图是私人的。 方法addDataToMyMap（），getDataFromMyMap（）和removeDataFromMyMap（）都是私有的。 只有入口点方法是公共的，除了类本身的静态getInstance（）之外别无其他。

您是否确认并发访问地图是线程安全的，因为没有其他方法可以使用地图但通过该方法？

如果它是Map的安全，我想这个原则可以应用于任何其他类型的共享资源。

非常感谢你的答案。

大卫

Answer 1

我需要看看你的方法的实现，但它可能就足够了。 但我建议你使用java的Collection API中的Map，然后除非你共享其他实例，否则你不需要同步你的方法。

阅读： http ： //www.java-examples.com/get-synchronized-map-java-hashmap-example

Answer 2

是的，只要唯一的入口点是doAction，您的类就是线程安全的。

Answer 3

如果您的cache类具有私有HashMap并且您有三个方法并且所有方法都是public synchronized而不是static并且如果您没有任何其他public实例变量，那么我认为您的缓存是thread-safe 。

最好发布您的代码。

Answer 4

这是完全安全的。 只要所有线程都使用公共锁访问它，在这种情况下是对象，那么它是线程安全的。 （其他答案可能更高效，但您的实施是安全的。）

Answer 5

您可以使用Collections.synchronizedMap来同步对Map访问。

Answer 6

因为很难确定代码是否是线程安全的。 您的示例中缺少的重要信息包括：

1. 方法是否公开
2. 方法是否同步
3. 它只能通过方法访问地图

我建议你研究同步 ，以掌握问题以及如何解决它们。 探索ConcurrentHashMap类将提供有关您的问题的更多信息。

Answer 7

您应该使用ConcurrentHashMap 。 与Collections.synchronizedMap（）相比，它提供了比同步doAction更好的吞吐量和更好的线程安全性。

Answer 8

这取决于您的代码。 正如其他人所说，您可以使用Collections.synchronizedMap。 但是，这只会同步地图上的各个方法调用。 因此，如果：

map.get(key);
map.put(key,value);

在两个不同的线程中同时执行，一个将阻塞，直到另一个退出。 但是，如果您的关键部分大于对地图的单个调用：

SomeExpensiveObject value = map.get(key);
if (value == null) {
   value = new SomeExpensiveObject();
   map.put(key,value);
}

现在让我们假设密钥不存在。 第一个线程执行，并返回null值。 调度程序产生该线程，并运行线程2，线程2也返回空值。 它构造新对象并将其放入地图中。 然后线程1恢复并执行相同的操作，因为它仍然具有空值。

这是您需要围绕关键部分的更大同步块的地方

SomeExpensiveObject value = null;

synchronized (map) {
  value = map.get(key);
  if (value == null) {
     value = new SomeExpensiveObject();
     map.put(key,value);
  }
}