Re: [轉錄] Code Review: 大家都應該做的事情
※ 引述《yy938559 (高個子)》之銘言：
: Code review?
: 別鬧了吧. 一家公司有能力做code review的有幾個?
: 好啦, 即使有人可以review, 並review出問題了, 有能力改嗎?
: 不可能吧. 就是因為經驗不足才會有這種狀況啊!
: 光是提出問題, 就能讓程度不夠的人立刻升級嗎? 不可能啊!
: 許多經驗不足, 但覺得自己很強的高手(通常是寫code不到十年的),
: 雖然寫code速度快, 博學多聞, 資料結構一把罩, 說起架構, design patterns
: 也是頭頭是道. 但寫起code來, 總是code架構不佳, 不好maintain.
: 這不是懂的多不多的問題, 也不是努力不努力的問題, 純綷是歷練的問題.
: 有些能力, 特別是code的架構安排,沒有長時間的體會, 就是不會到達那個火喉.
: 不服氣的人, 回去看看你去年寫的code, 如果你覺得去年寫的code讓
: 自己很驚艷, 那你就算是有經驗的programmer. 如果覺得自己程度明明很好,
: 但怎麼寫的code普普通通, 我說, 你就是經驗還不足.
: (我沒說未來無法達到這程度喔)
: 這麼說好了, code review之所以知易行難的原因是:
: review所出現的問題中, 無法處理的問題, 都是code的架構問題,
: 這些東西和programmer的經驗有關. 無法透過短期/長期教育來提升.
: 所以, 有能力改的人, 不用review, 自己也會在過程中修正.
: 沒能力改的人, 也不用改了. 多改只是create愈多問題.
: 至於那些像 SQL injection 要改成 bind variables,
: lookup table 用array, naming styles 不一致, code 亂成一團,
: 常常有超大method/class, copy-paste code一堆這些東西等問題.
: 如果到了要處理的程度, 那麼在處理之前, 先fire寫這些code的人吧.
: 唉...要被噓暴了.
為了方便讓人看懂
引用yy938559大的話來回應
"一家公司有能力做code review的有幾個?"
我想一間公司多少會有能力比較強的
很少會有那種完全沒有人可以code review的情況
只是有沒有辦法讓他們抽出時間而已
"光是提出問題, 就能讓程度不夠的人立刻升級嗎?"
不能說升級...但能發現錯誤並設法改正
你完全不告訴他有錯 他只會自我感覺良好
覺得這麼做就OK
當然 這前提是在code review有效
(有的人code review的目的只是罵人很爽 那就沒用了)
"review所出現的問題中, 無法處理的問題, 都是code的架構問題,
這些東西和programmer的經驗有關. 無法透過短期/長期教育來提升."
短期是沒辦法立即見效
但是不試著處理
怎麼改進?
我覺得code review最難的是...
1.找出適合的人code review
(心態要對 能力要強 還要能排得出時間)
一般老闆要員工加班都來不及了
怎麼還會讓能力強的員工有時間code review呢?
2.增加開發時間
code review的人需要多花時間來review
被code review 的人也會花時間被review
而且還要花時間來改
覺得台灣專案很多時程都很趕
我想除非非常care品質
不然一般公司不會這麼做的
--
※ 發信站: 批踢踢實業坊(ptt.cc)
◆ From: 114.42.233.65
推
08/17 12:28, , 1F
08/17 12:28, 1F
→
08/17 12:28, , 2F
08/17 12:28, 2F
推
08/17 12:52, , 3F
08/17 12:52, 3F
→
08/17 12:53, , 4F
08/17 12:53, 4F
推
08/18 02:34, , 5F
08/18 02:34, 5F
討論串 (同標題文章)