我正在开发一个将RequestDTO发送到Web服务的类。我需要在发送请求之前验证请求。
请求可以从3个不同的地方发送,并且每个“请求类型”有不同的验证规则,例如: request1必须有名称和phonenumber,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,所以我的问题是......是否有可能降低这种复杂性? - 如果是的话,我该怎么做呢?
最终我需要查看很多字段,如果没有大量的检查,我无法看到如何做到这一点:/
答案 0 :(得分:32)
一种简单的方法是将检查提升为单独的方法:
private String getAppendString(String value, String appendString) {
if (value == null || value.isEmpty()) {
return "";
}
return appendString;
}
然后您可以使用此方法代替if
块:
sb.append(getAppendString(request.getStreet(), "street,");
这会将复杂性从28降低到3.永远记住:高复杂度计数表明方法试图做太多。复杂性可以通过将问题分成更小的部分来处理,就像我们在这里所做的那样。
答案 1 :(得分:4)
另一种方法是在Request对象本身中强制执行该契约。如果字段是必需的或不能为空,请在创建请求时这样说。
以这样一种方式创建请求,使其100%有效,并在构造函数存在时准备就绪。
我还在Request toString()方法中创建了String版本。它应该知道如何渲染自己。
答案 2 :(得分:-1)
装饰 StringBuilder,添加名为 appendIfHasValue
的方法。