如何简化两种类似方法的代码？

如何解决如何简化两种类似方法的代码？

我想重构这两个方法，除了“ maxResult（）”外，它们实际上是相同的，这两个方法引用两个不同的get，一个返回我一个用户，另一个返回列表。我如何简化这两种方法（只要有意义） 这是两种方法：

第一种方法：

public List findFirstByTransactionId(String transactionId) {
        CriteriaBuilder builder = em.getCriteriaBuilder();
        CriteriaQuery<User> criteria = builder.createQuery(User.class);
        Root<User> root = criteria.from(User.class);
        criteria.select(root).where(builder.equal(root.get(User_.transactionId),transactionId));
        criteria.orderBy(builder.asc(root.get(User_.date)));
        TypedQuery<User> query = em.createQuery(criteria).setMaxResults(1);
         return query.getSingleResult();

第二种方法：

public List<User> findAllByTransactionId(String transactionId) {
        CriteriaBuilder builder = em.getCriteriaBuilder();
        CriteriaQuery<User> criteria = builder.createQuery(User.class);
        Root<User> root = criteria.from(User.class);
        criteria.select(root).where(builder.equal(root.get(User_.transactionId),transactionId));
        criteria.orderBy(builder.asc(root.get(User_.date)));
        TypedQuery<User> query = em.createQuery(criteria);
         return query.getResultList();

解决方法

更多附录。

建议添加布尔参数的其他答案是有效的，但是：干净的编码建议始终争取最少的参数数量。特别是不建议使用此类布尔参数。当然，这样做以避免代码重复在这里仍然有意义。

但是我会做什么：

• 是的，内部有一个使用布尔参数的private List<User> findFirstByTransactionId()
• 但是在您的 public 界面上，只需提供两种不同的方法，例如public List<User> findFirstUserByTransactionId()和`public List findUsersByTransactionId（）``

然后这两个公共方法可以调用内部方法并传递true / false。使用布尔值做出决定是实现的细节，您应该避免使它在事物的公共方面可见。

,

您在query上调用的方法与其他任何方法一样工作，因此您可以将某些调用放在if块中，如下所示：

public List<User> findByTransactionId(String transactionId,boolean onlyFirst) {
    CriteriaBuilder builder = em.getCriteriaBuilder();
    CriteriaQuery<User> criteria = builder.createQuery(User.class);
    Root<User> root = criteria.from(User.class);
    criteria.select(root).where(builder.equal(root.get(User_.transactionId),transactionId));
    criteria.orderBy(builder.asc(root.get(User_.date)));
    TypedQuery<User> query = em.createQuery(criteria);
    if (onlyFirst) {
      query=query.setMaxResults(1);
    }
    return query.getResultList();
}

,

最简单的选择通常是尝试将相同的代码移至其自己的方法。在这里，您有多个选择，其中之一可能是这样的：

1. 写一个新的（私有）方法：

private List<User> findFirstByTransactionId(String transactionId,boolean onlyOneResult) {
    CriteriaBuilder builder = em.getCriteriaBuilder();
    CriteriaQuery<User> criteria = builder.createQuery(User.class);
    Root<User> root = criteria.from(User.class);

    criteria.select(root).where(builder.equal(root.get(User_.transactionId),transactionId));
    criteria.orderBy(builder.asc(root.get(User_.date)));
    TypedQuery<User> query = em.createQuery(criteria);

    if (onlyOneResult) {
        query = query.setMaxResults(1);
    }
    
    return query.getResultList();
}

然后重构这样的现有方法：

public List<User> findFirstByTransactionId(String transactionId) {
    return findFirstByTransactionId(transactionId,false);
}

public List<User> findFirstByTransactionId(String transactionId) {
    return findFirstByTransactionId(transactionId,true);
}

现在您消除了8行重复的代码：）