背景

在日常開發(fā)中，IDEA可能會通過下方的problems窗口以及編輯窗口最右方的黃色標記提示給我們一些警告：

或者向git提交commit時同樣有可能提示警告：

大多數(shù)情況下，無視這些警告并不會影響我們的程序結(jié)果，但卻說明了我們的代碼或多或少有一些問題，還有優(yōu)化的空間，也有可能有潛在的風險，而且有強迫癥同學同樣也不想看到這些警告。這篇文章整理了我在平時開發(fā)中遇到的一些IDEA警告，以便后續(xù)碰到同樣的問題可以作為參考。

注意：不是所有警告都需要處理，需要自行判斷是否有處理的必要。(如我們對變量進行了縮寫，而IDEA認為是一個錯誤的單詞）。

如何檢查代碼

默認情況下，problems窗口的Current File頁簽中會展示的當前文件的錯誤以及警告等。

如果想查看整個項目、某一模塊或某個條件下的文件時，需要依次點擊最上方菜單欄上的Code -> Inspect Code

在彈出的窗口中可以選擇檢查哪些文件的代碼，如整個項目，某個模塊，未提交的文件等，也可以自己配置檢查哪些項：

警告分析

X can be simplified to Y

原因：寫法比較啰嗦，有簡化寫法。

public class Demo {
  boolean result;
  public boolean test() {
    return result == true;
  }
}
// 'result == true' can be simplified to 'result'

解決辦法：

按照提示將X修改為Y即可。

public class Demo {
  boolean result;
  public boolean test() {
    // 直接返回布爾值即可。
    return result;
  }
}

‘OptionalInt.getAsInt()’ without ‘isPresent()’ check

Pattern pattern = Pattern.compile("^(" + name + "\\()(\\d+)(\\))$");
int max = apps.stream()
    .mapToInt(app -> {
      Matcher matcher = pattern.matcher(app.getName());
      if (matcher.find()) {
        // 取第二部分表示版本號
        String group = matcher.group(2);
        // 正則表達式過濾都數(shù)字，不用擔心轉(zhuǎn)換異常
        return Integer.parseInt(group);
      }
      return 1;
    })
    .max()
    .getAsInt();
return max + 1;

示例代碼中，apps為一個對象列表，經(jīng)過一些列處理變?yōu)槟硞€int值最后取最大值。

提示的原因是因為map中有if的判斷，所以有可能全部不匹配，此時max()可能返回一個類似Optional為空的一個東西，需要進行類似處理，將getAsInt改為orElse即可

Pattern pattern = Pattern.compile("^(" + name + "\\()(\\d+)(\\))$");
int max = apps.stream()
    .mapToInt(app -> {
      Matcher matcher = pattern.matcher(app.getName());
      if (matcher.find()) {
        // 取第二部分表示版本號
        String group = matcher.group(2);
        // 正則表達式過濾都數(shù)字，不用擔心轉(zhuǎn)換異常
        return Integer.parseInt(group);
      }
      return 1;
    })
    .max()
    // 給一個默認值
    .orElse(1);
return max + 1;

Iteration can be replaced with bulk ‘Collection.addAll’ call

原因：采用循環(huán)一個一個向集合中添加，比較麻煩。

List<String> list1 = new ArrayList();
List<String> list2 = xxxx;
for (String str : list2) {
    list1.add(str);
 }

解決辦法：使用addAll方法。

List<String> list1 = new ArrayList();
List<String> list2 = xxxx;
list1.addAll(list2);

More arguments provided (1) than placeholders specified (0)

為SLF4J日志功能的警告

...
 LOGGER.error("user {0} is not found ", userId);
...

問題分析：

SL4FJ中占位符不像MessageFormat需要指定index，而是直接使用{}即可

解決辦法：直接使用{}

...
 LOGGER.error("user {} is not found ", userId);
...

Call to ‘asList()’ with only one argument

原因：使用Arrays.asList時只向其中添加了一個元素。

List<String> list = Arrays.asList("a");

解決方法：使用Collections.singletonList代替

List<String> list = Collections.singletonList("a");

Raw use of parameterized class xx

原因：一般為沒有指定泛型

例1：如有一個類定義為

public class ResponseDto<T> {
}

在實例化的時候按照如下所示

ResponseDto<String> response = new ResponseDto();

雖然引用加上了泛型，但是對象沒有加泛型，此時就會出現(xiàn)警告

解決方法：在對象上也加上泛型，但是無需寫全部，只需寫<>即可，否則IDE會覺得多余并發(fā)出警告

// 無需寫全
// ResponseDto<String> response = new ResponseDto<String>();
ResponseDto<String> response = new ResponseDto<>();

例2：在單元測試中使用mockito來虛擬一個feign客戶端時，如果參數(shù)為list那么會按照如下方式虛擬

