Question

我有一个用于构建本机sql查询的方法。我有4个String构建器作为输入，并且基于不同的条件，我需要构建查询。

这是代码

private void appendConditions(StringBuilder query, StringBuilder condition, StringBuilder condition2,
        StringBuilder condition3) {
    if (StringUtils.isNotEmpty(condition.toString())) {
        query.append(ApplicationConstants.AND+"(");
        query.append("( " + condition + " )");
        if (StringUtils.isNotEmpty(condition2.toString())) {
            query.append(ApplicationConstants.AND);
            query.append("( " + condition2 + " )");
            if (StringUtils.isNotEmpty(condition3.toString())) {
                query.append(ApplicationConstants.AND);
                query.append("( " + condition3 + " )");
            }

        } else {
            if (StringUtils.isNotEmpty(condition3.toString())) {
                query.append(ApplicationConstants.AND);
                query.append("( " + condition3 + " )");
            }
        }
        query.append(" )");
    } else {
        if (StringUtils.isNotEmpty(condition2.toString())) {
            query.append(ApplicationConstants.AND+"(");
            query.append("( " + condition2 + " )");
            if (StringUtils.isNotEmpty(condition3.toString())) {
                query.append(ApplicationConstants.AND);
                query.append("( " + condition3 + " )");
            }
            query.append(" )");
        } else {
            if (StringUtils.isNotEmpty(condition3.toString())) {
                query.append(ApplicationConstants.AND+"(");
                query.append("( " + condition3 + " )");
                query.append(" )");
            }
        }
    }
}

当我运行声纳报告时，它说这种方法的认知复杂性更高。我可以简化这种方法，避免很多if-else循环

谢谢！

Answer 1

简化可以归纳为：具有条件变量。然后，Stream解决方案只需几秒钟。

private void appendConditions(StringBuilder query, StringBuilder... conditions) {
    String ands = Stream.of(conditions)
            .map(StringBuilder::toString)
            .filter(StringUtils::isNotEmpty)
            .map(c -> "(" + c + ")")
            .collect(Collectors.joining(ApplicationConstants.AND));
    if (!ands.isEmpty()) {
        query.append(ApplicationConstants.AND).append(ands);
    }
}

如果需要，您可以使用Stream.of(condition1, condition2, condition3)。

Answer 2

if(StringUtils.isNotEmpty(condition.toString()) &&  StringUtils.isNotEmpty(condition.toString()) && StringUtils.isNotEmpty(condition.toString())) {
    // If all not empty
    query.append(ApplicationConstants.AND+"(");
    query.append("( ").append(condition).append(" )");
    query.append(ApplicationConstants.AND);
    query.append("( ").append(condition2).append(" )");
    query.append(ApplicationConstants.AND);
    query.append("( ").append(condition3).append(" )");
} else if(StringUtils.isNotEmpty(condition.toString()) && !StringUtils.isNotEmpty(condition.toString()) && StringUtils.isNotEmpty(condition.toString())) {
       // Two is empty.. skip two
        query.append(ApplicationConstants.AND+"(");
        query.append("( ").append(condition).append(" )");
        query.append(ApplicationConstants.AND);
        query.append("( ").append(condition3).append(" )");
} else if(!StringUtils.isNotEmpty(condition.toString()) && StringUtils.isNotEmpty(condition2.toString()) && StringUtils.isNotEmpty(condition3.toString())) {
    // One is empty , two and three are good to go
    query.append(ApplicationConstants.AND+"(");
    query.append("( ").append(condition2).append(" )");
    query.append(ApplicationConstants.AND);
    query.append("( ").append(condition3).append(" )");
    query.append(" )");

} else {
    // If only 3 is there.
    query.append(ApplicationConstants.AND+"(");
    query.append("( ").append(condition3).append(" )");
    query.append(" )");
}

简化嵌套的if else循环会导致Java的认知复杂性

2 个答案: