回看板
Disp BBS
作者 kingofsdtw (塔綠班)
標題 [心得] 花了很多時間重構卻被打槍用舊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 (臺灣)
※ 作者: kingofsdtw 2025-09-13 19:39:23
※ 文章代碼(AID): #1enLVqQp (Soft_Job)
※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1757763572.A.6B3.html
推 cancelpc: 維護的是你，不是他們。所以他們只想安全牌。不會管技術債換人厚，會多難接。一堆不知所以然的code。1F 09/13 19:41
所以心中五味雜陳
※ 編輯: kingofsdtw (123.241.68.207 臺灣), 09/13/2025 19:43:17
推 SHANGOYANYI: code直接丟github開源全世界共享 然後特休全壓放老人自己去解root cause啊 這還要教？3F 09/13 19:49
推 johnbill: 錢給到位嗎 有成長空間嗎 都沒有就走人XD5F 09/13 19:51
→ dildoe: 公司:能動最重要，你有看過醫院那些名醫看小病就要開刀的嗎??XD
特別是老人家，沒事就別亂開刀了萬一有糾紛
理由一大堆不用當真6F 09/13 20:03
→ kissmickey: 一堆公司都這樣 能動就好改這麼大做什麼 出問題你扛得住嗎10F 09/13 20:19
推 yamakazi: 可以重構啊，你不會等案子結束再重構？12F 09/13 20:24
→ kissmickey: 問題是你重構完 上頭買單?其他人接手會用會改？要多少時間熟悉你的code13F 09/13 20:29
就是新人同事(3-5y)解不掉才掉到我頭上的QwQ
※ 編輯: kingofsdtw (123.241.68.207 臺灣), 09/13/2025 20:33:47
→ kissmickey: 以上這只針對公司老人15F 09/13 20:34
推 hooll111: 可能只是不想要欠這種人情 也不想花錢請你重構 所以才這樣回16F 09/13 20:35
→ kissmickey: 很久以前我也跟你一樣 後來看開了 拿多少錢做多少事 除非上頭有交代
不然這些重構還是新東西自己改改玩玩 不會放檯面上，頂多找面試拿來講講18F 09/13 20:36
→ MoonCode: 那你就新舊都兼容啊 你的 pr 應該只有增加的行數沒有砍舊邏輯22F 09/13 20:44
→ NDark: 專案要先把責任切開 大雜燴下 對專案的風險感就會混雜24F 09/13 21:08
→ kingofsdtw: 程式已經亂到flag亂跳...
可讀性0..25F 09/13 21:08
推 abccbaandy: 老人還被質疑喔...27F 09/13 21:29
→ ericthree: 是說派你去救火的人 又不滿意你的方案嗎==28F 09/13 21:31
推 Bencrie: 他錢有給到你捨不得離職嗎 XD29F 09/13 21:56
→ kingofsdtw: 上面還有更老的的人啊...30F 09/13 22:02
→ keke0421: M有要你重構？如果沒有，你要重構，不應該先跟他討論?y說到底 IC 也只是 M 的資源，資源怎麼用是M的職責和權力事情發生後，建議可以去找M聊，解決問題 而不是想著離職31F 09/13 22:08
推 arhtur945: coding master是什麼鬼34F 09/13 22:31
→ GoalBased: 如果你不確定這個決定會不會被靠腰，你可以找比你懂公司狀況的人或者主管討論，而不是自己做決定35F 09/13 22:40
→ MoonCode: 那你就真擴充而不是順手重構 看行數最快37F 09/13 22:42
→ umum29: 除非你是決策者否則要重構要看大家意見 這不是技術好不好38F 09/13 22:45
推 accessdenied: 我有遇到遇到跟你一樣的狀況。明明團隊 wiki 有前人留下 guide line，寫童子軍原則：順手改掉周圍的爛 code。結果 review 後被要求全部 revert 回去，因為 reviewer 覺得跟需求無關的變動太多，造成他的負擔。39F 09/13 22:55
→ MoonCode: 順手要能改前提是有測試吧 不然應該是先補測試44F 09/13 23:21
推 CRPKT: 這是重寫，不是重構45F 09/14 00:24
推 viper9709: 推一樓~都到master了,講的話還沒人信喔@@...46F 09/14 00:27
→ bear1414: 原始任務是解bug。要開新任務（重構），請先和派任務的人溝通。47F 09/14 01:44
→ fanatics5566: 上線前：亂一點沒關係先把東西趕出來我們再回頭重構/ 上線後：好好的你改它幹嘛？49F 09/14 01:56
推 guanting886: 你的好意可能是他人的災難 有些東西還是要討論一下再決定 不要做無效工作 忙的要死得了一個非預期的結果 自己很挫折無意義51F 09/14 02:21
推 WaterLengend: 讓你認清現實這間公司是來甩鍋的不是來貢獻的54F 09/14 02:57
→ indexcome: 我只覺得coding是你 testing也是你 是一件很奇怪的事情55F 09/14 05:55

--
作者 kingofsdtw 的最新發文:
點此顯示更多發文記錄