ChatGPT解决这个技术问题 Extra ChatGPT

类型安全:未经检查的演员表

在我的 spring 应用程序上下文文件中,我有类似的内容:

<util:map id="someMap" map-class="java.util.HashMap" key-type="java.lang.String" value-type="java.lang.String">
    <entry key="some_key" value="some value" />
    <entry key="some_key_2" value="some value" />   
</util:map>

在 java 类中,实现如下所示:

private Map<String, String> someMap = new HashMap<String, String>();
someMap = (HashMap<String, String>)getApplicationContext().getBean("someMap");

在 Eclipse 中,我看到一条警告说:

类型安全:从 Object 到 HashMap 的未经检查的强制转换

什么地方出了错?

我想出了一个例程来实际检查转换为参数化的 HashMap,它消除了未经检查的转换警告:link 我会说这是“正确”的解决方案,但它是否值得可能值得商榷。 :)

J
Jon Skeet

问题是强制转换是运行时检查 - 但由于类型擦除,在运行时对于任何其他 FooBarHashMap<String,String>HashMap<Foo,Bar> 之间实际上没有区别。

使用 @SuppressWarnings("unchecked") 并捏住鼻子。哦,还有 Java 中的物化泛型运动 :)


我将采用 Java 的具体化泛型而不是无类型的 NSMutableWhatever,这感觉就像一个十年的飞跃,一周中的任何一天。至少 Java 正在尝试。
确切地。如果您坚持进行类型检查,则只能使用 HashMap 来完成,并且不会删除警告,因为它与不检查泛型类型相同。这不是世界末日,但令人讨厌的是,您要么压制警告要么忍受警告。
@JonSkeet 什么是具体化的泛型?
@SasQ 和所有其他想知道的人都应该看看这个 question
c
chelmertz

嗯,首先,新的 HashMap 创建调用会浪费内存。您的第二行完全忽略了对这个创建的 hashmap 的引用,使其可供垃圾收集器使用。所以,不要那样做,使用:

private Map<String, String> someMap = (HashMap<String, String>)getApplicationContext().getBean("someMap");

其次,编译器抱怨您将对象转换为 HashMap 而没有检查它是否是 HashMap。但是,即使你要这样做:

if(getApplicationContext().getBean("someMap") instanceof HashMap) {
    private Map<String, String> someMap = (HashMap<String, String>)getApplicationContext().getBean("someMap");
}

您可能仍会收到此警告。问题是,getBean 返回 Object,所以不知道类型是什么。将其直接转换为 HashMap 不会导致第二种情况出现问题(也许在第一种情况下不会出现警告,我不确定 Java 编译器对 Java 5 的警告有多迂腐)。但是,您将其转换为 HashMap<String, String>

HashMaps 实际上是将对象作为键并将对象作为值的映射,HashMap<Object, Object> 如果您愿意的话。因此,不能保证当您获得 bean 时它可以表示为 HashMap<String, String>,因为您可以有 HashMap<Date, Calendar>,因为返回的非泛型表示可以有任何对象。

如果代码可以编译,并且您可以执行 String value = map.get("thisString"); 而不会出现任何错误,则不必担心此警告。但是,如果映射不完全是字符串值的字符串键,您将在运行时获得 ClassCastException,因为在这种情况下泛型无法阻止这种情况发生。


这是不久前的事了,但我一直在寻找关于在强制转换之前检查 Set 的答案,并且您不能在参数化泛型上使用 instanceof。例如 if(event.getTarget instanceof Set) 您只能使用 ?这不会删除演员警告。例如 if(event.getTarget instanceof Set)
A
Amarghosh

如上面的消息所示,列表无法区分 List<Object>List<String>List<Integer>

我已经解决了类似问题的此错误消息:

List<String> strList = (List<String>) someFunction();
String s = strList.get(0);

具有以下内容:

List<?> strList = (List<?>) someFunction();
String s = (String) strList.get(0);

解释:第一次类型转换验证对象是 List 而不关心其中包含的类型(因为我们无法在 List 级别验证内部类型)。现在需要进行第二次转换,因为编译器只知道 List 包含某种对象。这将在访问列表时验证列表中每个对象的类型。


你是对的,我的朋友。而不是强制转换列表,只需迭代它并强制转换每个元素,警告就不会出现,太棒了。
这消除了警告,但我仍然不自信:P
是的,感觉就像蒙住编译器而不是运行时:D 所以我看不出这和 @SuppressWarnings("unchecked") 有什么区别
棒极了!使用 @SupressWarning 的主要区别在于,它使用注解消除了来自 IDE 和代码分析工具的警告,但如果您使用 -Werror 标志编译,您最终会出现错误。使用这种方法,两个警告都是固定的。
baeldung.com/java-warning-unchecked-cast 中解释了 ClassCastException 可能会在您的代码中的任何地方抛出,而无需参考这个有问题的未经检查的强制转换。解决方法是将强制转换移动到具有非常简洁的 ClassCastException 的东西上,因此如果需要可以更容易地修复。
D
David M. Karr

