这是一个创建于 2069 天前的主题，其中的信息可能已经有所发展或是发生改变。
楼主本科生，这周接手了校宣传部一个比赛网站的开发任务，代码是好几年前祖传下来的 PHP Yii 框架开发，之后每年这个比赛都会加一些新需求，然后每年学生都得修修补补
今年新加了几个分项，发现整个库表结构都得改，加上架构逻辑看起来很分散（不知道是框架的范式还是设计的问题），某个功能变了五六个文件得连带着改，还有不知道为什么某些表单结构和路由也放在数据库....。
部署到服务器后上传图片的接口崩了，半天也没定位到 Bug，易班认证的 OAuth2 接口也崩了，（说因为要推广易班所以需要先易班认证才能在本站注册），发现整个代码很多补丁的痕迹...头大 orz
老师本来告诉我有个功能前面几届都没给她实现，我当时没看项目代码，想着不是多复杂的功能，拍胸脯我今年肯定给她搞定...现在我明白为什么前几年没有实现了
第 1 条附言 · 2019-03-31 21:14:55 +08:00
感谢各位前辈的建议
 |
1
mamahaha 2019-03-31 11:36:07 +08:00  21
而且以后别人接手这个代码时，你也会被接手者划分到垃圾代码产生者来看待。
|
 |
2
CEBBCAT 2019-03-31 11:38:18 +08:00 via Android
先上 git 私有库，再上重构大法？
|
 |
3
lovedebug 2019-03-31 11:40:02 +08:00 via Android
重写可能更快。这类校内网站功能又不是很多很复杂。
|
 |
4
reus 2019-03-31 11:40:20 +08:00 1
重写吧
|
 |
5
whoami9894 OP |
|
6
CEBBCAT 2019-03-31 12:03:36 +08:00 via Android
@whoami9894 快跑！！！
|
 |
7
blackmirror 2019-03-31 12:10:01 +08:00 11
每个接手都想重写，到最后都是修修补补
|
 |
8
mscststs 2019-03-31 12:12:37 +08:00 via Android
@whoami9894 老师不懂开发，那么老师的意见很重要吗？？不懂你们的逻辑
|
 |
9
emmettwoo 2019-03-31 12:14:53 +08:00 via Android
@blackmirror 太真实了
|
|
10
whoami9894 OP @mscststs
这种项目大家其实都不愿意做，老师又催得紧，所以按自己的意思重写的话最后还得担责任，没有必要 |
|
11
whoami9894 OP @blackmirror
刚开始那几年修修补补应该是能用的，之后每年比赛要求都变更，今年变化大新加了几个分项...唉 |
 |
12
Mac 2019-03-31 12:41:01 +08:00 via Android
重构呗
|
 |
13
2589595915 2019-03-31 12:48:26 +08:00 via iPhone
老师算什么东西。你觉得该弄就弄。
|
|
14
mscststs 2019-03-31 13:02:32 +08:00 via Android
@whoami9894 允悲，学校里面做苦力其实还蛮难受的
|
 |
16
scukmh 2019-03-31 13:11:00 +08:00
又没有钱，没事干嘛给他打工。
|
 |
17
notreami 2019-03-31 13:48:54 +08:00 1
钱给够，时间给够。否则按需求改改完事，短板优化原则。
每维护个"屎山"都重写，浪费时间和精力，最关键 ROI 太低，怎么证明你的能力？ |
|
18
reus 2019-03-31 13:55:28 +08:00
@whoami9894 你先做出来，然后拿给老师看，如果做得好，自然就用你的了。学生时期，有这么一个机会，就要抓住。就算最终你的东西达不到要求，不用你的，你还是得到了宝贵的经验啊。
|
 |
19
okjb 2019-03-31 13:59:12 +08:00
三年三年又三年，缝缝补补又三年，能用就行
|
|
20
reus 2019-03-31 14:09:26 +08:00 1
如果你处理不了，那你就会成为老师口里的“前几届”其中之一
|
 |
