如何处理未经检查的强制转换警告?

Eclipse给我一个如下形式的警告:

类型安全:未检查从对象转换到HashMap

这是从一个API调用，我无法控制返回对象:

HashMap<String, String> getItems(javax.servlet.http.HttpSession session) {
  HashMap<String, String> theHash = (HashMap<String, String>)session.getAttribute("attributeKey");
  return theHash;
}

如果可能的话，我希望避免使用Eclipse警告，因为理论上它们至少表明存在潜在的代码问题。不过，我还没有找到消除这个问题的好方法。我可以将所涉及的单行单独提取到一个方法中，并向该方法添加@SuppressWarnings(“unchecked”)，从而限制忽略警告的代码块的影响。有更好的选择吗?我不想在Eclipse中关闭这些警告。

在我接触代码之前，它更简单，但仍然引起了警告:

HashMap getItems(javax.servlet.http.HttpSession session) {
  HashMap theHash = (HashMap)session.getAttribute("attributeKey");
  return theHash;
}

问题是在其他地方，当你试图使用散列时，你会得到警告:

HashMap items = getItems(session);
items.put("this", "that");

Type safety: The method put(Object, Object) belongs to the raw type HashMap.  References to generic type HashMap<K,V> should be parameterized.

当前回答

在强制转换之前检查一下。

Object someObject = session.getAttribute("attributeKey");
if(someObject instanceof HashMap)
HashMap<String, String> theHash = (HashMap<String, String>)someObject;

对于任何提问的人来说，收到你不确定类型的对象是很常见的。大量遗留的“SOA”实现传递各种对象，您不应该总是信任这些对象。(恐怖!)

EDIT修改了示例代码一次，以匹配海报的更新，在一些评论之后，我看到instanceof不能很好地使用泛型。然而，更改检查以验证外部对象似乎可以很好地使用命令行编译器。修订的例子现在发布。

2009-02-03 22:16:34

其他回答

我可能误解了这个问题(一个示例和几行代码就好了)，但是为什么您不总是使用合适的接口(和Java5+)呢?我看不出为什么你想要强制转换到HashMap而不是Map<KeyType,ValueType>。事实上，我想不出有什么理由将变量的类型设置为HashMap而不是Map。

为什么源是一个对象?是遗留集合的参数类型吗?如果是，请使用泛型并指定所需的类型。

2009-02-03 22:05:08

在强制转换之前检查一下。

Object someObject = session.getAttribute("attributeKey");
if(someObject instanceof HashMap)
HashMap<String, String> theHash = (HashMap<String, String>)someObject;

对于任何提问的人来说，收到你不确定类型的对象是很常见的。大量遗留的“SOA”实现传递各种对象，您不应该总是信任这些对象。(恐怖!)

2009-02-03 22:16:34

在Android Studio中，如果你想禁用检查，你可以使用:

//noinspection unchecked
Map<String, String> myMap = (Map<String, String>) deserializeMap();

2017-09-01 20:22:35

在HTTP Session世界中，您无法真正避免强制转换，因为API就是这样编写的(只接受并返回Object)。

不过，只要稍加努力，你就可以很容易地避免这种未经检查的转换。”这意味着在发生错误时，它将转换为传统的强制转换，并在那里给出ClassCastException)。未经检查的异常可能在以后的任何时候变成CCE，而不是在强制转换的时候(这就是为什么它是一个单独的警告)。

用专用类替换HashMap:

import java.util.AbstractMap;
import java.util.Collection;
import java.util.HashMap;
import java.util.Map;
import java.util.Set;

public class Attributes extends AbstractMap<String, String> {
    final Map<String, String> content = new HashMap<String, String>();

    @Override
    public Set<Map.Entry<String, String>> entrySet() {
        return content.entrySet();
    }

    @Override
    public Set<String> keySet() {
        return content.keySet();
    }

    @Override
    public Collection<String> values() {
        return content.values();
    }

    @Override
    public String put(final String key, final String value) {
        return content.put(key, value);
    }
}

然后转换为那个类，而不是Map<String,String>，所有内容都将在您编写代码的确切位置进行检查。以后不会出现意外的classcastexception。

2009-02-04 16:59:20

的对象。Esko Luontola上面回答的未检查的实用函数是避免程序混乱的好方法。

如果您不希望在整个方法上使用SuppressWarnings, Java会强制您将其放在本地方法上。如果你需要对一个成员进行强制转换，可能会导致这样的代码:

@SuppressWarnings("unchecked")
Vector<String> watchedSymbolsClone = (Vector<String>) watchedSymbols.clone();
this.watchedSymbols = watchedSymbolsClone;

使用这个实用程序要干净得多，而且你所做的事情仍然很明显:

this.watchedSymbols = Objects.uncheckedCast(watchedSymbols.clone());

注意: 我觉得有必要补充一下，有时候警告真的意味着你做错了什么，比如:

ArrayList<Integer> intList = new ArrayList<Integer>();
intList.add(1);
Object intListObject = intList; 

 // this line gives an unchecked warning - but no runtime error
ArrayList<String> stringList  = (ArrayList<String>) intListObject;
System.out.println(stringList.get(0)); // cast exception will be given here

编译器告诉您的是，在运行时不会检查此强制转换，因此在尝试访问泛型容器中的数据之前不会引发运行时错误。

2011-08-11 13:53:27

如何处理未经检查的强制转换警告?

推荐文章

最新文章

标签