在 Java 中使代码更通用

Question

我有一个 触发器管理器 场景，其中我委托 触发器（在其他情况下-单词订阅触发器）到不同的 处理程序 .

现在我有三种处理程序类型，我使用 switch-case 和 enum（这里的枚举是处理程序类型）来重定向到正确的处理程序。

但我的代码似乎不可扩展，它不是通用的，也不遵循 SOLID 原则。想象一下，如果我需要更多的处理程序

我最终会来编辑我的 switch case 代码，我会有更多的案例影响我的代码的圈复杂度

下面是我的确切代码片段

private static TriggerContext getTriggerContext(TriggerHandlerType triggerHandlerType) throws TriggerHandlerException {
    switch (triggerHandlerType) {
        case DASHBOARD_HANDLER:
            triggerContext = new TriggerContext(new DashboardTriggerHandler());
            return triggerContext;
        case COMPONENT_HANDLER:
            triggerContext = new TriggerContext(new ComponentTriggerHandler());
            return triggerContext;
        case WIDGET_HANDLER:
            triggerContext = new TriggerContext(new WidgetTriggerHandler());
            return triggerContext;
        default:
            LOGGER.error(MIS_CONFIGURED_REQUEST_IS_PROVIDED);
            throw new TriggerHandlerException(TRIGGER_HANDLER_TYPE_GIVEN_IS_NOT_CONFIGURED_IN_THE_LIST_OF_TRIGGER_HANDLERS);

    }

}

有人可以帮助我增强这段代码吗？我可以使它更通用并避免圈复杂度并遵循 SOLID 原则以及一些 设计模式.

Answer 1

我不确定整体设计结构，但是可以用枚举上的 newHandler() 方法替换开关。

private static TriggerContext getTriggerContext(TriggerHandlerType triggerHandlerType)
    throws TriggerHandlerException
{
    return new TriggerContext(triggerHandlerType.newHandler());
}

在枚举中，您 would implement 将每种类型枚举的方法设为

enum TriggerHandlerType {
    DASHBOARD_HANDLER
    {
        Handler newHandler() { return new DashboardHandler(); }
    },
    ...;
    abstract Handler newHandler();
}

Answer 2

我认为你的意思是“使代码更动态”，而你的问题来自使用对象作为基元。

您的枚举对象应包含要实例化的类型，而不是打开枚举对象：

enum TriggerHandlerType {
    DASHBOARD {
        @Override
        TriggerHandler create() {
            return new DashboardTriggerHandler();
        }
    },
    COMPONENT_HANDLER {
        //...
    };

    abstract TriggerHandler create();
}

getTriggerContext 然后可以调用 create() 来实例化处理程序：

private static TriggerContext getTriggerContext(TriggerHandlerType triggerHandlerType) throws TriggerHandlerException {
    return new TriggerContext(triggerHandlerType.create());
}

Answer 3

您可以为此使用配置图：

// All your triggers classes should implement this interface
interface TriggerHandler {}

// For example:
public static class DashboardTriggerHandler implements TriggerHandler {
}

// Create your configuration
static Map<TriggerHandlerType, Class> contexts;
static {
    contexts = new HashMap<>();
    contexts.put(TriggerHandlerType.DASHBOARD_HANDLER, DashboardTriggerHandler.class);
    contexts.put(TriggerHandlerType.COMPONENT_HANDLER, ComponentTriggerHandler.class);
    contexts.put(TriggerHandlerType.WIDGET_HANDLER, WidgetTriggerHandler.class);
}

// Return your instance through reflection
public static TriggerContext getTriggerContext(TriggerHandlerType triggerHandlerType) throws TriggerHandlerException, IllegalAccessException, InstantiationException {

    Class className = contexts.get(triggerHandlerType);

    if (className == null) {
        throw new TriggerHandlerException();
    }

    return new TriggerContext((TriggerHandler)className.newInstance());
}

在 Java 中使代码更通用

Make code more Generic in Java

java

design-patterns

enums

enumeration

solid-principles