Java 之HashMap.values()方法误用

大佬的解决方法
Java 之HashMap.values()方法误用
　1.出错

今天在测试代码的时候发现程序报错，看代码才知道是使用HashMap.values()方法的时候出错。因为项目中需要获取Map的值的集合然后进行遍历，所以就很自然的调用了HashMap.values()方法，如下所示
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26

package collections;

import java.util.HashMap;
import java.util.List;
import java.util.Map;

public class Test {

/**
 * @param args
 */
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");
    map.put("D", "D");
    map.put("E", "E");
    List<String> valuesList = (List<String>) map.values();
    for(String str:valuesList){
        System.out.println(str);
    }
}

}

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

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()方法所在的源码，信息如下：
1
2
3
4

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

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

3.解决方法

在ArrayList中，有一个构造函数
1
2
3
4
5
6
7

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；这样就达到了预期目的。代码如下
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28

package collections;

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

public class Test {

/**
 * @param args
 */
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");
    map.put("D", "D");
    map.put("E", "E");
    //List<String> valuesList = (List<String>) map.values();
    List<String> valuesList = new ArrayList<String>(map.values());
    for(String str:valuesList){
        System.out.println(str);
    }
}

}
另一位大佬的补充

public List<T> list(){
	//return (List<T>)map.values();//错误方法
	//return new ArrayList<>(map.values());//直接构造arraylist
	
	 List<T> list=(List<T>) Arrays.asList(map.values().toArray());//先转数组再转list
	 return list;
	
}