Re: [閒聊] Code Review 意見不合
先聲明一下，code review這真的是一個蠻有趣的串列，
我是隨便挑一篇回，並不是特定針對某篇文章。
不知道大家做code review是面對面看code還是offline?
我現在的公司code都是放在GitHub上面，code review自然也成為Pull Request的一環，
所以在Code review的過程基本上都是直接留言在GitHub上，也可以留個紀錄。
review完之後，就通知一下對方已經看完了，對方就會自己上去看，
畢竟大家都在忙，不一定什麼時候有空去看別人的code 也不定什麼時候會去看結果。
雖然並不是面對面的討論，不過可以避免不少口氣上的問題，
例如，我們公司習慣檔案結尾要空一行，不過有時候某些檔案就會漏掉，
這個時候留個言"new line"，就好了。自然就不會有情緒化的字眼。
所以關於格式上的東西，自然就可以順順的修正，順順的過去。
如果不是關於格式上的東西，例如API 的命名，或是某些東西上的實作，
在送給其他人review之前，我們也會自己留一些訊息給review的人，
例如，這個東西的命名，我想了很久，有沒有更好的想法。
這樣做這些訊息也不是真的寫在code 裡面，而且也可以凸顯code reivew的重點。
有時候，我們也會做面對面的review，不過主要會是針對比較大的架構上東西。
另外如果真的對於code 格式上真的有很多問題，那就找機會pair programming 一下，
寫的當下就立刻就抓出來，立刻就會記起來了。
最後跟大家分享一個影片: Laura Savino - Hate the Code, Love the Coder
https://vimeo.com/117105942
這是在講code review 畢竟我們是在review code 不是在review 人。
※ 引述《littlethe (東周小星星)》之銘言：
: code review的問題,
: 想到以前曾經遇過一件事,
: 我進到某間公司的時候,
: 還在了解這公司的coding standard,
: 因為規定很多,所以一開始還沒有把所有的規定都背起來,
: 我把程式寫好後,就交給同事review,
: 同事就開始罵我:
: 「你是沒看文件嗎?這地方要空1行,為什麼要空2行?」
: 「為什麼你刮號要寫成
: if(){
: }
: 而不是
: if()
: {
: }
: 這種寫法我沒看過,亂寫」
: 「你寫什麼,我都看不懂...」
: 我心裡就覺得很OOXX...
: 這個人的口氣真的很差,
: 後來再熟一點的時候,
: 也發現這位同事功力其實不如我,
: 而且最讓我不滿的是,
: 這位同事他自己也不是每個code都有符合規定的coding standard,
: 他懶的時候他也是沒照著standard來寫,
: 其實我非常認同要有code review這件事,
: 但是我給人code review只是想要去符合coding standard,
: 讓我的code,還有大家的code可以互相維護,
: 而不是要和同事比賽,
: 程式碼本身好不好,
: 這其實是很主觀的事,
: 但也許是人性問題吧,
: 發現有些人是抱著批評別人的心態在code review,
: 講出多餘的負面字眼也就算了,
: 還去要求別人做出coding standard以外的改變(私人要求),
: 也有人是完全不鳥coding standard,
: 被別人要求code review的時候,
: 就一直吵不肯改,這樣的人我也遇過...
: 以上兩種人都是個性很硬,對自己程式很有自信的人,
: 就像你所說的,很愛面子的人,進而不給別人面子,
: 弄到最後,code review也就變成政治活動了,
: 不爽的人就把別人拉入戰局,開始搞小圈圈,搞造神,搞文革,
: code review這件事也就被搞爛了,
: 這是我的感慨囉,
: 我現在蠻怕和「覺得自己程式寫很好」的人一起工作,
: 不管他的程式是不是真的寫得很好,
: 就像在打LOL,
: 若有人老是罵來罵去玩自己的,就算他打得很好,
: 你也不會想和他同一隊,因為他會破壞團隊氣氛導致輸場,
: 寫程式還是要抱著謙虛的心在寫比較好,
: 技術如此的多,世界如此的大,沒有人十全十美樣樣精通,
: 越學,會覺得自己越渺小
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 73.47.27.155
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1424364581.A.913.html
推
02/20 20:12, , 1F
02/20 20:12, 1F
討論串 (同標題文章)
本文引述了以下文章的的內容：
完整討論串 (本文為第 10 之 12 篇)：