MyBatis ＜if＞判断陷阱:单字符数字条件下的诡异行为揭秘!

经验文章nimo972024-12-15 16:34:0816A⁺A^-

引言

在日常的后端开发工作中，MyBatis凭借其强大的ORM功能和动态SQL能力，成为了众多Java开发者构建数据持久层的首选。然而，即便是经验丰富的开发者也可能在不经意间踩入一些鲜为人知的陷阱。本文将揭示一个容易被忽视的细节——在MyBatis的＜if＞条件判断中，当你遇到单字符数字时，可能会遭遇意想不到的结果。让我们一起揭开这个陷阱的面纱，并探索正确的解决之道。

陷阱案例重现

假设你正在编写一个MyBatis的XML映射文件，其中包含了一个＜if＞标签用于条件判断。代码片段如下所示：

<select id="selectUsers" parameterType="map" resultType="User">
    SELECT * FROM users
    <if test="status == '1'">
        WHERE status = #{status}
    </if>
</select>

这段代码的意图是在status参数等于1时，执行额外的WHERE子句进行过滤。然而，实际运行时，你会发现无论status参数是否为1，WHERE子句总是被忽略。

问题根源

这个问题的根本在于MyBatis使用OGNL（Object-Graph Navigation Language）表达式引擎来解析＜if＞标签中的测试条件。在OGNL中，单引号内的'1'被视为字符'1'（即Character类型），而非字符串"1"。由于Java是一种强类型语言，当比较一个Character和一个String时，它们永远不会相等，因此条件判断始终返回false。

看看源码,这个类：

org.apache.ibatis.ognl.OgnlOps

关键方法：

public static int compareWithConversion(Object v1, Object v2) {
        label75: {
            int result;
            if (v1 == v2) {
                result = 0;
            } else {
                int t1 = getNumericType(v1);
                int t2 = getNumericType(v2);
                int type = getNumericType(t1, t2, true);
                switch (type) {
                    case 6:
                        result = bigIntValue(v1).compareTo(bigIntValue(v2));
                        break;
                    case 7:
                    case 8:
                        break label75;
                    case 9:
                        result = bigDecValue(v1).compareTo(bigDecValue(v2));
                        break;
                    case 10:
                        if (t1 != 10 || t2 != 10) {
                            break label75;
                        }

                        if (v1 instanceof Comparable && v1.getClass().isAssignableFrom(v2.getClass())) {
                            result = ((Comparable)v1).compareTo(v2);
                            break;
                        }

                        if (!(v1 instanceof Enum) || !(v2 instanceof Enum) || v1.getClass() != v2.getClass() && ((Enum)v1).getDeclaringClass() != ((Enum)v2).getDeclaringClass()) {
                            throw new IllegalArgumentException("invalid comparison: " + v1.getClass().getName() + " and " + v2.getClass().getName());
                        }

                        result = ((Enum)v1).compareTo(v2);
                        break;
                    default:
                        long lv1 = longValue(v1);
                        long lv2 = longValue(v2);
                        return lv1 == lv2 ? 0 : (lv1 < lv2 ? -1 : 1);
                }
            }

            return result;
        }

        double dv1 = doubleValue(v1);
        double dv2 = doubleValue(v2);
        return dv1 == dv2 ? 0 : (dv1 < dv2 ? -1 : 1);
    }

解决方案

幸运的是，解决这个问题相对简单。有两种主要方法可以绕过这一陷阱：

使用双引号：将条件改为使用双引号包围的字符串。

<if test="status == \"1\"">

转换为字符串：将status参数显式转换为字符串。

<if test="status.toString().equals('1')">

示例代码

让我们通过一个完整的示例来看一下如何正确应用这些修正：

<select id="selectActiveUsers" parameterType="map" resultType="User">
    SELECT * FROM users
    <if test="status.toString().equals('1')">
        WHERE status = #{status}
    </if>
</select>