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

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.

当前回答

当然，最明显的答案是不要进行未经检查的强制转换。

如果绝对有必要，那么至少尝试限制@SuppressWarnings注释的范围。根据它的Javadocs，它可以访问局部变量;这样，它甚至不会影响整个方法。

例子:

@SuppressWarnings("unchecked")
Map<String, String> myMap = (Map<String, String>) deserializeMap();

没有办法确定Map是否真的应该有通用参数<String, String>。您必须事先知道参数应该是什么(否则当您得到ClassCastException时就会知道)。这就是代码生成警告的原因，因为编译器不可能知道是否安全。

2009-02-03 22:07:09

其他回答

在Eclipse首选项中，转到Java->编译器->错误/警告->泛型类型，并选中“忽略不可避免的泛型类型问题”复选框。

这满足了问题的意图，即。

我希望避免Eclipse警告……

如果不是精神。

2011-10-21 17:40:14

解决方案:在Eclipse中禁用此警告。不要@SuppressWarnings，完全禁用即可。

上面提出的几个“解决方案”都太离谱了，为了抑制愚蠢的警告而使代码不可读。

2013-02-28 20:27:30

不幸的是，这里没有很好的选择。记住，所有这些的目标都是保持类型安全。“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

您可以创建如下所示的实用程序类，并使用它来抑制未检查的警告。

public class Objects {

    /**
     * Helps to avoid using {@code @SuppressWarnings({"unchecked"})} when casting to a generic type.
     */
    @SuppressWarnings({"unchecked"})
    public static <T> T uncheckedCast(Object obj) {
        return (T) obj;
    }
}

你可以这样使用它:

import static Objects.uncheckedCast;
...

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

这里有更多关于这个问题的讨论: http://cleveralias.blogs.com/thought_spearmints/2006/01/suppresswarning.html

2009-02-03 22:54:33

哇;我想我找到了自己问题的答案。我只是不确定这是否值得!：）

问题是演员没有检查。所以，你必须自己检查。不能只使用instanceof检查参数化类型，因为参数化类型信息在运行时不可用，在编译时已被擦除。

但是，您可以使用instanceof对散列中的每一项执行检查，这样就可以构造一个类型安全的新散列。你不会引起任何警告。

多亏了mmyers和Esko Luontola，我将最初在这里编写的代码参数化了，所以它可以被包装在某个实用程序类中，并用于任何参数化的HashMap。如果您想更好地理解它，并且对泛型不是很熟悉，我建议您查看这个答案的编辑历史。

public static <K, V> HashMap<K, V> castHash(HashMap input,
                                            Class<K> keyClass,
                                            Class<V> valueClass) {
  HashMap<K, V> output = new HashMap<K, V>();
  if (input == null)
      return output;
  for (Object key: input.keySet().toArray()) {
    if ((key == null) || (keyClass.isAssignableFrom(key.getClass()))) {
        Object value = input.get(key);
        if ((value == null) || (valueClass.isAssignableFrom(value.getClass()))) {
            K k = keyClass.cast(key);
            V v = valueClass.cast(value);
            output.put(k, v);
        } else {
            throw new AssertionError(
                "Cannot cast to HashMap<"+ keyClass.getSimpleName()
                +", "+ valueClass.getSimpleName() +">"
                +", value "+ value +" is not a "+ valueClass.getSimpleName()
            );
        }
    } else {
        throw new AssertionError(
            "Cannot cast to HashMap<"+ keyClass.getSimpleName()
            +", "+ valueClass.getSimpleName() +">"
            +", key "+ key +" is not a " + keyClass.getSimpleName()
        );
    }
  }
  return output;
}

工作量很大，报酬可能很少……我不确定我是否会用它。我很感激任何关于人们是否认为它值得的评论。此外，我很感激改进建议:除了抛出AssertionErrors之外，还有什么更好的事情我可以做吗?有更好的东西让我扔吗?我应该让它成为一个受控异常吗?

2009-02-03 22:35:01

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

推荐文章

最新文章

标签