Re: [閒聊] Code Review 意見不合

[yauhh (小y寶貝)]

※ 引述《blabla123 (念不停 煩不煩？)》之銘言：
: 公司規定提交的 code 至少要經過另外二個人code review才可合入，
: 可能我比較死板，每次我一定先檢查變量名命名法則是否符合規範，
: 還有代碼執行效率，打的 Log 有沒有在前面加上 DEBUG開關，然後才
: 會開始檢查相關邏輯問題。有個同事常叫我幫他 code review，但是他
: 常覺得這沒必要，那沒關係，代碼執行正確就好了。今天，小的終於
: 忍不住了，和他說「我有我 review的標準，並且會努力提高這個標準
: ，如果你覺得我 review的不重要，那別讓我 review就好了，我真的
: 不在意(OS:你媽，幫你看代碼，花時間，到時出bug也是多少得負責，
: 老子不如多做其它的事，還聽你抱怨)」對方也就同意了，有點像不歡
: 而散的感覺。各位在 code review 的有發生過類似的事嗎？或者有其
: 它 code review 過程中的趣事可供分享？

我想你應該先判斷，自己的思維有沒有問題：

1. 你已經在記恨同事不幫你買帳，這種記恨變成下次有機會你就挑起爭端的
   藉口。

2. 你的說詞如果照字面來看，是強調因你的身份為 reviewer 而給品質
   帶來保障，而更甚於因你的 review 帶來品質的保障。

3. 你的說詞如果照字面來看，同事可以不要找你 review 而是找別人幫他
   review ，所以你根本不必對他警告什麼事嘛。

4. 你覺得 code review 是一種連帶要求別人按照你喜歡的方式寫程式
   的手段嗎？

    review, revision, bug, coding style 等等，這些詞彙彼此是有不同內涵。

我覺得你是錯的：

1. 記恨而會講出的那一番話，像是「我有我的原則，不要浪費我的時間」，
   應該是上次發生爭端之後講的話，而不是下一次 review 之前你先樹立的
   障礙。你的錯是在你把你自己的人格擋在同事之前，同事不經過你就不得
   涉入工作，於是你的人格妨礙工作，妨礙公司的運作。

2. 當你要強調你對現有的 code review 情況不滿的時候，應該客觀談那一件
   事情，而不應該把你自己的人格帶進來擺在上位。

3. 「管他去死」這樣的思維，在你的工作環境中，其實是代表你培養了可能
   讓你不適任的因素了。

4. 人家可能只是說 review ，多看一眼，至於什寫法叫做問題，是要另外有
   開放討論的。你光是叫人一定要改，假如不改，你就不接受，那其實，一
   方面這不是開放討論，另一方面，由你指揮別人一定要改的那些東西，
   如果有問題，應該是你要全權負責。



--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 114.42.70.234
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1424021801.A.8AB.html
推 cha122977:
→ Obama19: 我不想碰到原原po這種同事 講話不留餘地的人很難相處
推 Expsun: 推本篇&樓上
推 windgod124: 原PO到最後只是把別人當作他寫CODE的機器罷了~
推 d0808: 推
推 chairco: 認同本篇，推樓上
推 del680202: 推 跟原原PO當同事休養要很好
推 debut: 推 什麼都要照reviewer的方式去改，乾脆reviewer全部自己寫
推 giantwinter: agree
推 kewang: 好文推！