Java:输入安全性 - 未经检查的强制转换

时间:2009-09-29 01:28:01

标签: java type-safety

这是我的代码:

Object[] data = GeneComparison.readData(files);
MyGenome genome = (MyGenome) data[0];
LinkedList<Species> breeds = (LinkedList<Species>) data[1];

它为LinkedList提供此警告:

Type safety: Unchecked cast from Object to LinkedList<Species>

为什么它会抱怨链表而不是MyGenome?

2 个答案:

答案 0 :(得分:12)

当您将非参数化类型(Object)强制转换为参数化类型(LinkedList)时,Java会抱怨这种情况。它告诉你它可能是任何东西。它与第一个强制转换没有什么不同,除非第一个将生成ClassCastException,如果它不是那个类型但第二个不会。

这一切都归结为类型擦除。运行时的LinkedList实际上只是一个LinkedList。您可以在其中放置任何内容,它不会像第一个示例那样生成ClassCastException。

通常要摆脱这种警告,你必须做以下事情:

@SuppressWarning("unchecked")
public List<Something> getAll() {
  return getSqlMapClient.queryForList("queryname");
}

其中queryForList()返回一个List(非参数化),你知道内容将是Something类。

另一方面是Java中的数组是协变,这意味着它们保留了运行时类型信息。例如:

Integer ints[] = new Integer[10];
Object objs[] = ints;
objs[3] = "hello";

会抛出异常。但是:

List<Integer> ints = new ArrayList<Integer>(10);
List<Object> objs = (List<Object>)ints;
objs.add("hello");

完全合法。

答案 1 :(得分:3)

因为在这里:

MyGenome genome = (MyGenome) data[0];

您没有使用泛型

在这里

LinkedList<Species> breeds = (LinkedList<Species>) data[1];

您正在使用它们。

这只是一个警告,你在数据数组中混合类型。如果您知道自己在做什么(我的意思是,如果第二个元素包含 LinkedList ),您可以忽略该警告。

但更好的方法是拥有这样一个对象:

class Anything {
    private Object [] data;
    public Anything( Object [] data ) {
         this.data = data;
    }
    public Gnome getGnome() {
    .....
    }
    public List<Species> getBreeds() {
    ......
    }
 }

并且必须在正确转换之前返回正确的方法,以便最终得到:

Anything anything = new Anything( GeneComparison.readData(files) );
MyGenome genome             = anything.getGnome(); // similar to data[0]
LinkedList<Species> breeds = anything.getBreeds(); // similar to data[1];

在这些方法中,您必须进行适当的转换。