[心得] 花了很多時間重構卻被打槍用舊code

cancelpc09/13 19:41維護的是你，不是他們。所以他們只想安全牌。不會管技術

cancelpc09/13 19:41債換人厚，會多難接。一堆不知所以然的code。

SHANGOYANYI09/13 19:49code直接丟github開源全世界共享 然後特休全壓放老

SHANGOYANYI09/13 19:49人自己去解root cause啊 這還要教？

johnbill09/13 19:51錢給到位嗎 有成長空間嗎 都沒有就走人XD

dildoe09/13 20:03公司:能動最重要，你有看過醫院那些名醫看小病就要開刀的

dildoe09/13 20:03嗎??XD

dildoe09/13 20:05特別是老人家，沒事就別亂開刀了萬一有糾紛

dildoe09/13 20:09理由一大堆不用當真

kissmickey09/13 20:19一堆公司都這樣 能動就好改這麼大做什麼 出問題你扛得

kissmickey09/13 20:19住嗎

yamakazi09/13 20:24可以重構啊，你不會等案子結束再重構？

kissmickey09/13 20:29問題是你重構完 上頭買單?其他人接手會用會改？要多

kissmickey09/13 20:29少時間熟悉你的code

kissmickey09/13 20:34以上這只針對公司老人

hooll11109/13 20:35可能只是不想要欠這種人情 也不想花錢請你重構 所以才這

hooll11109/13 20:35樣回

kissmickey09/13 20:36很久以前我也跟你一樣 後來看開了 拿多少錢做多少事

kissmickey09/13 20:36 除非上頭有交代

kissmickey09/13 20:36不然這些重構還是新東西自己改改玩玩 不會放檯面上，

kissmickey09/13 20:36頂多找面試拿來講講

MoonCode09/13 20:44那你就新舊都兼容啊 你的 pr 應該只有增加的行數沒有砍

MoonCode09/13 20:44舊邏輯

NDark09/13 21:08專案要先把責任切開 大雜燴下 對專案的風險感就會混雜

kingofsdtw09/13 21:08程式已經亂到flag亂跳...

kingofsdtw09/13 21:08可讀性0..

abccbaandy09/13 21:29老人還被質疑喔...

ericthree09/13 21:31是說派你去救火的人 又不滿意你的方案嗎==

Bencrie09/13 21:56他錢有給到你捨不得離職嗎 XD

kingofsdtw09/13 22:02上面還有更老的的人啊...

keke042109/13 22:08M有要你重構？如果沒有，你要重構，不應該先跟他討論?y

keke042109/13 22:08說到底 IC 也只是 M 的資源，資源怎麼用是M的職責和權力

keke042109/13 22:09事情發生後，建議可以去找M聊，解決問題 而不是想著離職

arhtur94509/13 22:31coding master是什麼鬼

GoalBased09/13 22:40如果你不確定這個決定會不會被靠腰，你可以找比你懂公

GoalBased09/13 22:40司狀況的人或者主管討論，而不是自己做決定

MoonCode09/13 22:42那你就真擴充而不是順手重構 看行數最快

umum2909/13 22:45除非你是決策者否則要重構要看大家意見 這不是技術好不好

accessdenied09/13 22:55我有遇到遇到跟你一樣的狀況。明明團隊 wiki 有前人

accessdenied09/13 22:55留下 guide line，寫童子軍原則：順手改掉周圍的爛

accessdenied09/13 22:55 code。結果 review 後被要求全部 revert 回去，因

accessdenied09/13 22:55為 reviewer 覺得跟需求無關的變動太多，造成他的負

accessdenied09/13 22:55擔。

MoonCode09/13 23:21順手要能改前提是有測試吧 不然應該是先補測試

CRPKT09/14 00:24這是重寫，不是重構

viper970909/14 00:27推一樓~都到master了,講的話還沒人信喔@@...

bear141409/14 01:44原始任務是解bug。要開新任務（重構），請先和派任務的

bear141409/14 01:44人溝通。

fanatics556609/14 01:56上線前：亂一點沒關係先把東西趕出來我們再回頭重

fanatics556609/14 01:56構/ 上線後：好好的你改它幹嘛？

