求高手帮忙点拨一个“擦了小弟一个通宵”的“HashMap多线程安全性”的处理方案。

[chy_chy091]

高手，我弄了一个通信程序，其中遇到了C端用户好友列表登陆，“大约有1/3的概率”让用户看到没有好友的昵称的故障。
经过昨夜的请教，初步得到结论：“可能是由于小弟的全局HashMap没有进行多线程安全性”的处理造成的.
为了解决小弟的这个故障，一位高手为小弟做了一个解决这个故障的代码例子：

import java.util.ArrayList;
import java.util.Collections;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Map.Entry;

public class StaticVar {
private static Map<String, Object> map;

public static synchronized Map<String, Object> getMap() {
if(map==null||map.isEmpty()){
map=Collections.synchronizedMap(new HashMap<String, Object>());
}
return map;
}

public static synchronized boolean addItem(String key,Object value) {
boolean result=false;
if(!getMap().containsKey(key)&&value!=null){
getMap().put(key, value);
result=true;
}
return result;
}
public static synchronized boolean removeItem(String key,Object value) {
boolean result=false;
if(key!=null&&!"".equals(key)){
if(getMap().containsKey(key)){
getMap().remove(key);
result=true;
}
}else if(value!=null) {
if (getMap().containsValue(value)) {
List<String> removeKeys=new ArrayList<String>();
for (Entry<String, Object> entry : getMap().entrySet()) {
if(value.equals(entry.getValue())){
removeKeys.add(entry.getKey());  //  这里如果直接删的话，整个map的遍历将报错
}
}
for (String rmKey : removeKeys) {
getMap().remove(rmKey);
}
}
}

return result;
}
}

为了让高手给出的例子中的Map，能够“get方法也具备线程安全性”，小弟又求一位高手写了“getValue方法”添加到了上面的例子中，请高手过目：

import java.util.ArrayList;
import java.util.Collections;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
import java.util.Map.Entry;

public class StaticVar {
	private static Map<String, Object> map;
	public static HashMap<String,Integer> mapobj;

	public static synchronized Map<String, Object> getMap() {
		if(map==null||map.isEmpty()){
			map=Collections.synchronizedMap(new HashMap<String, Object>());
		}
		return map;
	}
	
public static synchronized Object getValue(String key) {
		
		if(key==null||key==""){//key 为空值
			throw new RuntimeException("Key can't be empty");		
		}
		
	return 	map.get(key);
		
		
		
	
	}
	
	
	public static synchronized boolean addItem(String key,Object value) {
		boolean result=false;
		if(!getMap().containsKey(key)&&value!=null){
			getMap().put(key, value);
			result=true;
		}
		return result;
	}
	public static synchronized boolean removeItem(String key,Object value) {
		boolean result=false;
		if(key!=null&&!"".equals(key)){
			if(getMap().containsKey(key)){
				getMap().remove(key);
				result=true;
			}
		}else if(value!=null) {
			if (getMap().containsValue(value)) {
				List<String> removeKeys=new ArrayList<String>();
				for (Entry<String, Object> entry : getMap().entrySet()) {
					if(value.equals(entry.getValue())){
						removeKeys.add(entry.getKey());  //  这里如果直接删的话，整个map的遍历将报错
					}
				}
				for (String rmKey : removeKeys) {
					getMap().remove(rmKey);
				}
			}
		}
		
		return result;
	}
	public static synchronized HashMap<String, Integer> getMapObj() {
		if(mapobj==null||mapobj.isEmpty()){
			mapobj=Collections.synchronizedMap(new HashMap<String, Integer>());
		}
		return mapobj;
	}
	
	public static synchronized boolean addItemObj(String key,Integer value) {
		boolean result=false;
		if(!getMap().containsKey(key)&&value!=null){
			getMap().put(key, value);
			result=true;
		}
		return result;
	}
	public static synchronized boolean removeItemObj(String key,Integer value) {
		boolean result=false;
		if(key!=null&&!"".equals(key)){
			if(getMap().containsKey(key)){
				getMap().remove(key);
				result=true;
			}
		}else if(value!=null) {
			if (getMap().containsValue(value)) {
				List<String> removeKeys=new ArrayList<String>();
				for (Entry<String, Object> entry : getMap().entrySet()) {
					if(value.equals(entry.getValue())){
						removeKeys.add(entry.getKey());  //  这里如果直接删的话，整个map的遍历将报错
					}
				}
				for (String rmKey : removeKeys) {
					getMap().remove(rmKey);
				}
			}
		}
		
		return result;
	}
	
public static synchronized Integer getValueObj(String key) {
		
		if(key==null||key==""){//key 为空值
			throw new RuntimeException("Key can't be empty");		
		}
		
	return 	mapobj.get(key);
		
		
		
	
	}
	
}

上面的初步修改稿，还没有把第66行的这个报错修改好：

希望得到高手点拨：
这个代码现场，小弟错在哪里？如何修改？
谢谢高手！！

[chy_chy091]

感谢蜗牛-大哥再次的相助！！ 感谢方老师再次的亲临指导！！提点，培养！ 感谢赵4赵老师的百忙中的亲自拔擢！！ 最后，感谢___________小P的前来捧场！！！！（再有一年的时间，所有的Java技术，可能我都会懂了） 有CSDN！！有CSDN精神！！！！！！有祖国软件工程屌丝，有祖国软件工程事业，最美丽，最壮丽的信息工程，上帝学习的工程专业，成为：世界NO.1！！！！

[姜小白-]

这个是map的遍历问题，使用for循环时，在循环中删除元素会破坏循环列表，有可能抛异常出来的。 使用Iterator 就可以了。 把

List<String> removeKeys=new ArrayList<String>();
for (Entry<String, Object> entry : getMap().entrySet()) {
if(value.equals(entry.getValue())){
removeKeys.add(entry.getKey());  //  这里如果直接删的话，整个map的遍历将报错
}
}
for (String rmKey : removeKeys) {
getMap().remove(rmKey);
}

改为如下形式

Iterator it = getMap().keySet().iterator();
		Map.Entry entry = null;
		while(it.hasNext()) {
			entry = (Entry) it.next();
			if (value.equals(entry.getValue())) {
				it.remove();
			}
		}

楼主试试效果

[花谢尊前不敢香]

Collections.synchronizedMap(new HashMap<String, Integer>())这个是实现一个线程安全的hashMap，但是你的mapobj是普通的hashmap，改为map接口或者强转一下。

[赵4老师]

加锁即可。

[___________小P]

悲观锁 乐观锁 你会吗