何如作代碼review?

明確代碼review的目標

代碼review能夠提升代碼質量,也能夠幫助開發人員成長,可是這些並不合適做爲目標。目標應該更加明確,建議根據項目須要設定目標。如下是常規代碼review目標:前端

  1. 完善需求用例,確保沒有遺漏的需求點。
  2. 保證邏輯正確,確保沒有明顯的代碼邏輯錯誤。
  3. 遵照代碼規範,確保項目代碼風格和可閱讀性,其中可能包括相關文檔。
  4. 規避生產性能問題,特別注意線程安全、集合類、大表SQL等容易形成性能的代碼。
  5. 經過平時的積累所總結出來的須要特別注意的代碼。

哪些項目適合代碼review

代碼review是一件頗有意義的事情,但並非適用於全部項目,由於它太耗時間了,並且有許多人不喜歡評審別人的代碼。哪些項目適合代碼review呢?後端

  1. 核心項目,特別是金融類和商品類,一切涉及到金錢的都須要額外注意。
  2. 影響用戶使用的項目,這裏用戶是指能帶來營收的用戶,也能夠理解爲公司所服務的用戶。
  3. 影響下游系統使用的項目,特別是基礎組件。

何如組織代碼review

項目中有許多角色,好比技術、產品、測試、運營。技術中又包括前端、後端,也許不一樣模塊也是由不一樣人來負責,人員衆多如何組織代碼review呢?安全

  1. 角色定位爲技術人員,而且先後端分別代碼review。
  2. 人數按代碼重要性和相關性來肯定,通常以6-8人爲佳。
  3. 評審團隊中應該包含較高評級的技術人員。
  4. 代碼評審時間應該在發版前,並預留修改錯誤代碼的時間。