32

我正在研究一个将 RequestDTO 发送到 Web 服务的类。我需要在发送之前验证请求。

请求可以从 3 个不同的地方发送，每个“请求类型”有不同的验证规则，例如 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()检查字符串是否为 nullisEmpty()并返回一个布尔值

在其中一种方法中，这给了我 28 的圈复杂度，所以我的问题是.. 是否有可能降低这种复杂度？- 如果是这样，我将如何去做？

最终我需要检查很多字段，如果没有大量检查，我看不到如何做到这一点：/

4

3 回答 3

48

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

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

然后你可以使用这个方法而不是if块：

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

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

于 2013-05-07T11:51:12.047 回答
5

另一种方法是在 Request 对象本身中强制执行该合同。如果某个字段是必需的或不能为空，请在创建请求时说明。

以这样的方式创建请求，当构造函数存在时，它是 100% 有效并准备就绪的。

我还将在 Request toString() 方法中创建该字符串版本。它应该知道如何渲染自己。

于 2013-05-07T11:51:30.020 回答
-1

装饰 StringBuilder，添加名为appendIfHasValue.

于 2021-04-10T21:33:30.560 回答