创建带参ViewModelFactory时遇unchecked cast警告，如何解决？

我正在创建一个ViewModelFactory，以便使用可接收参数的ViewModel构造方法。但在代码行return (T) new clubPageViewModel(mDataSource);上出现了“unchecked cast”警告。请问该如何修复？或者我的实现方式从根本上存在问题？以下是我的代码实现：

public class clubPageViewModelFactory implements ViewModelProvider.Factory {
    private final String mDataSource;

    public clubPageViewModelFactory(String dataSource) {
        mDataSource = dataSource;
    }

    @NonNull
    @Override
    public <T extends ViewModel> T create(@NonNull Class<T> modelClass) {
        if (modelClass.isAssignableFrom(clubPageViewModel.class)) {
            return (T) new clubPageViewModel(mDataSource);
        }
        throw new IllegalArgumentException("Unknown ViewModel class");
    }
}

是否有可行的解决办法？

解决方案 #

这个警告是Java泛型类型擦除机制导致的——编译器在编译期无法完全确认(T)强制转换的安全性，但你的ViewModelFactory实现思路本身是完全正确的，这是Android中创建带参数ViewModel的标准方案。下面提供两种修复方式：

方法一：使用Class.cast()替代强制类型转换（推荐）

Class类的cast()方法是泛型方法，它会在运行时检查类型并返回对应泛型类型的实例，编译器能识别这个转换的安全性，从而彻底消除警告。修改后的create方法如下：

@NonNull
@Override
public <T extends ViewModel> T create(@NonNull Class<T> modelClass) {
    if (modelClass.isAssignableFrom(clubPageViewModel.class)) {
        return modelClass.cast(new clubPageViewModel(mDataSource));
    }
    throw new IllegalArgumentException("Unknown ViewModel class");
}

方法二：添加抑制警告注解（应急场景使用）

如果你已经通过isAssignableFrom确认了类型匹配的安全性，可以添加@SuppressWarnings("unchecked")注解来隐藏警告。不过这种方式只是“掩盖”警告，并非从根源解决类型检查问题，所以仅推荐在特殊场景下使用：

@NonNull
@Override
@SuppressWarnings("unchecked") // 添加注解抑制警告
public <T extends ViewModel> T create(@NonNull Class<T> modelClass) {
    if (modelClass.isAssignableFrom(clubPageViewModel.class)) {
        return (T) new clubPageViewModel(mDataSource);
    }
    throw new IllegalArgumentException("Unknown ViewModel class");
}

总结 #

你的实现方式没有根本性问题，只需要调整类型转换的方式即可。优先选择第一种方法，它既安全又符合Java泛型的最佳实践。

内容的提问来源于stack exchange，提问作者dexter2019