PTT評價

重構的幾個迷思

TonyQ 發表於 2020/7/7 上午8:27:58
看板Soft_Job標題重構的幾個迷思作者
TonyQ
 (得理饒人)時間推噓24 推:25 噓:1 →:30

覺得最近很多文章都有些不求甚解的問題，來寫點論述。

1. 重構不是什麼了不起的事情

2. 變更程式碼，重寫舊的程式碼成自己爽的樣子，不一定是重構。

3. 重構是一種相對安全的工具型開發方法論，
但仍然有不少風險跟誘惑。

4. 如果你不應該或沒權力改的程式碼，
不要以為自稱是重構就能取得變更的權力。

5. 測試(單元或整合)，是一種快速驗證，
程式是否符合自己預期的工具，但並不是不會犯錯的保證。

因為，自己的預期可能就是錯的，也可能測試相依過深，
一時半刻沒有足夠好的介面，測不到真正的主要情境。


6. 不論重寫或重構，所有的程式碼變更都有一個本質，
要把時間花在刀口上，對重要的程式碼優先處理，
不重要的程式碼延後處理，程式碼都有優先權問題。


所謂的程式碼沒壞就不要修他，
本質上是「如果他的現況不影響別的事情，暫時不用管他」。

但如果他已經卡到別的功能的擴充或維護，
造成 DEBUG 困難，常出問題等，修改他就有了價值。

那個狀態就又是另一回事。


======

另外，有些話我覺得講的不夠白，做點翻譯：


1. 東西沒壞就不要改他：

你最近改壞的東西太多了/
你最近正常改好的東西太少了/
這段扣不關你的事情你沒有權限可以碰。


2. 開發應該要先做測試：

你最近改壞的東西太多了/
你最近正常改好的東西太少了


3. 要重構之前應該要先做測試：

你現在的 CODE 搞不好都已經在亂寫了，
再大規模改 CODE 會不會死的更難看。

4. 這 CODE 寫的很爛：

我想把 CODE 改成我看起來爽的樣子


=====

坦白說，你寫程式品質高的話，要怎麼炫技都可以，
凡人登山要確保，很多高手可以無確保登山，
但這些人每隔幾年也就會有一兩個摔死上新聞。


嗯，反正說到底改程式碼的權限是個政治問題。


打著重構或效能調整的名義變更是沒關係，
但有沒有做好，是一翻兩瞪眼的事情。


個人 Credit 喪失事小，
把使用這個工具的人給搞成白癡事大，

還麻煩大家，量力而為。


自己爛，不會開發，不要牽拖工具方法論。


要重構先還是要測試先，會問這種問題的，
還不如先練習看看什麼是重構，什麼是測試。



-----
Sent from JPTT on my Google Pixel 3 XL.

--

網頁上拉近距離的幫手 實現 GMail豐富應用的功臣

數也數不清的友善使用者體驗 這就是ｊａｖａｓｃｒｉｐｔ

歡迎同好到 AJAX 板一同討論。

--

※ PTT 留言評論
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 61.231.31.193 (臺灣)
※ PTT 網址
推

d128899907/07 08:48推推

推

yyhsiu07/07 08:51推前半，做事真的要看帶來的價值

推

bill020507/07 09:00推

推

Gaitz07/07 09:27推

推

WTFCN07/07 09:36台肯

推

lovez04wj0607/07 10:46除了半瓶水以外，真的有人喜歡沒事就重購嗎？

比起有沒有事(應該沒多少人是真的沒事的), 更重要的問題是很多人會看 code 看不順眼. (不過這個行為本身不見得是壞事) 看到一個問題就想要去挑戰更好的作法, 不論是求表現也好, 或是想找事情殺時間, 這是人之常情.

※ 編輯: TonyQ (210.61.209.201 臺灣), 07/07/2020 10:49:27
推

asd5105200007/07 10:53是不是少了[心得]?

→

x00003200107/07 11:05加功能就容易伴隨著重構 不然經常會變得疊床架屋

推

ian9091107/07 11:38中肯

推

jhengsiaomin07/07 12:02推

推

tbpfs07/07 12:37中肯
噓

iceman556607/07 12:45你的標題先重構一下吧

推

Menderca07/07 13:21中肯推

→

