java业务逻辑问题

Question

else if (time.equals("1")) {
if (!CharUtil.isEmpty(formtime )) {
condition += " and ( (inDate <= :formtime) )";
param.put("formtime",
CharUtil.stringToDate(DateUtil.addDay(formtime, 1), "yyyy-MM-dd"));
}
if (!CharUtil.isEmpty(totime) ) {
condition += " and ( (outDate < :totime) )";
param.put("totime",
CharUtil.stringToDate(totime, "yyyy-MM-dd"));
}
}else if (time.equals("1")) {
if (!CharUtil.isEmpty(formtime)) {
condition += " and (inDate > :formtime )";
param.put("formtime",
CharUtil.stringToDate(DateUtil.addDay(formtime, 1), "yyyy-MM-dd"));
}
if (!CharUtil.isEmpty(totime)) {
condition += " and ( (outDate < :totime ) or (outDate is null) or (outDate >= :totime) )";
param.put("totime",
CharUtil.stringToDate(totime, "yyyy-MM-dd"));
}
}
怎样把这两个代码段放在一起而不起冲突？

Answer 1

个人建议：你可以把判断的逻辑，两层if调换一下层级；
if(formtime){
if(time.equels("1")){ ... }

}
if(totime){
if(time.equels("2")){ ... }
}

想法：哪个逻辑大就在最外层先判断那个条件。

Answer 2

如果time 的是是常量，那么你可以试着使用 switch语句。
eg：
switch(time)
{
case 值1:
语句1 （这里可以写if）;
break;
case 值2:
语句2;
break;
default:
语句n;
break;

Answer 3

代码写的不清不楚啊。两段代码？两段代码之间为什么不加区别的符合？要合并，需要合成什么样子。从代码看来，只有if里面执行的操作不一样，判断条件是一样的。要如何合并？

追问

　　判断条件是一样的，第一个if (time.equals("1"))里面的if (!CharUtil.isEmpty(formtime ))和if (!CharUtil.isEmpty(totime) )现在是相连的，需要把第二个if (time.equals("1"))里面的操作也放入第一个的if (time.equals("1"))里 而不引起冲突。

Answer 4

你这代码不奇怪么，相同的条件却是不一样的处理？