1
kimchan 2016-07-04 14:32:26 +08:00
论代码 review 的重要性
|
2
mumuy 2016-07-04 14:33:40 +08:00
一起维护起一个代码库
|
3
linKnowEasy 2016-07-04 14:35:17 +08:00
写之前, 如果是一个需要消耗较多时间的,先沟通一下, 少的话, 无所谓吧
|
4
murmur 2016-07-04 14:44:04 +08:00
我现在维护的东西一份代码复制粘贴复制到了大家四五十个文件里吧 重复度 80%以上~
|
5
luluuulu4848 2016-07-04 14:59:33 +08:00
@kimchan 代码必须要 review 不然时间长了就是一个屎坑
|
6
lzjamao 2016-07-04 15:21:59 +08:00 1
1. 组长没有去 review
2. 组内沟通不足。 3. 太多的话谁敢去改拼这个烫手山 YU 。只怪最开头写的几名程序太赖。 |
8
lovedebug 2016-07-04 17:26:49 +08:00
不 review 怎么可以?可以搭建 review server 推荐 JetBrains 家的 自用小 team 免费
|
9
lovedebug 2016-07-04 17:27:12 +08:00
还有函数强制写注释是必需的
|
10
keinx 2016-07-04 17:27:33 +08:00
我手头有个代码库,现在都不敢动,一动就各种 bug ,你还没办法修,一修 bug 更多~~~
|
11
repus911 2016-07-04 17:58:28 +08:00
review 单元测试和重构
|
12
DevineRapier 2016-07-04 18:20:12 +08:00
代码我觉得精简一层就够了,如果你们项目的代码结构很清晰的话。
|
13
zonghua 2016-07-04 18:21:15 +08:00
代码审查
|
14
mhtt 2016-07-04 18:51:31 +08:00
1. 开发需求呢?
2. code review 呢? |
15
houshengzi 2016-07-04 18:56:48 +08:00
我发现公司代码重复率也很高:
1.没有 review 机制 2.老员工留下的坑,不能随便动 3.新员工进来,对整体不熟悉 4.需求很紧,以完成任务为优先。后续没有进行优化 |
16
sfree2005 2016-07-04 19:57:58 +08:00
设计阶段也很重要(各种 UML diagrams 和 use case ),因为在设计阶段,应该能看出部分代码逻辑需要重复使用,从一开始就预防。当然,之后还要加上楼上提到的 review 之类的。
原来的代码重复就重复吧,除非公司给出时间让优化什么的。但新的功能的话 就好好好设计和 review 了 |
17
yjxjn 2016-07-04 20:09:42 +08:00
@houshengzi 越是越大的公司这种情况越严重,有些公司的采购系统做的跟屎一样,我就不说哪家公司了。。。但是你去看代码,有些逻辑呢,写了 2 遍,你再去看 changelog 的时候,发现这个代码几乎都是>6 手代码。。。。所以很多人宁可去自己再写一遍,而不用 ex 程序员的代码,因为读代码耗时间太多,算下来各种 RISK 和成本都蛮高的( bug 风险,时间成本,经济成本 etc )
|
18
congeec 2016-07-04 20:18:36 +08:00
unzip 的源码就这样,一大堆重复的。。。。。嗯,这个是开源的
|
19
loading 2016-07-04 20:23:43 +08:00 via Android
我自己一个人写都会出这个问题…更何况几个人!
重构吧,少年。 要么,别管他… |
20
broadliyn 2016-07-04 21:30:03 +08:00
每个程序员追求不一样,有的人写代码只是为了工作,有的人写代码是因为兴趣、热情。再没有 code review 的公司,程序员为了完成任务造点互联网垃圾什么的,太正常不过了。差点的程序已经分不清 controller 、 service 、 dao 有什么区别了,到处都是业务代码。。
突然想起那张图, good design 又不会给你加工资。。 |
21
strwei 2016-07-04 21:52:09 +08:00
不要怜惜那些屎一样的代码,该重构时就重构
|
22
shot 2016-07-04 22:30:36 +08:00
sonarqube, checkstyle, findbugs, ReSharper, eslint, ...
|
23
aprikyblue 2016-07-04 23:03:10 +08:00 via Android
重构哪里可以方便添加功能,那就重构哪里。。
当然,重构前后做好测试。。不然改出 bug 背锅可不好 |
24
SoloCompany 2016-07-04 23:04:32 +08:00
不要舍不得删
|
25
pagict 2016-07-04 23:26:16 +08:00
说明该重新设计了
|
26
mingyun 2016-07-04 23:28:37 +08:00
员工变动频繁,老代码不敢动,只能往上加
|
27
georgema1982 2016-07-05 00:38:11 +08:00 1
楼上很多说因为没有 review 的根本就不知道自己在说什么。 review 不会解决代码重复的问题,因为 reviewer 可能不知到 review 的代码复制自其它地方,有些极端情况下,在设计 solution 的时候得出的共识就是复制粘帖代码,然后修改。这样做无非是因为害怕重构代码会带来 bug 。所以这种事情不是体现了 review 的重要性,而是体现了单元测试的重要性!
|
28
DT27 2016-07-05 09:15:11 +08:00
|
29
lovedboy 2016-07-05 14:07:04 +08:00
找到了同一个 stackoverflow 页面吗? 2333
|
30
sampeng 2016-07-05 17:18:42 +08:00
会怀孕么。。。?
明显不会。。管他干蛋。。。 |