Mybatis IF 标签整型判空潜在的坑

一时失言乱红尘 2023-09-24 16:59 128阅读 0赞

## 前言 ##

贴一张 来自 应届生的 瑟瑟发抖的 代码走读的 恐惧：

![651cbfc06b664a0da77cc6d4c4928f12.png][]

大家平时写代码，一定一定要明确，自测 是研发自己的事，不是自己写完就完事咯~

![20826016aca04532a45440d706b14db6.png][]

## **正文** ##

开始进入主题，一起看看怎么个事。

不能暴露业务代码，我就简单来举例复现一下。

大概是这样

小伙设计了一个表，表里面 设计了一个字段 代表 启用 停用 ：

> STATE 1:启用 0：停用 ，默认 1 启用

然后呢， 写了个根据 state 状态查询的 方法 ： getDistrictByState

根据枚举 启用 、停用 配合 mybatis 的if 标签判空 来触发 这个查询条件，做数据查询。

--------------------

表：

注意到，state 是 int 类型 （其实long类型也一样有该篇说的坑）  
  
![217f9b0a4b584576846cfd24a4a5cca1.png][]

mapper：  
  
![b447283cbfa249b098897534aa6fcdc5.png][]

xml ：

（红色框住的，if标签 对于 int long 这些数据的判空，这样写是有坑的！ 前排警告！！！）

![881a3323cae5423ca9274ec647d9ca06.png][]

然后代码是这样写的：  
  
![a72b09e9c97046b5b52702b9dc426063.png][]

看看数据表里面的数据 ：

![8cb0406c48454461938a4603bce16715.png][]

那么正常来说，就应该查出来这三条，因为 传入参数不是空，然后触发sql if 拼接：

![890249d0395e4d96a8d9daf0a6233209.png][]

而现实是全给查出来了， 因为条件没触发 ：

![7e49e0fcea4547809e5ad13cdeb7f184.png][]

![7b6b87076cd742a581b483a2b10950dd.png][]

## 为什么？ ##

在为什么之前，我先列出这种整型（int long） if标签，如果想写判空触发，正确写法是 ：

**方案一**

<if test="state!= null ">
                 STATE= #{state}
            </if>

![a7dbdf4ac5c94844a3ad0bfbd110a591.png][]  
![24385feb3fe44bdb9ada79d4e3cecec9.png][]

**方案二**

<if test="state!= null and state!= '' or state == 0">
                 STATE= #{state}
            </if>

![d6b428487fb2486eb7c857c4576ff50b.png][]

![28a767ea386b491089bfc4886b6bcd92.png][]

#### **再回到为什么？** ####

原理很简单，就是mybatis框架的代码的固定逻辑 ：

只要if 标签 传入的 是 Number 类型数据 ， 就会使用双精度浮点类型 跟 0 比较， 如果不等于 0 才 触发， 否则就是 不触发。

小伙写法传入的就是 0 ， 然后if 标签 的写法 state != null and state != '' " 。 就应证了上面的说法。

当然口说无凭，现在出来混，讲证据：

![3bd4c126af2c40ad83fd7b0f681eb3ad.png][]

> Any object can be used where a boolean is required. OGNL interprets objects as booleans like this:
> 
>  *  If the object is a Boolean, its value is extracted and returned;
>  *  If the object is a Number, its double-precision floating-point value is compared with zero; non-zero is treated as true, zero as false;
>  *  If the object is a Character, its boolean value is true if and only if its char value is non-zero;
>  *  Otherwise, its boolean value is true if and only if it is non-null.

翻译：  
如果对象是数字，则将其双精度浮点值与零进行比较；非零视为true，零视为false.

false的话 那当然就不触发咯。

经常看我文章的人，显然是知道的，我不止这么点。 我很干。

**我们接下来看看 mybatis 源码怎么说？**

![011fcd5e11a84c3d8f12a713e9ea61fe.png][]

debug，进源码，不啰嗦。

if标签解析，；来自于mybatisjar 里面的 IfSqlNode.java ：

位置 ：org.apache.ibatis.scripting.xmltags.IfSqlNode

![3833967e8179495392c72f12a4b812ff.png][]

看到了一开始，拿到了小伙 针对整型，可能是复制粘贴别人代码 写的if 标签 ：

![2aef71f97a134abba801d1a0c53b67b7.png][]

然后是 ExpressionEvaluator.java 的 evaluateBoolean函数 ：

位置 ： org.apache.ibatis.scripting.xmltags.ExpressionEvaluator

![85551b8d8f8e43f0980f5f9ce167eead.png][]

接着看 OgnlCache.java的 getValue函数 ：

OgnlCache.getValue(expression, parameterObject);

位置 ：org.apache.ibatis.scripting.xmltags.OgnlCache

![9b0d82ca0687430db4d1175b18f2a775.png][]

继续点进去，看看

parseExpression(expression)

![5ba3f9b4eabe4fe0b6b1a492414ccf54.png][]

把我们的 传参 state 为 0，传入 就是相当于是：  
![7ef4e6a4078c4635b4f01f05ab045dbe.png][]

