最近接手了一个老项目，“愉悦的心情”自然无以言表，做开发的朋友都懂，这里就不多说了，都是泪
接手老项目，自然是要先熟悉一下业务代码，然而在翻阅 mapper 文件时，发现了一个比较诡异的事情。这里给出简化后的业务代码：
<?xml version="1.0" encoding="UTF-8"?> <!DOCTYPE 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，那最简单的方式就是将它直接从代码中删除了，如下代码所示：
<?xml version="1.0" encoding="UTF-8"?> <!DOCTYPE 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"> name=#{name} </if> <if test="password!=null"> and password=#{password} </if> </select> </mapper>
以上代码删除了 1=1，并且把第一个 name 查询中的 and 去掉了，以防 SQL 查询报错。
但这样就没问题了吗？我们直接来看结果，当包含参数 name 查询时，结果如下：
一切顺理成章，完美的一塌糊涂。
然而，当省略 name 参数时（因为 name 为非必要参数，所以可以省略），竟然引发了以下异常：
又或者只有 password 查询时，结果也是一样：
都是报错信息，那肿么办呢？难不成把 1=1 恢复回去？