Question

我有一个方法，它接受一个名为capability id的输入。基于能力id，我执行以交换机案例形式提出的业务实现。我的功能正常运行，但遇到的问题是声纳报告显示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的“清洁代码”一书。

Answer 2

我建议您使用多态重构此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();
    }
}

更多有用的资源，谈论＆＃39;开关气味＆＃39;：

Answer 3

如果您的代码没有问题，为什么只是为了避免SonarQube警告而使其更复杂？您可能不被允许更改SonarQube规则，但您当然可以通过使用以下内容注释您的方法来禁止警告：

@SuppressWarnings("squid:MethodCyclomaticComplexity")

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语句处理代码的最佳方法是什么？

4 个答案: