Java 同步块与 Collections.synchronizedMap

Is the following code set up to correctly synchronize the calls on synchronizedMap?

public class MyClass {
  private static Map<String, List<String>> synchronizedMap = Collections.synchronizedMap(new HashMap<String, List<String>>());

  public void doWork(String key) {
    List<String> values = null;
    while ((values = synchronizedMap.remove(key)) != null) {
      //do something with values
    }
  }

  public static void addToMap(String key, String value) {
    synchronized (synchronizedMap) {
      if (synchronizedMap.containsKey(key)) {
        synchronizedMap.get(key).add(value);
      }
      else {
        List<String> valuesList = new ArrayList<String>();
        valuesList.add(value);
        synchronizedMap.put(key, valuesList);
      }
    }
  }
}

据我了解，我需要 addToMap() 中的同步块来防止另一个线程调用 remove() 或 containsKey()在我完成对 put() 的调用之前，但我不需要 doWork() 中的同步块，因为另一个线程无法进入 addToMap() 在 remove() 返回之前，因为我最初使用 Collections.synchronizedMap() 创建了地图.那是对的吗?有没有更好的方法来做到这一点?