读书人

复建题(2)-通过复制粘贴,获得第一份答

发布时间: 2012-07-24 17:47:58 作者: rapoo

重构题(2)--通过复制粘贴,获得第一份答案

public <t>list<t> query(simplequery simplequery,pager pager, rowmapper<t> rowmapper){string querysql = simplequery.getquerysql();object args = simplequery.getargs();long count = count(simplequery);pager.settotalobjects(count);if(count<=pager.getfirstresult()){return new arraylist<t>();}string pagingquerysql = createpagingquerysql(querysql, pager.getfirstresult(), pager.getpagesize());if(args == null){return jdbctemplate.query(pagingquerysql, rowmapper);}else{return jdbctemplate.query(pagingquerysql, rowmapper, args);}}public list <map<string,object>> query(simplequery simplequery,pager pager){string querysql = simplequery.getquerysql();object args = simplequery.getargs();long count = count(simplequery);pager.settotalobjects(count);if(count<=pager.getfirstresult()){return new arraylist<map<string,object>>();}string pagingquerysql = createpagingquerysql(querysql, pager.getfirstresult(), pager.getpagesize());if(args == null){return jdbctemplate.queryforlist(pagingquerysql);}else{return jdbctemplate.queryforlist(pagingquerysql, args);}}


接上回:

在对象村的春天公司里,大家讨论着:
添加这个功能,应该很简单...
恩,只要复制粘贴一下,然后引入rowmapper,很快就完成了这个功能...
但是这样就产生了重复代码...
看来原来的设计是存在问题的,通过这次添加功能,这个问题暴露了出来...
那么,准备开始重构吧...

读书人网 >软件架构设计

热点推荐