myabtis 数字+逗号 传参问题 $和#

大家好,我是烤鸭:


    遇到一个maybatis传参的问题。
    如果传参是数字+逗号的形式,比如1,2,3,4,5。

问题复现:

项目中查询sql和navicat查询sql结果不一致。

1    项目中:

如图所示,查到一条数据。

可以看到上面设置进去的参数是String类型的,也就是 IN ('1,2,3,4,5')。很显然,这不是我们想要的。

2.    navicat中:

同样的sql复制到navicat里,是5条。

3.     查找原因:

为什么会有这个问题呢。 为了找问题,将传入的参数写死是没有问题的。

问题在于#符号,#是先生成sql,后采用占位符的方式,将参数放进去。


4.    解决方式:

知道问题就比较好改了,将#换成$就可以了。

$是在sql生成时,把参数传入sql中。#是在sql后,将参数生成到占位符上。
这篇文章从源码角度说了两者的区别。

http://www.importnew.com/25791.html

$的第二种写法,更麻烦了,没必要:

<select id="selectByFlowIds" resultType="PreAuthorizeHnydExpendFlow">
    SELECT
    <include refid="preAuthorizeHnydExpendFlowColumns"/>
    FROM pre_authorize_hnyd_expend_flow a
    WHERE 1 = 1
    AND FIND_IN_SET(a.expend_id, #{flowIds})
    <if test="flowIds != null and flowIds!= ''">
        AND a.expend_id IN
        <foreach collection="flowIds.split(',')" item="flowId" index="index" open="(" close=")" separator=",">
        '#{flowId}'
        </foreach>
    </if>
</select>

但是我们都知道$会有sql注入的风险,用#改写这个,采用mysql的FIND_IN_SET函数。

<select id="selectByFlowIds" resultType="PreAuthorizeHnydExpendFlow">
    SELECT
    <include refid="preAuthorizeHnydExpendFlowColumns"/>
    FROM pre_authorize_hnyd_expend_flow a
    WHERE 1 = 1
    AND FIND_IN_SET(a.expend_id, #{flowIds})
</select>

总结一下:

来源: 

https://www.cnblogs.com/friends-wf/p/4227999.html

mybatis中的#和$的区别

#相当于对数据 加上 双引号,$相当于直接显示数据

1. #将传入的数据都当成一个字符串,会对自动传入的数据加一个双引号。如:order by #user_id#,如果传入的值是111,那么解析成sql时的值为order by "111", 如果传入的值是id,则解析成的sql为order by "id".
  
2. $将传入的数据直接显示生成在sql中。如:order by $user_id$,如果传入的值是111,那么解析成sql时的值为order by user_id,  如果传入的值是id,则解析成的sql为order by id.
  
3. #方式能够很大程度防止sql注入。
  
4.$方式无法防止Sql注入。

5.$方式一般用于传入数据库对象,例如传入表名.
  
6.一般能用#的就别用$.

MyBatis排序时使用order by 动态参数时需要注意,用$而不是#

字符串替换
默认情况下,使用#{}格式的语法会导致MyBatis创建预处理语句属性并以它为背景设置安全的值(比如?)。这样做很安全,很迅速也是首选做法,有时你只是想直接在SQL语句中插入一个不改变的字符串。比如,像ORDER BY,你可以这样来使用:
ORDER BY ${columnName}
这里MyBatis不会修改或转义字符串。

重要:接受从用户输出的内容并提供给语句中不变的字符串,这样做是不安全的。这会导致潜在的SQL注入攻击,因此你不应该允许用户输入这些字段,或者通常自行转义并检查。

猜你喜欢

转载自blog.csdn.net/Angry_Mills/article/details/84799789