警告就是这样。一个警告。有时警告是无关紧要的,有时则不是。它们用于提醒您注意编译器认为可能是问题但可能不是问题的事情。

在强制转换的情况下,它总是会在这种情况下发出警告。如果您绝对确定特定演员表是安全的,那么您应该考虑在该行之前添加这样的注释(我不确定语法):

@SuppressWarnings (value="unchecked")

-1:永远不应该接受警告。或抑制此类警告或修复它。会有这样的时刻,您将不得不收到许多警告,而您一次也看不到相关警告。
在转换参数化泛型(即 Map)时,您无法真正避免类转换警告,因此这是原始问题的最佳答案。
D
David Nehme

您收到此消息是因为 getBean 返回一个 Object 引用,并且您正在将其转换为正确的类型。 Java 1.5 给你一个警告。这就是使用 Java 1.5 或更好的代码来运行这样的代码的本质。 Spring 有类型安全的版本

someMap=getApplicationContext().getBean<HashMap<String, String>>("someMap");

在它的待办事项清单上。


R
Rabbit

如果您真的想摆脱警告,您可以做的一件事是创建一个从泛型类扩展的类。

例如,如果您尝试使用

private Map<String, String> someMap = new HashMap<String, String>();

您可以像这样创建一个新类

public class StringMap extends HashMap<String, String>()
{
    // Override constructors
}

然后当你使用

someMap = (StringMap) getApplicationContext().getBean("someMap");

编译器确实知道(不再是通用的)类型是什么,并且不会发出警告。这可能并不总是完美的解决方案,有些人可能会认为这种方式违背了泛型类的目的,但是您仍然在重用泛型类中的所有相同代码,您只是在编译时声明什么类型你想用。


A
Andy

下面的代码导致类型安全警告

Map<String, Object> myInput = (Map<String, Object>) myRequest.get();

解决方法

创建一个新的地图对象而不提及参数,因为未验证列表中保存的对象类型。

第 1 步:创建一个新的临时地图

Map<?, ?> tempMap = (Map<?, ?>) myRequest.get();

第二步:实例化主地图

Map<String, Object> myInput=new HashMap<>(myInputObj.size());

第 3 步:迭代临时 Map 并将值设置到主 Map

 for(Map.Entry<?, ?> entry :myInputObj.entrySet()){
        myInput.put((String)entry.getKey(),entry.getValue()); 
    }

J
João Neto

避免未经检查的警告的解决方案:

class MyMap extends HashMap<String, String> {};
someMap = (MyMap)getApplicationContext().getBean("someMap");

看起来像黑客而不是解决方案。
- 可序列化类 MyMap 未声明 long 类型的静态最终 serialVersionUID 字段:{
@Ulterior 然后添加这样一个字段:stackoverflow.com/questions/2288937/…
J
Jeremy

另一种解决方案是,如果您发现自己经常投射相同的对象并且不想在代码中乱扔 @SupressWarnings("unchecked"),则可以使用注释创建一个方法。这样你就可以集中演员表,并希望减少出错的可能性。

@SuppressWarnings("unchecked")
public static List<String> getFooStrings(Map<String, List<String>> ctx) {
    return (List<String>) ctx.get("foos");
}

d
davidxxx

我做错了什么?我该如何解决这个问题?

这里 :

Map<String,String> someMap = (Map<String,String>)getApplicationContext().getBean("someMap");

您使用了我们通常不想使用的旧方法,因为它返回 Object

Object getBean(String name) throws BeansException;

有利于从 bean 工厂获取(对于单例)/创建(对于原型)bean 的方法是:

<T> T getBean(String name, Class<T> requiredType) throws BeansException;

使用它,例如:

Map<String,String> someMap = app.getBean(Map.class,"someMap");

将编译但仍带有未经检查的转换警告,因为所有 Map 对象不一定都是 Map<String, String> 对象。

但是 <T> T getBean(String name, Class<T> requiredType) throws BeansException; 在 bean 泛型类(例如泛型集合)中是不够的,因为它需要指定多个类作为参数:集合类型及其泛型类型。

在这种情况下,一般来说,更好的方法是不直接使用 BeanFactory 方法,而是让框架注入 bean。

bean声明:

@Configuration
public class MyConfiguration{

    @Bean
    public Map<String, String> someMap() {
        Map<String, String> someMap = new HashMap();
        someMap.put("some_key", "some value");
        someMap.put("some_key_2", "some value");
        return someMap;
    }
}

豆子注入:

@Autowired
@Qualifier("someMap")
Map<String, String> someMap;