2013年5月9日 星期四

code review 123


  最近做的雖然也被叫code review,但偶而看到文章發現做法差很多,有任務性質跟常態性質的code review。反正目前菜嗶巴,多看看也好。code review的主要目的是為了暸解並改善目前現有的程式,故理想是能有多一點人提供意見想法來改善目前的程式。這是好事,但因為大多是看別人的code,又需要提供意見來變更目前的程式,通常一開始很難溝通(因為不是原參與者)且還有可能因此產生一些問題:得罪人或是只由強勢的一方來決定程式的變更,有些公司在產品沒出包前也不太會做code review。

參考: (這文章提出的方式應該是屬於常態而非任務性質的)
Effective learning through code workshops
http://tech.blog.box.com/2013/05/effective-learning-through-code-workshops/

看起來,code review最好
1.有現有規格及其需求說明
2.討論前參與的人要先準備且有主要說明者
3.討論要有重點(看是要看哪部份或改善什麼)
4.最好有專人負責記錄整理
5.討論過程是針對程式碼而不是作者

--
看完文章是覺得最近做的不叫code review而是針對code review的結論來進行程式變更,其他部份只能自己主動提出,但若是只針對結論目前的時程似乎就有問題a。

沒有留言: