Java 之HashMap.values()方法误用

1.出错

　　　今天在测试代码的时候发现程序报错，看代码才知道是使用HashMap.values()方法的时候出错。因为项目中需要获取Map的值的集合然后进行遍历，所以就很自然的调用了HashMap.values()方法，如下所示

public class Test {
    public static void main(String[] args) {   
        Map<String,String> map = new HashMap<String,String>();
        map.put("A", "A");
        map.put("B", "B");
        map.put("C", "C");
        List<String> valuesList = (List<String>) map.values();
        for(String str:valuesList){
            System.out.println(str);
        }
    }
}

运行时候抛出异常，异常信息如下：

Exception in thread "main" java.lang.ClassCastException: java.util.HashMap$Values cannot be cast to java.util.List
    at collections.Test.main(Test.java:20)

2.错误原因分析

　　首先找到了values()方法所在的源码，信息如下：

public Collection<V> values() {
      Collection<V> vs = values;
      return (vs != null ? vs : (values = new Values()));
  }

原来values()方法只是返回了一个Collection集合，可是如程序中的用法所示，在向下转型的时候出现了类型转换错误。那我们应该怎么才能获取自己想要的结构呢？

3.解决方法

　　在ArrayList中，有一个构造函数

public ArrayList(Collection<? extends E> c) {
    elementData = c.toArray();
    size = elementData.length;
    // c.toArray might (incorrectly) not return Object[] (see 6260652)
    if (elementData.getClass() != Object[].class)
        elementData = Arrays.copyOf(elementData, size, Object[].class);
}

可以接受一个集合类型的参数，然后返回一个list；这样就达到了预期目的。代码如下

public class Test {
    public static void main(String[] args) { 
        Map<String,String> map = new HashMap<String,String>();
        map.put("A", "A");
        map.put("B", "B");
        map.put("C", "C");
        //List<String> valuesList = (List<String>) map.values();
        List<String> valuesList = new ArrayList<String>(map.values());
        for(String str:valuesList){
            System.out.println(str);
        }
    }
}