xxxhelloworld 2010-11-25 11:59
浏览 204
已采纳

if else重构问题

[code="java"]public List getEptData(Params params) {
short leixingId = Short.valueOf(params.getTypeId());
String userType = params.getUserType();
List listBean = null;
if (leixingId == EptType.EPT) {// 如果点击根节点,则电业局节点
listBean = getDyjBeanData(userType);
} else if (leixingId == EptType.DYJ) {// 如果点击电业局节点,则显示变电站节点
listBean = getBdzBeanData(params, userType);
} else if (leixingId == EptType.BDZ) { // 如果点击变电站节点,显示间隔节点和环境节点
listBean = getJgAndHjBeanData(params, userType);
} else if (leixingId == EptType.JG) { // 如果点击间隔节点,则显示设备节点
listBean = getShebeiBeanData(params, userType);
} else {
return null;
}
return listBean;
}[/code]
这是对应业务逻辑层的一个方法,Params params是Action传过来的从页面获取的一些参数经过封装的对象。
对于以上方法,我感觉可以利用多态代替if else,我想用map保存leixingId 与 (调用方法所需的)实例的对应关系,用listBean = map.get(leixingId).getXXXdata() ([color=red]接口中定义getXXXdata方法[/color])代替if else语句,这样合适吗?这个map放在哪里好呢?就是说放在哪里便于以后修改,添加新的扩展?
还有,大家一般是怎样重构if else的?谢谢!

  • 写回答

4条回答 默认 最新

  • weixin_42532992 2010-11-26 14:53
    关注

    在根据你的那个ID 用工厂创建他们就行了

    本回答被题主选为最佳回答 , 对您是否有帮助呢?
    评论
查看更多回答(3条)

报告相同问题?

悬赏问题

  • ¥15 乘性高斯噪声在深度学习网络中的应用
  • ¥15 运筹学排序问题中的在线排序
  • ¥15 关于docker部署flink集成hadoop的yarn,请教个问题 flink启动yarn-session.sh连不上hadoop,这个整了好几天一直不行,求帮忙看一下怎么解决
  • ¥30 求一段fortran代码用IVF编译运行的结果
  • ¥15 深度学习根据CNN网络模型,搭建BP模型并训练MNIST数据集
  • ¥15 C++ 头文件/宏冲突问题解决
  • ¥15 用comsol模拟大气湍流通过底部加热(温度不同)的腔体
  • ¥50 安卓adb backup备份子用户应用数据失败
  • ¥20 有人能用聚类分析帮我分析一下文本内容嘛
  • ¥30 python代码,帮调试,帮帮忙吧