
一堆注释代码,毫无代码格式可言,大部分时间都浪费在理逻辑,我接手前项目经过好几个人手,改的乱七八糟,每天都想砸键盘,辞职的心都有了,怎么办?
1 allgy OP 老项目是 php 框架 tp3.1 写的 |
2 xeis Sep 7, 2017 via Android 重构 |
4 Light3 Sep 7, 2017 幸好只是维护..开发咋整 你都不知道有没有写公共方法~ 不过我觉得砸键盘倒是不用 tp 至少还是有序可循的.. |
5 66beta Sep 7, 2017 重构 哦不,换框架重做! |
6 roys Sep 7, 2017 走人吧~ 我页经历过类似项目。 |
10 UnisandK Sep 7, 2017 换 HHKB,砸之前想一想价格 |
12 scys Sep 7, 2017 没法,截断一个模块出来,看看能不能一个个模块重构。 大项目一个人要重构太难。 |
13 Adamla Sep 7, 2017 给你备多个键盘以防之 |
17 dream7758522 Sep 7, 2017 via Android windows 运行里输入 osk |
18 Marlon Sep 7, 2017 via iPhone 慢慢理,维护还好说吧, |
19 chinvo Sep 7, 2017 直接把键盘砸掉避免之后再费劲砸 |
20 allgy OP @dream7758522 不明觉厉 |
23 Sanko Sep 7, 2017 via Android 删库跑路 |
24 t123yh Sep 7, 2017 via Android 如果浏览器兼容不用怎么考虑的话,可以尝试一下前端直接强上 Vue 进行渲染 |
25 TuxcraFt Sep 8, 2017 告诉老板 要么涨工资 要么走人…… |
26 cxbig Sep 8, 2017 写新的 > 额外津贴 > 走人 |
27 msg7086 Sep 8, 2017 维护啥?改 bug ?那就慢慢改咯。烂项目改动耗时自然偏多,平时改一个 Bug 半天,换上烂项目改一个 Bug 要两周,这很正常的。心态放平,慢慢做就是了。跟上司或者老板说清楚情况,让他们充分了解工作的难点,给你充足的时间,我觉得就足够了。毕竟是按照时间发你工钱的,同样的时间,同样的收入啊。 |
28 cmonday Sep 8, 2017 建议要求立项重做,好歹有个盼头。维护恶心的老代码除了培养耐心之外对个人提升没什么益处,远不如维护设计合理的项目或者开发有挑战的新项目。 如果不答应的话,除非薪水特别诱人,还是走人吧。 |
29 peneazy Sep 8, 2017 via Android 看看你的上级人怎么样,人不行的话,果断走人 |
30 sfree2005 Sep 8, 2017 via Android 如果不能全部重构,至少可以提议部分独立的业务重构。就算是我自己的项目,看到自己一两年前写的代码我也有砸键盘的冲动,重构会是我的唯一选择。 |
31 cxh116 Sep 8, 2017 via Android 你是没有接手过更加糟糕的,一个仓库的 php 项目,维护了 5 年多,里面有用三种框架来实现不同的功能。 |
32 AEANWspPmj3FUhDc Sep 8, 2017 via Android 再把代码恶化一下。 等着下一个接盘侠。 |
33 rason Sep 8, 2017 以我多年经验,换一家很大可能也是这样的,骚年 |
34 xomix Sep 8, 2017 …………没什么啊,就当学习一下反面教程就好了。 |
35 zlhsvc Sep 8, 2017 维护还好点,开发新功能才要你命 |
36 klgd Sep 8, 2017 |
37 jadetang Sep 8, 2017 via Android 我觉得那些劝你走人的都是在搞笑,工作中肯定会碰到遗留代码的。所谓的重写,其实也就是在造新的遗留代码罢了。大家都看过 重构 那本书,这是实践的好机会。当然,重要的是要管理好老板的预期,把事情说的严重一点。 |
39 FYK Sep 8, 2017 我们的主站版本还在 3.1 之前,不过我只是个新人,还在做基础工作,涉及不到源代码。这里就祝福一下你啦。 |
40 allgy OP @t123yh 私下跟前端讨论过,为什么不搞分离,他说不想碰,最令我惊讶的是一个前端归产品部管,上级不懂技术,部门聚餐都是跟产品设计一起 |
42 caijihui11 Sep 8, 2017 理理逻辑也是程序员的工作之一。 注释先别管,万一别人有用呢 |
46 ferstar Sep 8, 2017 劝走的真是够了,讲道理工作中给前人擦屁股基本上避无可避,逃得了这家逃不了下家。说说我的经历吧,前阵子刚把一个全用 Shell 和 Perl 拼起来的烂摊子收拾完毕,用 Python 重写,搞完系统性能提高了 300 倍(没错是 300 倍,主要是前面的摊子实在太烂,随便加个多线程然后并行计算就能提速)。刚接盘时全局变量满天飞,各种 function1,function2,function3 简直马勒戈壁,Perl 的那些我了个去的#$%,当时我也很绝望啊,还好坚持下来了,为啥能坚持?是因为上家辞职闪人就是接了个要把 demo 速度拉上生产环境上线的烂项目,当时走的那叫一个潇洒,裸辞近一个月才找到这家,没想到还是坑,还好填平了。 PS:偶尔逛了下拉钩,发现上家 HR 还在招接盘侠。推荐楼主看下这本书《凤凰项目:一个 IT 运维的传奇故事》,我是很收鼓舞的 |
48 Felldeadbird Sep 8, 2017 既定事实,基本没办法再修改风格的了。只能按照:先在小部分进行优化,确保之前的没错误。一点一点迁移到新架构中。。。 反正,这事情很漫长的。楼主你遇到一点事情就这样,和我三年前差不多情况。不过那时候我没打算辞职。只想着重构他。 |
49 allgy OP |
53 Qlccks2 Sep 8, 2017 多买几个键盘旁边放着,省得砸坏了没得用。 |
54 l00t Sep 8, 2017 一边砸键盘一边慢慢改吧。这都走人的话还真没多少工作合适的了。维护老代码是再常见不过的工作内容了啊。 |
56 x86 Sep 8, 2017 via Android 跟我比惨吗?手上还有个 tp3.2 的就项目,代码看的我想打人,真的是用 tp 不可怕,写的那么烂才是真的可怕 |
58 abujj Sep 8, 2017 via Android 哈哈哈哈哈 |
60 allgy OP @caijihui11 之前看过有人讨论过,为什么一些人喜欢留注释代码,好像是 防御性开发,我也忘记了怎么说 |
61 jswh Sep 8, 2017 重构也很有成就感啊。看一件事慢慢完成,和看一件事慢慢变好,对我来说都挺好的 |
62 allgy OP @ferstar 哈哈,理解你在填坑中的心情,现在情况是 bug 接踵而来,一个压一个,他们根本不关心代码是如果优雅,只要结果就是你解决了 bug |
63 815lbh Sep 8, 2017 有注释都不错了,你要知道没有注释的老项目,会让你哭。 |
64 PazuLee Sep 8, 2017 其实更坑的是你老板不知道。。。。所以有机会的话先跟老板诉诉苦吧,起码让她知道你干了啥。。。这样再难怼出来也算有价值的。 |
65 2ME Sep 8, 2017 所以是所有逻辑都写在控制器 然后模型没有一行代码的那种项目吗 .. |
66 nullcoder Sep 8, 2017 那些说重构的,你们给 PO 主写测试? |
67 rswl Sep 8, 2017 你看过一份 20 多 M 的 c 文件吗没有注释 |
69 nicevar Sep 8, 2017 维护性的工作,不要说的那么夸张,况且才四五年前的,主要算史前那一些运营商的项目都是大爆炸时代的了,不要听上面的人瞎扯,轻易的去重构,先熟悉透了做合理的小块修改,以前有个同事,年轻人太冲动,有个维护的项目跑着 100 万级别的用户,让他别乱动他非要去改一个不太熟悉地方,结果造成大面积用户无法使用,投诉电话打爆了 |
70 SilentDepth Sep 8, 2017 如果除了接盘别无选择的话: 1. git init && git commit --all 2. 按自己喜欢的风格全文格式化代码,然后 git commit 3. 如果可以,移除掉所有未被实际使用的引用(没开发过 PHP,一些 IDE 应该有这个功能) 4. 如果可以,整理出所有模块的依赖情况和业务线(如果这个项目存在业务线的概念相对独立的功能链) 5. 从被依赖数最小的模块开始整理业务逻辑 & 注释 & 重构,注释写输入输出即可 5.1. 如果可能,配合一点简单的单元测试(为了防止 Willow 无意点火饥荒梗) 6. 调整作息,保证稳定的工作节奏,非工作时间果断避开所有跟这个项目有关的事情 6.1. 心态决定一切 当然,如果你有别的选择,或者实际情况已经明显超出了你的能力范围,或者老板要求明天就上线,或者有任何其他重要因素促使你拒绝 take this shit,珍爱生命,果断走人(丢锅) |
71 qiqico Sep 8, 2017 管理好上级预期,列一个计划并随时更新,让上级了解你工作的艰巨,以及你付出的努力。。 当一天和尚,撞一天钟,反正按时间领薪水的。。 |
72 allgy OP @Felldeadbird 嗯,有时候其实没有太多选择,自己也是个普通人,只能凭这个吃饭,不是什么的大神,随便跳就有橄榄枝 |
77 wolffn Sep 8, 2017 刚工作的时候,都会不爽 等工作一段时间发现,这就是工作的主旋律。 |
78 allgy OP @SilentDepth 嗯,感谢给出这么详细的建议 |
85 silov Sep 8, 2017 问个镜像问题:项目第一个 /批开发者,如何避免后人接手的时候产生此类情绪? 我现在就是团队的后段项目第一个开发者,尽量完善一些代码和注释,尽量规范化,但是业务变更有时候是很快的,项目要跟上进度需求很容易产生一些逻辑混乱的代码。。。这种事情又需要怎么避免呢。。。 |
87 fishman Sep 8, 2017 via Android 说走人的,你们都不维护项目吗? |
89 qiumaoyuan Sep 8, 2017 有能力和权力就直接改。没能力或者没权力就走。抱怨不解决问题。 |
90 msg7086 Sep 8, 2017 对于 Append 我跟你说,单个回复里可以回复很多人,不需要一层一层回,避免触发 1800 大法。 |
91 wangcansun Sep 8, 2017 via iPhone 要不要来一次重构啊,多爽 |
92 silov Sep 8, 2017 |
95 server Sep 8, 2017 忍 或者 滚, |
96 fhefh Sep 8, 2017 慢慢把坑填好~~ 我填了好几次坑了 反正能用 稳定运行 |
97 xzg1993 Sep 8, 2017 我真觉得我是个好人把代码都重构了恶心的代码自己都看不下去 |
98 ashin Sep 8, 2017 我也经历过类似项目,维护过的人最后都离职了 |
99 Michelle1991 Sep 8, 2017 不是好正常的事情?如果做不了赞同忍或滚,所有工作都是自己分析衡量是否可以做,是否决定做,决定就不要考虑太多,抱怨无法解决问题,尽力解决问题,尽力后无法解决自己觉得无法接受了那就滚,只是别滚到同样的坑~忌抱怨心态 |
100 em84 Sep 8, 2017 胃镜做起来多爽(滑稽) |