作者kingofsdtw (塔绿班)
看板Soft_Job
标题[心得] 花了很多时间重构却被打枪用旧code
时间Sat Sep 13 19:39:23 2025
最近案子快收尾在收敛bug
身为救援大队长的老人我被指派到维护一个很老的API
老API的设计已经无法满足扩充需求
新的扩充功能造成BUG
於是我花了大量时间甚至debug到天亮甚至请无薪假
新的API经过我反覆测试各种case都完美无缺
但是code review却被质疑:
1. 是不是没找到root cause
2. 干嘛改动如此大? 只不过新加一点点功能干嘛改架构?
心中五味杂陈...
好歹我也是coding master，我说该重构了就是该开始还技术债了
更上头还是希望用最鸵鸟的方法继续用旧架构一堆workaound当作root cause
是该离职了吗? QwQ
--
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 123.241.68.207 (台湾)
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1757763572.A.6B3.html
1F：推 cancelpc: 维护的是你，不是他们。所以他们只想安全牌。不会管技术 09/13 19:41
2F：→ cancelpc: 债换人厚，会多难接。一堆不知所以然的code。 09/13 19:41
所以心中五味杂陈
※ 编辑: kingofsdtw (123.241.68.207 台湾), 09/13/2025 19:43:17
3F：推 SHANGOYANYI: code直接丢github开源全世界共享 然後特休全压放老 09/13 19:49
4F：→ SHANGOYANYI: 人自己去解root cause啊 这还要教？ 09/13 19:49
5F：推 johnbill: 钱给到位吗 有成长空间吗 都没有就走人XD 09/13 19:51
6F：→ dildoe: 公司:能动最重要，你有看过医院那些名医看小病就要开刀的 09/13 20:03
7F：→ dildoe: 吗??XD 09/13 20:03
8F：→ dildoe: 特别是老人家，没事就别乱开刀了万一有纠纷 09/13 20:05
9F：→ dildoe: 理由一大堆不用当真 09/13 20:09
10F：→ kissmickey: 一堆公司都这样 能动就好改这麽大做什麽 出问题你扛得 09/13 20:19
11F：→ kissmickey: 住吗 09/13 20:19
12F：推 yamakazi: 可以重构啊，你不会等案子结束再重构？ 09/13 20:24
13F：→ kissmickey: 问题是你重构完 上头买单?其他人接手会用会改？要多 09/13 20:29
14F：→ kissmickey: 少时间熟悉你的code 09/13 20:29
就是新人同事(3-5y)解不掉才掉到我头上的QwQ
※ 编辑: kingofsdtw (123.241.68.207 台湾), 09/13/2025 20:33:47
15F：→ kissmickey: 以上这只针对公司老人 09/13 20:34
16F：推 hooll111: 可能只是不想要欠这种人情 也不想花钱请你重构 所以才这 09/13 20:35
17F：→ hooll111: 样回 09/13 20:35
18F：→ kissmickey: 很久以前我也跟你一样 後来看开了 拿多少钱做多少事 09/13 20:36
19F：→ kissmickey: 除非上头有交代 09/13 20:36
20F：→ kissmickey: 不然这些重构还是新东西自己改改玩玩 不会放台面上， 09/13 20:36
21F：→ kissmickey: 顶多找面试拿来讲讲 09/13 20:36
22F：→ MoonCode: 那你就新旧都兼容啊 你的 pr 应该只有增加的行数没有砍 09/13 20:44
23F：→ MoonCode: 旧逻辑 09/13 20:44
24F：→ NDark: 专案要先把责任切开 大杂烩下 对专案的风险感就会混杂 09/13 21:08
25F：→ kingofsdtw: 程式已经乱到flag乱跳... 09/13 21:08
26F：→ kingofsdtw: 可读性0.. 09/13 21:08
27F：推 abccbaandy: 老人还被质疑喔... 09/13 21:29
28F：→ ericthree: 是说派你去救火的人 又不满意你的方案吗== 09/13 21:31
29F：推 Bencrie: 他钱有给到你舍不得离职吗 XD 09/13 21:56
30F：→ kingofsdtw: 上面还有更老的的人啊... 09/13 22:02
31F：→ keke0421: M有要你重构？如果没有，你要重构，不应该先跟他讨论?y 09/13 22:08
32F：→ keke0421: 说到底 IC 也只是 M 的资源，资源怎麽用是M的职责和权力 09/13 22:08
33F：→ keke0421: 事情发生後，建议可以去找M聊，解决问题 而不是想着离职 09/13 22:09
34F：推 arhtur945: coding master是什麽鬼 09/13 22:31
35F：→ GoalBased: 如果你不确定这个决定会不会被靠腰，你可以找比你懂公 09/13 22:40
36F：→ GoalBased: 司状况的人或者主管讨论，而不是自己做决定 09/13 22:40
37F：→ MoonCode: 那你就真扩充而不是顺手重构 看行数最快 09/13 22:42
38F：→ umum29: 除非你是决策者否则要重构要看大家意见 这不是技术好不好 09/13 22:45
39F：推 accessdenied: 我有遇到遇到跟你一样的状况。明明团队 wiki 有前人 09/13 22:55
40F：→ accessdenied: 留下 guide line，写童子军原则：顺手改掉周围的烂 09/13 22:55
41F：→ accessdenied: code。结果 review 後被要求全部 revert 回去，因 09/13 22:55
42F：→ accessdenied: 为 reviewer 觉得跟需求无关的变动太多，造成他的负 09/13 22:55
43F：→ accessdenied: 担。 09/13 22:55
44F：→ MoonCode: 顺手要能改前提是有测试吧 不然应该是先补测试 09/13 23:21
45F：推 CRPKT: 这是重写，不是重构 09/14 00:24
46F：推 viper9709: 推一楼~都到master了,讲的话还没人信喔@@... 09/14 00:27
47F：→ bear1414: 原始任务是解bug。要开新任务（重构），请先和派任务的 09/14 01:44
48F：→ bear1414: 人沟通。 09/14 01:44
49F：→ fanatics5566: 上线前：乱一点没关系先把东西赶出来我们再回头重 09/14 01:56
50F：→ fanatics5566: 构/ 上线後：好好的你改它干嘛？ 09/14 01:56
51F：推 guanting886: 你的好意可能是他人的灾难 有些东西还是要讨论一下 09/14 02:21
52F：→ guanting886: 再决定 不要做无效工作 忙的要死得了一个非预期的结 09/14 02:21
53F：→ guanting886: 果 自己很挫折无意义 09/14 02:21
54F：推 WaterLengend: 让你认清现实这间公司是来甩锅的不是来贡献的 09/14 02:57
55F：→ indexcome: 我只觉得coding是你 testing也是你 是一件很奇怪的事 09/14 05:55
56F：→ indexcome: 情 09/14 05:55
57F：推 ku399999: 这件事的问题是 事前沟通。没说服就做 浪费彼此时间 09/14 08:52
58F：推 hduek153: 你这也不是一天的工作吧 中间没人反应？？ 09/14 09:26
59F：→ VScode: 该离职了 09/14 09:36
60F：→ pot1234: 重构前稍微跟别人提一下吧… 09/14 09:41
61F：推 OriginStar: 看过一些沟通方面的书籍，原PO上层还有决策者的话，要 09/14 09:44
62F：→ OriginStar: 先说服或告知决策者，让他们心里有预期，看起来你做的 09/14 09:45
63F：→ OriginStar: 和决策者的预期有所出入，才会被打枪 09/14 09:46
64F：推 longlyeagle: 没先沟通好的问题 09/14 09:54
65F：推 dream1124: 原始任务是除错对吗？ 这样的话，设计烂做不了就回报吧 09/14 10:27
66F：→ dream1124: 不然把除错做成更花资源且异动更大的重构人家也不领情 09/14 10:28
67F：→ dream1124: 说重构可能还客气了。如果直接变成新API，那算是改写或 09/14 10:29
68F：→ dream1124: 重新设计……这样如果人家不收其实也不令人非常意外 09/14 10:30
69F：→ DrTech: 原本维护API可以很多人维护，你这一改，只剩你知道了。这 09/14 11:23
70F：→ DrTech: 样真的是只有自己对吗？ 不一定喔。 09/14 11:23
71F：→ DrTech: 你有权限负责整个专案，或整个部门的考绩吗？ 没权限的话 09/14 11:26
72F：→ DrTech: ，这样改。即即使技术没问题，千万不要认为是对的。到任何 09/14 11:26
73F：→ DrTech: 公司都可能得罪人。 09/14 11:26
74F：推 ILoveAMD: 取暖喔 自以为是的重构 09/14 11:29
75F：推 tsaigi: 傻子才自己在那边重构 09/14 11:33
76F：嘘 pig2014: 通常这种情况代表没实力的怕事装逼仔在上位，会让这种逼 09/14 11:44
77F：→ pig2014: 洨上去的部门主管方向感也不是很好，如果薪资不是特别好 09/14 11:44
78F：→ pig2014: 应该可以闪人了 09/14 11:44
79F：→ pig2014: 不过还是要看规模，如果是一千行以内我都觉得还好。超过 09/14 11:45
80F：→ pig2014: 一千行就真的要思考了 09/14 11:45
81F：嘘 pig2014: 不用屌楼上一堆嘴重构的嫩逼，techjob都是搞硬体的废材 09/14 11:47
82F：→ pig2014: ，而且科技业95%都是冗员废材，所以留言有95%怕事废材也 09/14 11:47
83F：→ pig2014: 合理 09/14 11:47
84F：→ pig2014: 干原来是softjob，那更惨了，大部分都是台湾系新创小规 09/14 11:49
85F：→ pig2014: 模公司废材，薪水大概半导体业1/2，更不能参考 09/14 11:49
86F：→ DrTech: 没人嘴重构好吗。我们嘴的是：重构前，不先沟通。 09/14 11:58
87F：推 alihue: 有共识、排进去时程的重构才比较不会出现这个问题。自己 09/14 12:12
88F：→ alihue: 重构通常都是小规模、PR review 容易看懂的规模 09/14 12:12
89F：推 tsaigi: 怎麽有人留言看一看自己破防XDD 09/14 12:24
90F：推 lturtsamuel: 你如果这麽资深了 东西又有做出来 怎麽还会有人在程 09/14 12:49
91F：→ lturtsamuel: 式码层面质疑你？感觉很怪 是不是有牙膏没挤 09/14 12:49
92F：→ lturtsamuel: 回到这两个问题都很合理 而且都不难回答吧 你有没有 09/14 12:51
93F：→ lturtsamuel: 找到根因？修正那个根因需不需要这麽大的改动？一百 09/14 12:51
94F：→ lturtsamuel: 字以内就应该回答清楚的问题 答不出来先去训练表达能 09/14 12:51
95F：→ lturtsamuel: 力== 09/14 12:51
96F：推 jimmytzeng: 你上一位接手可能也是这样想，然後每新来一位每一位 09/14 13:06
97F：→ jimmytzeng: 都在重构，每次专案的程式码都不一样 09/14 13:06
98F：嘘 watashino: 问题是review前为啥不暴露一下你要做这件事 09/14 13:08
99F：→ watashino: 大家讨论一下有没有价值 09/14 13:08
100F：→ watashino: 你就自己单干但是上面觉得没用那就是没用啊 09/14 13:08
101F：推 crazwade: 老问题了 你想扛上面不想扛 09/14 16:32
102F：→ crazwade: 我也遇过就是做ppt跟上面报告一轮 09/14 16:33
103F：嘘 Satomisan: 看到debug到天亮就想笑 09/14 19:32
104F：→ Satomisan: 鬼岛惯老板这麽多就是你这种人养的 09/14 19:32
105F：推 Suleika: 重构要有计画跟目标，而且定期，不是遇到问题重构 09/14 20:51
106F：→ Suleika: 你这样搞下去有问题怎麽知道是新问题还是原问题 09/14 20:52
107F：→ Suleika: 跟质疑其实也没啥关系，就是其他人听了会觉得很危险 09/14 20:53
108F：→ lucky4283: 没上头指示干嘛重构，不够忙吧 09/14 21:02
109F：→ cdy815: 如果是我，就先做ppt、拉会议安排code review，最终更上 09/14 21:37
110F：→ cdy815: 面说要怎麽做就怎麽做，反正我把决策责任丢出去了，不重构 09/14 21:37
111F：→ cdy815: 我也乐得轻松 09/14 21:37
112F：→ stepnight: 资历是老人，思维跟做事方式像社会新鲜人 09/14 21:59
113F：→ stepnight: 所以同一间待太久也不好 09/14 21:59
114F：推 justaID: 理解原po的无奈，code落到自己头上，为了改得动和长久维 09/14 22:01
115F：→ justaID: 护的动，愿意吃亏花时间去重构，但反而被不是在第一线负 09/14 22:01
116F：→ justaID: 责维护的reviewer质疑而觉得沮丧。只能说这种情况是政治 09/14 22:01
117F：→ justaID: 和文化问题，开发文化是由有话语权和决策权的人说了算， 09/14 22:01
118F：→ justaID: 如果沟通无果，要就加入这种文化，要不心里的坎过不去的 09/14 22:01
119F：→ justaID: 话，那就好好打算吧 09/14 22:01
120F：推 stupid0319: 你的实力压不过别戈才会这样 09/14 22:27
121F：→ shooter555: 提到老api xxx 看来你这不是重构唷 改api被打枪不是 09/14 23:59
122F：→ shooter555: 很合理吗 09/14 23:59
123F：推 viper9709: 推justaID 09/15 01:25
124F：推 Aidan79225: 开新api就好 旧的标deprecated 09/15 01:46
125F：推 Csongs: 这行多的是文人相轻 09/15 02:38
126F：→ Csongs: api 改spec 出事一定扛不住 09/15 02:39
127F：推 jamesho8743: 用你的新架构有风险 你要从头维护到底吗 再来你明显 09/15 05:48
128F：→ jamesho8743: 不够厉害 找不到root cause以及用最小的改动解决问 09/15 05:48
129F：→ jamesho8743: 题 09/15 05:48
130F：→ love99067333: 你以为〈只要能动就不要改〉是开玩笑的时候： 09/15 06:16
132F：推 matrixki: 做改动前有先向上沟通吗？或是跨组沟通？ 09/15 09:58
133F：→ matrixki: 获得同意才做的还是你就直接做下去了? 09/15 09:58
134F：推 newkkloo2: 这代表你在公司credit还不够吧..够力的话谁会挡... 09/15 12:17
135F：推 GooglePixel: 沟通能力有待加强 美其名想解决问题 其实只是底层码 09/15 13:38
136F：→ GooglePixel: 农的美好幻想 在产品先行/功能先行的团队就是这样 09/15 13:38
137F：→ GooglePixel: 也不见得要离职啦 可以找其他方式实现自我 参加程式 09/15 13:41
138F：→ GooglePixel: 小作坊之类的 不要用工作来实现理想 那是赚钱的地方 09/15 13:41
139F：→ ma721: 完美无缺是你自认为的，隐藏没爆的可能比你想像的多 09/15 15:34
140F：→ windlll: 以前的公司发生过，要求重构→开需求→写完测完→「还能 09/15 17:18
141F：→ windlll: 跑就不用换了」，浪费我时间 09/15 17:18
142F：→ ssccg: 有给你薪水就没浪费你时间啦，别学原PO没事自干就好 09/15 18:04
143F：推 RINPE: 老气 没先确认过就自己改了吗？ 09/16 06:35
144F：推 skizard: 如果已经在公司扛这麽久，说明清楚後还不被上头信任 我 09/16 23:27
145F：→ skizard: 是会直接走人 09/16 23:27
146F：→ kiwijang: 这种上头的 code review 当耳边风就好，看有没有机会加 09/17 10:16
147F：→ kiwijang: 薪继续忍上头，或升迁为上头，没机会就换了吧 09/17 10:16
148F：→ popcool: 除非你改的地方code base只有你一个人在写跟维护，多人都 09/17 15:40
149F：→ popcool: 会动到的地方你翻掉以前都没先design review过？改完了才 09/17 15:40
150F：→ popcool: 说就是想硬上，本来大家熟悉的东西你确定你改了还有人看 09/17 15:40
151F：→ popcool: 的懂？上面要你修bug结果你整个翻掉，时程上都没人发现？ 09/17 15:40
152F：→ popcool: 整件事都很怪啊 master大大 09/17 15:40
153F：推 Nitricacid: 拿多少钱做多少事 没要你做的就别做了 09/18 23:30
154F：→ knives: 首先 有人叫你重构吗?没有干嘛做 09/19 16:42
155F：推 prag222: 重构你负责的部份就好，重构主要是为了自己後续方便，自 09/21 01:34
156F：→ prag222: 己舒服而已，搞太多别人酸被人嫌，正常的刚好 09/21 01:34
157F：推 prag222: 说真的不懂设计模式跟oop实际经验，别跟我提重构两字，一 09/21 01:39
158F：→ prag222: 律当笑话 09/21 01:39
159F：→ MoonCode: 我最害怕懂设计模式的人了 09/21 01:43
160F：推 okgogogo: 假如原本市场测了10年，你改完经得起考验吗？ 10/04 19:55
161F：推 new122851: 耍废摆烂即可，反正公司不能随便开除 10/06 14:33