类型安全：从 Object 到 List<MyObject> 的未经检查的强制转换

Question

我有一个 ListView 列出了一个自定义对象（比如说MyObject）。

我想通过 EditText 动态过滤它，所以我必须使用 publishResults 方法实现 getFilter()：

@Override
protected void publishResults(CharSequence constraint, FilterResults results) {
    MyObjectAdapter.this.setItems((List<MyObject>) results.values);
    MyObjectAdapter.this.notifyDataSetChanged();
}

此时，Eclipse 报错：Type safety: Unchecked cast from Object to List<MyObject>

我确信这个演员表永远是正确的，但 Eclipse 只建议添加@SuppressWarnings("unchecked")，但我完全反对SuppressWarnings，因为它只是隐藏问题，而不是解决方案......

我尝试添加：

if(results.values instanceof List<MyObject>)

但是 Eclipse 再次抱怨，这并没有解决任何问题......

Cannot perform instanceof check against parameterized type List<MyObject>. Use the form List<?>

我知道强制转换总是正确的，但这是确保results.values 实际上是List<MyObject> 的正确方法？

提前致谢！

Answer 1

如果您只需要使用Object，那么您就无法在运行时检查您实际上是否拥有List<MyObject>，因为泛型类型MyObject 仅用于编译时类型检查，它在运行时不可用。这就是您尝试添加instanceof 检查时收到错误的原因。

如果您确定您的 Object 确实始终是 List<MyObject>，那么我会说 @SuppressWarnings 没问题，如果您记录了为什么您确定这不是问题。

如果您绝对想避免警告，您可以创建自己的List 实现（例如MyObjectList），它本身不是通用的，但实现了List<MyObject>。然后您可以在运行时对MyObjectList 进行instanceof 检查。

另一个选项是检查 List<?> 并将其转换为 instanceof 错误提示。然后您可以遍历列表中的元素并检查它们是否实际上都是 MyObject 的所有实例，并将它们复制到新的List<MyObject>。

Answer 2

嗯，我终于找到了解决办法。

正如@Medo42 所说：

另一种选择是检查并转换为 List 作为 instanceof 错误提示。然后您可以遍历列表中的元素并 检查它们是否实际上是 MyObject 的所有实例，并将它们复制到 一个新的列表。

尽管我没有经历过创建一个全新对象的过程以使这个特殊情况“无警告”地工作，但这是正确的方向。

所以我采用了@lokoko 的想法并将其用于新的setItems() 方法，使用Object 参数而不是List<MyObject> 以确保

结果代码如下：

public void setItems(List<MyObject> var){
    this.list = var;
}

public void setItems(Object var){
    List<MyObject> result = new ArrayList<MyObject>();
    if (var instanceof List){
        for(int i = 0; i < ((List<?>)var).size(); i++){
            Object item = ((List<?>) var).get(i);
            if(item instanceof MyObject){
                result.add((MyObject) item);
            }
        }
    }
    setItems(result);
}

感谢大家的帮助！

Answer 3

试试这样的：

List<?> result = (List<?>) results.values;
for (Object object : result) {
    if (object instanceof MyObject) {
        tempList.add((MyObject) object); // <-- add to temp
    }
}

filteredItems = tempList; // <-- set filtered

Answer 4

您可以在将其传递给setItems()之前执行检查。

final Object myListObj = reuslts.values;
if(myListObj instanceof List<?>) {
    if(((List<?>)myListObj).get(0) instanceof MyObject)
        // You can safely suppress the warning here because you made sure it is a List containing MyObject
        MyObjectAdapter.this.setItems((List<? extends MyObject>) myListObj);

}

但是，您需要相应地更改您的 setItems() 方法：

public void setItems(List<? extends MyObject> list) {
    // Your code here
}