2020年11月18日 星期三

[Java] Collapsible "if" statements should be merged (java:S1066)

 Merging collapsible if statements increases the code's readability.

Noncompliant Code Example

if (file != null) {
  if (file.isFile() || file.isDirectory()) {
    /* ... */
  }
}

Compliant Solution

if (file != null && isFileOrDirectory(file)) {
  /* ... */
}

private static boolean isFileOrDirectory(File file) {
  return file.isFile() || file.isDirectory();
}
以上內容的意思是~
當if條件句中, 又只有一個if條件句
這兩個if條件句就可以合併成一個
中間用&&隔開~真是有時候並不會發現的小毛病呢~

沒有留言:

張貼留言

2026年新年新目標

從今年開始預計2.5-3年後成為能應徵國際大廠的頂尖工程師 今年在程式上 1 每個月看看能不能都做專案, 一年12專案 2 刷題每周4題 3 跑個人scrum, 並組讀書會 4規律讀英文ex每天早上8:30-9:30 5規律運動7:35-8:30 6減肥97->85 而下班...