java isnoneblank,java – 此方法的Cyclomatic Complexity大于授权

您需要分析重复的代码并将其重构为可重用的方法.

鉴于您的原始片段,

public boolean isVoNotNull() {

return null != this.cardNo && StringUtils.isNotBlank(this.cardNo)

&& null != this.otp && StringUtils.isNotBlank(this.otp)

&& null != this.password && StringUtils.isNotBlank(this.password)

&& null != this.userid && StringUtils.isNotBlank(this.userid)

&& null != this.type && StringUtils.isNotBlank(this.type)

&& null != this.walletMobileNo && StringUtils.isNotBlank(this.walletMobileNo);

}

我们可以识别以下重复部分：

null != this.xxx && StringUtils.isNotBlank(this.xxx)

鉴于StringUtils#isNotBlank()already检查为null,我们可以进一步简化它.

StringUtils.isNotBlank(this.xxx)

鉴于你需要多次调用它,最好将它重构为一个采用variable number of arguments的方法,该方法在循环中检查它们.

public static boolean isNoneBlank(String... strings) {

for (String string : strings) {

if (!StringUtils.isNotBlank(string)) {

return false;

}

}

return true;

}

或者,如果您已经使用Streams and Lambda支持Java 8：

public static boolean isNoneBlank(String... strings) {

return Arrays.stream(strings).allMatch(StringUtils::isNotBlank);

}

现在您可以使用它,如下所示：

public boolean isVoNotNull() {

return isNoneBlank(this.cardNo, this.otp, this.password, this.userid, this.type, this.walletMobileNo);

}

您可以通过删除不必要的内容来进一步减少样板.

public boolean isVoNotNull() {

return isNoneBlank(cardNo, otp, password, userid, type, walletMobileNo);

}

也就是说,正如msandiford所指出的那样,事实证明,Apache Commons Lang StringUtils从版本3.2 already开始就是这种方法.因此,如果您还没有,请考虑将Apache Commons Lang升级到至少3.2.

public boolean isVoNotNull() {

return StringUtils.isNoneBlank(cardNo, otp, password, userid, type, walletMobileNo);

}