更多文章参考我的笔记: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>
该配置问题出在 <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 就可以了。