作者howdiun (Howdiun)
看板Soft_Job
標題Re: [閒聊] Code Review 意見不合
時間Mon Feb 16 10:11:44 2015
基本上我看別人的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
推 idoz5: 好招! 02/16 12:17
推 zo4j4: good 02/16 13:04
推 fortinet: Push 02/16 13:50
推 yyc1217: 可以用三明治講法:整體還OK,但這裡...,應該改一下就好 02/16 19:55
→ viper9709: 推~高招XD 02/16 23:58