Map的奇怪问题

Question

public class ContextHolder {
public static Map<String, ScriptSession> onlineUsers = new HashMap<String, ScriptSession>(); public static void removeMapItemByKey(Map map,Object key){
Iterator<Map.Entry<Object, Object>> it = map.entrySet().iterator();  
        while(it.hasNext()){  
            Map.Entry<Object, Object> entry=it.next();  
            Object k=entry.getKey();  
            if(key.equals(k)){  
                //System.out.println("delete this: "+k+" = "+key);  
                it.remove();        //OK   
            }  
        }
}
}
我这里有个好奇怪的问题，我用一个静态的HashMap（onlineUsers），用来记录在线的用户，当一个用户登录系统时就用onlineUsers.put(userName,ScriptSession)，进入这个HashMap,然后当一个用户退出系统时就通过removeMapItemByKey(Map map,Object key)这个方法来删除HashMap内的这个用户，但现在来了一个比较奇怪的问题，就是我要历遍这个HashMap时没有拿到东西，用onlineUsers.size()是有50多个的！知道问题在哪里吗？！ 历遍的方法为JsonObject jo = new JsonObject();
String onlineUser = "";
Map<String, ScriptSession> m = new IdentityHashMap<String, ScriptSession>();
Iterator it = ContextHolder.onlineUsers.entrySet().iterator();
while (it.hasNext()) {
        Map.Entry<String, ScriptSession> pairs = (Map.Entry<String, ScriptSession>)it.next();
        m.put(pairs.getKey(),pairs.getValue());
    }
    it = m.entrySet().iterator();
    while (it.hasNext()) {
        Map.Entry<String, ScriptSession> pairs = (Map.Entry<String, ScriptSession>)it.next();
        onlineUser += "".equals(onlineUser)?"'"+pairs.getKey()+"'":",'"+pairs.getKey()+"'";
    }
jo.addProperty("onlineUser", "["+onlineUser+"]");
json = jo.toString();前台看到的 这个json{onlineUser:[]}；初步估计是 这个map在历遍的时候同时有其他用户对它进行了put或者调用了removeMapItemByKey(Map map,Object key)方法！但是都已经用了Iterator了要怎样才可以没有问题！？

Answer 1

你这个删除代码写的真复杂，为啥不直接：
public static void removeMapItemByKey(Map map,Object key){
  map.remove(key);
}以至于我怀疑你各地方对map的操作都是存在问题的。

Answer 2

首先谢谢您的关注我一开始的时候就是这样写 对Map历遍时也只是单纯的 用.keySet() 然后历遍 之后就出现了现在这个问题 然后在网上了解到 当一个Map在历遍时 对它进行map.remove(key) 会有问题的 
因为这里的Map是一个静态的变量，在内存只有一份。
当一个用户A对它进行历遍，另外一个用户B对它进行map.remove(key)可能就会影响到用户A的历遍。
所以改成这样子试一下会不会解决问题 但……
由于对Java Map的底层实现原理不熟。可能也要补一下，最好有高手来指导一下说一下Map底层的实现原理吧！嘻嘻不过授之于鱼不如授之于渔 最好说一下怎样可以查到类似这些底层的东西 也可以介绍一些书 供学习学习

Answer 3

hashmap，是非线程安全的，并发的情况下，会有问题。用concurrentmap，它是线程安全的，写入的时候，会锁map，不允许读。这样就不会产生脏数据了。楼主不妨一试

Answer 4

楼主目测一下是不是其他代码操作了静态Map变量啊！！！

Answer 5

谢谢的建议 这次 我通过用复制对象的方法 解决了问题 
就是历遍的时候先复制一个Map的对象 再历遍这个复制的对象
至于 hashmap，是非线程安全的 在增删方面 我换成了hashTable 
至于用concurrentmap的话因为他会锁map 高并发的时候 可能会慢

Answer 6

1.5以后有个并发包java.util.concurrent，这个是线程安全的，在遍历时可以修改删除数据：
ConcurrentHashMap