刚入职,要熟悉现有代码(没有文档)好头痛...

2016-10-15 20:11:17 +08:00
 mortonnex
代码量非常大,没有文档

只有劝自己死磕了...
4320 次点击
所在节点    程序员
26 条回复
tilv37
2016-10-17 11:07:27 +08:00
接手新公司的代码,前人 1 年前早已离职,无文档,公司里唯一“懂”该程序的人,只能告诉我程序的使用方法。其他的全都不知道。我都不知道我那一阵是怎么度过的。
RiceNoodle
2016-10-17 13:04:15 +08:00
@ainimuyan 刚入坑就重构,会不会太自信了点?
我觉得重构起码要基于对于业务有足够的了解。而且重构要确信带来优势,无论是阅读上的优势还是维护上的优势,这个优势不能只你一个人说了算,起码得到大多数项目组的认可才行吧。
再一点,你重构了代码,也要和测试同事沟通清楚。不测试上线的话,是对项目的不负责,测试的话,就涉及到测试同事的工作安排了。
我以为,想清楚了重构的目的,有了足够的资源(时间,同事对重构基本方案的认可,测试同事的时间),再去重构会更合理一点。
不赞成动不动就重构的做法,我那样更大概率只会留下更多的坑而不是减少坑。。
ainimuyan
2016-10-17 22:30:08 +08:00
@RiceNoodle 你说的很多点都是对的,各种因素自然会考虑的,并且业务也不是很复杂,不然我傻啊给自己挖坑。。😁
ainimuyan
2016-10-17 22:39:27 +08:00
@palytoxin 当然不能忍,我是接手了一段时间了,项目不大也增加了不少功能了,业务的各个方面感觉理解的还算透彻并且也提了不少建议被接受了,基于这些前提以及后续业务方向的调整,同时原有的代码是一个刚学几个月 java 的同事为了赶进度写的。。。自然会有一些问题的,因此。。感觉要是你的脾气,你会比我还想改一改。另外自信谈不上,我更愿意向大家学习,各种前辈的代码也是看了又看,或者有问题的话尽早提出来一起讨论避免埋下什么隐患。。。大家何必这么激动呢。。😁
ainimuyan
2016-10-17 22:43:08 +08:00
@palytoxin 我也应该反思反思,或许我一些地方做的比较激进,应该慢慢来,考虑大家的感受,每个人的感受,循序渐进,比如等到问题出现了大家会更容易接受一点。。。
palytoxin
2016-10-18 00:57:26 +08:00
@ainimuyan 别小瞧任何业务逻辑,很可能有你根本想不到的坑。测试不是百分百覆盖的情况下,除了性能特别不能忍,尽量别动生产代码。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/313030

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX