使用Integer与''进行比较的坑

前段时间遇到一个bug，就是修改某一个integer类型的值的时候，除了修改为0 其他的值都可以进行修改。只要想把这个字段改成0 就似乎没有走update这个字段的语句。(前段时间比较忙也就没有记下这个笔记)

相信很多朋友在使用Mybatis的时候都会去使用通用一点的insert或者update语句，类似加入以下这种判断来进行复用。

<if test="orgName != null and orgName != ''">
    orgName =#{orgName ,jdbcType=VARCHAR},
</if>

也就是只有orgName有有效值的时候才会进入set语句，但是在对于Integer类型下的判断就会有一定的影响。

经过我的测试发现mybatis会将Integer值的0等于空字符串''

例如传入的num的值为0，if标签里面一样写

num != null and num != ''

当num的值为0 ，mybatis会当做0等于空串，也就不会进入此if里面的代码块。

比如update语句，如果传进来的值为0那么这个字段就不会被update掉。

也就是说在对Integer类型的值进行判断的时候，只需要加入!= null 即可。

Integer类型参数在Mybatis中的比较问题

案例

当使用mybatis传入参数为Integer类型，使用条件构造语句时偶尔会导致判断错误。如下

select * from tbl_user where 1=1
<if test="flag != null and flag != '' ">
    and flag = #{flag}
</if>

当flag传入参数为0，会导致条件构造失败；SQL执行结果不是我们想要的

原因分析

mybatis在预编译sql时，使用OGNL表达式来解析if标签，对于Integer类型属性，在判断不等于''时，会返回''的长度，也就是0

源码：

(s.length() == 0) ? 0.0 : Double.parseDouble( s )

因此表达式 flag != ''被当做 flag != 0 来判断，所以当flag为0时，if条件判断不通过

总结

if条件判断number类型，没必要判断''的情况，只需判断null的情况即可；

如果非要判断''的情况，那么要考虑到等于0的情况，即<if test="flag != '' or flag == 0">

最后