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扩展用户>从接口并将其应用于这样的方法:
<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找不到这种方式自动装配的bean,因为JpaUserRepository是UserRepository< JpaUser>.服务层不了解存储库实现.
如果你展示了如何声明userRepository会有所帮助,因为它缺失了.但问题是存储库具有通用类型< T extends User>,这与< User>不同,这就是您的代码发出警告的原因.问题不在于警告,而是关于通用类型的正确使用.我猜你的userRepository声明如下:
@Autowired private JpaUserRepository userRepository;
但这意味着名称错了.毕竟,它不是User对象的存储库,而是JpaUser对象的存储库.
现在,您可能会认为JpaUser源自User.但这不是Java泛型的工作方式.那里有一个很好的资源,Java Generics FAQ.这真的值得一读.
现在我将推测一点,从某种意义上说,我认为你将存储库暴露给用户,但你不想公开JpaUser类.但这没有意义,因为存储库是一个非常基本的接口,您不应该在库中公开它.
因此,如果没有所有信息,但通过做出有根据的猜测,我可以看到两种情况:
>您不会将存储库公开给用户,在这种情况下,您只需处理JpaUser对象.
>您确实希望用户使用User对象,在这种情况下,您应该构建一个隐藏存储库的façade.
编辑:我很快就制作了一个你可能想要用作起点的外观课程.
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()); }
它编译时没有任何警告,因为编译器会推断出正确的类型.我承认我发现它缺乏某种优雅,但它有效.