Lambda和清洁代码的一个重构案例 - frankel

20-06-25 banq

原代码:

List<Person> persons = ...;
persons.stream().filter(p -> {
    if (p.getGender() == Gender.MALE) {
        return true;
    }
    LocalDate now = LocalDate.now();
    Duration age = Duration.between(p.getBirthDate(), now);
    Duration adult = Duration.of(18, ChronoUnit.YEARS);
    return age.compareTo(adult) > 0) {
        return true;
    }
    return false;
}).map(p -> p.getFirstName() + " " + p.getLastName())
  .collect(Collectors.toList());

它很长,甚至令人费解。

第一步是应用正确的命名,并将逻辑移到它所属的位置。

public class Person {

    // ...

    public boolean isMale() {
        return getGender() == Gender.MALE;
    }

    public boolean isAdult(LocalDate when) {
        Duration age = Duration.between(birthDate, when);
        Duration adult = Duration.of(18, ChronoUnit.YEARS);
        return age.compareTo(adult) > 0;
    }
}

这个小的重构已经提高了lambda的可读性:

persons.stream().filter(p -> {
    if (p.isMale()) {
        return true;
    }
    LocalDate now = LocalDate.now();
    return p.isAdult(now);
}).map(p -> p.getFirstName() + " " + p.getLastName())
        .collect(Collectors.toList());

命名lambda:

// Implementation details
Predicate<Person> isMaleOrAdult = p -> {
    if (p.isMale()) {
        return true;
    }
    LocalDate now = LocalDate.now();
    return p.isAdult(now);
};
Function<Person, String> concatenateFirstAndLastName = p -> p.getFirstName() + " " + p.getLastName();

// Core
persons.streams()
    .filter(isMaleOrAdult)
    .map(concatenateFirstAndLastName)

流(最后一行)变得更具可读性,而不是隐藏在实现细节的后面。它不会阻止开发人员阅读它们,只有在必要时才阅读。

                   

1