作者darkness0727 (72727)
看板java
標題[問題]不遵循開發規範
時間Tue Sep 13 19:03:42 2016
不知道有沒有前輩在也碰過這種問題
會有資深同事在一個講求速度和共用性的專案內
1.不管專案設計好的介面又另開介面搞高科技(自定義annotation一類的
2.用了新技術但是不精熟就用也沒補捉錯誤 造成錯誤出在底層
3.明明有共用可參考的做法 還是要再寫一套包起來
因為部分同事會抽調到別的案子
有時候就會有A照時程掰不完他的高科技 留給B去接
結果B發現沒做完的部分難以銜接要花大量時間處理
像這一類的問題有辦法透過Git一類的發現嚴格控管嗎?
比如說每一層架構都有固定強制要繼承的介面 沒繼承就是無法執行
因為小弟會參與專案開發規範
希望下個案子不會有再有人跟我抱怨有人寫了高科技 變成只有他能改比較順
這個問題在這次結案前檢討會中引起部分資淺組員"非常大"的反應
不只是後端 前端部分也有這個反應
難道就只能靠多派個人code review柔性勸導嗎
又不可能叫硬要寫高科技的就自己從開發測試到維護都掛他名字
--
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 118.171.22.209
※ 文章網址: https://webptt.com/m.aspx?n=bbs/java/M.1473764625.A.883.html
1F：→ qrtt1: 有沒有去掉情緒後的，實際程式比較版本！？ 09/13 19:20
2F：推 ripple0129: 開好介面又改自己爽的根本就是雷啊 09/13 19:40
3F：→ james732: 使用gerrit要review之後才可以merge？ 09/13 20:30
4F：推 LPH66: 就 code review 了吧, 最好是同一組內的人 review 09/13 22:59
5F：→ lagunawang: 好好的寫好測試，測不過就沒什麼好說的了... 09/14 00:59
6F：→ cryinglove: 每個人都要為自己新增的func寫好測試 09/14 08:18
7F：→ tacovirus: 這種事情應該用 LINT 去管理，用人管只會增加無謂衝突 09/14 09:59
8F：→ windincloud: 可以使用git hook的方式當commit時去檢查是否符合規 09/14 17:27
9F：→ windincloud: 範，只是要自己寫定義跟parser 09/14 17:28