21
qinkangdeid 2019-03-31 14:27:17 +08:00 via Android
就像楼上一位兄台说的
每一位接收者在看代码之后 心里都想着重构 重构了几个方法后 放弃的概率越来越大 重构好了 表面上看和以前一样 老板（老师）又不看代码 觉得有什么区别 你说以前的版本在这个地方有个 bug 老板说你不能修复这个 bug 吗 怎么要重做 还要那么多时间 你说我这次重构为了以后应对新需求上有更多的灵活性 老板可能心里还没有任何需求 ....... 总之 领导不挑头的事情 很难重做 团队团结还好 不团结 重构导致新问题了 锅就是你的了 大家都说：以前的又不是不能用 的时候就非常尴尬了 题主倒是可以提一提 看看老师的支持程度 学习实践的话 题主可以自己私下有时间自己重构 一部分模块 别动老版本就行 |
|
22
qinkangdeid 2019-03-31 14:34:07 +08:00
@blackmirror 太真实了 真实得不行 每个接手者刚看到代码都会指点一遍前任 这个地方怎么这么写？这里怎么不抽出来 这里多此一举啊 要是我我就不这么做 这代码不能看 垃圾 我来改 改完几个方法后 最后 大都默默的 Ctrl+z......
|
 |
23
xiaomingVTEX 2019-03-31 14:47:37 +08:00
@qinkangdeid 太真实了
|
 |
24
kindjeff 2019-03-31 14:56:29 +08:00 2
 |
 |
25
vcluopeng 2019-03-31 15:01:36 +08:00
如果你的代码可读性高,并能留下详细的开发注释和文档.那么就整个项目重构吧.
当然也要有信心在老师规定的时间内完成. 不然就继续舔砖加瓦,毕竟有句古话:没有金刚钻，不揽瓷器活. 还有记住 霍夫斯塔特定律 |
 |
26
cz5424 2019-03-31 15:25:20 +08:00 via iPhone
新开个项目写新功能，旧的他被慢慢的抛弃
|
 |
27
deepkolos 2019-03-31 15:34:37 +08:00
我比较幸运，上来就重写
|
 |
28
farseeraliens 2019-03-31 15:48:30 +08:00 via iPhone
@CEBBCAT 不明白重构跟 git 什么关系？
|
 |
29
yuhr123 2019-03-31 15:50:04 +08:00 via iPhone
把需求梳理一遍，用 python 重写。
|
 |
30
e2c 2019-03-31 16:03:49 +08:00
新三年，旧三年，缝缝补补又三年
看你的了啦 |
|
31
CEBBCAT 2019-03-31 16:25:11 +08:00 via Android
@farseeraliens 一来可以快速回到原来的状态，二来正式开发也需要 git 不是
|
 |
32
encro 2019-03-31 19:14:27 +08:00
最厉害的程序员就是能将一堆垃圾代码理出条例来。
我以十多年的经验告诉你。 |
|
33
encro 2019-03-31 19:20:46 +08:00 2
我的经验还告诉我，要舍得删除代码，看不懂的都删，直到程序跑不通了再去看（前提是有版本控制），可能你就能看明白以前的代码了。
明显感觉你是开发功力还不够，需求不了解，不敢去问 boss，不愿意深入思考最有效的解决问题的办法，潜意识找理由放弃。 如果你把这个任务当做是你毕业前最好的一个练手机会，那么相信你能成为导师最看重的学生，也能在毕业时超过大部分同学。 |
 |
34
x7395759 2019-03-31 19:29:37 +08:00
就一个建议，重写！！！
|
 |
35
Hyperion 2019-03-31 19:33:23 +08:00 1
当然是偷梁换柱釜底抽薪，写一套寄生代码附加在原来的代码上（在前端还是后端动手脚都可以），然后另外开一个目录跑新写的程序，来提供新的接口啦。
你说要在页面上新加两行分数？原始模板上直接对着结构写 Vue 的结构和标记，对着主体容器直接一个 new Vue，然后该怎么请求新结构就怎么请求。 重构这种事情，说说当然是容易的，但上学阶段根本不可能嘛，一没动力二没精力，最后还是吃力不讨好。这种东西刚开始很有热情，但你社团活动不搞了嘛？四六级单词不要背嘛？什么晨跑锻炼不要搞了嘛？ |
 |
