我发现我所有有查询的controller都要调用service层的两个方法 getById和getByKey 参数大致一样
写的时候基本都是复制粘帖。。。 于是idea老提示大量重复代码 好烦啊 于是就改进了一下
让所有有关查询的service实现一个QueryService的接口
接口定义俩方法
1 import com.jfinal.plugin.activerecord.Page; 2 3 public interface QueryService<T> { 4 5 Page<T> getById(String id, String flag); 6 7 Page<T> getByKey(int page, int pageSize, String value, String flag); 8 }
肯定要用泛型啊 不然强转容易出错 鬼知道我刚才调用的哪的这个方法
然后把service层的那些方法都统一一下参数
然后在一个公共控制器里写个静态方法 那个注解应该可以不用写 反正控制器不认静态方法
1 @NotAction 2 static <T> Page<T> query(QueryService<T> service,Controller controller, String key, String value, String flag, int page, int pageSize) { 3 value = value == null ? "" : value; 4 Page<T> list; 5 if ("id".equals(key)) { 6 list = service.getById(value, flag); 7 } else if ("key".equals(key)) { 8 list = service.getByKey(page, pageSize, value, flag); 9 } else { 10 list = null; 11 } 12 if (list == null) { 13 controller.renderJson("result", "{'code':'203','info':'unknown key'}"); 14 } else { 15 controller.renderJson("result", "{'code':'103','info':" + Json.getJson().toJson(list) + "}"); 16 } 17 return list; 18 }
用的时候在控制器里第一个写调用的service对象,这个service一定要实现QueryService接口,第二个就传this,因为查询结束要传给前台的,我也是懒,都写这里了
如果render不写这里的话就把第二个参数删掉,把12-16行的业务代码扔回控制器的方法里写
最后还返回list 我也不知道我想干啥 等我想开了就把返回值删掉算了