重构题(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,很快就完成了这个功能...
但是这样就产生了重复代码...
看来原来的设计是存在问题的,通过这次添加功能,这个问题暴露了出来...
那么,准备开始重构吧...

你可能感兴趣的:(重构)