（0 ！= null ）&& （ 0!= ''）

![6606da3cb2594ed3b0fee75b575d8101.png][]

自己玩一玩，当前情况的if 标签， 我们传入 0 还有传入 1，是怎么会回事？

就是硬核，我们直接把代码拿出来玩！

pom 文件 加入 ：

写个测试方法，眼见为实：

public static void main(String[] args) throws ognl.OgnlException {
    
            Object o = ognl.Ognl.parseExpression("0 != null and 0 != ''");
            Object result = Ognl.getValue(o, null);
            System.out.println("触发结果是 ：" + result);
    
            Object o1 = ognl.Ognl.parseExpression("1 != null and 1 != ''");
            Object result1 = Ognl.getValue(o1, null);
            System.out.println("触发结果是 ：" + result1);
        }

![15268eecabdc4b5dbc05b89453d9cadb.png][]

该篇就到这 ，每个人不同角度看我的文章，关注点也许不一样，但是我希望我能传达、分享更多的东西给到大家。 点赞、收藏、评论，让我看看你们在支持我。

[651cbfc06b664a0da77cc6d4c4928f12.png]: https://img-blog.csdnimg.cn/651cbfc06b664a0da77cc6d4c4928f12.png
[20826016aca04532a45440d706b14db6.png]: https://img-blog.csdnimg.cn/20826016aca04532a45440d706b14db6.png
[217f9b0a4b584576846cfd24a4a5cca1.png]: https://img-blog.csdnimg.cn/217f9b0a4b584576846cfd24a4a5cca1.png
[b447283cbfa249b098897534aa6fcdc5.png]: https://img-blog.csdnimg.cn/b447283cbfa249b098897534aa6fcdc5.png
[881a3323cae5423ca9274ec647d9ca06.png]: https://img-blog.csdnimg.cn/881a3323cae5423ca9274ec647d9ca06.png
[a72b09e9c97046b5b52702b9dc426063.png]: https://img-blog.csdnimg.cn/a72b09e9c97046b5b52702b9dc426063.png
[8cb0406c48454461938a4603bce16715.png]: https://img-blog.csdnimg.cn/8cb0406c48454461938a4603bce16715.png
[890249d0395e4d96a8d9daf0a6233209.png]: https://img-blog.csdnimg.cn/890249d0395e4d96a8d9daf0a6233209.png
[7e49e0fcea4547809e5ad13cdeb7f184.png]: https://img-blog.csdnimg.cn/7e49e0fcea4547809e5ad13cdeb7f184.png
[7b6b87076cd742a581b483a2b10950dd.png]: https://img-blog.csdnimg.cn/7b6b87076cd742a581b483a2b10950dd.png
[a7dbdf4ac5c94844a3ad0bfbd110a591.png]: https://img-blog.csdnimg.cn/a7dbdf4ac5c94844a3ad0bfbd110a591.png
[24385feb3fe44bdb9ada79d4e3cecec9.png]: https://img-blog.csdnimg.cn/24385feb3fe44bdb9ada79d4e3cecec9.png
[d6b428487fb2486eb7c857c4576ff50b.png]: https://img-blog.csdnimg.cn/d6b428487fb2486eb7c857c4576ff50b.png
[28a767ea386b491089bfc4886b6bcd92.png]: https://img-blog.csdnimg.cn/28a767ea386b491089bfc4886b6bcd92.png
[3bd4c126af2c40ad83fd7b0f681eb3ad.png]: https://img-blog.csdnimg.cn/3bd4c126af2c40ad83fd7b0f681eb3ad.png
[011fcd5e11a84c3d8f12a713e9ea61fe.png]: https://img-blog.csdnimg.cn/011fcd5e11a84c3d8f12a713e9ea61fe.png
[3833967e8179495392c72f12a4b812ff.png]: https://img-blog.csdnimg.cn/3833967e8179495392c72f12a4b812ff.png
[2aef71f97a134abba801d1a0c53b67b7.png]: https://img-blog.csdnimg.cn/2aef71f97a134abba801d1a0c53b67b7.png
[85551b8d8f8e43f0980f5f9ce167eead.png]: https://img-blog.csdnimg.cn/85551b8d8f8e43f0980f5f9ce167eead.png
[9b0d82ca0687430db4d1175b18f2a775.png]: https://img-blog.csdnimg.cn/9b0d82ca0687430db4d1175b18f2a775.png
[5ba3f9b4eabe4fe0b6b1a492414ccf54.png]: https://img-blog.csdnimg.cn/5ba3f9b4eabe4fe0b6b1a492414ccf54.png
[7ef4e6a4078c4635b4f01f05ab045dbe.png]: https://img-blog.csdnimg.cn/7ef4e6a4078c4635b4f01f05ab045dbe.png
[6606da3cb2594ed3b0fee75b575d8101.png]: https://img-blog.csdnimg.cn/6606da3cb2594ed3b0fee75b575d8101.png
[15268eecabdc4b5dbc05b89453d9cadb.png]: https://img-blog.csdnimg.cn/15268eecabdc4b5dbc05b89453d9cadb.png