Mybatis踩坑记录——if从句中的赋值乌龙

最近遇到一个有点意思的bug，特此记录一下。

问题背景

• 前台参数：前端请求后台接口，上送参数进行查询，上传参数如下：
  
  可知上送为两个参数：activityType: “3”，merchantId: “3a9d21a0d18a4e1c819dac57b1730992”
• 后台逻辑

String activityType = vo.getActivityType();
		String merchantId = vo.getMerchantId();
		List<TbMarketActivity> activity = activityMapper.getActivityByMerchantId(activityType, merchantId);

• SQL语句

<select id="getActivityByMerchantId"
            resultType="cn.buyforyou.aipc.small.api.entity.marketActivity.TbMarketActivity">
        SELECT
        <include refid="tbMarketActivityColumns"/>,
        b.product_discount as "productDiscount"
        FROM
        tb_market_activity a
        left join tb_market_activity_product b on a.activity_id = b.activity_id
        <where>
            a.status = '2'
            and a.activity_type = #{activityType}
            and a.merchant_id = #{merchantId}
        </where>
        <if test="activityType != null and activityType = '2'.toString()">
			order by product_discount
		</if>
		limit 1
    </select>

后台直接使用前台参数进行数据库查询，sql日志如下：

 ==> Preparing: SELECT a.activity_id AS "activityId", b.product_discount as "productDiscount" FROM tb_market_activity a left join tb_market_activity_product b on a.activity_id = b.activity_id WHERE a.status = '2' and a.activity_type = ? and a.merchant_id = ? order by product_discount limit 1 
 ==> Parameters: 2(String), 3a9d21a0d18a4e1c819dac57b1730992(String)
 <==      Total: 0

那么问题就来了，中间并没有进行任何的上传参数处理，但是到了SQL语句中，activity_type的值是2，而不是上传的3

问题分析

• 排查一：确认上送参数
  
  如图，上传参数确实为3，但是到了sql中还是2，很纳闷，确定不是上送参数的问题

• 排查二：SQL语句
  本次sql并不复杂，就是一个简单的条件过滤查询，而且根据SQL日志来看，也已经成功绑定了

• 排查三：特么没有可排查的啦，简直就没找到错误，暂定为悬案

反思：为啥上传和代码处理都正确，但是SQL赋值出现错误，明明上传的activityType为3，可最后的条件是2；但又为啥不是其他偏偏是2呢。。。。。。

问题解决

问题的关键点就在于为啥偏偏是2身上，仔细看一下sql语句，这个activityType还是和2产生过联系的。如下：

这是一个简单的条件判断，并决定排序字段的语句，但是中间却使用了等号作为判断条件，显然这儿是不成立的，大抵已经找到原因了。
activityType就是在这个if判断语句中，被赋值为2的，即使在Mybatis的初始绑定值为3，那这么说的话，activityType不为null时，它的值会一直为2。
继续往下面想,这个if语句的执行顺序，应该是在执行SQL查询之前，那可以推断的是，Mybatis在执行SQL时，首先是先将所有的标签转化组装为SQL，然后在执行SQL语句。所以这里就已经被赋错了值。

解决办法也特别简单，就是再加一个=就可以了，标准的判断语句而已。

补充 ：mybatis的映射文件中，使用的是ognl表达式，所以在字符串进行判等的时候，可使用以下两种方法

<if test="sex=='Y'.toString()">
或者
<if test = 'sex== "Y"'>

唯独不能使用

<if test="sex=='Y'">

因为mybatis会把’Y’解析为字符，java是强类型语言，所以不能这样写。

总结

确实觉得很有意思，一个很小的错误，发生的那么不经意，SQL不报错，页面也没有错误，排查起来确实费了点心，以后写代码还是得多加注意呀！！！共勉共勉