有没有办法在此代码中避免@SuppressWarnings？

Question

有没有办法避免以下使用@SuppressWarnings，并保持相同的功能，而不警告“类型安全：未选中从AbstractDO []转换为E []”：

public MyClass { 
    ... 
    private Map<Class<? extends AbstractDO>, AbstractDO[]> map; 
    ... 
    private void saveConcreteDOs(AbstractDO[] theEntities) {   
    entityMap.put(theEntities[0].getClass(), theEntities); 
    } 

    @SuppressWarnings("unchecked") 
    protected <E extends AbstractDO> E[] getConcreteDOs(Class<E> theType) { 
    return (E[]) map.get(theType); 
    } 
    ... 
} 

也许提高地图的声明？

Answer 1

您可以选择：抑制您知道总是会成功的演员的警告，或者避免警告并验证演员是否通过了try/catch块。

只有这两种选择。

maybe there is a way to enhance map declaration?

就你而言，我会说你有几个选择。

我认为最好的办法是到throws ClassCastException条款添加到您的getConcreteDOs方法，让来电处理无效的投带来的无效使用的方法 - 假设他们可以得到它来编译在你extends AbstractDO条款。这具有强制消费者在try/catch块中包装调用或者声明他们自己的throws子句以强制try/catch块在堆栈上更高的不利副作用。

你可以用一个空的catch块吞下异常;坦率地说，我更喜欢@SuppressWarning。

或者您可以完全放弃该方法，只处理抽象实体，从而有效地使存储库的使用者处理演员阵容。底线：每当您尝试构建通用存储库时，都会遇到这些问题。使用每个实体类型的具体存储库模式可能会更好。

Answer 2

你可以通过让你的类变为一般的方法来避免未经检查的转换。

public class MyClass<E extends AbstractDO> { 

    private Map<Class<? extends AbstractDO>, E[]> map; 

    public void saveConcreteDOs(E[] theEntities) {   
     map.put(theEntities[0].getClass(), theEntities); 
    } 

    public E[] getConcreteDOs(Class<E> theType) { 
     return map.get(theType); 
    } 
} 

Answer 3

首先，您的代码不是类型安全的。它可以在运行时抛出类转换异常。您应该有

 
    private void saveConcreteDOs(AbstractDO[] theEntities) {   
    entityMap.put(theEntities.getClass().getComponentType(), theEntities); 
    } 

您可能在运行时只有同类数组，并且元素[0]与数组元素类型的类型相同。然而，单靠考察这门课是无法知道的。

有了这个更正的代码，超级智能编译器可以证明getConcreteDOs()是类型安全的。但是，javac并不那么聪明。语言规范要求提供警告。

通常，在Java中，无法表示键和值之间更复杂的关系。不变量，一个值是一个数组，其组件类型是关键字，只保留在你的头上。

现在，看看这个非数组版本：

private Map<Class<? extends AbstractDO>, AbstractDO> map; 

protected <E extends AbstractDO> E getConcreteDOs(Class<E> theType) 
{ 
    AbstractDO obj = map.get(theType); 
    return theType.cast(obj); 
} 

这有没有报警，但它的那种作弊。 Class.cast()隐藏了我们的警告，就这些。

它没有帮助阵列版本，Class<T>中没有T[] castArray(Object[])。你可以自己制作一种方法，有效地隐藏它的警告。

或者你可以做到这一点，但它确实是不必要的俗气。如果您知道自己在做什么，并且您仔细检查了程序以确保类型安全，请不要担心未经检查的警告。

protected <E extends AbstractDO> E[] getConcreteDOs(Class<E[]> arrayType) 
{ 
    AbstractDO[] array = map.get(arrayType.getComponentType()); 
    return arrayType.cast(array); 
} 
... 
X[] array = getConcreteDOs(X[].class);