起因

最近在寫CRUD的時候，發(fā)現(xiàn)有個分頁的VO寫的健壯性比較差，一時手癢改了一下，沒想到改了之后好幾個功能都出現(xiàn)了問題。

原VO關(guān)鍵代碼如下：

public class PageVo implements Serializable{
  // ...省略所有無關(guān)代碼
  Map<String, String> query
  
}

這個VO是用于從前端分頁查詢時傳參，而query是用于傳遞查詢條件的（這里不討論用Map傳參是否合理）。當(dāng)前端無查詢條件時則會導(dǎo)致query為null，如果不注意容易出現(xiàn)NPE。

所以我就改造成下面這樣了。

public class PageVo implements Serializable{
  // ...省略所有無關(guān)代碼
  Map<String, String> query=new HashMap<>
  
}

但是沒想到就是這么簡單的改造居然都翻車(・ε・｀)

沒辦法，只好去排查問題。

發(fā)現(xiàn)問題

想過很多種原因，但是我真沒想到居然是因為這樣(／‵Д′)／~ ╧╧，不多說了，問題關(guān)鍵代碼如下：

if (StringUtils.isEmpty(page.getQuery())) {
  // 省略處理邏輯
}

居然用StringUtils去判斷一個Map是否為空，好歹也換個CollectionUtils啊(╬￣皿￣)凸

雖然是前人挖坑，但是為什么Spring的`StringUtils居然設(shè)計成支持Object入?yún)⒛豲_o ....

想了一下，還是去看看源碼吧

源碼分析

StringUtils的isEmpty()方法源碼超級簡單，三行代碼搞定（其實(shí)嚴(yán)格來說就一行代碼）：

public static boolean isEmpty(@Nullable Object str) {
  return (str == null || "".equals(str));
}

既然我的Map對象不為null，那么問題應(yīng)該是因為String的equals()方法。不多說，繼續(xù)跟蹤源碼

public boolean equals(Object anObject) {
    if (this == anObject) {
      return true;
    }
    // 問題出在這里
    if (anObject instanceof String) {
      String anotherString = (String)anObject;
      int n = value.length;
      if (n == anotherString.value.length) {
        char v1[] = value;
        char v2[] = anotherString.value;
        int i = 0;
        while (n-- != 0) {
          if (v1[i] != v2[i])
            return false;
          i++;
        }
        return true;
      }
    }
    return false;
  }

這個equals()方法的邏輯很簡單

1. 入?yún)閚ull則返回true
2. 入?yún)⒉粸镾tring類型返回false
3. 入?yún)ο蠛蛅his對象都為String就比較它們內(nèi)置的char[]數(shù)組長度和每個char元素是否相同，滿足則返回true，否則返回false

而我的問題就是由第二點(diǎn)引起的，因為類型不相同┴─┴︵╰（‵□′╰）

教訓(xùn)總結(jié)

1. 不建議使用Spring的StringUtils的isEmpty()對非String類型的對象判空。（這里建議換成apache common的StringUtils或者Google Guava的Strings，這兩個工具包都是類型強(qiáng)約束的）
2. 無論是修改哪處的代碼都最好檢查一下引用，避免修復(fù)小問題引出大問題

以上就是本文的全部內(nèi)容，希望對大家的學(xué)習(xí)有所幫助，也希望大家多多支持我們。