个人随笔
PageHelper不安全分页
2020-09-11 22:10:51

经常偶尔会发现,进入我的博客个人页面的时候,加载用户标签竟然分页了,我的天,怎么hi突然分页了呢,然后再刷新又不分页了,无数次检查代码都没有发现问题.

  1. public MsgReturn getMenus(Long userid, String visible) {
  2. List<Menu> menus = menuMapper.getMenusByUserid(userid,visible);
  3. MsgReturn msgReturn = this.getMsgReturn("0000","查询成功");
  4. msgReturn.setObj(menus);
  5. return msgReturn;
  6. }

我敢保证,绝对没有加入任何
PageHelper.startPage(page, size);
这种分页提示。
并且,也不是进程发生!这怎么搞!


然后想着按事实说话,看日志输出,绝对是分页了,既然我代码上没写,然后又想起PageHelper分页好像是产生一个LocalThread来的,想着会不会我有些地方前面有PageHelper.startPage(page, size);这个代码,然后没有执行mybatis查询,导致这个线程一直存在?想着首页好像是查redis没有直接查数据库,应该是这个原因!

参考:https://blog.csdn.net/qq_38082304/article/details/94405415 说的:
PageHelper 方法使用了静态的 ThreadLocal 参数,分页参数和线程是绑定的。

只要你可以保证在 PageHelper 方法调用后紧跟 MyBatis 查询方法,这就是安全的。因为 PageHelper 在 finally 代码段中自动清除了 ThreadLocal 存储的对象。

如果代码在进入 Executor 前发生异常,就会导致线程不可用,这属于人为的 Bug(例如接口方法和 XML 中的不匹配,导致找不到 MappedStatement 时), 这种情况由于线程不可用,也不会导致 ThreadLocal 参数被错误的使用。

但是如果你写出下面这样的代码,就是不安全的用法:

  1. PageHelper.startPage(1, 10);
  2. List<Country> list;
  3. if(param1 != null){
  4. list = countryMapper.selectIf(param1);
  5. } else {
  6. list = new ArrayList<Country>();
  7. }

这种情况下由于 param1 存在 null 的情况,就会导致 PageHelper 生产了一个分页参数,但是没有被消费,这个参数就会一直保留在这个线程上。当这个线程再次被使用时,就可能导致不该分页的方法去消费这个分页参数,这就产生了莫名其妙的分页。

上面这个代码,应该写成下面这个样子:

  1. List<Country> list;
  2. if(param1 != null){
  3. PageHelper.startPage(1, 10);
  4. list = countryMapper.selectIf(param1);
  5. } else {
  6. list = new ArrayList<Country>();
  7. }

这种写法就能保证安全。

如果你对此不放心,你可以手动清理 ThreadLocal 存储的分页参数,可以像下面这样使用:

  1. List<Country> list;
  2. if(param1 != null){
  3. PageHelper.startPage(1, 10);
  4. try{
  5. list = countryMapper.selectAll();
  6. } finally {
  7. PageHelper.clearPage();
  8. }
  9. } else {
  10. list = new ArrayList<Country>();
  11. }

这种bug真抽象,我去修改下代码!

 1
有疑问发邮件到 : suibibk@qq.com 侵权立删
Copyright : 个人随笔   备案号 : 粤ICP备18099399号-2