今天学习重构没怎么开窍，这有一段代码，大家帮忙Refactor

今天学习重构没怎么开窍，这有一段代码，大家帮忙Refactor

 

StringBuffer sb =new StringBuffer("select d.*  from table d where 1=1 ");
  if(vo.getC() != null && !vo.getC().equals("")){
   sb.append(" and d.c like '%"+vo.getC()+"%'");
  }
  if(vo.getT() != null && !vo.getT().equals("")){
   sb.append(" and d.t = '"+vo.getT()+"'");
  }
  if(vo.getB()!=null && !vo.getB().equals("")){
   sb.append(" and d.B like '%"+vo.getB()+"%'");
  }
  if(vo.getD()!=null && !vo.getD().equals("")){
   sb.append(" and d.d = '"+vo.getD()+"'");
  }

SQLQuery query = this.getSession().createSQLQuery(sb.toString());

 

这是一段简单的条件查询语句，vo是参数，table是数据库里的表名

 

这段代码应该怎么重构，如果以后查询条件多了该怎么办，谢谢

 

 

评论

8 楼 hdp2010 2011-10-24  

    这个看你系统怎么设计，如果你组装这样的sql语句很多，完全可以写一个接口，提供将实体bean转换成一个sql语句的一部分；
interface ITransBeanToStr<T>{
  String beanToSql(T obj);
}利用反射实现

7 楼 Just_抱怨 2011-10-16  

hyj1254 写道

引用

这段提出来作为一个独立的函数，一可封装出一个独立的功能且有机会解释其作用；二可避免使两个不同的功能散落在同一个函数内。

引用

SQLQuery query = this.getSession().createSQLQuery(sb.toString());

因为这也可以看作一个功能。
至于getC,getT之类的确实可以优化一下命名，便于理解。

至于命名，是我把后面的删了，公司有保密制度，代码严禁外泄的

6 楼 hyj1254 2011-10-15  

引用

StringBuffer sb =new StringBuffer("select d.*  from table d where 1=1 ");
  if(vo.getC() != null && !vo.getC().equals("")){
   sb.append(" and d.c like '%"+vo.getC()+"%'");
  }
  if(vo.getT() != null && !vo.getT().equals("")){
   sb.append(" and d.t = '"+vo.getT()+"'");
  }
  if(vo.getB()!=null && !vo.getB().equals("")){
   sb.append(" and d.B like '%"+vo.getB()+"%'");
  }
  if(vo.getD()!=null && !vo.getD().equals("")){
   sb.append(" and d.d = '"+vo.getD()+"'");
  }

这段提出来作为一个独立的函数，一可封装出一个独立的功能且有机会解释其作用；二可避免使两个不同的功能散落在同一个函数内。

引用

SQLQuery query = this.getSession().createSQLQuery(sb.toString());

因为这也可以看作一个功能。
至于getC,getT之类的确实可以优化一下命名，便于理解。

5 楼 木易有峰 2011-10-15  

还是有些地方可以重构的吧。
在append里面在用+操作。
如果是JDK1.5的话 且不是多线程环境考虑用StringBuilder吧。
另外为啥不用变量代替 getT getB等呢。你这个里面每次都差不多get了三次吧。

4 楼 Just_抱怨 2011-10-15  

我看坏味道里面举得例子就和我这个相似，可他没有对它重构，只是提出来了

3 楼 Just_抱怨 2011-10-15  

287854442 写道

没什么可重构的。
至少

vo.getB()!=null && !vo.getB().equals("")

这一句能提取出来一个类似

public static boolean isEmpty(String value){
    return value == null || value.trim().equals("")
}

的方法。

哦 因为这个就三个条件 所以我就没写 很多地方我都写了

2 楼 287854442 2011-10-14  

没什么可重构的。
至少

vo.getB()!=null && !vo.getB().equals("")

这一句能提取出来一个类似

public static boolean isEmpty(String value){
    return value == null || value.trim().equals("")
}

的方法。

1 楼 Just_抱怨 2011-10-14  

自己先顶下，习惯了