新入职被安排维护前人留下的'史前'项目,怎么防止砸键盘?

2017-09-07 18:16:38 +08:00
 allgy

一堆注释代码,毫无代码格式可言,大部分时间都浪费在理逻辑,我接手前项目经过好几个人手,改的乱七八糟,每天都想砸键盘,辞职的心都有了,怎么办?

9646 次点击
所在节点    职场话题
112 条回复
jswh
2017-09-08 09:37:33 +08:00
重构也很有成就感啊。看一件事慢慢完成,和看一件事慢慢变好,对我来说都挺好的
allgy
2017-09-08 09:37:57 +08:00
@ferstar 哈哈,理解你在填坑中的心情,现在情况是 bug 接踵而来,一个压一个,他们根本不关心代码是如果优雅,只要结果就是你解决了 bug
815lbh
2017-09-08 09:39:43 +08:00
有注释都不错了,你要知道没有注释的老项目,会让你哭。
PazuLee
2017-09-08 09:45:46 +08:00
其实更坑的是你老板不知道。。。。所以有机会的话先跟老板诉诉苦吧,起码让她知道你干了啥。。。这样再难怼出来也算有价值的。
2ME
2017-09-08 09:57:44 +08:00
所以是所有逻辑都写在控制器 然后模型没有一行代码的那种项目吗 ..
nullcoder
2017-09-08 10:00:55 +08:00
那些说重构的,你们给 PO 主写测试?
rswl
2017-09-08 10:13:23 +08:00
你看过一份 20 多 M 的 c 文件吗没有注释
ferstar
2017-09-08 10:14:29 +08:00
@allgy 给老板讲清楚状况, 加钱加时间加 leader, 然后必须让他知道你的辛苦
nicevar
2017-09-08 10:27:37 +08:00
维护性的工作,不要说的那么夸张,况且才四五年前的,主要算史前那一些运营商的项目都是大爆炸时代的了,不要听上面的人瞎扯,轻易的去重构,先熟悉透了做合理的小块修改,以前有个同事,年轻人太冲动,有个维护的项目跑着 100 万级别的用户,让他别乱动他非要去改一个不太熟悉地方,结果造成大面积用户无法使用,投诉电话打爆了
SilentDepth
2017-09-08 10:30:12 +08:00
如果除了接盘别无选择的话:

1. git init && git commit --all
2. 按自己喜欢的风格全文格式化代码,然后 git commit
3. 如果可以,移除掉所有未被实际使用的引用(没开发过 PHP,一些 IDE 应该有这个功能)
4. 如果可以,整理出所有模块的依赖情况和业务线(如果这个项目存在业务线的概念——相对独立的功能链)
5. 从被依赖数最小的模块开始整理业务逻辑 & 注释 & 重构,注释写输入输出即可
5.1. 如果可能,配合一点简单的单元测试(为了防止 Willow 无意点火——饥荒梗)
6. 调整作息,保证稳定的工作节奏,非工作时间果断避开所有跟这个项目有关的事情
6.1. 心态决定一切

当然,如果你有别的选择,或者实际情况已经明显超出了你的能力范围,或者老板要求明天就上线,或者有任何其他重要因素促使你拒绝 take this shit,珍爱生命,果断走人(丢锅)
qiqico
2017-09-08 10:41:01 +08:00
管理好上级预期,列一个计划并随时更新,让上级了解你工作的艰巨,以及你付出的努力。。

当一天和尚,撞一天钟,反正按时间领薪水的。。
allgy
2017-09-08 11:01:38 +08:00
@Felldeadbird 嗯,有时候其实没有太多选择,自己也是个普通人,只能凭这个吃饭,不是什么的大神,随便跳就有橄榄枝
allgy
2017-09-08 11:02:36 +08:00
@abujj 😭
allgy
2017-09-08 11:04:03 +08:00
@rswl 👍
allgy
2017-09-08 11:04:48 +08:00
@ferstar 才来没多久,老板面都见不到
allgy
2017-09-08 11:06:04 +08:00
@nicevar 嗯,很多 bug 就是被前人生生改出来的
wolffn
2017-09-08 11:09:26 +08:00
刚工作的时候,都会不爽

等工作一段时间发现,这就是工作的主旋律。
allgy
2017-09-08 11:09:33 +08:00
@SilentDepth 嗯,感谢🙏给出这么详细的建议
allgy
2017-09-08 11:10:32 +08:00
@jswh 好痛苦....
allgy
2017-09-08 11:11:24 +08:00
@PazuLee 见不到老板人...

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

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

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

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

© 2021 V2EX