Mybatis识别Integer 0为空字符串问题

最新推荐文章于 2024-09-05 17:45:44 发布

城主J

最新推荐文章于 2024-09-05 17:45:44 发布

阅读量2.3k

点赞数 1

分类专栏：工作总结文章标签： mybatis Integer 0 OGNL 解决

工作总结专栏收录该内容

4 篇文章 0 订阅

订阅专栏

项目中的实体用到大量的状态位，一般都是用Integer 来标识，在项目中遇到一个问题，就是传入的状态值为0，但是拼接的SQL却没有拼接进去。

起初看了mybatis的源码，以为是转BigDecimal的时候，将空字符串和0都识别成了0，转念一想不对，都识别成0也要能够判断不是空，而且敲了一下代码实验一下，new BigDecimal("");会抛出

throw new NumberFormatException("Bad offset or len arguments for char[] input.");异常，

实验代码段如下，不是BigDecimal的问题，转而一看是OGNL表达式的问题

BigDecimal string0 = new BigDecimal("");
BigDecimal integer0 = new BigDecimal(0);

下面引用一个比较详细的解释记录一下

原地址：https://www.jianshu.com/p/91ed365c0fdd

POJO

private Integer status;//状态，可能为0、1、2、3。
//...省略其他

Mapper XML

<sql>
  <trim prefix="where" prefixOverrides="and | or ">
      //...省略其他
      <if test="status != null and status !=''">and status = #{status}</if> 
  <trim prefix="where" prefixOverrides="and | or ">
</sql>

当status的值为 0时该where SQLand status = 0并未正常拼接，也就是说test内的表达式为false，从而导致查询结果错误。但是，显然该值（Integer :0）！= null也！= ' '，应该为true才对。

通过Debug MyBatis源码顺藤摸瓜找到了IfSqlNode类，该类用来处理动态SQL的<if>节点，方法public boolean apply(DynamicContext context)用来构造节点内的SQL语句。if (evaluator.evaluateBoolean(test, context.getBindings())该代码便是解析<if test="status !=null and status !=''">test内表达式的关键，如果表达式为true则拼接SQL，否则忽略。

public class IfSqlNode implements SqlNode {
  private ExpressionEvaluator evaluator;
  private String test;
  private SqlNode contents;

  public IfSqlNode(SqlNode contents, String test) {
    this.test = test;
    this.contents = contents;
    this.evaluator = new ExpressionEvaluator();
  }

  public boolean apply(DynamicContext context) {
    if (evaluator.evaluateBoolean(test, context.getBindings())) {
      contents.apply(context);
      return true;
    }
    return false;
  }
}

打开ExpressionEvaluator 类，发现解析表达式使用的是OGNL，如果你使用过古老的Struts框架你应该对它不陌生。通过OgnlCache.getValue(expression, parameterObject);可以看到表达式的值是从缓存中获取的，由此可知MyBatis竟然对表达式也做了缓存，以提高性能。

public class ExpressionEvaluator {  
  public boolean evaluateBoolean(String expression, Object parameterObject) {  
    Object value = OgnlCache.getValue(expression, parameterObject);  
    if (value instanceof Boolean) return (Boolean) value;  
    if (value instanceof Number) return !new BigDecimal(String.valueOf(value)).equals(BigDecimal.ZERO);  
    return value != null;  
  }

跟进去看看，终于找到了解析表达式的方法private static Object parseExpression(String expression),该方法会先从缓存取值，如果没有便进行解析并放入缓存中，然后调用Ognl.getValue(parseExpression(expression), root)获得表达式的值。

public class OgnlCache {

  private static final Map<String, ognl.Node> expressionCache = new ConcurrentHashMap<String, ognl.Node>();

  public static Object getValue(String expression, Object root) throws OgnlException {
    return Ognl.getValue(parseExpression(expression), root);
  }

  private static Object parseExpression(String expression) throws OgnlException {
    try {
      Node node = expressionCache.get(expression);
      if (node == null) {
        node = new OgnlParser(new StringReader(expression)).topLevelExpression();
        expressionCache.put(expression, node);
      }
      return node;
    } catch (ParseException e) {
      throw new ExpressionSyntaxException(expression, e);
    } catch (TokenMgrError e) {
      throw new ExpressionSyntaxException(expression, e);
    }
  }

至于Ognl.getValue(parseExpression(expression), root)是如何运作的，如果你有兴趣可以自行跟下去一探究竟，本文就不赘述了。到此为止，我们已经知道MyBatis的表达式是用OGNL处理的了，这一点已经够了。下面我们去OGNL官网看看是不是我们的表达式语法有问题从而导致该问题的发生。

Interpreting Objects as Booleans

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.

果然，如果对象是一个Number类型，值为0时将被解析为false，否则为true，浮点型0.00也是如此。OGNL对于boolean的定义和JavaScript有点像，即'' == 0 == false。这也就不难理解<if test="status != null and status !=''">and status = #{status}</if>当status=0时出现的问题了，显然0！=''是不成立的，导致表达式的值为false。

将表达式修改为<if test="status != null">and status = #{status}</if>该问题便迎刃而解。该问题的根源还是来自编码的不规范，只有String类型才需要判断是否！=''，其他类型完全没有这个必要，可能是开发人员为了省事直接复制上一行拿过来改一改或是所使用的MyBatis生成工具不严谨导致该问题的发生。

这里有必要再提一个“坑”，如果你有类似于String str ="A"; <if test="str!= null and str == 'A'">这样的写法时，你要小心了。因为单引号内如果为单个字符时，OGNL将会识别为Java 中的 char类型，显然String 类型与char类型做==运算会返回false，从而导致表达式不成立。解决方法很简单，修改为<if test='str!= null and str == "A"'>即可。