[問題]不遵循開發規範

作者: darkness0727 (72727)   2016-09-13 19:03:42
不知道有沒有前輩在也碰過這種問題
會有資深同事在一個講求速度和共用性的專案內
1.不管專案設計好的介面又另開介面搞高科技(自定義annotation一類的
2.用了新技術但是不精熟就用也沒補捉錯誤 造成錯誤出在底層
3.明明有共用可參考的做法 還是要再寫一套包起來
因為部分同事會抽調到別的案子
有時候就會有A照時程掰不完他的高科技 留給B去接
結果B發現沒做完的部分難以銜接要花大量時間處理
像這一類的問題有辦法透過Git一類的發現嚴格控管嗎?
比如說每一層架構都有固定強制要繼承的介面 沒繼承就是無法執行
因為小弟會參與專案開發規範
希望下個案子不會有再有人跟我抱怨有人寫了高科技 變成只有他能改比較順
這個問題在這次結案前檢討會中引起部分資淺組員"非常大"的反應
不只是後端 前端部分也有這個反應
難道就只能靠多派個人code review柔性勸導嗎
又不可能叫硬要寫高科技的就自己從開發測試到維護都掛他名字
作者: qrtt1 (有些事,有時候。。。)   2016-09-13 19:20:00
有沒有去掉情緒後的,實際程式比較版本!?
作者: ripple0129 (perry tsai)   2016-09-13 19:40:00
開好介面又改自己爽的根本就是雷啊
作者: james732 (好人超)   2016-09-13 20:30:00
使用gerrit要review之後才可以merge?
作者: LPH66 (-6.2598534e+18f)   2016-09-13 22:59:00
就 code review 了吧, 最好是同一組內的人 review
作者: lagunawang   2016-09-14 00:59:00
好好的寫好測試,測不過就沒什麼好說的了...
作者: cryinglove (毓™)   2016-09-14 08:18:00
每個人都要為自己新增的func寫好測試
作者: tacovirus (小璋丸)   2016-09-14 09:59:00
這種事情應該用 LINT 去管理,用人管只會增加無謂衝突
作者: windincloud (虹霓之戀)   2016-09-14 17:27:00
可以使用git hook的方式當commit時去檢查是否符合規範,只是要自己寫定義跟parser
作者: james732 (好人超)   2016-09-15 10:23:00
https://goo.gl/gl0MCp Android的gerrit管理

Links booklink

Contact Us: admin [ a t ] ucptt.com