如何重构并修复冗余的空检查

t5zmwmid 于 2021-07-06 发布在 Java

关注(0)|答案(2)|浏览(328)

在以下代码块中获取冗余的空检查错误。有人能帮我重构一下吗：

private Date pickRecent(Date d1, Date d2, Date d3, Date crd) {
        logger.debug("d1: " + d1 + " d2: " + d2 + " d3: " + d3 + " crd: " + crd);
        // all null
        if (d1 == null && d2 == null && d3 == null) return crd;
        logger.debug("through all");
        // two null
        if (d1 == null && d2 == null) return d3;
        if (d1 == null && d3 == null) return d2;
        if (d2 == null && d3 == null) return d1;
        logger.debug("through two");
        // one null
        if (d1 == null) if (d2.after(d3)) return d2; else return d3;
        if (d2 == null) if (d1.after(d3)) return d1; else return d3;
        if (d3 == null) if (d1.after(d2)) return d1; else return d2;
        logger.debug("through one");
        // none null
        if (d1.after(d2) && d1.after(d3)) return d1;
        if (d2.after(d1) && d2.after(d3)) return d2;
        if (d3.after(d1) && d3.after(d2)) return d3;
        if (d1.equals(d2) && d1.equals(d3)) return d1;
        logger.debug("through none");
        return null;
    }

Java refactoring null-check

来源：https://stackoverflow.com/questions/63476020/how-do-i-refactor-do-fix-redundant-null-check

2条答案

按热度按时间

pobjuy321#

这里有一个可能的解决方案。。。

import java.util.Arrays;
import java.util.Comparator;
import java.util.Date;

public class RecentDate
{
  public static void main(String[] args)
  {
    System.out.println(pickRecent(new Date(1, 1, 1), new Date(2, 2, 2), new Date(3, 3, 3), new Date(4, 4, 4)));
    System.out.println(pickRecent(null, null, null));
  }

  private static Date pickRecent(Date...dates)
  {
    return Arrays.stream(dates).filter(d -> d != null).max(Comparator.naturalOrder()).orElse(null);
  }
}

输出：

Wed May 04 00:00:00 CST 1904
null

赞(0）回复(0）举报 2021-07-06

nwo49xxi2#

public static Date pickRecent(Date one, Date two, Date three, Date crd) {
    Set<Date> unique = new TreeSet<>(Comparator.reverseOrder());

    if (one != null)
        unique.add(one);
    if (two != null)
        unique.add(two);
    if (three != null)
        unique.add(three);

    return unique.size() == 1 ? unique.iterator().next() : crd;
}

赞(0）回复(0）举报 2021-07-06

我来回答

如何重构并修复冗余的空检查

2条答案

相关问题

热门标签

最新问答