36
Alpha486 2019-03-31 19:45:35 +08:00 via iPhone
这不接手则已，一接手就明白为啥前人连简单的功能都做不好！😂
|
 |
37
sazima 2019-03-31 20:39:42 +08:00
给工资吗
|
|
38
whoami9894 OP @scukmh 没酬劳，团队的任务哈哈
|
|
39
whoami9894 OP @notreami
按需求修修补补感觉头大，比重写还累 |
|
40
whoami9894 OP @reus
感谢您的建议，决定重写了 |
|
41
whoami9894 OP @qinkangdeid 重构就不了，本来就不太喜欢 PHP，打算直接重写了。明天先探探老师口风
|
|
42
whoami9894 OP @vcluopeng
能不能来得及我心里也没底，我自己不会前端，感觉工作量挺大的 |
|
43
whoami9894 OP |
 |
44
ifxo 2019-03-31 21:22:14 +08:00
不用管了，拉泡屎走人
|
|
45
whoami9894 OP @x7395759
重写重写走起 |
|
46
whoami9894 OP @Hyperion
让我改原来的可能更花时间，而且还痛苦，不如重写了 |
|
47
whoami9894 OP |
|
48
encro 2019-03-31 21:43:10 +08:00
@whoami9894
连开发都不会，还会安全了？ |
|
49
encro 2019-03-31 21:53:27 +08:00 2
整天说重构的，有几种人：
1，代码没看懂，反正要重构，潜意识已经做好走人准备； 2，代码看懂了，借重构争取更多时间； 3，代码看懂了，而且知道重构对自己来说是小 case，时间也能自己搞定和争取到，确信能写出以后更容易维护的代码来。 切莫忘记很多时候你是无法重构的，比如你面对整个淘宝的代码，你只能一点一点平滑升级，这时才是考验你的架构和能力。 就如人生不能重来，打好手上现有的牌才是最佳途径。 |
 |
50
ytmsdy 2019-03-31 22:33:01 +08:00
如果就维护几个月，那就忍忍吧。
如果要维护几年，那就长痛不如短痛直接重构吧 |
 |
52
xuanyuanaosheng 2019-03-31 22:43:05 +08:00 via Android
重写吧，正好练手
|
 |
53
siriussilen 2019-03-31 22:48:19 +08:00 via iPhone
如果是团委学生会之类的老师 我的教训就是千万别干. 他们拿你当苦力还不给钱 变了方法的给你提需求 一旦完不成 他们还会质疑你的能力……
|
|
54
notreami 2019-03-31 22:51:45 +08:00
@whoami9894 完全不需要考虑成本。。。年轻，真好～～～
|
 |
55
learningman 2019-03-31 22:55:32 +08:00
@Hyperion 记得原来我在一个 C 项目里边藏了一个 Go 的二进制文件，至今还能用 qwq
|
 |
56
metabot 2019-03-31 23:00:43 +08:00 via Android
易班。。这玩意儿。。令人怀念的大学生活
|
|
57
whoami9894 OP @encro
您这样的逻辑就有问题了，安全和开发确实联系紧密比如一些业务逻辑漏洞，但毕竟侧重点相差很远，并没有一个先行后继关系。开发要的工程能力，架构，优化等等安全是不需要涉及的，而安全我举个 bypass 的例子: `"".__class__.__mro__[-1].__subclasses__()[60].__init__.__globals__['__builtins__']['eval']('__import__("os").system("ls")')` 再有您这句"连...都不会，还会...了？"让人听起来很不是滋味 |
|
58
whoami9894 OP @ytmsdy
维护确实只有几个月，然后就丢给下一届做了，可是据去年的说这老师经常加需求 |
|
59
whoami9894 OP @siriussilen
您猜对了，党委宣传部老师，据说往届的说这老师"屁事"很多，三天两头加需求，确实像个苦力，不过也就当积累经验了 |
|
60
whoami9894 OP @learningman
popen 调用二进制吗，为何不编译成动态库呢 |
|
61
whoami9894 OP @metabot
和超星学习通一样，处处被诟病，几乎没人用，都是上级一厢情愿的推广 |
 |
