PageHelper不安全分页

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

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

我敢保证，绝对没有加入任何
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 参数被错误的使用。

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

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

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

上面这个代码，应该写成下面这个样子：

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

这种写法就能保证安全。

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

List<Country> list;
if(param1 != null){
    PageHelper.startPage(1, 10);
    try{
        list = countryMapper.selectAll();
    } finally {
        PageHelper.clearPage();
    }
} else {
    list = new ArrayList<Country>();
}

这种bug真抽象，我去修改下代码！