Re: [閒聊] Code Review 意見不合
看板Soft_Job作者dritchie (Dennis MjY3OTk5)時間10年前 (2015/02/20 23:54)推噓0(0推 0噓 2→)留言2則, 1人參與討論串12/12 (看更多)
這種問題竟然可以吵一整串文
原原po除了愛用大陸話跟脾氣比較差一點外
整個review的方向是對的.....
1~變數命名不合規範:錯誤命名的變數會害別人誤會
2~執行效率:如果你覺得執行效率不重要,那是你寫的程式太小or不夠重要
光是一個call by reference和call by value的差異就可以影響整個產品好幾個%
3~debug開關:當你負責維護or處理客戶case的時候自然就會理解...
今天你放水讓它過 就是挖洞給別人跳 運氣不好還害以後的自己挫屎
那你不好好review嗎?? 這樣想的話 看到爛code翻桌跟他拼了還是剛好而已
※ 引述《blabla123 (念不停 煩不煩？)》之銘言：
: 公司規定提交的 code 至少要經過另外二個人code review才可合入，
: 可能我比較死板，每次我一定先檢查變量名命名法則是否符合規範，
: 還有代碼執行效率，打的 Log 有沒有在前面加上 DEBUG開關，然後才
: 會開始檢查相關邏輯問題。有個同事常叫我幫他 code review，但是他
: 常覺得這沒必要，那沒關係，代碼執行正確就好了。今天，小的終於
: 忍不住了，和他說「我有我 review的標準，並且會努力提高這個標準
: ，如果你覺得我 review的不重要，那別讓我 review就好了，我真的
: 不在意(OS:你媽，幫你看代碼，花時間，到時出bug也是多少得負責，
: 老子不如多做其它的事，還聽你抱怨)」對方也就同意了，有點像不歡
: 而散的感覺。各位在 code review 的有發生過類似的事嗎？或者有其
: 它 code review 過程中的趣事可供分享？
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 180.217.29.129
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1424447678.A.EE3.html
→
02/22 00:21, , 1F
02/22 00:21, 1F
→
02/22 00:22, , 2F
02/22 00:22, 2F
討論串 (同標題文章)
本文引述了以下文章的的內容：
完整討論串 (本文為第 12 之 12 篇)：