如何在没有未经检查的赋值的情况下从通用接口检索对象列表？

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>从界面中删除并将其应用于以下方法:

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

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

我也尝试从接口和方法中删除泛型,同时保持返回列表的通用性:

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无法像JpaUserRepository那样找到以这种方式自动装配的bean UserRepository<JpaUser>.服务层不了解存储库实现.

Answer 1

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

\n\n

问题不在于警告，而在于泛型类型的正确使用。我猜你的userRepository声明如下：

\n\n

@Autowired\nprivate JpaUserRepository userRepository;\n

\n\n

但这意味着这个名字是错误的。毕竟，它不是User对象的存储库，而是JpaUser对象的存储库。

\n\n

现在，您可能会争辩说它JpaUser源自User。但这不是 Java 泛型的工作原理。Java 泛型常见问题解答是一个非常好的资源。这确实值得一读。

\n\n

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

\n\n

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

\n\n

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

\n\n

编辑：我很快就创建了一个 fa\xc3\xa7ade 类，您可能想将其用作起点。

\n\n

public class RepositoryFacade {\n\n    private final UserRepository<? extends User> repository;\n\n    public RepositoryFacade(UserRepository<? extends User> repository) {\n        this.repository = repository;\n    }\n\n    public List<User> findAll(final UserCriteria userCriteria, final PageDetails pageDetails) {\n        return repository.findAll(userCriteria, pageDetails)\n                .stream()\n                .collect(Collectors.toList());\n    }\n\n    public User findByEmail(final String email) {\n        return repository.findByEmail(email);\n    }\n}\n\npublic RepositoryFacade getJpaUserFacade() {\n    return new RepositoryFacade(new JpaUserRepository());\n}\n

\n\n

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

\n