java - 降低此方法的 boolean 表达式复杂度？

Question

此 isValid() 方法最多可设置 6 个属性以进行验证。如果未设置特定属性，则不需要对其进行验证。逻辑似乎按照我希望的方式工作。如何降低此方法的 boolean 表达式复杂度？

import java.util.ArrayList;
import java.util.List;

public class Valid {

private int foo;
private int bar;
private int foobar;
private String foofoo;
private String barbar;
private String foobarfoobar;

private int myfoo;
private int mybar;
private int myfoobar;
private String myfoofoo;
private List<String> mybarbarlist = new ArrayList<String>();
private String myfoobarfoobar;

private boolean validate;

public boolean isValid() {

    validate = false;

    if ((((foo > 0) && (foo == myfoo)) || (foo == 0))
            && (((bar > 0) && (bar == mybar)) || (bar == 0))
            && (((foobar > 0) && (foobar == myfoobar)) || (foobar == 0))
            && (((foofoo != null) && (foofoo.equals(myfoofoo))) || (foofoo == null))
            && (((barbar != null) && (mybarbarlist.contains(barbar))) || (barbar == null))
            && (((foobarfoobar != null) && (foobarfoobar
                    .equals(myfoobarfoobar))) || (foobarfoobar == null)))
        validate = true;
    return validate;
}

public static void main(String[] args) {

    Valid valid = new Valid();

    // valid.foo = 1;
    // valid.bar = 1;
    // valid.foobar = 1;
    // valid.foofoo = "1";
    // valid.barbar = "1";
    // valid.foobarfoobar = "1";

    // valid.myfoo = 1;
    // valid.mybar = 1;
    // valid.myfoobar = 1;
    // valid.myfoofoo = "1";
    // valid.mybarbarlist.add("1");
    // valid.myfoobarfoobar = "1";

    System.out.println(valid.isValid());
}
}

Answer 1

我会首先为每个子表达式赋予有意义的名称并将结果分配给变量，然后在变量之间进行 && 检查。这将大大增加可读性。

而且你的条件似乎都有这种形式

(X != null && X == value) || X == null

可以使用短路评估将其简化为

X == null || X == value

因为如果X == null评估为真则不需要检查右侧，或者如果 X == null计算结果为 false，它需要检查右侧，您已经知道它不为 null，因此您不需要检查右侧表达式中的 null。

结合两者你会得到这样的东西:

public boolean isValid() {

    validate = false;

    boolean fooCond = (foo == 0) || (foo == myfoo);
    boolean barCond = (bar == 0) || (bar == mybar);
    boolean foobarCond = (foobar == 0) || (foobar == myfoobar);
    boolean foofooCond = (foofoo == null) || foofoo.equals(myfoofoo);
    boolean barbarCond = (barbar == null) || mybarbarlist.contains(barbar);
    boolean foobarfoobarCond = (foobarfoobar == null) || foobarfoobar.equals(myfoobarfoobar);

    if (fooCond
          && barCond
          && foobarCond
          && foofooCond;
          && barbarCond
          && foobarfoobarCond) {
        validate = true;
    }

    return validate;
}

编辑

这对您要检查的表达式不太有效

((foo > 0) && (foo == myfoo)) || (foo == 0)

因为如果foo < 0它返回错误。如果你真的想

((foo > 0) && (foo == myfoo)) || (foo <= 0)

那么这个方法就如写的那样工作。