java - 以 Java 8 风格重构 if-else if 链

Question

关闭。此题需要 details or clarity 。目前不接受答案。

---

想要改进这个问题吗？通过 editing this post 添加详细信息并澄清问题.

已关闭 8 年前。

Improve this question

我有一个带有以下 if-else-if 链的方法:

        if(downstreamActual.getNumber() <= downstreamRecommended.getNumber()){
            downstreamActual.setInvalid();
            showErrorWindow();
            throw new NumberFormatException();
        }else if(upstreamActual.getNumber() <= upstreamRecommended.getNumber()){
            upstreamActual.setInvalid();
            showErrorWindow();
            throw new NumberFormatException();
        }else if(biUpstreamActual.getNumber() <= biUpstreamRecommended.getNumber()){
            biUpstreamActual.setInvalid();
            showErrorWindow();
            throw new NumberFormatException();
        }else if(biDownstreamActual.getNumber() <= biDownstreamRecommended.getNumber()){
            biDownstreamActual.setInvalid();
            showErrorWindow();
            throw new NumberFormatException();
        }

每一步我们都做相同的工作(为 boolean 表达式中使用的第一个对象调用相同的方法，调用 showErrorWindow() 并抛出 Exception )

有哪些好的技术，特别是使用 Java 8 来使代码更易于管理？

Answer 1

根据您的评论，我认为您不需要 Java 8 构造。

只需使用一个方法:

 public void validate (NumberTextBox actual, NumberTextBox recommended) 
 {
     if(actual.getNumber() <= recommended.getNumber()) {
        actual.setInvalid();
        showErrorWindow();
        throw new NumberFormatException();
     }
 }

然后调用它 4 次:

 validate (downstreamActual,downstreamRecommended);
 validate (upstreamActual,upstreamRecommended);
 ...

由于第一个失败的会抛出异常，从而阻止其余的测试，因此您不需要 if else-if 结构。