在工作上，我最近對一個現有的Java項目代碼進行了清理。完成之后，我發現了一些反復出現的不規范代碼。所以，我把它們整理成了一個列表出來分享給我的同行希望能引起注意并改善代碼的質量和可維護性。

這個列表不區分順序，全部來自一些代碼質量檢查工具，如 CheckStyle, FindBugs 和 PMD。

在Eclipse中格式化源代碼并管理import語句

Eclipse提供了自動格式化源代碼和管理import語句的功能（并移除未使用的語句）。你可以使用下面的快捷鍵來使用這些功能。

• Ctrl + Shift + F – 格式化源代碼。
• Ctrl + Shift + O – 管理import語句并移除未使用的語句。

除了手動執行這兩個功能外，你還可以讓Eclipse在保存文件的時候自動格式化源代碼并自動管理import語句。要做到這個，在Eclipse中，到 Window -> Preferences -> Java -> Editor -> Save Actions并啟用 Perform the selected actions on save，選中 Format source code和 organize imports。

避免在方法中出現多個return語句（退出點）：

在你的方法中，確保只有一個 退出點。不要在一個方法中使用多于一個return語句。

如，下面的代碼是 不推薦的，因為它有多個退出點（return語句）。

private boolean isEligible(int age){  
  if(age > 18){  
    return true;  
  }else{  
    return false;  
  }  
} 

上面的代碼可以這么寫（當然，下面的代碼還可以改進，后面再說）。

private boolean isEligible(int age){  
  boolean result;  
  if(age > 18){  
    result = true;  
  }else{  
    result = false;  
  }  
  return result;  
} 

 簡化if-else方法：

我們寫了一些只要一個參數的工具方法，檢查一些條件并根據條件返回一個值。如，在上面見到的 isEligible方法。

private boolean isEligible(int age){  
  boolean result;  
  if(age > 18){  
    result = true;  
  }else{  
    result = false;  
  }  
  return result;  
} 

可以只使用一個return語句來重寫此方法。

private boolean isEligible(int age){  
  return age > 18;  
} 

不要為Boolean，Integer或String創建新的實例：

避免創建新的Boolean，Integer，String等實例。使用Boolean.valueOf(true)代替new Boolean(true)。兩種寫法效果差不多但卻可以改善性能。

在代碼塊周圍使用大括號:

永遠不要忘記在塊類型語句（如：if,for,while）周圍使用大括號。這可以減少代碼歧義并且避免在你修改代碼塊的時候產生新的bug。

不推薦

if(age > 18)  
  return true;  
else 
  return false; 

推薦

if(age > 18){  
  return true;  
}else{  
  return false;  
} 

把方法的參數聲明為final類型:

總是在所有兼容的地方把把方法的參數聲明為final。這樣做的話，當你無意中修改了參數的值，編譯時會得到警告，并且編譯生成的字節碼也會得到優化。

推薦

private boolean isEligible(final int age){ ... } 

用大寫命名public static final類型成員變量：

總是使用大寫命名用public static final類型變量。這樣可以讓你很容易區分常量和本地變量。

不推薦

public static final String testAccountNo = "12345678"; 

推薦

public static final String TEST_ACCOUNT_NO = "12345678"; 

把多個if語句合并成一個：

下面的代碼

if(age > 18){  
  if( voted == false){  
    // eligible to vote.  
  }  
} 

可以使用一個if語句改寫為：

if(age > 18 && !voted){  
  // eligible to vote  
} 

別忘了給switch添加default語句：

總是給switch添加一個default語句。

避免重復使用同樣的字符串，創建一個常量吧：

如果你需要在多個地方使用同一個字符串，那就創建一個字符串常量來使用吧。

下面的代碼：

private void someMethod(){  
  logger.log("My Application" + e);  
  ....  
  ....  
  logger.log("My Application" + f);  
} 

可以創建一個常量代替字符串"My Application"：

public static final String MY_APP = "My Application";  
 
private void someMethod(){  
  logger.log(MY_APP + e);  
  ....  
  ....  
  logger.log(MY_APP + f);  
} 

其它資源：Java***實踐 , 代碼規范檢查 , PMD規則列表

原文鏈接,OSCHINA編譯