[閒聊] Code Review 意見不合

看板Soft_Job作者 (念不停 煩不煩？)時間9年前 (2015/02/14 00:55), 9年前編輯推噓15(16推 1噓 30→)
留言47則, 25人參與, 最新討論串1/12 (看更多)
公司規定提交的 code 至少要經過另外二個人code review才可合入， 可能我比較死板，每次我一定先檢查變量名命名法則是否符合規範， 還有代碼執行效率，打的 Log 有沒有在前面加上 DEBUG開關，然後才 會開始檢查相關邏輯問題。有個同事常叫我幫他 code review，但是他 常覺得這沒必要，那沒關係，代碼執行正確就好了。今天，小的終於 忍不住了，和他說「我有我 review的標準，並且會努力提高這個標準 ，如果你覺得我 review的不重要，那別讓我 review就好了，我真的 不在意(OS:你媽，幫你看代碼，花時間，到時出bug也是多少得負責， 老子不如多做其它的事，還聽你抱怨)」對方也就同意了，有點像不歡 而散的感覺。各位在 code review 的有發生過類似的事嗎？或者有其 它 code review 過程中的趣事可供分享？ -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 222.130.164.45 ※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1423846558.A.63D.html
→
02/14 01:06, , 1F
你沒什麼錯, 讓你 review 又不聽 feedback 就沒意義了
02/14 01:06, 1F
→
02/14 01:07, , 2F
雖然也是可以婉轉一點啦, 可能之後再好好溝通一下比較好
02/14 01:07, 2F
→
02/14 01:38, , 3F
這樣還教review嗎XD
02/14 01:38, 3F
→
02/14 01:46, , 4F
都沒有沒有錯 如果談不隴直接請管理者訂下規則 大
02/14 01:46, 4F
→
02/14 01:46, , 5F
家遵守就好
02/14 01:46, 5F
其實我覺得不是誰對誰錯的問題，簡單二個字，就是隨緣，但我不想因為他降低我的 標準，才不想幫他看 code。 ※ 編輯: blabla123 (222.130.164.45), 02/14/2015 01:51:34
推
02/14 04:54, , 6F
其實code review這東西也很看個人，除非有講規則，不然
02/14 04:54, 6F
→
02/14 04:54, , 7F
其實跟coding style很像，各人有各人想法
02/14 04:54, 7F
→
02/14 07:53, , 8F
看看對方是想要讓程式好還是想提交而已
02/14 07:53, 8F
→
02/14 08:31, , 9F
看你主管阿 如果主管覺得沒差 你也別太認真
02/14 08:31, 9F
推
02/14 10:42, , 10F
coding style是團隊規範還是個人堅持?
02/14 10:42, 10F
推
02/14 10:55, , 11F
團隊沒有review的規範嗎?照規定來就好了吧
02/14 10:55, 11F
推
02/14 12:45, , 12F
那你的code需不需要別人review? 有沒有被刁的時候?
02/14 12:45, 12F
→
02/14 12:46, , 13F
還是建立共同規範比較好 免得成為彼此刁來刁去的藉口
02/14 12:46, 13F
推
02/14 13:39, , 14F
review是要符合專案的要求,不是符合個人標準
02/14 13:39, 14F
推
02/14 13:50, , 15F
你很努力不過工程師性格太重囉 要符合公司規範不是你自己
02/14 13:50, 15F
→
02/14 13:51, , 16F
你可以跟主管反映 主管如果覺得OK自然就成為公司規範
02/14 13:51, 16F
→
02/14 13:52, , 17F
如果主管不OK 你就別堅持了 合群比個人風格重要
02/14 13:52, 17F
→
02/14 13:53, , 18F
你覺得受不了 那可以跳槽到你心目中的公司 不要孤高自賞
02/14 13:53, 18F
這麼說吧，我們團隊有很多人，他可以去找其它願意替他看 code的人。我的標準其實還 是比公司的規範要鬆的。而且，還是有很多其它人要我替他們看代碼的，我覺得這是一種 肯定。 ※ 編輯: blabla123 (222.130.164.45), 02/14/2015 13:58:06
推
02/14 13:57, , 19F
就用軟體處理吧（茶）....
02/14 13:57, 19F
推
02/14 14:04, , 20F
比公司規範鬆的話，那唯一的問題就是語氣問題了
02/14 14:04, 20F
其實我不打算在這點上 back off 就讓其它的兄弟來幫他 code review 吧。你們 code review 有發生比較有趣的事嗎？ ※ 編輯: blabla123 (222.130.164.45), 02/14/2015 14:13:49
→
02/14 15:46, , 21F
其實有些是在系統上 code review 的，幾個人投票有超過才
02/14 15:46, 21F
→
02/14 15:46, , 22F
能放行。不一定需要每次同一個人來覺得互相為難。
02/14 15:46, 22F
推
02/14 15:56, , 23F
code review就是一場工程師之間的聖戰 XD 意見不合很正常
02/14 15:56, 23F
→
02/14 15:58, , 24F
覺得code review應該著重在 合進去後會不會產生系統其他問題
02/14 15:58, 24F
→
02/14 16:00, , 25F
他能隨意去找人review這點很奇怪 每個元件應該要是要特定人
02/14 16:00, 25F
→
02/14 16:01, , 26F
不過幫人review就是幫人背書 有自己的堅持也是常態
02/14 16:01, 26F
→
02/14 16:06, , 27F
還有你應該也不能拒絕幫人review 只能不同意他這樣做
02/14 16:06, 27F
推
02/14 20:44, , 28F
review不就是為了更好嗎...
02/14 20:44, 28F
推
02/14 23:54, , 29F
有review過真的比較好...
02/14 23:54, 29F
推
02/15 00:26, , 30F
看了好多code review吵架的事
02/15 00:26, 30F
推
02/15 00:57, , 31F
review rule 寫下來。
02/15 00:57, 31F
→
02/15 09:34, , 32F
有看過因為code review不合而離職 或是被lay off的...
02/15 09:34, 32F
→
02/15 09:37, , 33F
或是因為大小寫 最後變成十幾個人吵的不可開交的XDDD
02/15 09:37, 33F
推
02/15 16:00, , 34F
最後都會變成信仰問題(菸
02/15 16:00, 34F
其實我覺得為了 code review 離職太扯了…了不起別人不希望你看，那就別看就行了… lay off 應該是有其它的原因吧 Orz ※ 編輯: blabla123 (222.128.159.32), 02/15/2015 20:58:04
→
02/15 23:11, , 35F
是被review的人離職 每個元件都有指定的負責人負責review
02/15 23:11, 35F
→
02/15 23:12, , 36F
負責人不同意 rd就不能check in code 耽誤的時間都算那rd的
02/15 23:12, 36F
→
02/15 23:13, , 37F
這種情形久了之後自然就...
02/15 23:13, 37F
→
02/15 23:14, , 38F
被lay off是負責人不同意 然後rd不管 直接check in code
02/15 23:14, 38F
→
02/15 23:14, , 39F
然後該rd就被...
02/15 23:14, 39F
→
02/15 23:22, , 40F
總之就公司大了 rd寫code容易變得沒樂趣 感覺只是螺絲而已
02/15 23:22, 40F
→
02/15 23:23, , 41F
大概就這種感覺吧...
02/15 23:23, 41F
→
02/16 01:31, , 42F
科科，貼段code上來再討論
02/16 01:31, 42F
推
02/16 01:34, , 43F
jily 那邊看起來比較像是被 review 的人有問題啊
02/16 01:34, 43F
→
02/16 01:35, , 44F
至於原Po其實是對的，真要說錯，是沒有第一次就說你的標準
02/16 01:35, 44F
推
02/16 15:20, , 45F
我在前公司是不想給某人review,但他就是喜歡自己跳進來revi
02/16 15:20, 45F
噓
02/17 13:19, , 46F
兩變水準一樣，沒所謂的review，不過是問道於盲罷了
02/17 13:19, 46F
→
02/22 17:06, , 47F
我覺得主管有問題 主管應該要求公司規範被大家遵守才對
02/22 17:06, 47F
‣ 更多 blabla123 的文章
文章代碼(AID): #1KtYoUOz (Soft_Job)
討論串 (同標題文章)
以下文章回應了本文 (最舊先)：
完整討論串 (本文為第 1 之 12 篇)：
排序： 最舊先 | 最新先 | 留言數
在新視窗開啟完整討論串 (共12篇)
‣ 更多 blabla123 的文章
文章代碼(AID): #1KtYoUOz (Soft_Job)