背景

我们的springboot应用程序的持久层，是用jeecgboot框架生成的代码。mybatisplus版本是3.1.2。

在一次对当前程序的sql性能优化时，我重写了BaseMapper的selectPage方法。其中，为Wrapper<T>参数加上了id限制，以提高sql执行性能。

import com.baomidou.mybatisplus.core.mapper.BaseMapper;

public interface SbhPlatOrderMapper extends BaseMapper<SbhPlatOrder> {    

    @Override
    default IPage<SbhPlatOrder> selectPage(IPage<SbhPlatOrder> page, @Param(Constants.WRAPPER) Wrapper<SbhPlatOrder> queryWrapper){
        PrePageDto prePageDto = selectCountCache(queryWrapper);
        page.setTotal(prePageDto.getRowCount());
        if (prePageDto.getRowCount()>0) {
            if (queryWrapper instanceof LambdaQueryWrapper) {
                ((LambdaQueryWrapper<SbhPlatOrder>) queryWrapper).between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
            } else if (queryWrapper instanceof QueryWrapper) {
                ((QueryWrapper<SbhPlatOrder>) queryWrapper).lambda().between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
            }
            page.setRecords(selectPageList((page.getCurrent() - 1) * page.getSize(), page.getSize(), queryWrapper));
        }
        return page;
    }
    
    @Cacheable(cacheNames = RedisConfig.SBH_PLAT_ORDER_COUNT_CACHE_KEY,
            key = "T(com.emax.zhenghe.common.util.security.MD5Util).md5Encode(#queryWrapper.customSqlSegment)")
    PrePageDto selectCountCache(@Param(Constants.WRAPPER) Wrapper<SbhPlatOrder> queryWrapper);

    。。。
}

问题描述

程序上线后运行了一段时间。后来，发现了一个问题。见下面日志截图

原来呢，service类里调用这个分页的方法里，并不是每次调用分页时都new一个Wrapper对象，而是重复使用一个Wrapper对象。 所以，出现上图的问题就不难理解了。

解决办法 -Wrapper#getCustomSqlSegment

头疼医头的方式，是修改service类里调用这个分页的方法，每次调用分页前都new一个Wrapper对象。

显然，这样解决问题只是一时。以后再有调用这个分页的地方，依然会存在这个问题呀。

所以，我的解决办法是利用 Wrapper#getCustomSqlSegment， 上面sql里重复出现的是 id BETWEEN。所以，修正的代码如下：

    // 外面调用处可能复用这个queryWrapper对象。所以，为避免重复追加条件，这里先做判读再追加。
    if (!queryWrapper.getCustomSqlSegment().contains("id BETWEEN")) {
        if (queryWrapper instanceof LambdaQueryWrapper) {
            ((LambdaQueryWrapper<SbhPlatOrder>) queryWrapper).between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
        } else if (queryWrapper instanceof QueryWrapper) {
            ((QueryWrapper<SbhPlatOrder>) queryWrapper).lambda().between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
        }
    }

更好的解决办法-Wrapper#clone

利用clone方法。mybatisplus的Wrapper的抽象父类AbstractWrapper，重写了超类Object的clone方法。如下是源码：

package com.baomidou.mybatisplus.core.conditions;

public abstract class AbstractWrapper ...{

    @Override
    @SuppressWarnings("all")
    public Children clone() {
        return SerializationUtils.clone(typedThis);
    }

}

相比上面的解决办法，我认为这是更好的解决办法，这样修正的代码如下：

import com.baomidou.mybatisplus.core.mapper.BaseMapper;

public interface SbhPlatOrderMapper extends BaseMapper<SbhPlatOrder> {    

    @Override
    default IPage<SbhPlatOrder> selectPage(IPage<SbhPlatOrder> page, @Param(Constants.WRAPPER) Wrapper<SbhPlatOrder> queryWrapper){
        Wrapper<SbhPlatOrder> queryWrapperClone = queryWrapper.clone();
        PrePageDto prePageDto = selectCountCache(queryWrapperClone);
        page.setTotal(prePageDto.getRowCount());
        if (prePageDto.getRowCount()>0) {
            if (queryWrapperClone instanceof LambdaqueryWrapperClone) {
                ((LambdaQueryWrapper<SbhPlatOrder>) queryWrapperClone).between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
            } else if (queryWrapperClone instanceof QueryWrapper) {
                ((QueryWrapper<SbhPlatOrder>) queryWrapperClone).lambda().between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
            }
            page.setRecords(selectPageList((page.getCurrent() - 1) * page.getSize(), page.getSize(), queryWrapperClone));
        }
        return page;
    }

    。。。
}

EOF-thanks!