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

看板Soft_Job (軟體人)作者yauhh (小y寶貝)時間11年前 (2015/02/16 01:36)推噓9(9推 0噓 1→)

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

※ 引述《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