XXClient client = mock(XXClient.class);
when(client.listByIds(anyList())).thenAnswer(invocation -> {
 // Do something
}

此時IDE會給予警告

因為我們的接口參數(shù)指定了list的泛型

List<XX> listByIds(List<Long> ids);

但是mock時調(diào)用的方法anyList的返回值并沒有指定泛型

所以IDE給出警告

解決辦法：調(diào)用mockito中提供的其他可以指定泛型的方法，如

when(client.listByIds(anyListOf(Long.class))).thenAnswer(invocation -> {
	...
}

Passing a non-null argument to ‘Optional’

原因：向Optional.ofNullable傳遞一個非null對象。

User user = xxxx;
if (user != null) {
  Optional.ofNullable(user).xxx;
}

解決辦法，如果可以明確對象非null，則直接使用of。

User user = xxxx;
if (user != null) {
  Optional.of(user).xxx;
}

Method ‘xx()’ recurses infinitely, and can only end by throwing an exception

原因：提示為無限遞歸，只能通過觸發(fā)異常的方式結(jié)束。

public void method1() {
  method1();
}

解決辦法：

一般來說為錯誤的調(diào)用了自己（如想調(diào)用重載的其他方法，最后由于馬虎又重新調(diào)用了自己），或是遞歸方法寫的有問題，需要具體檢查代碼。

‘if’ statement replaceable with ‘switch’ statement

原因：為if…else…判斷條件太多導致，IDEA認為應(yīng)該替換為swtich語句

if (condition1) {
  // Do something
} else if (condition2) {
  // Do something
} else if (condition3) {
  // Do something
} else if (condition4) {
  // Do something
} else if (condition5) {
  // Do something
} else if (condition6) {
  // Do something
} else {
  // Do something
}

解決辦法：建議通過表驅(qū)動、多態(tài)、設(shè)計模式等方式消除這么多的判斷，如果不得不進行判斷，那么可以根據(jù)提示修改為switch。

switch(xxx) {
  case condition1:
    xxx;
    break;
   case condition2:
    xxx;
    break;
   ... 
}

Result of ‘XX’ is ignored

原因：調(diào)用一個有返回值的方法，但是沒有使用該返回值。

File folder = new File("xx");
if (!folder.exists()) {
  folder.mkdir();
}

由于mkdir()方法會返回boolean類型的返回值，但是我們沒有使用，所以編譯器會報出警告。

解決辦法：可以使用返回值輸出一些日志等，如果實在不想使用返回值，那么可以在方法上標注以下注解來壓制警告。

@SuppressWarnings("ResultOfMethodCallIgnored")

Statement lambda can be replaced with expression lambda

users.forEach(u -> {
  u.setState(xxx);
})

當有以上寫法時會有警告，可以看出lambda表達式方法體內(nèi)只有一行代碼，此時可以將花括號去掉進行簡寫

解決辦法：去掉大括號

users.forEach(u -> u.setState(xxx))

Lambda can be replaced with method reference

names.stream().
  map(n -> n.toUpperCase())
  ...

當有如上寫法時會出現(xiàn)警告。

解決辦法：可以替換為方法調(diào)用的簡寫形式

names.stream().
  map(String::toUpperCase)
  ...

Condition ‘x != null’ covered by subsequent condition ‘x instanceof List’

原因：后面的判斷條件包含了前面的判斷條件

public boolean test(Object obj) {
  if (obj != null && obj instanceof String) {
      // Do sth...
    }
  }

上例中，如果obj為null，則自然obj instanceof String為false，所以不用重復判斷

解決辦法：去掉多余判斷

public boolean test(Object obj) {
  if (obj instanceof String) {
      // Do sth...
    }
  }

Actual value of parameter ‘xxx’ is always ‘yyy’

原因：調(diào)用某個方法時，某個參數(shù)的值目前來說一直是固定的一個值。

解決辦法：

1. 考慮是否一直是固定值，如果是，則將傳參改為局部變量或常量等。
2. 如果該參數(shù)確實需要傳遞，只不過目前來說是一個值，將來不確定，可以考慮將其壓制

@SuppressWarnings("SameParameterValue")

Unchecked generics array creation for varargs parameter

解決辦法：在方法上添加注解

@SafeVarargs

XXX is deprecated

原因：調(diào)用某個標記為@Deprecated方法或?qū)傩詴r，會報此警告。意為提示調(diào)用者這是個過時的方法或?qū)傩?，再未來的某個版本有可能會消失，不應(yīng)該調(diào)用該方法。

但是某種情況下我們確實需要調(diào)用過時的方法，比如我們開發(fā)工具類需要兼容老版本，或我們將一個方法標記了過時之后，單元測試依舊調(diào)用此方法。

解決辦法：

• 如果可能的話，嘗試使用替代方法。一般來說在此方法的Javadoc中指引出新的方法。
• 如果確實需要調(diào)用過時的方法或?qū)傩?，可以考慮將其壓制

@SuppressWarnings("deprecation")

總結(jié)

以上為個人經(jīng)驗，希望能給大家一個參考，也希望大家多多支持腳本之家。

最新評論