在某個工程團隊的新人訓練裡,主管花了整整一小時談 程式碼審查,但沒講任何技術細節。他只談一件事:態度。
他說,程式碼審查 最容易出錯的地方,是把它當成挑毛病的場合。當 審查者 的語氣帶著「你怎麼會這樣寫」,對話就變成防禦戰,雙方都學不到東西。
真正有效的 審查,是對話:先問「這裡是基於什麼考量?」再給建議;把「這樣寫不好」換成「這樣寫會不會比較清楚?」;遇到真的有問題的地方,解釋原因,而不是只丟下判決。
程式碼的好壞當然重要,但人和人之間的信任更貴。一個會讓人願意主動 請求審查 的團隊,永遠比一個 審查 嚴格卻沒人敢貼 合併請求 的團隊走得更遠。

發佈留言