看板
[ Soft_Job ]
討論串[閒聊] Code Review 意見不合
共 12 篇文章
內容預覽:
這種問題竟然可以吵一整串文. 原原po除了愛用大陸話跟脾氣比較差一點外. 整個review的方向是對的...... 1~變數命名不合規範:錯誤命名的變數會害別人誤會. 2~執行效率:如果你覺得執行效率不重要,那是你寫的程式太小or不夠重要. 光是一個call by reference和call by
(還有98個字)
內容預覽:
我有點不太明白，原PO公司code review的目的是甚麼?. 我想採取一種作法一般來說都有其目的與原因. 看起來對方的要求只是看邏輯是否正確. 而原PO的要求則是先看邏輯以外的東西. 假設code review的目的就只是確定邏輯是否正確. 那這就有點像有的公司會說徵PG. 但實際上職責是PM+
(還有421個字)
內容預覽:
先聲明一下，code review這真的是一個蠻有趣的串列，. 我是隨便挑一篇回，並不是特定針對某篇文章。. 不知道大家做code review是面對面看code還是offline?. 我現在的公司code都是放在GitHub上面，code review自然也成為Pull Request的一環，所以
(還有599個字)
內容預覽:
你這個齁，不是coding style問題，也不是公司問題. 是人的問題. 今天同樣你被抓包沒照規範走，但你同事跟你提醒的口氣是這樣. 哈囉，你剛進公司對不對??我看到你有些小地方，好像沒. 照著公司的規範寫喔!!. 程式的部份沒什麼大問題啦，雖然我還沒實際測過，不過這樣寫. 蠻OK的，就剩下版面格
(還有1192個字)
內容預覽:
想起曾經有過的類似 code review 的經驗。. 在一家新進的公司裡，要給軟體做個設定檔，當然就用 WinAPI 處理. private profile 的函數囉，所以做出來的設定檔就是 Windows 慣用的格式，. 有 section, key 和 value 那樣。. 本來想，大家的習慣
(還有994個字)