如何降低圈复杂度？
我正在开发一个将RequestDTO发送到Web服务的类。 我需要在发送之前validation请求。
请求可以从3个不同的地方发送，每个“请求类型”有不同的validation规则，例如request1必须有名称和phonenumber，request2必须有地址，等等）
我有一个DTO，其中包含很长的字段列表（名称，地址，城市，电话号码等），无论是哪种类型的请求都是相同的DTO。
我创建了3种不同的validation方法，并根据类型调用适当的方法。
在每个方法中，我都有一长串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的圈复杂度，所以我的问题是……是否有可能降低这种复杂性？ – 如果是的话，我该怎么做呢？
最终我需要检查很多字段，如果没有大量的检查，我无法看到如何做到这一点：/
一种简单的方法是将检查提升为单独的方法：
private String getAppendString(String value, String appendString) { if (value == null || value.isEmpty()) { return ""; } return appendString; }
然后你可以使用这个方法而不是if块：
sb.append(getAppendString(request.getStreet(), "street,");
这将复杂性从28降低到3.永远记住：高复杂度计数表明方法试图做太多。 复杂性可以通过将问题分成更小的部分来处理，就像我们在这里所做的那样。
另一种方法是在Request对象本身中强制执行该契约。 如果字段是必需的或不能为空，请在创建请求时这样说。
以这样的方式创建请求，使其100％有效并且在构造函数存在时准备就绪。
我还在Request toString（）方法中创建了String版本。 它应该知道如何渲染自己。