美文网首页
策略模式代替switch...case

策略模式代替switch...case

作者: 一只yami | 来源:发表于2021-07-20 15:18 被阅读0次

直接上主题
这里是原来的一段代码,通过一个type判断怎么分发业务,然后返回一个公共的对象
优点:
写起来很快,所有处理方式全部列举出来

缺点:
所有代码堆叠在一起,导致一个方法行数超多,如果以后继续加业务,这个方法就会越来越大
在case很多的时候,易读性很差

switch (cmdType) {
                case REPORT_PARAM:
                    downProtocolLog = reportParamHandle(ctx, upProtocol, deviceGuid);
                    break;
                case HEARTBEAT:
                    downProtocolLog = heartbeatHandle(ctx, upProtocol, deviceGuid);
                    break;
                case SET_PARAM:
                    downProtocolLog = handleUpSetParam(ctx, upProtocol, deviceGuid);
                    break;
                case REPORT_PARAM_WITHOUT_SAVE:
                    downProtocolLog = reportParamWithoutSaveHandle(ctx, upProtocol, deviceGuid);
                    break;
                case REPORT_CHILD_GUID:
                    downProtocolLog = reportChildGuidHandle(ctx, upProtocol, deviceGuid);
                    break;
                case SOFTWARE_VERSION:
                    downProtocolLog = softVersionHandle(ctx, upProtocol, deviceGuid);
                    break;
                case SET_TIME:
                    downProtocolLog = setTimeHandle(ctx, upProtocol, deviceGuid);
                    break;
                case REPORT_OFFLINE_PARAM:
                    downProtocolLog = reportOfflineDataHanlde(ctx, upProtocol, deviceGuid);
                    break;
                case WAYBILL_INFORMATION:
                    downProtocolLog = waybillInformationHandle(ctx, upProtocol, deviceGuid);
                    break;
                case POWER_ON_OFF:
                    downProtocolLog = downOnOffHandle(ctx, upProtocol, deviceGuid);
                    break;
                case DEVICE_UPGRADE:
                    downProtocolLog = deviceUpgradeHandle(ctx, upProtocol, deviceGuid);
                    break;
                default:
                    downProtocolLog = null;
                    brea

开始改进

核心思想:不同的业务类实现一个接口,新增一个接口实现的分发类,根据type找到对应的业务实现

1. 统一的接口

public interface CallBackService {
    
    public CmdType assetType();

    public DownProtocol process(ChannelHandlerContext ctx, UpProtocol upProtocol, String deviceGuid);
    
}

2. 具体业务的实现

@Service
@Slf4j
public class HeartbeatCallback implements CallBackService {

    @Override
    public CmdType assetType() {
        return CmdType.HEARTBEAT;
    }

    @Override
    public DownProtocol process(ChannelHandlerContext ctx, UpProtocol upProtocol, String deviceGuid) {
            log.info("处理业务");
            return DownProtocol;
    }

}

3. 业务处理分发路由

@Service
public class CallBackServiceRoute {

    @Resource
    private CallBackService[] callBackServices;

    public CallBackService route(CmdType cmdType) {
        for (CallBackService service : callBackServices) {
            if (Objects.equals(cmdType, service.assetType())) {
                return service;
            }
        }
        return null;
    }
}

改进之后的代码使用方式

CallBackService callback = callBackServiceRoute.route(cmdType);
                    
if(null != callback) {
        downProtocolLog = callback.process(ctx, upProtocol, deviceGuid);
}

通过改进后,再有新增的case直接写自己的业务实现类就可以了,不用改公共代码,不用怕改错别人的代码或者跟别人有冲突了
代码有删减,不能直接使用,但是方法和思想已经表达出来了
使用策略模式代替switch...case仅限于switch或者if else较多的情况下,如果case本身较少,使用策略模式反而显得臃肿易读性差,各位按需使用

相关文章

网友评论

      本文标题:策略模式代替switch...case

      本文链接:https://www.haomeiwen.com/subject/hvusmktx.html