当前位置 : 主页 > 网络推广 > seo >

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

来源:互联网 收集:自由互联 发布时间:2021-06-16
我有以下界面: public interface UserRepositoryT extends User { ListT findAll(UserCriteria userCriteria, PageDetails pageDetails); T findByEmail(String email);} 它的实施: @Repositorypublic class JpaUserRepository implements User
我有以下界面:

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());
}

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

网友评论