使用 Java 8 Predicates 改进代码 - 比较多个参数

Question

我不确定我有正确的标题。 我想对请求执行一些验证。 我在现有代码的很多地方都有这个条件测试。 它位于实用程序类中。 该方法是DoesRequestLineFallsWithinDateRange(object)。

我想知道我是否可以通过使用谓词并停用该方法来改进代码并在 Java 8 中更加优雅。

处理请求：我想检查每一行是否满足条件。 所以调用方法调用这个实用程序类来检查每一行是否在数据范围内。 在此示例中，是订阅期内的项目 (requestLine)。

一个请求进来，一些处理将订阅成员对象添加到请求行。 然后在每一行上调用实用程序方法以检查请求行是否在订阅周期内。

该方法首先检查它应该使用的日期字段是否不为空，以防止出现空异常。

然后使用请求行日期，与订阅成员开始/结束日期字段进行比较以确定日期是否在范围内。

我想用 Java 8 Predicate 函数调用替换该方法。

这是测试集合中每一行是否在数据范围内的方法示例。

public static boolean DoesRequestLineFallsWithinDateRange(RequestLine requestLine) {
    if (isDatesNotNull(requestLine)) {
      return requestLine.getServiceStartDate().compareTo(requestLine.getSubscription().getStartDate()) >= 0
          && requestLine.getServiceStartDate().compareTo(requestLine.getSubscription().getEndDate()) <= 0;
    }
   return false;
  }

public class RequestLine(){
   private Date serviceStartDate;
   private Subscription subscription;
.....

}

Answer 1

这是使用 Java 8 Predicates和LocalDate API 编写的示例代码

    public boolean doesRequestLineFallsWithinDateRange(RequestLine requestLine)
    {
        BiPredicate<LocalDate, LocalDate> afterDatePredicate = (startData, endDate) -> (startData.isAfter(endDate));
        BiPredicate<LocalDate, LocalDate> beforeDatePredicate = (startData, endDate) -> (startData.isBefore(endDate));

        if (Objects.isNull(requestLine))
        {
            return false;
        }

        LocalDate serviceStartDate = requestLine.getServiceStartDate();
        LocalDate startDate = requestLine.getSubscription().getStartDate();
        LocalDate endDate = requestLine.getSubscription().getEndDate();

        return afterDatePredicate.test(serviceStartDate, startDate) && beforeDatePredicate.test(serviceStartDate,
                endDate);

    }

引用的 Java 代码

请求线

public class RequestLine

{
    private Subscription subscription;
    private LocalDate serviceStartDate;

    // getters and setters
}

订阅.java

public class Subscription
{
    private LocalDate startDate;
    private LocalDate endDate;

    // getters and setters
}

Answer 2

我尝试集成您的代码以使用方法引用作为谓词并过滤 RequestLine 流。 有趣的方法是execute ：它接收一个流并使用对 doRequestLineFallsWithinDateRange 方法的引用对其进行过滤。

import java.util.Date;
import java.util.stream.Stream;

public class RequestLine {

    public void execute(Stream<RequestLine> stream) {
        // this is the interesting code
        stream.filter(this::doesRequestLineFallsWithinDateRange);
    }

    public boolean doesRequestLineFallsWithinDateRange(RequestLine requestLine) {
        if (isDatesNotNull(requestLine)) {
            return requestLine.getServiceStartDate().compareTo(requestLine.getSubscription().getStartDate()) >= 0
                    && requestLine.getServiceStartDate().compareTo(requestLine.getSubscription().getEndDate()) <= 0;
        }
        return false;
    }

    //code created to complete the example
    public static class Subscription {

        private Date startDate;

        public Date getEndDate() {
            return endDate;
        }

        private Date endDate;

        public Date getStartDate() {
            return  startDate;
        }
    }



    private boolean isDatesNotNull(RequestLine requestLine) {
        return requestLine.serviceStartDate!=null;
    }


    public Date getServiceStartDate() {
        return serviceStartDate;
    }

    public Subscription getSubscription() {
        return subscription;
    }

    private Date serviceStartDate;
    private Subscription subscription;

}

Answer 3

使用Predicate::and链接多个谓词

Predicate<RequestLine> rlPred = this::isDatesNotNull;

rlPred = rlPred.and(p -> p.getServiceStartDate().compareTo(p.getSubscription().getStartDate()) >= 0)
               .and(p -> p.getServiceStartDate().compareTo(p.getSubscription().getEndDate()) <= 0);