重构题(3) -- 第一个问题所在

接上回:
:我们可以Extract Method(提炼函数)啊,只要把重复部分提炼出来就ok了.
:别把问题想的太简单,你看这两个函数,主要是在返回的地方不同,没办法直接提炼,我们应该使用模板方法模式.
:我们已经使用了模板方法模式了,现在要再次引入模板方法模式,不觉得太复杂了吗? :的确这样,看来我们给这个类赋予了过多职责.
:嗯,我们把查询的功能分离出去吧,只剩下构造SQL的职责就没问题了


public abstract class SimplePagingQueryJdbcTemplate{

public static final Pattern fromPattern = Pattern.compile("from",Pattern.CASE_INSENSITIVE);

public static final String countSQLHead = "select count(*) from ";

public static final Pattern orderByPattern = Pattern.compile("order(\\s)+by",Pattern.CASE_INSENSITIVE);

public static final Pattern selectPattern = Pattern.compile("select",Pattern.CASE_INSENSITIVE);

public void prepareQuery(SimpleQuery query,Pager pager) {
query.setQuerySQL(createPagingQuerySQL(query.getQuerySQL(), pager.getFirstResult(), pager.getPageSize()));
}

public void prepareCount(SimpleQuery query) {
query.setCountSQL(createCountSQL(query.getQuerySQL()));
}
private String createCountSQL(String querySQL){
String[] SQLParts = fromPattern.split(querySQL, 2);
if(SQLParts.length!=2){
throw new IllegalArgumentException("The SQL query maybe miss the word \"from\".");
}
String countSQLbody = cutLastOrderBy(SQLParts[1])[0];
return countSQLHead+countSQLbody;
}

protected String[] cutLastOrderBy(String sql){
Matcher matcher = orderByPattern.matcher(sql);
if(!matcher.find()){
return new String[]{sql,null};
}
int orderByIndex = matcher.start();
while(matcher.find()){
orderByIndex = matcher.start();
}
return new String[] {sql.substring(0,orderByIndex),sql.substring(orderByIndex)};
}


protected abstract String createPagingQuerySQL(String querySQL, long firstResult, int maxResult);
}


:现在清爽多了,连原来的重复代码都消除了
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值