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

更多文章参考我的笔记：https://www.yuque.com/yinjianwei/vyrvkf

 

复现异常

我们先通过案例复现该类异常，测试项目地址：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>

该配置问题出在  这段代码，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 就可以了。