MyBatis 中为什么不建议使用 where 1=1?
共 1900字,需浏览 4分钟
·
2021-11-19 15:21
作者 | 王磊
来源 | Java中文社群(ID:javacn666)
转载请联系授权(微信ID:GG_Stone)
最近接手了一个老项目,“愉悦的心情”自然无以言表,做开发的朋友都懂,这里就不多说了,都是泪...
接手老项目,自然是要先熟悉一下业务代码,然而在翻阅 mapper 文件时,发现了一个比较诡异的事情。这里给出简化后的业务代码:
mapper PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN" "http://mybatis.org/dtd/mybatis-3-mapper.dtd">
<mapper namespace="com.example.demo.mapper.UserMapper">
<select id="list" resultType="com.example.demo.model.User">
select * from user
where 1=1
<if test="name!=null">
and name=#{name}
if>
<if test="password!=null">
and password=#{password}
if>
select>
mapper>
机智的小伙伴可能已经看出了问题,在众多 mapper 中发现了一个相同的想象,几乎所有的 mapper 中都包含了一个无用的拼接 SQL:where 1=1。作为一个几乎有代码洁癖症的人,自然是忍不住动手改造一番了。
错误的改造方式
既然是去掉 where 1=1,那最简单的方式就是将它直接从代码中删除了,如下代码所示:
"1.0" encoding="UTF-8"?>
"-//mybatis.org//DTD Mapper 3.0//EN" "http://mybatis.org/dtd/mybatis-3-mapper.dtd">
"com.example.demo.mapper.UserMapper">
以上代码删除了 1=1,并且把第一个 name 查询中的 and 去掉了,以防 SQL 查询报错。
但这样就没问题了吗?我们直接来看结果,当包含参数 name 查询时,结果如下:一切顺理成章,完美的一塌糊涂。
然而,当省略 name 参数时(因为 name 为非必要参数,所以可以省略),竟然引发了以下异常:又或者只有 password 查询时,结果也是一样:都是报错信息,那肿么办呢?难不成把 1=1 恢复回去?
正确的改进方式
其实不用,在 MyBatis 中早已经想到了这个问题,我们可以将 SQL 中的 where 关键字换成 MyBatis 中的
mapper PUBLIC "-//mybatis.org//DTD Mapper 3.0//EN" "http://mybatis.org/dtd/mybatis-3-mapper.dtd">
<mapper namespace="com.example.demo.mapper.UserMapper">
<select id="list" resultType="com.example.demo.model.User">
select * from user
<where>
<if test="name!=null">
and name=#{name}
if>
<if test="password!=null">
and password=#{password}
if>
where>
select>
mapper>
代码改造完成之后,接下来我们来测试一下所有的请求场景。
不传任何参数的请求
此时我们可以不传递任何参数(查询所有数据),如下图所示:生成的 SQL 语句如下:
传递 1 个参数的请求
也可以传递 1 个参数,根据 name 进行查询,如下图所示:生成的 SQL 如下图所示:也可以只根据 password 进行查询,如下图所示:生成的 SQL 如下图所示:
传递 2 个参数的请求
也可以根据 name 加 password 的方式进行联合查询,如下图所示:生成的 SQL 如下图所示:
用法解析
我们惊喜的发现,在使用了
首先,
总结
在 MyBatis 中,建议尽量避免使用无意义的 SQL 拼接 where 1=1,我们可以使用
推荐阅读:
欢迎关注微信公众号:互联网全栈架构,收取更多有价值的信息。