SAY討論區

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

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

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

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

審判廳：

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

責任任務：

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

我和我的代碼

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

[英文原文：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/)