多程序员开发老是写重复的代码大家又遇到吗？

论代码 review 的重要性

一起维护起一个代码库

写之前, 如果是一个需要消耗较多时间的,先沟通一下, 少的话, 无所谓吧

我现在维护的东西一份代码复制粘贴复制到了大家四五十个文件里吧 重复度 80%以上~

@kimchan 代码必须要 review 不然时间长了就是一个屎坑

1. 组长没有去 review
2. 组内沟通不足。
3. 太多的话谁敢去改拼这个烫手山 YU 。只怪最开头写的几名程序太赖。

@lzjamao 是不是还加上主动性够强？

不 review 怎么可以？可以搭建 review server 推荐 JetBrains 家的 自用小 team 免费

还有函数强制写注释是必需的

我手头有个代码库，现在都不敢动，一动就各种 bug ，你还没办法修，一修 bug 更多~~~

review 单元测试和重构

代码我觉得精简一层就够了，如果你们项目的代码结构很清晰的话。

代码审查

1. 开发需求呢？
2. code review 呢？

我发现公司代码重复率也很高：
1.没有 review 机制
2.老员工留下的坑，不能随便动
3.新员工进来，对整体不熟悉
4.需求很紧，以完成任务为优先。后续没有进行优化

设计阶段也很重要（各种 UML diagrams 和 use case ），因为在设计阶段，应该能看出部分代码逻辑需要重复使用，从一开始就预防。当然，之后还要加上楼上提到的 review 之类的。
原来的代码重复就重复吧，除非公司给出时间让优化什么的。但新的功能的话 就好好好设计和 review 了

@houshengzi 越是越大的公司这种情况越严重，有些公司的采购系统做的跟屎一样，我就不说哪家公司了。。。但是你去看代码，有些逻辑呢，写了 2 遍，你再去看 changelog 的时候，发现这个代码几乎都是>6 手代码。。。。所以很多人宁可去自己再写一遍，而不用 ex 程序员的代码，因为读代码耗时间太多，算下来各种 RISK 和成本都蛮高的（ bug 风险，时间成本，经济成本 etc ）

unzip 的源码就这样，一大堆重复的。。。。。嗯，这个是开源的

我自己一个人写都会出这个问题…更何况几个人！

重构吧，少年。

要么，别管他…

每个程序员追求不一样，有的人写代码只是为了工作，有的人写代码是因为兴趣、热情。再没有 code review 的公司，程序员为了完成任务造点互联网垃圾什么的，太正常不过了。差点的程序已经分不清 controller 、 service 、 dao 有什么区别了，到处都是业务代码。。

突然想起那张图， good design 又不会给你加工资。。

不要怜惜那些屎一样的代码，该重构时就重构

sonarqube, checkstyle, findbugs, ReSharper, eslint, ...

重构哪里可以方便添加功能，那就重构哪里。。
当然，重构前后做好测试。。不然改出 bug 背锅可不好

不要舍不得删

说明该重新设计了

员工变动频繁，老代码不敢动，只能往上加

楼上很多说因为没有 review 的根本就不知道自己在说什么。 review 不会解决代码重复的问题，因为 reviewer 可能不知到 review 的代码复制自其它地方，有些极端情况下，在设计 solution 的时候得出的共识就是复制粘帖代码，然后修改。这样做无非是因为害怕重构代码会带来 bug 。所以这种事情不是体现了 review 的重要性，而是体现了单元测试的重要性！

难道不是一段代码重复写才会提高熟练度吗

找到了同一个 stackoverflow 页面吗？ 2333

会怀孕么。。。？
明显不会。。管他干蛋。。。