减少圈复杂的switch语句 - 声纳

Question

我想减少我的开关罩的圈复杂度 我的代码是：

public String getCalenderName() { 
     switch (type) { 
    case COUNTRY: 
     return country == null ? name : country.getName() + HOLIDAY_CALENDAR; 
    case CCP: 
     return ccp == null ? name : ccp.getName() + " CCP" + HOLIDAY_CALENDAR; 
    case EXCHANGE: 
     return exchange == null ? name : exchange.getName() + HOLIDAY_CALENDAR; 
    case TENANT: 
     return tenant == null ? name : tenant.getName() + HOLIDAY_CALENDAR; 
    default: 
     return name; 
    } 
} 

此代码块的复杂性是16，并希望将其降低到10 国家， ccp，exchange和tenant是我不同的对象。基于类型我会调用它们各自的方法。

Answer 1

据我所知，不要在switch语句中使用return语句。在switch语句之后使用变量应用该逻辑。 创建一个检查空值的方法，并从开关调用该方法，那么你将能够减少圆环复杂性

Answer 2

我相信这是一个Sonar警告。我认为Sonar警告不是必须做的规则，而只是指南。您的代码块是READABLE和MAINTAINABLE原样。它已经很简单了，但如果你真的想改变它，你可以尝试下面的两种方法，看看复杂度是否变低：

注意：我现在没有编译器，所以可能会出现错误，对不起关于那个提前。

第一种方法：

Map<String, String> multipliers = new HashMap<String, Float>(); 
    map.put("country", country); 
    map.put("exchange", exchange); 
    map.put("ccp", ccp); 
    map.put("tenant", tenant); 

然后我们就可以使用地图抢权元素

return map.get(type) == null ? name : map.get(type).getName() + HOLIDAY_CALENDAR; 

第二个办法：

你所有的对象都具有相同的方法，这样你就可以在其中添加一个Interface和getName()方法，并更改您的方法签名，如：

getCalendarName(YourInterface yourObject){ 
    return yourObject == null ? name : yourObject.getName() + HOLIDAY_CALENDAR; 
} 

Answer 3

您可以删除所有空比较，并在切换大小写前检查它。在这种情况下，复杂性会减少4或更多。

Answer 4

如果您的对象：country，cpp，exchange和tenant共享相同的界面，例如ObjectWithGetName你可以重构你的代码如下：

public String getCalenderName() { 
    ObjectWithGetNameMethod calendarType = null; 
    switch (type) { 
     case COUNTRY: 
      calendarType = country; 
      break; 
     case CCP: 
      calendarType = cpp; 
      break; 
     case EXCHANGE: 
      calendarType = exchange; 
      break; 
     case TENANT: 
      calendarType = tenant; 
      break; 
     default: 
      calendarType = null; 
    } 
    return (calendarType != null ? (calendarType.getName() + HOLIDAY_CALENDAR) : name); 
    } 

而且我认为这将是很好的开关移动到不同的方法，因为它看上去就像巫婆会在许多不同的地方使用。

Answer 5

public String getName() { 
    if (type == null) { 
     return name; 
    } 
    if (type == BusinessCalendarType.COUNTRY) { 
     return country == null ? name : country.getName() + HOLIDAY_CALENDAR; 
    } else if (type == BusinessCalendarType.CCP) { 
     return ccp == null ? name : ccp.getName() + " CCP" + HOLIDAY_CALENDAR; 
    } else if (type == BusinessCalendarType.EXCHANGE) { 
     return exchange == null ? name : exchange.getName() + HOLIDAY_CALENDAR; 
    } else if (type == BusinessCalendarType.TENANT) { 
     return tenant == null ? name : tenant.getName() + HOLIDAY_CALENDAR; 
    } else { 
     return name; 
    } 
} 

这个工作对我来说

Answer 6

如果你的第一个目标，是唯一减少圈复杂度，你应该创建一些方法来获取的名称，如下面的各种方式。在具体的例子

public String getCalenderName() { 
    switch (type) { 
    case COUNTRY: 
     return getCountryName(); 
    case CCP: 
     return getCcpName(); 
    case EXCHANGE: 
     return getExchangeName(); 
    case TENANT: 
     return getTenantName(); 
    default: 
     return name; 
    } 
} 

private String getCountryName() { 
    return country == null ? name : country.getName() + HOLIDAY_CALENDAR; 
} 

private String getCcpName() { 
    return ccp == null ? name : ccp.getName() + " CCP" + HOLIDAY_CALENDAR; 
} 

private String getExchangeName() { 
    return exchange == null ? name : getName.toString() + HOLIDAY_CALENDAR; 
} 

private String getTenantName() { 
    return tenant == null ? name : getName.toString() + HOLIDAY_CALENDAR; 
} 

---

注意，我假设你有1类，聚（至少）4点十分相似的行为。重构肯定会更有意义，例如为了实现一个基本实现（抽象与否）和4个其他继承类。