重构题(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,很快就完成了这个功能...
但是这样就产生了重复代码...
看来原来的设计是存在问题的,通过这次添加功能,这个问题暴露了出来...
那么,准备开始重构吧...
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值