java - 如何在没有未经检查的分配的情况下从通用接口(interface)检索对象列表？

Question

我有以下界面:

public interface UserRepository<T extends User> {
    List<T> findAll(UserCriteria userCriteria, PageDetails pageDetails);
    T findByEmail(String email);
}

及其实现:

@Repository
public class JpaUserRepository implements UserRepository<JpaUser> {
    public List<JpaUser> findAll(UserCriteria userCriteria, PageDetails pageDetails) {
       //implementation
    }

    public JpaUser findByEmail(String email) {
       //implementation
    }
}

现在，当我打电话时:

User user = userRepository.findByEmail(email);

在我的服务类中，一切都很好。

但是当我打电话时:

List<User> users = userRepository.findAll(userCriteria, pageDetails);

我收到未经检查的分配警告，原因是 userRepository 具有原始类型，因此 findAll 的结果被删除。如果确实如此，不应该findByEmail表现一样吗？它只是看起来不太一致。

在这种情况下如何消除原始类型？我尝试了一些事情:

删除 <T extends User>从接口(interface)并将其应用于这样的方法:

<U extends User> List<U> findAll(UserCriteria userCriteria, PageDetails pageDetails);

这对服务来说工作正常，但存储库实现现在会给出关于未经检查的覆盖的警告(返回类型需要未经检查的转换)。

我还尝试从接口(interface)和方法中删除泛型，同时保持返回列表的泛型:

List<? extends User> findAll(UserCriteria userCriteria, PageDetails pageDetails);

这解决了问题，没有警告，但需要我这样编写服务:

List<? extends User> users = userRepository.findAll(userCriteria, pageDetails);

感觉有点笨拙(也许只有我这样，所以请让我知道从“良好编程”的角度来看这是否可以接受)。

无论哪种方式，是否有可能得到 List<User>没有原始类型警告，同时保持存储库不变？

非常感谢您的宝贵时间。

编辑:我不是在寻找转换名单的方法。

编辑 2: 存储库声明:

private final UserRepository userRepository;

有些人建议将该声明更改为 UserRepository<User> userRepository;并且那成功地删除了警告但是 Spring 找不到以这种方式 Autowiring 的 bean，因为 JpaUserRepository 是 UserRepository<JpaUser> .服务层不知道存储库实现。

Answer 1

如果你能展示如何userRepository 会有所帮助被声明，因为它丢失了。但问题是存储库具有通用类型 <T extends User> ，这不与<User>相同，这就是您的代码发出警告的原因。

问题与其说是关于警告，不如说是关于正确使用泛型类型。我猜你的 userRepository声明如下:

@Autowired
private JpaUserRepository userRepository;

但这意味着名称是错误的。毕竟，它不是 User 的存储库。对象，但属于 JpaUser对象。

现在，您可能会争论 JpaUser源自 User .但这不是 Java 泛型的工作方式。那里有一个很好的资源，Java Generics FAQ .非常值得一读。

现在我要推测一下，我认为您正在向用户公开存储库，但您不想公开 JpaUser类(class)。但这没有意义，因为存储库是一个非常基本的接口(interface)，您不应该在库中公开它。

因此，在没有所有信息的情况下，通过有根据的猜测，我可以看到两种情况:

1. 您不向用户公开存储库，在这种情况下您只需处理 JpaUser对象。
2. 您确实希望用户使用 User对象，在这种情况下，您应该构建一个隐藏存储库的外观。

编辑:我已经快速制作了一个外观类，您可能想将其用作起点。

public class RepositoryFacade {

    private final UserRepository<? extends User> repository;

    public RepositoryFacade(UserRepository<? extends User> repository) {
        this.repository = repository;
    }

    public List<User> findAll(final UserCriteria userCriteria, final PageDetails pageDetails) {
        return repository.findAll(userCriteria, pageDetails)
                .stream()
                .collect(Collectors.toList());
    }

    public User findByEmail(final String email) {
        return repository.findByEmail(email);
    }
}

public RepositoryFacade getJpaUserFacade() {
    return new RepositoryFacade(new JpaUserRepository());
}

它编译时没有任何警告，因为编译器会推断出正确的类型。我承认我发现它缺乏某种优雅，但它确实有效。