分配了findbug修正的任務，以下是自己findbug中錯誤型別的整理。

ME_ENUM_FIELD_SETTER
將方法改成private或者是移除

ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD
描述：Write to static field 通過例項方法更新靜態屬性
常見於常量類，直接通過類名.常量名獲取的方式違背了封裝的原則，findbugs不提倡使用，而如果將常量改成靜態成員變數，又因為spring不支援靜態注入導致不能實現，解決方法是非靜態的setter呼叫靜態的setter方法給靜態成員變數賦值。
解決方法：
常量類F：

class F{
public static String a = “123”;
}

常量a改為靜態成員變數，通過F.getA()獲取，且由於spring不支援靜態注入，改為：

class F{
private static String a;
public static Integer getA() {
return a;
}
public void setA(String a) {
setAValue(a);
}
public static void setAValue(String a) {
F.a = a;
}
}

STCAL_INVOKE_ON_STATIC_DATE_FORMAT_INSTANCE
DateFormats 是非執行緒安全的， 其實要避免這個問題方法很簡單，不使用SimpleDateFormat，或者不使用成員變數/靜態成員變數的SimpleDateFormat物件即可。
參考：

NM_METHOD_NAMING_CONVENTION
方法應該是動詞，與第一個字母小寫混合的情況下，與每個單詞的首字母大寫的內部。
方法名不以小寫字母開始

方法應該是動詞,與第一個字母小寫字母大小寫混合,與內部每個詞的第一個字母大寫。

引數名大小寫注意

RV_RETURN_VALUE_IGNORED_BAD_PRACTICE
關於一個方法邏輯執行是否成功，有兩種方式，一種是丟擲異常，一種是提供boolean型別的返回值。舉一個例子，使用者登入，某些人將login方法的返回值定義為int，然後枚舉出各個值的含義，比如0代表成功，1代表使用者名稱不存在等等；而有些人，把這些列舉值看成是use case中的異常流，將它們定義為異常物件，遇到“異常”情況直接丟擲異常從而實現分支的流程。第一種方式是典型的C語言面向過程風格，第二種方式，帶有強烈的面向物件味道，特別是java提供了checked Exception，貌似偏離主題了。
java中很多方法的執行成功依賴於異常的分支實現，但也有提供返回值的實現，比如這裡的File.delete方法，上面的寫法忽略了返回值（如果呼叫某個方法卻不使用其返回值要特別注意），刪除一個檔案很可能不成功，但是從程式碼裡並沒有看到這一層面的意思。
解決方法：
檔案刪除不成功該怎麼辦？現在能處理就處理，現在不能處理就把父類的方法也改成有返回值的，然後向上傳遞，這跟處理異常的道理是一樣的，當然，你也可以把它封裝成一個異常物件。

OBL_UNSATISFIED_OBLIGATION_EXCEPTION_EDGE
方法可能因為checked exception導致清理流或資源失敗
參考：http://just4java.iteye.com/blog/2160704

1. 

• public static void closeResources (Connection con, PreparedStatement ps,ResultSet rs){
• try {
• if (rs != null) {
• rs.close();
• }
• } catch (SQLException e) {
• LOG.error(“釋放ResultSet出錯”, e);
• } finally {
• try {
• if (ps != null){
• ps.close();
• }
• } catch (SQLException e) {
• LOG.error(“釋放PreparedStatement出錯”, e);
• } finally {
• if (con != null) {
• try {
• con.close();
• } catch (SQLException e) {
• LOG.error(“釋放Connection出錯”, e);
• }
• }
• }
  }

• }

DM_STRING_VOID_CTOR
使用沒有引數的構造方法去建立新的String物件是浪費記憶體空間的，因為這樣建立會和空字串“”混淆。Java中保證完成相同的構造方法會產生描繪相同的String物件。所以你只要使用空字串來建立就可以了。

DM_STRING_CTOR
使用java.lang.String(String)建構函式會浪費記憶體因為這種構造方式和String作為引數在功能上容易混亂。只是使用String直接作為引數的形式

DM_BOXED_PRIMITIVE_FOR_PARSING
使用parseInt更有效率一點。
valueOf().intValue():轉換過程是
parseInt:

EI_EXPOSE_REP
描述：may expose internal representation by returning getter方法返回引用型別
eclipse自動生成的引用型別(Object、陣列、Date等)的getter、setter方法會得到或通過對可變物件的引用操作而暴露程式碼內部實現，解決方法很多，只要返回的或賦值的物件不是原引用物件即可。
解決方法：
以Date型別為例：

public Date getHappenTime() {
if(happenTime != null){
return (Date) happenTime.clone();
}else{
return null;
}
}

EI_EXPOSE_REP2
描述：may expose internal representation by storing an externally mutable object into setter方法返回引用型別
eclipse自動生成的引用型別(Object、陣列、Date等)的getter、setter方法會得到或通過對可變物件的引用操作而暴露程式碼內部實現，解決方法很多，只要返回的或賦值的物件不是原引用物件即可。
解決方法：
以Date型別為例：

public void setHappenTime(Date happenTime) {
if(happenTime != null){
this.happenTime = (Date) happenTime.clone();
}else{
this.happenTime = null;
}
}

BX_UNBOXING_IMMEDIATELY_REBOXED
描述： Boxed value is unboxed and then immediately reboxed 裝箱的值被拆箱，然後立刻重新裝箱了
常見的是三目運算時，同時存在基本型別和包裝型別。
解決方法：

Integer a = null;
//…
a = (a == null)?0:a;

此問題在於a不為null時，會被拆箱，賦值時再裝箱。這是自動裝箱拆箱的特性，只要運算中有不同型別，當涉及到型別轉換時，編譯器就會向下轉型，再進行運算。修改方法，統一型別：

Integer a = null;
//…
a = (a == null)?Integer.valueOf(0):a;

DLS_DEAD_LOCAL_STORE

public static void main(String args[]) throws Exception{
Object str = new Object() ;　　//報錯處
str = new Object() ;
System.out.println(str);
}
Object str = new Object();是無用的程式碼，因為在下面有一句str= new Object();，很多語言編譯器它都會做優化，比如：去除一些無用的程式碼以提高效率。JAVA編譯器也會做一些優化，但是，Java編譯器對上面這段程式碼卻沒有做優化（你可以DComplie確認一下），編譯後的.class檔案還是new了兩次，具體什麼原因導致它不去做這個優化我還不能確定，我覺得難做這種優化不是藉口，起碼不應該是Sun的藉口。
修改這段程式碼方法很簡單，隨便去掉一行new Object();就可以了。

SBSC_USE_STRINGBUFFER_CONCATENATION

方法在迴圈中使用+進行字串拼接
For example:
// This is bad

String s =”“;

for (int i = 0; i< field.length; ++i) {

s = s +field[i];

}
// This is better

StringBuffer buf= new StringBuffer();

for (int i = 0; i< field.length; ++i) {

buf.append(field[i]);

}

String s = buf.toString();

NP_NULL_ON_SOME_PATH_EXCEPTION
可能存在為空的情況，必須進行處理，而不是產生null指標

public void test(){
//
ResultSet rs = cmd.executeQuery();
if (rs != null && rs.next()) {
goodsId = rs.getInt(“goods_id”);
}
rs.close();　　//rs可能為null
//
}
可以用try…catch…finally…處理可能出現的異常，也可以在rs.close之前，先判斷rs是否為null

ICAST_INT_CAST_TO_DOUBLE_PASSED_TO_CEIL
integral的值轉換為double後使用了Math.ceil方法

注意前後int 和double的轉換

DM_DEFAULT_ENCODING
編碼格式問題，沒有設定預設編碼格式，這可能導致在不同平臺上出現編碼不一致的問題

RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE
對非空的不斷重複檢查

NP_ALWAYS_NULL

IP_PARAMETER_IS_DEAD_BUT_OVERWRITTEN
將忽略此引數的初始值和引數是覆蓋。這通常表明一種錯誤的觀念就是,寫入引數將被返回給呼叫者。

UC_USELESS_OBJECT
報錯，無用的物件。檢查物件是否有使用

REC_CATCH_EXCEPTION

RCN_REDUNDANT_NULLCHECK_WOULD_HAVE_BEEN_A_NPE
一個多餘的空判斷
檢查一個值是否為空,但這個值不能是零,因為以前的引用時,如果是空一個空指標異常發生在前面的廢棄。從本質上講,這段程式碼和前面的廢棄不同意這個值是否允許空值。檢查是多餘的或以前的廢棄是錯誤的。

RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT

SA_LOCAL_SELF_ASSIGNMENT
public void fun(){
int x =3;
x = x;
}

SF_DEAD_STORE_DUE_TO_SWITCH_FALLTHROUGH

MS_CANNOT_BE_FINAL
增加final

UWF_UNWRITTEN_FIELD
欄位永遠沒被複制，值一直為空

DMI_HARDCODED_ABSOLUTE_FILENAME
包含一個硬編碼的絕對路徑

NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE
方法的返回值沒有進行是否為空的檢查就重新賦值，這樣可能會出現空指標異常。

BC_VACUOUS_INSTANCEOF
在instanceof判斷中，除非為空，否則永遠是true，如果是為了判斷空，用更好的方法判斷。使用instanceof判斷會使得程式不利於別人理解。

DC_DOUBLECHECK
多執行緒錯誤 - 可能對屬性進行了雙重檢測

IS2_INCONSISTENT_SYNC

RCN_REDUNDANT_NULLCHECK_OF_NULL_VALUE

NP_NULL_ON_SOME_PATH
可能存在為空的引用，應考慮其解決