复现异常

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

  1. <!-- 根据条件获取学生信息-->
  2. <select id="listByConditions" parameterType="studentQuery" resultMap="BaseResultMap">
  3.     select
  4.     <include refid="Base_Column_List" />
  5.     from t_student
  6.     <where>
  7.         <if test="ids != null and ids.size() > 0">
  8.             AND id IN
  9.             <foreach collection="ids" item="item" open="(" close=")" separator=",">
  10.                 #{item}
  11.             </foreach>
  12.         </if>
  13.         <if test="name != null and name != ''">
  14.             AND name LIKE CONCAT('%', #{name}, '%')
  15.         </if>
  16.         <if test="sex != null and sex != ''">
  17.             AND sex = #{sex}
  18.         </if>
  19.         <if test="selfcardNo != null">
  20.             AND selfcard_no = #{selfcardNo}
  21.         </if>
  22.     </where>
  23. </select>

该配置问题出在 这段代码，sex 在传入的参数中是 Byte 类型，属于数值类型，而 sex != ‘’ 适用于字符串类型判断，我们误将数值类型当成字符串类型配置了。

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

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

image.png

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

image.png

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

分析源码

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

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

image.png

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

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

image.png

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

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

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

image.png

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

image.png

结论

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

作者：殷建卫 链接：https://www.yuque.com/yinjianwei/vyrvkf/tuoakh 来源：殷建卫 - 架构笔记 著作权归作者所有。商业转载请联系作者获得授权，非商业转载请注明出处。