使用具有高度圈复杂度的if/switch语句处理代码的最佳方法是什么？
使用具有高度圈复杂度的if/switch语句处理代码的最佳方法是什么？
提问于 2016-10-03 06:04:29
我有一个方法，它接受一个名为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
}
}更新：我能够解决我的问题，我在下面的答案中发布了一个解决方案。
Stack Overflow用户
发布于 2016-10-04 05:25:56
我建议您使用多态重构这个开关语句(这是一个代码嗅觉)。一种可能的方法是这样做：
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();
}
}讨论“开关气味”的更有用的资源：
页面原文内容由Stack Overflow提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接：
https://stackoverflow.com/questions/39825386
复制相关文章
相似问题
腾讯云开发者
