判断一个数组是否存在空的元素

超、凢脫俗 2022-02-01 10:37 324阅读 0赞

来源： [https://stackoverflow.com/questions/37046882/the-cyclomatic-complexity-of-this-method-is-greater-than-authorized][https_stackoverflow.com_questions_37046882_the-cyclomatic-complexity-of-this-method-is-greater-than-authorized]

public static boolean isNoneBlank(String... strings) {
        for (String string : strings) {
            if (!StringUtils.isNotBlank(string)) {
                return false;
            }
        }
        return true;
    }

Or if you're already on Java 8 with [Streams and Lambda][] support:

public static boolean isNoneBlank(String... strings) {
        return Arrays.stream(strings).allMatch(StringUtils::isNotBlank);
    }

Now you can make use of it as below:

public boolean isVoNotNull() {
        return isNoneBlank(this.cardNo, this.otp, this.password, this.userid, this.type, this.walletMobileNo);
    }

You could further reduce the boilerplate by removing the unnecessary `this`.

public boolean isVoNotNull() {
        return isNoneBlank(cardNo, otp, password, userid, type, walletMobileNo);
    }

This all was an appliance of the [Don't Repeat Yourself (DRY)][Don_t Repeat Yourself _DRY] software engineering principle.

That said, as [msandiford][] pointed out, it turns out that Apache Commons Lang `StringUtils`has since version 3.2 [already][] exactly this method. So if you don't have it yet, consider upgrading Apache Commons Lang to at least 3.2.

public boolean isVoNotNull() {
        return StringUtils.isNoneBlank(cardNo, otp, password, userid, type, walletMobileNo);
    }

[share][][improve this answer][]

[https_stackoverflow.com_questions_37046882_the-cyclomatic-complexity-of-this-method-is-greater-than-authorized]: https://stackoverflow.com/questions/37046882/the-cyclomatic-complexity-of-this-method-is-greater-than-authorized
[Streams and Lambda]: http://www.oracle.com/technetwork/articles/java/ma14-java-se-8-streams-2177646.html
[Don_t Repeat Yourself _DRY]: https://en.wikipedia.org/wiki/Don%27t_repeat_yourself
[msandiford]: https://stackoverflow.com/a/37047826
[already]: https://commons.apache.org/proper/commons-lang/apidocs/org/apache/commons/lang3/StringUtils.html#isNoneBlank%28java.lang.CharSequence...%29
[share]: https://stackoverflow.com/a/37047926
[improve this answer]: https://stackoverflow.com/posts/37047926/edit