[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的？谢谢！
在根据你的那个ID 用工厂创建他们就行了
考虑一下switch case的方式，性能应该比这个要好。
貌似你需要的是一个工厂方法而已
那就是工厂类了
public interface IServices{
public List getData(Params params);
}
public class DyjData implements IServices{
public List getData(Params params){
}
}
public class BdzData implements IServices{
public List getData(Params params){
}
}