java - 降低 if-else 语句代码的复杂性

Question

我的代码分析插件提示包含以下代码的方法中的代码复杂性。我注意到以下代码看起来可以组合，但我不知道如何做到这一点:

for(Command command : commands) {
    if (command instanceof AddCommand || command instanceof UpdateCommand) {
        if (!isMaturityDateInPast() && !paymentDueDate().isAfter(LocalDate.now())) {
            command.execute(request);
        }
    } else {
        command.execute(request);
    }
}

我尝试引入 boolean 变量并在 if 和 else 语句中设置它，但这只是增加了更多代码行。我不太擅长逻辑地放置具有共同点的代码部分。我可以告诉这个 if-else 可以组合起来，但我不知道该怎么做。有人可以透露一些信息吗？

Answer 1

我会尽早继续以避免重复command.execute()。我认为不值得为了它而组合一个函数中的条件或创建另一个函数。

for(Command command : commands) {
  if (command instanceof AddCommand || command instanceof UpdateCommand) {
    if (isMaturityDateInPast() || paymentDueDate().isAfter(LocalDate.now())) {
        continue;
    }
  }
  command.execute();
}