guanting88609/14 02:21你的好意可能是他人的災難 有些東西還是要討論一下

guanting88609/14 02:21再決定 不要做無效工作 忙的要死得了一個非預期的結

guanting88609/14 02:21果 自己很挫折無意義

WaterLengend09/14 02:57讓你認清現實這間公司是來甩鍋的不是來貢獻的

indexcome09/14 05:55我只覺得coding是你 testing也是你 是一件很奇怪的事

indexcome09/14 05:55情

ku39999909/14 08:52這件事的問題是 事前溝通。沒說服就做 浪費彼此時間

hduek15309/14 09:26你這也不是一天的工作吧 中間沒人反應？？

VScode09/14 09:36該離職了

pot123409/14 09:41重構前稍微跟別人提一下吧…

OriginStar09/14 09:44看過一些溝通方面的書籍，原PO上層還有決策者的話，要

OriginStar09/14 09:45先說服或告知決策者，讓他們心裡有預期，看起來你做的

OriginStar09/14 09:46和決策者的預期有所出入，才會被打槍

longlyeagle09/14 09:54沒先溝通好的問題

dream112409/14 10:27原始任務是除錯對嗎？ 這樣的話，設計爛做不了就回報吧

dream112409/14 10:28不然把除錯做成更花資源且異動更大的重構人家也不領情

dream112409/14 10:29說重構可能還客氣了。如果直接變成新API，那算是改寫或

dream112409/14 10:30重新設計……這樣如果人家不收其實也不令人非常意外

DrTech09/14 11:23原本維護API可以很多人維護，你這一改，只剩你知道了。這

DrTech09/14 11:23樣真的是只有自己對嗎？ 不一定喔。

DrTech09/14 11:26你有權限負責整個專案，或整個部門的考績嗎？ 沒權限的話

DrTech09/14 11:26，這樣改。即即使技術沒問題，千萬不要認為是對的。到任何

DrTech09/14 11:26公司都可能得罪人。

ILoveAMD09/14 11:29取暖喔 自以為是的重構

tsaigi09/14 11:33傻子才自己在那邊重構

pig201409/14 11:44通常這種情況代表沒實力的怕事裝逼仔在上位，會讓這種逼

pig201409/14 11:44洨上去的部門主管方向感也不是很好，如果薪資不是特別好

pig201409/14 11:44應該可以閃人了

pig201409/14 11:45不過還是要看規模，如果是一千行以內我都覺得還好。超過

pig201409/14 11:45一千行就真的要思考了

pig201409/14 11:47不用屌樓上一堆嘴重構的嫩逼，techjob都是搞硬體的廢材

pig201409/14 11:47，而且科技業95%都是冗員廢材，所以留言有95%怕事廢材也

pig201409/14 11:47合理

pig201409/14 11:49幹原來是softjob，那更慘了，大部分都是台灣系新創小規

pig201409/14 11:49模公司廢材，薪水大概半導體業1/2，更不能參考

DrTech09/14 11:58沒人嘴重構好嗎。我們嘴的是：重構前，不先溝通。

alihue09/14 12:12有共識、排進去時程的重構才比較不會出現這個問題。自己

alihue09/14 12:12重構通常都是小規模、PR review 容易看懂的規模

tsaigi09/14 12:24怎麼有人留言看一看自己破防XDD

lturtsamuel09/14 12:49你如果這麼資深了 東西又有做出來 怎麼還會有人在程

lturtsamuel09/14 12:49式碼層面質疑你？感覺很怪 是不是有牙膏沒擠

lturtsamuel09/14 12:51回到這兩個問題都很合理 而且都不難回答吧 你有沒有

lturtsamuel09/14 12:51找到根因？修正那個根因需不需要這麼大的改動？一百

lturtsamuel09/14 12:51字以內就應該回答清楚的問題 答不出來先去訓練表達能

lturtsamuel09/14 12:51力==

jimmytzeng09/14 13:06你上一位接手可能也是這樣想，然後每新來一位每一位

jimmytzeng09/14 13:06都在重構，每次專案的程式碼都不一樣

watashino09/14 13:08問題是review前為啥不暴露一下你要做這件事

