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

[thinkniht (不下棋=.=)]

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

看起來對方的要求只是看邏輯是否正確
而原PO的要求則是先看邏輯以外的東西

假設code review的目的就只是確定邏輯是否正確
那這就有點像有的公司會說徵PG
但實際上職責是PM+SA+SD+PG+QA(權限與薪資就只有PG的權限與薪資)

我也不曉得 規範到底是甚麼?
是公司訂出來的?還是reviewer自己想的?
這不就又有點像...
規格訂得很模糊，之後就怪PG做的成果不合需求(即使都有滿足規格)
這算誰的問題咧?

當規範不清楚，要求的責任與遵守的限制可以隨意增加時
不就變成可以去整寫的人、
藉由污辱與貶低別人來顯示自己有多厲害...等等
衝突的發生...絕對是正常現象

我相信很多作法本身是好的，只是用的人不會用或亂用時
就會使得做法可能除了破壞性外，一點建設性都沒有...

--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 59.115.121.78
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1424397019.A.9A2.html
推 NDark: 請看下篇 "同儕審查機制已經變成政治算計的遊戲"
推 littlethe: 建構後宮鬥爭模型-以程式設計師code review為例
推 Ayukawayen: 做一個code review的動作
我以前有份工作，職缺上有提到code review
進去後主管說他們都用resharper code review...
另外有次面試，面試官也說他們會做code review，不過是只用工具code review

我的感想是...code review 都不code review了XD

※ 編輯: thinkniht (59.115.126.152), 02/20/2015 22:05:13