如何在不能更改接口的情況下重構此方法？

Question

如果無法修改Event接口，如何重構以下方法？ PMD報告太複雜，findbugs報告ITC_INHERITANCE_TYPE_CHECKING。還有幻數，如3,4,5等。

public int getEventCode(Event event) { 
     if (event instanceof OneEvent) { 
      return 1; 
    } 
     if (event instanceof TwoEvent) { 
      return 2; 
     } 
     if (event instanceof ThreeEvent) { 
      return 3; 
     } 
     if (event instanceof FourEvent) { 
      return 4; 
     } 
     if (event instanceof FiveEvent) { 
      return 5; 
     } 
     if (event instanceof SixEvent) { 
      return 6; 
     } 
     if (event instanceof SevenEvent) { 
      return 7; 
     } 
     if (event instanceof EightEvent) { 
      return 8; 
     } 
     if (event instanceof NineEvent) { 
      return 9; 
     } 
     if (event instanceof TenEvent) { 
      return 10; 
     } 
     return event.getClass().hashCode() + 10; 
    } 

Answer 1

可以使用例如一個List<Class<?>>：

private static final List<Class<? extends Event>> EVENT_CLASSES 
    = Arrays.asList(OneEvent.class, ...); 

然後：

public int getEventCode(final Event event) 
{ 
    final Class<? extends Event> c = event.getClass(); 
    final int index = EVENT_CLASSES.indexOf(c); 
    return index != -1 ? index + 1 : c.hashCode() + 10; 
} 

注：需要事件的確切類的，不是得到的（即OneEvent而不是OneDerivedEvent）。否則，測試稍微複雜一點，但仍然可行。

至於：

FindBugs的報告ITC_INHERITANCE_TYPE_CHECKING

是的，這是因爲instanceof檢查。

但是：代碼有一個基本缺陷。 不能保證.hashCode()在兩個不同的JVM執行之間返回相同的值。更重要的是，它可以返回負值。這意味着它可以返回，例如-4作爲一個值;這意味着6將返回「其他事件」，因此與SixEvent衝突。

考慮一個重構！

Answer 2

這些類型的塊是否有一些在代碼庫中浮動？如果是這樣的話，可能會有所幫助。

Answer 3

嗯，這是不好的instanceof使用，但如果你不能改變事件接口甚至添加「詮釋的getType（）」或類似的，至少你可以調整你的代碼：

使用「否則如果「結構，而不是結構。我建議首先放置更深的繼承的事件類型，然後再考慮更一般的事件聲明（如果測試事件instanceof事件，您將始終成爲true）。

這種方式至少可以降低複雜性。

假設所有的活動直接實現事件接口，而且沒有其他繼承問題進行檢查：

public int getEventCode(Event event) { 
    int result = event.getClass().hashCode() + 10; 
    if (event instanceof OneEvent) { 
     result = 1; 
    } 
    else if (event instanceof TwoEvent) { 
     result = 2; 
    } 
    else if (event instanceof ThreeEvent) { 
     result = 3; 
    } 
    else if (event instanceof FourEvent) { 
     result = 4; 
    } 
    else if (event instanceof FiveEvent) { 
     result = 5; 
    } 
    else if (event instanceof SixEvent) { 
     result = 6; 
    } 
    else if (event instanceof SevenEvent) { 
     result = 7; 
    } 
    else if (event instanceof EightEvent) { 
     result = 8; 
    } 
    else if (event instanceof NineEvent) { 
     result = 9; 
    } 
    else if (event instanceof TenEvent) { 
     result = 10; 
    } 
    return result; 
} 

我也改變了你的方法，因此只能申報一個返回變量，這通常被認爲是一個更好的編碼實踐，因爲它也有助於降低複雜性，即使它在這種特殊情況下沒有真正的技術優勢。

但正如我所說，如果TenEvent延伸FiveEvent你也可以有問題。在這種情況下，我會建議使用您收到事件實例的類，像這樣：

public int getEventCode(Event event) { 
    int result = event.getClass().hashCode() + 10; 
    String eventClass = event.getClass().getSimpleName(); 
    if ("OneEvent".equals(eventClass) { 
     result = 1; 
    } 
    else if ("TwoEvent".equals(eventClass)) { 
     result = 2; 
    } 
    return result; 
} 

我懶，我只寫了前兩個樣品中，所以你的想法...

甚至更​​好：

public int getEventCode(Event event) { 
    int result = event.getClass().hashCode() + 10; 
    Class eventClass = event.getClass(); 
    if (OneEvent.class.equals(eventClass) { 
     result = 1; 
    } 
    else if (TwoEvent.class.equals(eventClass)) { 
     result = 2; 
    } 
    return result; 
} 

Answer 4

public int getEventCode(OneEvent event) { 
    return 1; 
} 
public int getEventCode(TwoEvent event) { 
    return 2; 
} 
// etc. 

這是不好的 - 但是如果你不能改變事件類，這可能是解決你的需求是最面向對象的方式。這是replacing conditional with polymorphism，沒有改變有問題的類