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

感觉有点笨拙（也许它只是我，所以请告诉我这是否可以接受＆＃34;良好的编程和＃34;观点）。

无论哪种方式，是否可以在没有原始类型警告的情况下获取List<User>，同时保持存储库不受影响？

非常感谢你的时间。

编辑：我不是在寻找投射名单的方法。

编辑2：存储库声明：

private final UserRepository userRepository;

有些人建议将该声明更改为UserRepository<User> userRepository;并且成功删除了警告，但是由于JpaUserRepository为UserRepository<JpaUser>，因此Spring无法通过这种方式找到自动装配的bean。服务层不了解存储库实现。

Answer 1

如果您展示了如何声明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());
}

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

Answer 2

以下情况可以正常运行，无例外：

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

作为旁注，与您的问题无关，您可能需要考虑如下定义您的课程：

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

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

由于返回的List应该是不可修改的，因此它为您的实现提供了更多的灵活性。它允许子类返回包含JpaUser

子类的列表

因此，例如，如果有一个名为AwesomeJpaUser的子类，则子类可以安全地返回List。对于您当前的实现，这是不允许的。

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

2 个答案: