代碼走查25條疑問

代碼走查(Code Review) 是一個開發人員與架構師集中討論代碼的過程。通過代碼走查可以提高代碼的

質量,同時減少Bug出現的機率。但是在小公司中並沒有代碼走查的過程在這裏總結和記錄一些代碼走

查的要求,時刻提醒自己注重代碼質量,每天下班前自己走查一下自己的代碼。代碼走查一些注意事項

如下:

  1. 代碼的註釋與代碼是否一致?註釋是否是多餘的?
  2. 是否存在超過3層嵌套的循環與/或判斷?
  3. 變量的命名是否代表了其作用?
  4. 所有的循環邊界是否正確?
  5. 所有的判斷條件邊界是否正確?
  6. 輸入參數的異常是否處理了?
  7. 程序中所有的異常是否處理了?
  8. 是否存在重複的代碼?
  9. 是否存在超過25行的方法?
  10. 是否存在超過7個方法的類?
  11. 方法的參數是否超過3個?
  12. 是否有多種原因導致修改某個類?
  13. 當發生某個功能變化時,是否需要修改多個類?
  14. 代碼中的常量是否合適?
  15. 一個方法是否訪問了其他類的多個屬性?
  16. 某幾項數據是否總是同時出現,而又不是一個類的屬性?
  17. switch語句是否可以用類來替代?
  18. 是否有一類的職責很少?
  19. 是否有一個類的某些屬性或者方法沒有被其他類所使用?
  20.  在類的方法中是否存在如下的調用形式:a.b().c()? 
  21. 是否某個類的方法總是調用另外一個類的同名方法?  
  22.  是否某個類總是訪問另外一個類的屬性與方法? 
  23.   是否兩個類完成了類似的工作,使用了不同的方法名,卻沒有擁有同一個父類? 
  24.  是否某個類僅有字段和簡單的賦值方法與取值方法構成?
  25.  是否某個子類僅使用了父類的部分屬性或方法?

這25條疑問就是代碼規範的修煉手冊,這25條疑問是別人總結出來的,記錄在此用於提醒自己,