一个臃肿的老项目,强制 CODE REVIEW 没有按照规定执行 不能提交代码，团队成员强烈反弹抱怨

[pence2019]

作为 top leader 安排 我是执行者。

要求 所有路过的方法增加方法 Java 注释
iBatis 方法 加注释
类加注释

一个老员工强烈反弹 情绪化 说 我不提交代码了，浪费多少时间 等等等。。。。

[woodensail]

对于老项目，一般原则是保持原有风格，尽量少动，同时新增代码也要尽量贴近原有风格。要不然一个项目经手几回就乱成一团糟了。
当然要是想当做一个长期项目来做，建议重构，不仅仅是代码风格，连项目架构一起按自己想要的方式来调。

[pence2019]

@woodensail 原有风格是 java 类 和方法 代码没有注释 iBatis 没有注释，那我们也要按照原风格走,
现在就是一个焦油坑 没有碰，但是 top leader 不懂技术 把这个作为长期项目对待，

你公司有 code review 吗?

[KentY]

所有路过的方法... 什么叫"路过"的方法?
如果是指所有方法, 强制所有方法都加 javadoc 是不明智的.

[woodensail]

@pence2019 有，但是看具体项目，这边 code review 的第一追求是减少无用改动，其次才是代码风格。特别是对于老项目，不允许在不经测试的情况下私自改动。
实在忍不了就提重构，自己重新做，测试重新测。

[pence2019]

@KentY 改的功能模块 类呀...
你公司有执行 CODE REVIEW 吗?

[Justin13]

加点注释就不行了?
我们这还要补 Spec,写 Requirement,本来的 JS 转 TS，漏的单元测试要补上。

[KentY]

@pence2019
还是不懂, 改的模块, 类? 你意思就是这个项目的方法以前没有 javadoc, 现在都要加上是么?

有.现在很少有没有 review 的吧.

[eGlhb2Jhb2Jhbw]

不愿意可以，但是也要给个合理的理由啊，什么叫浪费时间？公司给他发的工资难道不就是买他时间的么，公司都没嫌他浪费时间，他自己先急了。不想一起玩就滚蛋，最烦这种事精了。

[pence2019]

@woodensail 目标:增加 注释 优化代码结构 提高方法重用性 多使用设计模式

[woodensail]

@pence2019 注释基本上强制推行没问题，写注释基本不会翻车。
但是后面几个就算是重构级别的了，动到哪儿了都得全面回归测试。事前也得做全面的分析，确定影响范围。

[bxb100]

估计没算进工时

[woodensail]

其实我感觉楼主的公司就是想做重构又没时间，只好搞这种渐进式重构。但是渐进式重构需要对项目有很强的掌控能力，要不然各种翻车。比如遗漏影响范围导致依赖方挂了，忽略特殊场景导致部分业务场景异常等。

[hehheh]

算进工时里就好了，注释也算代码量。

[winterbells]

我之前也想对这个项目做各种更新
后来发现自己太天真了。。
1、你提出来的，那就你来改吧
2、你改的，出问题找你

折腾了我一星期，现在谁爱干嘛干嘛去，我不会多说一句话。

[zarte]

给时间都好说，应该是只算开发时间，然后做这些的时间加上就要义务加班了。

[pence2019]

@KentY 没有注释 焦油坑呀
@eGlhb2Jhb2Jhbw 一个女技术组长的情绪化反应，我表示很无语，你是技术组长呀，。。。可能人家就是想当农民
@woodensail 现在大家的场景就是修复 BUG 的时候 增加相关方法和类的注释 这都不行。
@hehheh 一个女技术组长的情绪化反应，我表示很无语，你是技术组长呀，。。。可能人家就是想当农民。
@bxb100 和工时没有关系，只是情绪化反应

[pence2019]

@winterbells 不负责任的态度，你每天给你白发工资吧，你怎么提升，？？？

[pence2019]

@zarte 就没有做 bug 让你多久完成，你可以提出增加时间呀，。。。这个和时间无关

[woodensail]

@pence2019 加注释都不肯就比较麻烦了，没有威信的后果(doge)

[zarte]

@pence2019 你有跟他们说这个的时间可以算进工时么？可以根据实际情况加？你没说的话换我也不会提，等下被怼加个注释要这么久时间？