SAY討論區

標題: 代碼審查不是用來…… - 程式設計 [打印本頁]

作者: Flibs    時間: 2013-10-24 11:08:07     標題: 代碼審查不是用來…… - 程式設計

1500435s0ipdbt2dc.jpg 在我們公司里要執行代碼審查。這是我們每天的例行工作。事實上,今天的我們正是從這種一直堅持探索的漫長道路上走出來的。我們嘗試各種技術、方法和工具,直到我們走到今天的成就(但這並不是說我們就此停步)。
在這旅途中,我們發現了很多的陷阱和危險,它們等待新手們上鉤。這篇文章就是關於它們的:代碼審查中的陷阱和誤解。
代碼控制:

很多公司都把代碼審查當成控制代碼的方法。很多這樣的公司都使用預提交策略。這種策略大多時候都是開源項目中使用,因為會有成百上千的提交者。可在一般的公司里,很少會有這種情況。如果你僱用一個人,這意味這你要完全信任他,允許他將代碼提交到代碼庫里。我知道有些公司會忍不住制定一些規程,要求程序員在提交代碼前必須進行「審查」和「批准」,但這並不能保證代碼的質量。而且,程序員很快就會把這種代碼審查當作一種「愚蠢」的公司形式過程,會開始抵制它(例如,每月改一次密碼。例如,使用像mypass1,mypass2等的密碼)。
150044nep5h3hgxfb.png
 
審判廳:

不要把代碼審查當成尋找替罪羊和追究責任的工具。比如說,這有一個錯誤,你找到「審查」這段代碼的人,並責備他沒有發現這個問題。這種事情會給公司里的開發工作帶來嚴重的影響。人們會挑出每個分號放置不正確的地方,因為他們擔心會被當成替罪羊。團隊成員開始缺乏信心,並最終失去互信。
150045zmvijgijjui.jpg
責任任務:

不要過分要求程序員做代碼審查。如果你強迫他們每天做一小時的代碼審查,他們很快就會痛恨它,把它當成一種無趣的任務。代碼審查是一種學習,是表揚,是獲得反饋,是一種十分社交性的活動。代碼審查應該是有趣的,不要讓它變的無聊。
150045yxdb02bssxd.jpg
我和我的代碼

如果你的代碼被某人審查,他會留下一些注釋(有時是不那麼友好的話),不要生氣。他並不是在說你是一個很爛的程序員。這不是他的本意,也不是代碼審查的目的。他的所作所為是在批評代碼(而不是作者)。代碼審查是針對代碼,不是針對你。不要把代碼審查當成互相諷刺的論壇和相互批判的工具。當你寫審查注釋時,努力保持不要粗魯,也不要太苛刻。努力站在作者的立場上看待這些代碼。
總結一下,很多種錯誤都會使代碼審查變味。上面4種是我經歷過或預料到的,請警惕。我承諾還會寫一篇描述「代碼審查是來…..」的文章。
如果你想嘗試一種代碼審查工具,請訪問codebrag.com,這是我們的經驗和實踐的成果。 150045wgo11tati2g.jpg

[英文原文:Code Review is not about... ]

附件: [1500435s0ipdbt2dc.jpg] 1500435s0ipdbt2dc.jpg (2013-10-24 11:08:07, 0 Bytes) / 下載次數 0
http://say.go2tutor.com/forum.php?mod=attachment&aid=NDQ1MjY0fDFjZjYxNDA4fDE3ODE2ODgwMDF8MHww

附件: [150044nep5h3hgxfb.png] 150044nep5h3hgxfb.png (2013-10-24 11:08:07, 0 Bytes) / 下載次數 0
http://say.go2tutor.com/forum.php?mod=attachment&aid=NDQ1MjY1fDRmNWI5OGVhfDE3ODE2ODgwMDF8MHww

附件: [150045zmvijgijjui.jpg] 150045zmvijgijjui.jpg (2013-10-24 11:08:07, 0 Bytes) / 下載次數 0
http://say.go2tutor.com/forum.php?mod=attachment&aid=NDQ1MjY2fGI5MzA3NDEyfDE3ODE2ODgwMDF8MHww

附件: [150045yxdb02bssxd.jpg] 150045yxdb02bssxd.jpg (2013-10-24 11:08:07, 0 Bytes) / 下載次數 0
http://say.go2tutor.com/forum.php?mod=attachment&aid=NDQ1MjY3fDM5ZGY3ZWJhfDE3ODE2ODgwMDF8MHww

附件: [150045wgo11tati2g.jpg] 150045wgo11tati2g.jpg (2013-10-24 11:08:07, 0 Bytes) / 下載次數 1
http://say.go2tutor.com/forum.php?mod=attachment&aid=NDQ1MjY4fGI2Mzg5YzA3fDE3ODE2ODgwMDF8MHww
作者: alvinsiusiu    時間: 2013-10-24 14:58:35

