buguge - Keep it simple,stupid

知识就是力量,但更重要的,是运用知识的能力why buguge?

导航

通过一次性能优化,再次记牢了mybatisplus的QueryWrapper、LambdaQueryWrapper、AbstractWrapper这三者之间的关系...

QueryWrapper/LambdaQueryWrapper/AbstractWrapper/Wrapper... 一图看懂mybatisplus中各个Wrapper类的关系图

背景源于一次性能优化

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

 

在一次对交易数据的分页查询代码做性能优化时,我在交易Mapper类-SbhPlatOrderMapper里重写了父接口BaseMapper的selectPage。其中,调用Wrapper<T>参数对象的between操作,为最终的sql加上了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);

    List<SbhPlatOrder> selectPageList(long offset, long pageSize, @Param(Constants.WRAPPER) Wrapper<SbhPlatOrder> queryWrapper);
}

SbhPlatOrderMapper.xml:

    <select id="selectCountCache" resultType="com.emax.etl.provider.modules.dto.PrePageDto">
        select count(1) as rowCount, max(id) as maxId, min(id) as minId from sbh_plat_order ${ew.customSqlSegment}
    </select>
    <select id="selectPageList" resultMap="sbhPlatOrderLimitResultMap">
        select
        <include refid="Base_Column_List"/>
        from
        sbh_plat_order
        ${ew.customSqlSegment}
        limit #{offset}, #{pageSize}
    </select>

 

问题-小瑕疵来了

程序上线后,性能的确有改观。不过呢,发现了一个小瑕疵。见下面日志截图。where条件里的“id BETWEEN ? AND ?”重复了N遍。

 

原来呢,在业务service类中在使用while循环调用这个selectPage时,并不是每次都new一个Wrapper参数,而是重复使用同一个Wrapper对象。见下图。 这样一来,就不难理解 where条件里重复多次 “id BETWEEN ? AND ?” 了。

  

最直接的解决办法 -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());
        }
    }

 

更好的解决办法-对象克隆

上面的解决办法存在两点不足:①使用了字符串contains操作,不利于维护,如果日后id字段重命名,而忽略了这里的修改,就出现bug;②限定了Wrapper只调用一次between操作,而在某些情况下maxId/minId可能是会发生变化的,这样就又会出现隐患。mybatisplus的Wrapper也不提供移除某个已经存在的条件的操作api。

我注意到,QueryWrapper和LambdaQueryWrapper的clone方法。经测试,可行。

下面源码,是mybatisplus的抽象类AbstractWrapper,重写的超类Object#clone。

package com.baomidou.mybatisplus.core.conditions;
public abstract class AbstractWrapper ...{ @Override @SuppressWarnings("all") public Children clone() { return SerializationUtils.clone(typedThis); } }

如此一来,上面的selectPage方法就可以进一步改造一番了。写这篇博文是子夜11:20,我即将关机休息。三下五除二,我就在这篇blog的文本编辑器里敲击键盘,写出下面的selectPage代码:

    @Override
    default IPage<SbhPlatOrder> selectPage(IPage<SbhPlatOrder> page, @Param(Constants.WRAPPER) Wrapper<SbhPlatOrder> queryWrapper){
        Wrapper<SbhPlatOrder> queryWrapperClone = queryWrapper.clone();
        // 下面的代码同上,只不过都访问的是queryWrapperClone ,而不再是queryWrapper。
        。。。
    }

第二天上班后,我开始沿着上面的思路改工程里的代码。却发现行不通!

经过梳理才发现,原来mybatisplus中,各种Wrapper的关系是下面这样子滴

这个类关系图传递如下信息:

  • QueryWrapper与LambdaQueryWrapper 两者本身不存在继承关系,两者都继承自AbstractWrapper。
  • QueryWrapper和LambdaQueryWrapper 与 他们的抽象父类AbstractWrapper 的泛型不同,AbstractWrapper类有3个泛型参数。 所以,试图将上面selectPage方法的第二个参数类型由Wrapper改为子类AbstractWrapper,然后再在方法第一行调用其clone方法是不可以的。

 

结合起来,将上面selectPage方法代码做如下改动:

@Override
default IPage<SbhPlatOrder> selectPage(IPage<SbhPlatOrder> page, @Param(Constants.WRAPPER) Wrapper<SbhPlatOrder> queryWrapper) {
    PrePageDto prePageDto = selectCountCacheEnhance(queryWrapper);
    page.setTotal(prePageDto.getRowCount());
    if (prePageDto.getRowCount() > 0) {
        // 外面调用处可能复用这个queryWrapper对象。所以,为避免重复调用between等操作追加条件,做如下处理
        if (queryWrapper instanceof LambdaQueryWrapper) {
            LambdaQueryWrapper<SbhPlatOrder> clonedWrapper = ((LambdaQueryWrapper<SbhPlatOrder>) queryWrapper).clone();
            clonedWrapper.between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
            page.setRecords(selectPageList((page.getCurrent() - 1) * page.getSize(), page.getSize(), clonedWrapper));
        } else if (queryWrapper instanceof QueryWrapper) {
            QueryWrapper<SbhPlatOrder> clonedWrapper = ((QueryWrapper<SbhPlatOrder>) queryWrapper).clone();
            clonedWrapper.lambda().between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
            page.setRecords(selectPageList((page.getCurrent() - 1) * page.getSize(), page.getSize(), clonedWrapper));
        }
    }
    return page;
}

代码改成下面这样子,能更直观体现出来QueryWrapper、LambdaQueryWrapper、AbstractWrapper这三者之间的关系。不过,因为AbstractWrapper是泛型类,这里IDE会提示:Raw use of parameterized class 'AbstractWrapper' 

@Override
default IPage<SbhPlatOrder> selectPage(IPage<SbhPlatOrder> page, @Param(Constants.WRAPPER) Wrapper<SbhPlatOrder> queryWrapper) {
    PrePageDto prePageDto = selectCountCacheEnhance(queryWrapper);
    page.setTotal(prePageDto.getRowCount());
    if (prePageDto.getRowCount() > 0) {
        // 外面调用处可能复用queryWrapper参数。所以,为避免重复调用between等操作追加条件,做如下处理
        AbstractWrapper clonedWrapper = null;
        if (queryWrapper instanceof LambdaQueryWrapper) {
            clonedWrapper = ((LambdaQueryWrapper<SbhPlatOrder>) queryWrapper).clone().between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
        } else if (queryWrapper instanceof QueryWrapper) {
            clonedWrapper = ((QueryWrapper<SbhPlatOrder>) queryWrapper).clone().lambda().between(SbhPlatOrder::getId, prePageDto.getMinId(), prePageDto.getMaxId());
        }
        page.setRecords(selectPageList((page.getCurrent() - 1) * page.getSize(), page.getSize(), clonedWrapper));
    }
    return page;
}

 

 

EOF-thanks for reading.

posted on 2022-11-30 22:45  buguge  阅读(957)  评论(0编辑  收藏  举报