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

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.

当前回答

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

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

2009-02-03 22:05:08

其他回答

这很难，但以下是我目前的想法:

如果你的API返回Object，那么你什么也做不了——无论如何，你都将盲目地强制转换对象。您可以让Java抛出classcastexception，或者您可以自己检查每个元素并抛出断言或illegalargumentexception或诸如此类的东西，但这些运行时检查都是等效的。无论在运行时做什么，都必须抑制编译时未检查的强制转换。

我只是更喜欢盲强制转换，让JVM为我执行它的运行时检查，因为我们“知道”API应该返回什么，并且通常愿意假设API工作。如果需要，可以在类型转换上方的任何地方使用泛型。您实际上并没有购买任何东西，因为您仍然拥有单一的盲强制转换，但至少您可以从这里开始使用泛型，因此JVM可以帮助您避免在代码的其他部分使用盲强制转换。

在这个特殊的例子中，假设您可以看到对SetAttribute的调用，并看到进入的类型，因此在退出时将类型盲强制转换为same并不是不道德的。添加一个引用SetAttribute的注释并完成它。

2011-03-01 23:04:58

不幸的是，这里没有很好的选择。记住，所有这些的目标都是保持类型安全。“Java泛型”提供了一种处理非泛型遗留库的解决方案，在8.2节中有一种特别的方法叫做“空循环技术”。基本上，使不安全施放，并压制警告。然后像这样循环地图:

@SuppressWarnings("unchecked")
Map<String, Number> map = getMap();
for (String s : map.keySet());
for (Number n : map.values());

如果遇到意外类型，您将得到一个运行时ClassCastException，但至少它将发生在问题的根源附近。

2009-02-03 22:46:42

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

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

2017-09-01 20:22:35

以这个为例，它比创建一个新的HashMap要快得多，如果它已经是一个HashMap，但仍然是安全的，因为每个元素都根据它的类型进行检查…

@SuppressWarnings("unchecked")
public static <K, V> HashMap<K, V> toHashMap(Object input, Class<K> key, Class<V> value) {
       assert input instanceof Map : input;

       for (Map.Entry<?, ?> e : ((HashMap<?, ?>) input).entrySet()) {
           assert key.isAssignableFrom(e.getKey().getClass()) : "Map contains invalid keys";
           assert value.isAssignableFrom(e.getValue().getClass()) : "Map contains invalid values";
       }

       if (input instanceof HashMap)
           return (HashMap<K, V>) input;
       return new HashMap<K, V>((Map<K, V>) input);
    }

2010-10-14 09:28:19

警告抑制不是解决办法。你不应该在一个语句中执行两层类型转换。

HashMap<String, String> getItems(javax.servlet.http.HttpSession session) {

    // first, cast the returned Object to generic HashMap<?,?>
    HashMap<?, ?> theHash = (HashMap<?, ?>)session.getAttribute("attributeKey");

    // next, cast every entry of the HashMap to the required type <String, String>
    HashMap<String, String> returingHash = new HashMap<>();
    for (Entry<?, ?> entry : theHash.entrySet()) {
        returingHash.put((String) entry.getKey(), (String) entry.getValue());
    }
    return returingHash;
}

2015-03-09 23:43:39

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

推荐文章

最新文章

标签