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

看板Soft_Job作者 (Howdiun)時間10年前 (2015/02/16 10:11), 編輯推噓4(401)
留言5則, 5人參與, 最新討論串4/12 (看更多)
基本上我看別人的code也有很多OS 「這排版也太醜了吧,IDE不是可以自動對齊嗎,連按都不按?」 「這個單字拼錯了吧,而且還是對外參數,是要逼其他人一起拼錯嗎?」 「到底要擁有怎樣的腦袋,才有辦法寫出這種東西?」 「我常常聽別人說腦袋裝屎,但現在我是看到一坨屎.....」 「這不是重複了嗎...這不是多餘的嗎...這有必要另外寫嗎....」 「你是希望寫到只剩你看得懂吧...為什麼我還看得懂....」 「你其實是新手吧?前十年其實在寫word對吧?為什麼薪水那麼高?」 「你是傳遞魔人嗎?你不覺得這裡傳這些變數很詭異嗎?還是你本身就很詭異?」 以上都是內心戲亂演, 現實上我最多說出:「嗯哼、嗯哼」 最後跟同事說: 「寫得還不錯嘛!」 一般人聽到被誇獎, 都會直接接受不會拒絕, 所以我等於是第一句話就讓他接受我的論點了(點頭) 「是說這裏怎麼這樣寫」(指) 因為他已經肯定了我的第一句話, 他很有可能就會繼續肯定我第二句話, 「對啊,這裏怎麼這樣寫,我去改掉」 ※ 引述《blabla123 (念不停 煩不煩?)》之銘言: : 公司規定提交的 code 至少要經過另外二個人code review才可合入, : 可能我比較死板,每次我一定先檢查變量名命名法則是否符合規範, : 還有代碼執行效率,打的 Log 有沒有在前面加上 DEBUG開關,然後才 : 會開始檢查相關邏輯問題。有個同事常叫我幫他 code review,但是他 : 常覺得這沒必要,那沒關係,代碼執行正確就好了。今天,小的終於 : 忍不住了,和他說「我有我 review的標準,並且會努力提高這個標準 : ,如果你覺得我 review的不重要,那別讓我 review就好了,我真的 : 不在意(OS:你媽,幫你看代碼,花時間,到時出bug也是多少得負責, : 老子不如多做其它的事,還聽你抱怨)」對方也就同意了,有點像不歡 : 而散的感覺。各位在 code review 的有發生過類似的事嗎?或者有其 : 它 code review 過程中的趣事可供分享? -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 211.75.221.136 ※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1424052706.A.0D6.html

02/16 12:17, , 1F
好招!
02/16 12:17, 1F

02/16 13:04, , 2F
good
02/16 13:04, 2F

02/16 13:50, , 3F
Push
02/16 13:50, 3F

02/16 19:55, , 4F
可以用三明治講法:整體還OK,但這裡...,應該改一下就好
02/16 19:55, 4F

02/16 23:58, , 5F
推~高招XD
02/16 23:58, 5F
文章代碼(AID): #1KuL7Y3M (Soft_Job)
討論串 (同標題文章)
文章代碼(AID): #1KuL7Y3M (Soft_Job)