直接上主题
这里是原来的一段代码，通过一个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本身较少，使用策略模式反而显得臃肿易读性差，各位按需使用
