您好,登錄后才能下訂單哦!
代碼審查會減緩自己代碼的進度,但也是一個提升自己代碼,并向其他聰明、經驗豐富的工程師學習的機會。那么怎么才能充分利用這個機會呢?
盡量讓你的代碼變動出現在第一輪審查中,只有輕微的評論。如果你的代碼審查通常需要多輪的意見,這些技巧可以節省你的時間。
有效的利用你的代碼審查者的時間,這是一種有限的資源。如果他們發現了那種你自己很容易就能發現的問題,這會降低你的團隊的整體工作效率。
重新評估你的代碼:不要像以前測試時一樣隨便把評審代碼送去審查。至少要重新從整體上思考一遍。尤其是如果當天時間已經晚了,看看第二天早上是否會有更好的想法。雖然這樣可能會拖慢個人代碼的變化,這將從長期上帶來更大的平均吞吐量。
考慮做一次非正式的設計探討:如果你有不確定的事,和審查者面對面交流,或提前提出異議并要求一次整體設計上的“預審”。
自行審查更改:試著從一個什么都不知道的人的角度,盡可能嚴格的看自己的代碼。你的代碼審查工具會比IDE更能給你根本上的不同看法。這將很容易的為你免去一次往返。
使差異容易理解:一下子出現的多種改變讓代碼更難審查。當你自行審查,尋找能減少差異的簡單更改。例如,保證重要的重構或格式變化。
不要隱藏提交消息中的重要信息:把它也放在代碼中。之后閱讀代碼的人是不可能看提交的消息的。
處理完重要意見后,重新評估你的代碼:退一步并用真正全新的眼光去看代碼。一旦完成了一組更改,你會發現,這些更改經常會帶來額外的改進。正如任何的重構,它可能需要幾個步驟來達到最佳設計。
弄明白審查者做出的每一個注釋:如果你不明白注釋背后的原因,不要僅是修改代碼——去找審查者弄清楚,以便自己能學習新的東西。
回復審查者在代碼上的問題:不只是回復——讓代碼更容易理解(例如,提高一個變量名、更改一個布爾值等等)——或添加注釋。以免之后其他人也會有同樣的問題。
免責聲明:本站發布的內容(圖片、視頻和文字)以原創、轉載和分享為主,文章觀點不代表本網站立場,如果涉及侵權請聯系站長郵箱:is@yisu.com進行舉報,并提供相關證據,一經查實,將立刻刪除涉嫌侵權內容。