java - @Singleton 缓存实现

Question

我想知道我的缓存实现是否正确，并希望得到任何反馈。资源分配了一些字符串（客户端地址）值。

我必须同步 addEntityRegistration() 方法吗？或者这个用例有更好的方法吗？

在此先感谢，米

@Singleton
@ConcurrencyManagement(ConcurrencyManagementType.BEAN)
public class Cache{

   private Map<Object, Set<String>> registeredClients = new ConcurrentHashMap<Object,Set<String>>();

   .....

   protected void addEntityRegistration(Object key, String fullJid){
    Set<String> registered = registeredClients.get(key);
    if(registered == null){
        registered = Collections.newSetFromMap(new ConcurrentHashMap<String,Boolean>());
        registeredClients.put(key, registered);
    }

    registered.add(fullJid);
   }