作者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/cn.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