longlyeagle07/07 13:21E

→

shooter55507/07 13:24加功能伴隨重構 那應該會想打寫架構的人

推

TAKADO07/07 13:32改扣之前先問問自己，這段程式這麼爛大家都有看到，為什麼

→

TAKADO07/07 13:32沒有前人去改它，凡事必有因果。

我自己改過很多次, 通常原因都是前人能力不足/膽量不足. 所以要問問的應該是自己有沒有比前人厲害.

→

shooter55507/07 13:33不過真的就是權限問題 權限夠想把所有功能改掉 不用說

→

shooter55507/07 13:34物件化 擬人化都可以 每個功能幫它取個名子

推

jixiang07/07 13:39中肯

→

x00003200107/07 13:39哪來的萬用架構都不用動就可以加功能 不需要dirty wo

→

x00003200107/07 13:39rk 請務必讓我見識怎麼設計出來的

→

KeyFSN07/07 13:54就是有阿 看過才會讚嘆公司花大錢請 senior 不是沒有原因

→

as3038543807/07 14:08能搞出萬用架構應該也不是senior, 是神了

→

as3038543807/07 14:08不然就是一堆overdesign的garbage code自以為很神

※ 編輯: TonyQ (210.61.209.201 臺灣), 07/07/2020 14:11:04
推

Darkword198707/07 14:18我覺得要refactor要有理由 醜不算

→

shooter55507/07 14:29架構總要保留可以擴充跟相容的空間吧 加一個功能就要

→

shooter55507/07 14:30這就是有問題的

→

shooter55507/07 14:30重構

推

hichcock07/07 15:12年輕人應該多鼓勵重構當練功, 但是請私下做

→

hichcock07/07 15:12不要影響到大家工作的環境, 重構下去你才會發現很多問題

→

hichcock07/07 15:13包含自己缺少的, 還有舊架構的涵義等等

推

leveger090307/07 15:57如果該專案幾乎沒有後續需求的話 只是醜我可以

→

leveger090307/07 15:58但是常常有後續一堆需求 加上前人刻意不照公司寫程

→

leveger090307/07 15:58式規範走 留下很多坑 東西又在線上 不得不選這條路

→

lazarus112107/07 17:37站在管理者的立場就是沒壞不要改，因為錯了要擔責任

→

lazarus112107/07 17:38但站在開發者的角度，這東西不改我會很難維護跟除錯

我是管理者而且我會改，反正我扛責任沒人會說什麼。 這篇真正的立場叫做掂掂自己斤兩。

→

lazarus112107/07 17:39這篇的立場只是偏前者

推

joery07/07 18:42推能跑沒問題再爛也不要動code,很難改不知道是否有地雷，可

→

joery07/07 18:423會付出代價

→

airtsubasa07/07 20:35如果公司沒有自己的規範呢？ 呵呵…
推

simpleplanya07/07 21:36推唷

推

ericjc07/07 22:24推一個

推

clamperni07/08 00:03說真的我到現在還沒遇到真的懂重構的 2倒是一堆XD

推

Csongs07/08 00:05很多人只是看不慣別人寫得而已

※ 編輯: TonyQ (61.231.78.150 臺灣), 07/08/2020 05:24:43
推

nenpow07/08 08:44這篇整理得很清楚, 但多數人還是只會擷取自己想聽的

推

LuyTe07/08 09:53premature optimization is the root of all evil

→

LuyTe07/08 09:57重構跟講英文很像，你會看到一堆英文很爛的人很有自信開口

→

LuyTe07/08 09:57講，也會看到英文很好卻不敢開口的人。你會看到一堆該重構

→

LuyTe07/08 09:57的人找理由不去重構，也會看到不該重構的人OCD發作改些沒意

→

LuyTe07/08 09:57義的東西

推

smily13407/08 11:01推

→

zx198607/08 13:49> 凡人登山要確保 超中肯！Testing 先起來再說重構！

推

overhead07/11 15:36抱持code沒壞就不要改的，是爛主管。通常就是他自己cod

→

overhead07/11 15:36ing能力差，不勤練，不好好寫test。還因為他是這種人，

→

overhead07/11 15:36整個團隊也慢慢變這種風格。
其他人也閱讀了
PTT 熱門相關