如何降低圈复杂度？

一尘不染

如何降低圈复杂度？

java

我正在研究将RequestDTO发送到Web服务的类。我需要先验证请求，然后再发送。

可以从3个不同的地方发送请求，每个“
requesttype”都有不同的验证规则，例如request1必须具有名称和电话号码，request2必须具有地址，等等）

我有一个DTO，其中包含很长的字段列表（名称，地址，城市，电话号码等），无论请求是哪种类型，DTO都发送相同的消息。

我创建了3种不同的验证方法，并根据类型调用了适当的方法。

在每种方法中，我都有很长的if-else清单，用于检查每种请求类型所需的字段。

private void validateRequest1(Request request) {
    StringBuilder sb = new StringBuilder();
    if (null == request) {
        throw new IllegalArgumentException("Request is null");
    }
    if (isFieldEmpty(request.getName())) {  *see below
        sb.append("name,"));
    }
    if (isFieldEmpty(request.getStreet())) {
        sb.append("street,"));
    }
    ...

isFieldEmpty()检查字符串是否为null isEmpty()并返回一个布尔值

在这些方法之一中，这给我带来了28的圈复杂度，所以我的问题是..可以降低这种复杂度吗？-如果是这样，我将如何去做？

最终，我需要检查很多字段，而且如果不进行大量检查，就看不到如何做到这一点：/

阅读 661

2020-12-03

共1个答案

一尘不染

一种简单的方法是将检查升级为单独的方法：

private String getAppendString(String value, String appendString) {
    if (value == null || value.isEmpty()) {
        return "";
    }
    return appendString;
}

然后可以使用此方法代替if块：

sb.append(getAppendString(request.getStreet(), "street,");

这会将复杂度从28降低到3。永远记住：高复杂度计数表明一种方法正在尝试做太多事情。像我们在这里所做的那样，可以通过将问题分成多个小部分来解决复杂性。

2020-12-03