Re: [轉錄] Code Review: 大家都應該做的事情

看板Soft_Job (軟體人)作者Pjack (pjack)時間14年前 (2011/08/19 10:23)推噓2(2推 0噓 0→)

留言2則, 2人參與討論串13/18 (看更多)

※ 引述《yy938559 (高個子)》之銘言： : ※ 引述《yoco315 (眠月)》之銘言： : : 實在看不下去。 : : 首先一間公司如果能作 code review 的人沒幾個，這種公司不待也罷。 : : 因為 code review 並不難，如果一間公司沒幾人能作 code review， : : 那除非整間公司都是白痴。 : 有能力做code review的人本來就不多啊. 這會很奇怪嗎? : 整間公司都程度不好也不是沒有吧. 的確讓資深的人做 code review 會讓你得到更多東西但也不代表一定只能讓資深的人做 code review 做 code review 的方式也可以很多變一種是請一個人單獨幫你看, 然後給你 feedback => 這種還是請老手吧~ 另一種是開一個 code review meeting, 找同組內的人一起看很適合在公司草創時代, 大家都還在摸索階段雖然加總起來很花時間 (ex: 1 小時 * n 個人 ) 但其實可以同時達到 Code review + Knowledge Sharing 未必不划算 : : 但是你說的沒錯，這些東西無法靠 code review 完成，為什麼？ : : 「因為 code review 根本不該看這些」 : 你該不會以為code的架構是Visio/PowerPoint隨便幾個矩型, 塗上 : 幾個顏色就是架構吧? 還是像用UML一樣, 以為畫幾個diagrams, : 可以產生insert/delete/update, 就是架構? : classes/methods的安排都是在coding時產生的.可以說是依經驗加一點 : 推理在人腦中runtime寫出的. : 那些在沒有寫code前, 就說程式架構(指程式碼的安排)巳經設計好的, : 不是太天真說大話, 或者騙子, 不然就是傳說中萬中選一的那位. : 所以... code review 不review架構要review什麼? 可能要看你對架構的定義, code review 的確有時會探討到一些架構問題, 但對我來說, 大部份都屬於 "微調", 而不會是大改變如果會需要大改變, 那表示當初的 design review 沒有做紮實或是在撰寫 code 之前沒有把自己的 design 用各種面像去看一下才會在這個階段還爆出大問題不過的確, 如果一個案子用的技術大家都很不熟的狀況下 code review 很容易就變成在討論架構, 但我覺得這算是過渡時期, 只能期待未來大家的能力都提升後, 這樣的狀況能減少, 畢竟每個人都不希望打掉重練 : 語法不好, 演算法不佳, 邏輯不周, 或不夠了解tools的特性而使用不當等問題, : 這些問題花點時間, 通常就可以處理. 如果真的處理不來, 也可以問別人. : 可以處理/問人的東西, 又何需review呢? 使用的人不一定知道自己使用不當, 或是知道自己這樣寫演算法不佳所以才需要做 code review / knowledge sharing : 沒有人會故意寫爛code的, 是因為實在handle不了, 才會出怪招. : 而程度不夠的人, 每天所產生爛code的速度, 遠超過你能review的. : 不要讓這種人繼續寫code, 才能加速開發的速度不是嗎? : 所以, 也不需要review爛code啊! 停止產生才是正道. 這段我就有點不能理解你的想法.. 每個人又不是一開始就是高手總是會從爛 code 開始寫起, 慢慢一點一滴的把 code 寫好而且除非你是老闆.. 怎麼樣才能讓你的同事停止生產 ? 我寧可正向一些, 和同事一起討論該怎麼寫才能把 code 寫好或許現在階段很花時間, 但或許半年後我就多一個可以合作無間的好同事 : 有經驗的人都知道, 遇到一個困難時, 有時候花時間努力也不見得能處理. : 這和人的經驗/資質/能力/運氣/靈感是有關的. : code reivew 在我的觀點就是這樣: : 除了review架構之外, 其他東西是不需review的. : 不是這方法不對. 而是在現實世界中, : 你很難(應該說不可能發生)同時找到一個有能力review : 又同時有一個有能力做相對應改變的人. : 如果有這樣子的配對, 以這樣子的程度, 那可能還需要review? : 不實用的東西, 不是學院派的話, 難道是實務派? : 我知道又要被噓了. 沒辦法我的眼界就只到這裡. 如果鬧了笑話, : 請各位多包涵啊. 這段我也不太理解, 你可以很清楚的告訴對方應該要怎麼改甚至給個 sample, 如果在這種狀況下對方還是不知道怎麼改那我也不知道說什麼了因為這就好像有人問路, 你說前面紅綠燈右轉就到了, 但對方卻回答他看不到, 所以還是不會走那我覺得就變成是對方態度的問題.....他不願意試只想要你帶他走到目的地再來是如果是我, 其實我也很願意讓能力和我相當的人來幫我 review 因為每個人善長的領域不同, 你熟 A 領域, 他熟 B 領域是可以互通有無的 -- ※ 發信站: 批踢踢實業坊(ptt.cc) ◆ From: 60.251.132.28