如果接口无法更改，如何重构此方法？

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）。否则测试会稍微复杂一些，但仍然可行。

至于：

查找错误报告 ITC_INHERITANCE_TYPE_CHECKING

是的，这是因为 instanceof 检查。

但是：一开始的代码存在一个根本性缺陷。 不能保证.hashCode() 在两个不同的 JVM 执行之间返回相同的值。更重要的是，它允许返回负值。这意味着它可以返回例如 -4 作为值；这意味着“其他事件”将返回 6，因此与 SixEvent 发生冲突。

考虑重构！

Answer 2

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

这不好 - 但如果您不能更改 Event 类，这可能是满足您要求的最面向对象的方式。这是replacing conditional with polymorphism，无需更改相关类

Answer 3

代码库中是否存在一些此类类型块？如果是这样，用多态性替换条件http://refactoring.com/catalog/replaceConditionalWithPolymorphism.html 可能会有所帮助。

Answer 4

好吧，使用instanceof不好，但如果你不能改变事件接口，甚至添加一个“int getType()”或类似的，至少你可以重构你的代码：

使用“else if”结构而不是just if结构。我建议将继承更深的事件类型放在首位，最后考虑更通用的事件声明（如果您测试事件 instanceof Event，您将始终为真）。

这样至少可以降低复杂性。

假设您的所有事件都直接实现事件接口并且没有其他继承问题需要检查：

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，您也会遇到问题。在这种情况下，我建议使用您收到的事件实例的 Class，如下所示：

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;
}