watashino09/14 13:08大家討論一下有沒有價值

watashino09/14 13:08你就自己單幹但是上面覺得沒用那就是沒用啊

crazwade09/14 16:32老問題了 你想扛上面不想扛

crazwade09/14 16:33我也遇過就是做ppt跟上面報告一輪

Satomisan09/14 19:32看到debug到天亮就想笑

Satomisan09/14 19:32鬼島慣老闆這麼多就是你這種人養的

Suleika09/14 20:51重構要有計畫跟目標，而且定期，不是遇到問題重構

Suleika09/14 20:52你這樣搞下去有問題怎麼知道是新問題還是原問題

Suleika09/14 20:53跟質疑其實也沒啥關係，就是其他人聽了會覺得很危險

lucky428309/14 21:02沒上頭指示幹嘛重構，不夠忙吧

cdy81509/14 21:37如果是我，就先做ppt、拉會議安排code review，最終更上

cdy81509/14 21:37面說要怎麼做就怎麼做，反正我把決策責任丟出去了，不重構

cdy81509/14 21:37我也樂得輕鬆

stepnight09/14 21:59資歷是老人，思維跟做事方式像社會新鮮人

stepnight09/14 21:59所以同一間待太久也不好

justaID09/14 22:01理解原po的無奈，code落到自己頭上，為了改得動和長久維

justaID09/14 22:01護的動，願意吃虧花時間去重構，但反而被不是在第一線負

justaID09/14 22:01責維護的reviewer質疑而覺得沮喪。只能說這種情況是政治

justaID09/14 22:01和文化問題，開發文化是由有話語權和決策權的人說了算，

justaID09/14 22:01如果溝通無果，要就加入這種文化，要不心裡的坎過不去的

justaID09/14 22:01話，那就好好打算吧

stupid031909/14 22:27你的實力壓不過別戈才會這樣

shooter55509/14 23:59提到老api xxx 看來你這不是重構唷 改api被打槍不是

shooter55509/14 23:59很合理嗎

viper970909/15 01:25推justaID

Aidan7922509/15 01:46開新api就好 舊的標deprecated

Csongs09/15 02:38這行多的是文人相輕

Csongs09/15 02:39api 改spec 出事一定扛不住

jamesho874309/15 05:48用你的新架構有風險 你要從頭維護到底嗎 再來你明顯

jamesho874309/15 05:48不夠厲害 找不到root cause以及用最小的改動解決問

jamesho874309/15 05:48題

love9906733309/15 06:16你以為〈只要能動就不要改〉是開玩笑的時候：

s3136466309/15 09:43https://www.youtube.com/watch?v=HvLXaAle5jw

matrixki09/15 09:58做改動前有先向上溝通嗎？或是跨組溝通？

matrixki09/15 09:58獲得同意才做的還是你就直接做下去了?

newkkloo209/15 12:17這代表你在公司credit還不夠吧..夠力的話誰會擋...

GooglePixel09/15 13:38溝通能力有待加強 美其名想解決問題 其實只是底層碼

GooglePixel09/15 13:38農的美好幻想 在產品先行/功能先行的團隊就是這樣

GooglePixel09/15 13:41也不見得要離職啦 可以找其他方式實現自我 參加程式

GooglePixel09/15 13:41小作坊之類的 不要用工作來實現理想 那是賺錢的地方

ma72109/15 15:34完美無缺是你自認為的，隱藏沒爆的可能比你想像的多

windlll09/15 17:18以前的公司發生過，要求重構→開需求→寫完測完→「還能

windlll09/15 17:18跑就不用換了」，浪費我時間

ssccg09/15 18:04有給你薪水就沒浪費你時間啦，別學原PO沒事自幹就好

RINPE09/16 06:35老氣 沒先確認過就自己改了嗎？

skizard09/16 23:27如果已經在公司扛這麼久，說明清楚後還不被上頭信任 我

skizard09/16 23:27是會直接走人

kiwijang09/17 10:16這種上頭的 code review 當耳邊風就好，看有沒有機會加

kiwijang09/17 10:16薪繼續忍上頭，或升遷為上頭，沒機會就換了吧