MyBatis 数值类型 where 条件配置的坑

2019-11-26 来源: 殷建卫 发布在  https://www.cnblogs.com/yinjw/p/11933361.html

复现异常

我们先通过案例复现该类异常,测试项目地址:https://gitee.com/yin_jw/demo/tree/master/mybatis-demo/springboot-mybatis-demo,StudentMapper.xml 中根据条件获取学生信息的 SQL 配置如下所示。

<!-- 根据条件获取学生信息-->
<select id="listByConditions" parameterType="studentQuery" resultMap="BaseResultMap">
    select
    <include refid="Base_Column_List" />
    from t_student
    <where>
        <if test="ids != null and ids.size() > 0">
            AND id IN
            <foreach collection="ids" item="item" open="(" close=")" separator=",">
                #{item}
            </foreach>
        </if>
        <if test="name != null and name != ''">
            AND name LIKE CONCAT('%', #{name}, '%')
        </if>
        <if test="sex != null and sex != ''">
            AND sex = #{sex}
        </if>
        <if test="selfcardNo != null">
            AND selfcard_no = #{selfcardNo}
        </if>
    </where>
</select>

该配置问题出在 <if test="sex != null and sex != ''"> 这段代码,sex 在传入的参数中是 Byte 类型,属于数值类型,而 sex != '' 适用于字符串类型判断,我们误将数值类型当成字符串类型配置了。

当我们错误的配置了 sex 字段的判断条件,传入 sex = 0 时,sex != '' 会返回 false,传入 sex = 1 时,sex != '' 会返回 true,同样是传入数字,为什么会出现两种不同的结果呢?

sex = 0 的执行效果如下图所示:

sex = 1 的执行效果如下图所示:

sex = 0 的执行返回内容明显不是我们需要的结果,下面我们通过源码分析来解开这个谜题。

分析源码

通过源码分析篇中的“MyBatis 源码篇-SQL 执行的流程”章节,我们知道 MyBatis 执行 SQL 最终都会交给 Executor 类执行。

前面的调试步骤省略,直接进入 CacheExecutor 类的 query 方法。

猜测 MyBatis 根据参数和映射配置文件生成 boundSql 的时候,出现了一些问题。我们一路往下 DEBUG,发现问题出在 MyBatis 对 OGNL 表达式处理上面。

org.apache.ibatis.ognl.OgnlOps#compareWithConversion(Object v1, Object v2) 该方法在比较 (Byte)0 和 "" 时,返回的是 true,也就是该方法认为两者是相同的,执行结果如下图所示。

所以,sex = 0 的条件没有出现在生成的 SQL 中。

那么当 sex = 1 的时候,compareWithConversion(Object v1, Object v2) 方法的执行结果是怎样的呢?

修改参数,调试测试接口,执行结果如下图所示:

compareWithConversion(Object v1, Object v2) 方法返回的结果是 false,其实该问题的本质是,org.apache.ibatis.ognl.OgnlOps#doubleValue(Object value) 方法当 value 是空字符串时返回 0.0D。

结论

对于数值类型的参数,在配置 where 条件的时候要注意,不能把它当成字符串类型判断,数值型只需要判断 sex != null 就可以了。

相关文章