62
0TSH60F7J2rVkg8t 2019-03-31 23:36:08 +08:00 via iPhone
如果这项目给钱，而且钱够多，值得你花时间做的话，那就重写一遍；反之随便改改就完了
|
 |
63
zjsxwc 2019-04-01 07:09:23 +08:00 via Android
我的建议是请把代码写的更烂一点，最好来个全局混淆，让后来者无法维护
|
 |
64
loading 2019-04-01 07:27:02 +08:00 via Android
在现有数据库结构魔改，用特殊技巧使用数据库字段，似的能支持新需求，我觉得这个比重写更有挑战性。
重写其实是另一种逃避方式。 |
|
65
ytmsdy 2019-04-01 08:57:08 +08:00 via iPhone
又不给钱！重写什么啊！！
|
 |
66
spotfg 2019-04-01 09:17:55 +08:00
我今年运气不错，换了家新公司，上来上班第一天上司就说老板要求重构代码，这个月搞定。。。现在 4 月了，搞得差不多了……所以就加班一个月
|
|
67
learningman 2019-04-01 09:19:01 +08:00
@whoami9894 在本地再开一个端口，用 socks 通信
是社团的项目，基本没有架构。我刚刚扫了一眼发现还有可疑的 pyc 文件 |
 |
68
annielong 2019-04-01 09:23:11 +08:00
有时候只能魔改，新建表，把特殊配置全写进去
|
 |
69
guoyuchuan 2019-04-01 09:28:21 +08:00
重构大法
|
 |
70
Seney 2019-04-01 09:39:14 +08:00
@whoami9894 那你就直接说开发 1.3/1.4/1.5/。。。 学校这种一般都是编程很粗浅的同学写的，当时可能不知道什么叫工程
|
|
71
encro 2019-04-01 09:39:27 +08:00
@whoami9894
这个算是浅层次安全吧，语法上的，未来很多都要交给机器， 未来这样的安全岗位应该有限（人数和发展前景）， 高级点的应该是架构级别的。 架构级别别说不懂开发。 解决安全问题更多是架构级别的解决。 比如架构级别防止注入，网络通信安全，防止爬虫。都需要开发基础。 |
 |
73
mogutouer 2019-04-01 09:59:52 +08:00
所以你以为 重构 这个词是怎么来的
|
 |
74
liuzuo 2019-04-01 10:00:05 +08:00
如果原代码能做到解藕，就按照要改的功能重写，微服务化。
如果做不到，就在原有基础上魔改吧。 |
 |
75
cwx391497 2019-04-01 10:02:14 +08:00
代码是屎山已经不恐怖了, 恐怖的是, 测试和文档也都是屎山
|
 |
76
gscoder 2019-04-01 10:05:35 +08:00
没必要重构，重构会发现有很多细小的功能点，到时候他们会说，之前可以这样这样现在怎么不行了，又不是要长期维护的项目，随便改改就行了
|
 |
77
alexmy 2019-04-01 10:35:24 +08:00
我这也是 5.6, yii1，性能部分重构下，就一个人，才没有精力去重写，现在毕竟稳定，老板其实只要能用就好，并不喜欢重头开发一遍，万一开发出来的又像之前 si 一样怎么办。
|
 |
78
no1xsyzy 2019-04-01 10:36:34 +08:00 1
@whoami9894 #5
简单地说：这不是 1.1，这是 1.1.0.0.α.zebra.Ⅳ.㎏.勹.г.\frac{ω5}{在吗？} 要改成 1.2 需要在不改动功能的情况下把那些东西都去掉，也就是需要做很多次换心手术。 ——每一次重构都是一个未知生物的换心手术：确定边界，取代边界（写测试），替换部件，确认新部件状态良好，还原边界。 |
 |
