此 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());
}
}
最佳答案
我会首先为每个子表达式赋予有意义的名称并将结果分配给变量,然后在变量之间进行 && 检查。这将大大增加可读性。
而且你的条件似乎都有这种形式
(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)
那么这个方法就如写的那样工作。
关于java - 降低此方法的 boolean 表达式复杂度?,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/25209026/