为何IntelliJ提示将Object强制转换为String属于冗余操作?
为什么Object转String的强转会被IntelliJ判定为冗余?
你编写的Java代码如下,其中(String) v被IntelliJ提示Casting to String is redundant:
public class Utils { private static Map<Class, Function<Object, ?>> functions = new HashMap<>(); static { functions.put(String.class, v -> (String) v); functions.put(Foo.class, v -> { Foo foo = new Foo(); foo.setA((String) ((Map<String, Object>) v).get("A")); foo.setB((String) ((Map<String, Object>) v).get("B")); return foo; }); } }
冗余判定的原因
针对v -> (String) v的强转
这段lambda是对应String.class的转换逻辑,设计意图是把Object输入转成String输出。但从类型逻辑和IDE的上下文推断来看:
- 如果输入的
v本身就是String类型,这个强转完全没用——String是Object的子类,对象本身就是目标类型时,显式强转不会改变任何状态,属于纯冗余写法。 - 如果输入的
v不是String类型,这个强转直接会抛出ClassCastException,但IDE能推断出在这个上下文下,传入的v理应是String(否则这个转换逻辑本身就不合理),所以判定这个强转是多余的。
简单来说:当对象实际类型已经是目标类型时,显式强转属于无意义的冗余操作,IDE通过代码上下文识别出这一点,所以给出提示。
关于setA/setB中的强转
如果这两处也被提示冗余,说明IDE通过上下文推断出Map中"A"/"B"对应的value本身就是String类型,此时显式强转同样多余;如果没有提示,说明IDE无法确定value的实际类型,强转就是必要的类型校验操作。
内容的提问来源于stack exchange,提问作者IJR
相关产品推荐
相关产品推荐