79
hellowmykami 2019-04-01 11:00:46 +08:00
魔改
|
 |
80
zarte 2019-04-01 11:30:35 +08:00
有追求就重构
求稳就在现有基础上打补丁留给下一任。 |
 |
81
jackmod 2019-04-01 11:36:01 +08:00
之前用 jekyll 重构一个大点的静态网站搞了好几天。这活不太好做。
|
 |
82
leekafai 2019-04-01 11:37:18 +08:00
每个人都抱着重写的愿望去维护，所以最后四不像，各有各的风格
|
 |
83
jimrok 2019-04-01 11:38:28 +08:00
注释里不要忘了加"// fuck xxx, FU"
|
 |
84
JimiJimi 2019-04-01 16:56:41 +08:00 1
最讨厌学校分配不懂技术还喜欢瞎命令的老师来管理技术部
所以当年我解散了技术部 是真的闹心 |
|
85
whoami9894 OP 1
@loading 嗯确实我有点逃避的意味在里面，还是经验能力不足吧....魔改的话对我来说感觉要花更多的精力
|
|
86
whoami9894 OP @zjsxwc 所以干脆在我手上重写一遍，但随着未来加新的需求修修补补，肯定又会变成现在的屎山
|
|
87
whoami9894 OP @Seney 我看他的代码风格架构等等可能算是学生中不错的了，但跟工作多年的前辈们比肯定有很大差距。变成现在这样还是因为每年修修补补加上维护者水平不一
|
|
88
whoami9894 OP @encro 您可能不太了解，说的只是安全的一个片面，pentesting 工程师、reverse 工程师等等，很多安全问题没有办法从大的架构层面解决。而且您关注一下 defcon 等等安全会议，安全研究员们研究的大多就是我说的语言框架层面的新式漏洞，比如 PHP 的 phar 协议
我们做过 ML 的 WAF 的玩具项目，也了解过市面上想关产品，被 AI 取代完全取代还是很遥远的（真有这一天那么应该 90%职业已经被取代了） > 架构级别防止注入，网络通信安全，防止爬虫 架构级别怎么防止注入我还真不知道，愿闻其详。而举例 SQLi，即使有参数化查询能完全防止，但依然能在各大 SRC 平台看见 SQLi 漏洞，所以基础安全人员不会衰减的。后两样是基础，不太明白您说的不会是什么意思 而且您说了这么半天依然没论证您“开发和安全的先行关系”的逻辑 |
|
89
whoami9894 OP @liuzuo 代码整体感觉确实是分成不同组件层面，但是某些逻辑分散纠结在一起，感觉很难改单个功能
|
|
90
whoami9894 OP @alexmy 唉我对 PHP 仅仅停留在语法层面，日常也没有拿它开发过项目，所以看着那套范式还是很头疼的
|
|
91
whoami9894 OP @no1xsyzy 不重构了，感觉我也没能力去重构的比现在好，还是换个熟悉的语言重写吧
|
|
92
whoami9894 OP @jimrok 哈哈哈不太好不太好
|
 |
93
sannyzeng 2019-04-01 19:37:18 +08:00
写代码最重要的还是第一代，不然后面都会给带偏。。。
|
|
94
encro 2019-04-02 09:14:45 +08:00
@whoami9894 “代码都看不懂能做安全？”
|
|
95
alexmy 2019-04-02 10:10:26 +08:00
@whoami9894 我接手的时候，还看了半天语法，我本身是做游戏开发的。
|
|
96
whoami9894 OP @encro
你觉得我看不懂就看不懂吧，你这逻辑能力我也是服了，回复全程不讲逻辑无脑杠，block |
|
97
whoami9894 OP @alexmy 老师催着这周四前要上线，再慢慢踩坑怕是肯定来不及了
|
Select Language