Map（或者说HashMap）可以说是Java中最常用的一个类，事实上，这个类的使用却存在很多陷井，比如下面的写法：
public class Demo {
private Map attributes=new HashMap();
/**
* @return the attributes
*/
public Map getAttributes() {
return attributes;
}
/**
* @param attributes the attributes to set
*/
public void setAttributes(Map attributes) {
this.attributes = attributes;
}
}
上面是最常见的写法，它最大的问题在于，直接将Map暴露出去，而且允许外面的代码直接修改。比如说：
Demo demo=new Demo();
demo.setAttributes(null);
这时候，demo.getAttributes()就会返回null，于是我们很多人的代码，就必须加上null判断。
这样的代码，犯了几个错误：
1、违反了如Effecitve Java中的第12条：使类和成员的可访问能力最小化。这样，通过Map map=demo.getAttributes(); map.put(…)就可以随意修改Map中的数据，完全不可控。
2、不必要地强迫很多人写上各种null判断的代码，而且因为程序执行路径的不同，可能引发NPE错误。
建议的写法如下：
private final Map<String, String> attributes = new HashMap<String, String>();
public String getAttribute(String key) {
return attributes.get(key);
}
public void setAttribute(String key, String value) {
this.attributes.put(key, value);
}
public void clearAttributes() {
this.attributes.clear();
}
public void addAllAttributes(Map<String, String> attributes) {
if(null!=attributes){
this.attributes.putAll(attributes);
}
}
public Map<String, String> getUnmodifiablsAttributes() {
return Collections.unmodifiableMap(this.attributes);
}
通过上述写法，对外封闭了内部的Map信息，同时明确地告诉外部，只能通过哪些方法来修改数据。