使用具有高圈复杂度的 if/switch 语句处理代码的最佳方法是什么？

Question

我有一个方法接受名为功能 ID 的输入。基于能力id，我以switch case的形式执行提出的业务实现。我有我的功能，但遇到了一个问题，即声纳报告显示 12-14 左右的高圈复杂度。我工作的公司使用 10 作为最大圈复杂度的标准。我的考虑是，如果我碰巧将代码分成太多块，代码的可读性就会受到影响。

注：- 我没有更改声纳规则的权限。

很遗憾，我无法共享代码。不过，代码将采用以下方式，

// Dependency Injected
private someService;

public void processCapability(..., String capabilityId) {
    switch (capabilityId) {
        case ORDER_DISPENSED_WITH_SOURCE1:
            someService.doDispense1();
        case ORDER_DISPENSED_WITH_SOURCE2:
            someService.doDispense2();
        case ORDER_REJECTED:
            someService.doReject();
        case ORDER_CANCEL:
            someService.doCancel();
        case ORDER_PURGE:
            someService.doPurge();
            ...
            default: throw exception
    }
}

更新：我能够解决我的问题，我在下面的答案中发布了解决方案。

Answer 1

有多种方法可以避免大型 if-else 语句。

也许最常见的模式是 template method pattern 和 strategy pattern。

由于您没有提供任何代码，因此很难帮助您降低圈复杂度，但我绝对确定，有一种方法可以避免那些 if-else-s。

我建议您阅读一些有关该主题的内容；我将为您提供一些我认为有用的资源：

Using strategy to replace if else
Using command pattern to replace conditional logic
我鼓励你使用枚举，它们在条件逻辑中比你想象的更有用：using enums instead of switch
此外，了解各种设计模式也很好，也许您可以找到合适的here

此外，可读代码意味着它易于理解。我同意@Erwin Bolwidt 的评论，如果您创建更多具有好名字的方法，它将更易于阅读。要阅读有关该主题的更多信息，请查看 Robert C. Martin 的书 "Clean Code"。

Answer 2

如果你的代码没有问题，为什么要让它更复杂只是为了避免 SonarQube 警告？您可能不允许更改 SonarQube 规则，但您当然可以通过使用以下注释您的方法来抑制警告：

@SuppressWarnings("squid:MethodCyclomaticComplexity")

Answer 3

我建议您使用多态重构此 switch 语句（这是一种代码味道）。一种可能的方法是这样的：

public interface Order {
    void processCapability(...);
}

public final class RejectedOrder implements Order {
    private final SomeService someService;

    public RejectedOrder(SomeService someService) {
        this.someService = someService;
    }

    @Override
    public void processCapability(...) {
        someService.doReject();
    }
}

public final class CancelledOrder implements Order {
    private final SomeService someService;

    public RejectedOrder(SomeService someService) {
        this.someService = someService;
    }

    @Override
    public void processCapability(...) {
        someService.doCancel();
    }
}

更有用的资源说'switch smell'：

Answer 4

我想出了一种处理大型 if/switch 语句的方法。基于这些想法，我从你们每个人那里得到了。我编译成一个简单的解决方案，易于理解和修改，也处理了复杂的部分。请在下面找到我的解决方案，

// Dependency Injected
private someService;

public void processCapability(..., String capabilityId) {

  Boolean isCapabilityProcessed = processDispenseCapabilities(...) || processUpdateCapabilities(..);

  if(isCapabilityProcessed) {
    throw exception("Invalid Capability");
  }
}

private Boolean processDispenseCapabilities(..,String capabilityId) {

  Boolean result = false;

  switch (capabilityId) {
        case ORDER_DISPENSED_WITH_SOURCE1:
            someService.doDispense1();
            result = true;
        case ORDER_DISPENSED_WITH_SOURCE2:
            someService.doDispense2();
            result = true;
        case ORDER_REJECTED:
            someService.doReject();
            result = true;
            ...
        default: //do nothing
    }

  return result;
}

private Boolean processUpdateCapabilities(..,String capabilityId) {
  Boolean result = false;

    switch (capabilityId) {
        case ORDER_CANCEL:
            someService.doCancel();
            result = true;
        case ORDER_PURGE:
            someService.doPurge();
            result = true;
            ...
        default: //do nothing
    }

  return result;
}

使用具有高圈复杂度的 if/switch 语句处理代码的最佳方法是什么？

What is the best way to handle code with if/switch statements having high cyclomatic complexity?

java

static-analysis

sonarqube