代碼審查會(huì)減緩自己代碼的進(jìn)度,但也是一個(gè)提升自己代碼,并向其他聰明、經(jīng)驗(yàn)豐富的工程師學(xué)習(xí)的機(jī)會(huì)。那么怎么才能充分利用這個(gè)機(jī)會(huì)呢?
盡量讓你的代碼變動(dòng)出現(xiàn)在第一輪審查中,只有輕微的評(píng)論。如果你的代碼審查通常需要多輪的意見(jiàn),這些技巧可以節(jié)省你的時(shí)間。
有效的利用你的代碼審查者的時(shí)間,這是一種有限的資源。如果他們發(fā)現(xiàn)了那種你自己很容易就能發(fā)現(xiàn)的問(wèn)題,這會(huì)降低你的團(tuán)隊(duì)的整體工作效率。
在你的代碼進(jìn)行審查之前:
重新評(píng)估你的代碼:不要像以前測(cè)試時(shí)一樣隨便把評(píng)審代碼送去審查。至少要重新從整體上思考一遍。尤其是如果當(dāng)天時(shí)間已經(jīng)晚了,看看第二天早上是否會(huì)有更好的想法。雖然這樣可能會(huì)拖慢個(gè)人代碼的變化,這將從長(zhǎng)期上帶來(lái)更大的平均吞吐量。
考慮做一次非正式的設(shè)計(jì)探討:如果你有不確定的事,和審查者面對(duì)面交流,或提前提出異議并要求一次整體設(shè)計(jì)上的“預(yù)審”。
自行審查更改:試著從一個(gè)什么都不知道的人的角度,盡可能?chē)?yán)格的看自己的代碼。你的代碼審查工具會(huì)比IDE更能給你根本上的不同看法。這將很容易的為你免去一次往返。
使差異容易理解:一下子出現(xiàn)的多種改變讓代碼更難審查。當(dāng)你自行審查,尋找能減少差異的簡(jiǎn)單更改。例如,保證重要的重構(gòu)或格式變化。
不要隱藏提交消息中的重要信息:把它也放在代碼中。之后閱讀代碼的人是不可能看提交的消息的。
當(dāng)你處理代碼審查意見(jiàn):
處理完重要意見(jiàn)后,重新評(píng)估你的代碼:退一步并用真正全新的眼光去看代碼。一旦完成了一組更改,你會(huì)發(fā)現(xiàn),這些更改經(jīng)常會(huì)帶來(lái)額外的改進(jìn)。正如任何的重構(gòu),它可能需要幾個(gè)步驟來(lái)達(dá)到最佳設(shè)計(jì)。
弄明白審查者做出的每一個(gè)注釋?zhuān)?/b>如果你不明白注釋背后的原因,不要僅是修改代碼——去找審查者弄清楚,以便自己能學(xué)習(xí)新的東西。
回復(fù)審查者在代碼上的問(wèn)題:不只是回復(fù)——讓代碼更容易理解(例如,提高一個(gè)變量名、更改一個(gè)布爾值等等)——或添加注釋。以免之后其他人也會(huì)有同樣的問(wèn)題。