這個站不錯!!  
作者: hopakngai    時間: 2013-10-24 15:01:46

這篇帖子,我還是第一次在這看到
作者: 前程    時間: 2013-10-24 15:09:56

沒人理我。。
作者: 砵仔糕    時間: 2013-10-24 15:15:52

似曾相識的感覺
作者: easonchan    時間: 2013-10-24 15:22:22

我等你喲
作者: ng015856    時間: 2013-10-24 15:22:36

這樣的貼子,不頂說不過去啊
作者: 鳥不起點算    時間: 2013-10-24 15:23:13

我回不回呢 考慮再三 還是不回了吧 ^_^  
作者: 小妖怪    時間: 2013-10-24 15:30:52

看看好不好看
作者: ILOVEU5    時間: 2013-10-24 15:31:23

拿把椅子看表演
作者: hopakngai    時間: 2013-10-24 15:40:37

真的有麼
作者: 花家姐    時間: 2013-10-24 15:42:15

真是有你的!
作者: 太陽穴    時間: 2013-10-24 15:48:00

經過你的指點 我還是沒找到在哪 ~~~
作者: a_c_e06    時間: 2013-10-24 15:48:59

希望大家都有好運
作者: ccseeker2012    時間: 2013-10-24 20:16:16

今天沒事來逛逛,看了一下,感覺相當的不錯。
作者: 太陽穴    時間: 2013-10-24 21:07:20

厲害!強~~~~沒的說了!  
作者: yan4327    時間: 2013-10-25 02:50:35

好啊,,不錯、
作者: 北極蝦    時間: 2013-10-25 03:45:28

快樂看帖,認真回覆
作者: chrity__summer    時間: 2013-10-25 07:16:28

加油啊!!!!頂哦!!!!!  
作者: mic167    時間: 2013-10-26 05:09:55

謝謝分享了!
作者: Some1    時間: 2013-10-26 09:15:08

希望大家能幫忙多多分享
作者: junhau    時間: 2013-10-26 18:36:58

默默的看完,安靜的回覆
作者: hkgordan    時間: 2013-10-27 06:18:59

@,@..是什麼意思呀?
作者: plas_hoz    時間: 2013-10-28 08:13:29

樓主分享的好東西,我幫你宣傳
作者: coffyma    時間: 2013-10-28 19:13:08

謝謝分享了!
作者: cb_reptile    時間: 2013-10-29 04:16:42

希望我回覆這一帖,能在前幾名
作者: kensiu123    時間: 2013-10-29 09:06:08

賺點分不容易啊
作者: juny0385    時間: 2013-10-29 19:11:42

估計你是沒見識過~` 呵呵  
作者: 尊貴的基佬    時間: 2013-10-30 02:01:52

加油!樓主,謝謝分享
作者: hillmen    時間: 2013-10-30 08:46:38

希望大家幫我把這個帖發給你身邊的人,謝謝!
作者: easonchan    時間: 2013-10-30 18:16:32

我看不太到喔
作者: 曾冇權    時間: 2013-10-31 02:15:40

樓上的話等於沒說~~~
作者: hkgordan    時間: 2013-10-31 15:26:11

感覺好像在哪裡看過了,汗~
作者: skykingdom    時間: 2013-10-31 23:56:39

文筆流暢,修辭得體,深得魏晉諸朝遺風,更將唐風宋骨發揚得入木三分,能在有生之年看見樓主的這個帖子。實在是我三生之幸啊。
作者: k2a3i4    時間: 2013-11-1 06:51:53

這樣的貼子,不頂說不過去啊
作者: 弒殺滅神    時間: 2013-11-1 13:16:37

Thank you for your sharing,回覆囉
作者: chrity__summer    時間: 2013-11-1 20:03:25

我看不太到喔
作者: ng015856    時間: 2013-11-2 01:58:41

遇到好論壇中的好帖子
作者: 傻QQ    時間: 2013-11-2 07:03:09

回復一下
作者: 正義西門慶    時間: 2013-11-2 12:53:19

真是佩服得六體投地啊
作者: ~Muscle~    時間: 2013-11-2 18:12:44

有些帖子,真的不錯
作者: nh88    時間: 2013-11-3 03:33:05

初來乍到,請多多關照。。。
作者: mydear888    時間: 2013-11-3 06:05:41

一頭霧水…
作者: 切雞飯    時間: 2013-11-3 07:48:37

長時間沒來看了 ~~
作者: katchan0481    時間: 2013-11-3 09:44:28

不是吧




歡迎光臨 SAY討論區 (http://say.go2tutor.com/) Powered by Discuz! X2