作者TonyQ (得理饒人)
看板Soft_Job
標題重構的幾個迷思
時間Tue Jul 7 08:27:58 2020
覺得最近很多文章都有些不求甚解的問題，來寫點論述。
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.cc), 來自: 61.231.31.193 (臺灣)
※ 文章網址: https://webptt.com/m.aspx?n=bbs/Soft_Job/M.1594081682.A.E16.html
1F：推 d1288999: 推推 07/07 08:48
2F：推 yyhsiu: 推前半，做事真的要看帶來的價值 07/07 08:51
3F：推 bill0205: 推 07/07 09:00
4F：推 Gaitz: 推 07/07 09:27
5F：推 WTFCN: 台肯 07/07 09:36
6F：推 lovez04wj06: 除了半瓶水以外，真的有人喜歡沒事就重購嗎？ 07/07 10:46
比起有沒有事(應該沒多少人是真的沒事的),
更重要的問題是很多人會看 code 看不順眼. (不過這個行為本身不見得是壞事)
看到一個問題就想要去挑戰更好的作法,
不論是求表現也好, 或是想找事情殺時間, 這是人之常情.
※ 編輯: TonyQ (210.61.209.201 臺灣), 07/07/2020 10:49:27
7F：推 asd51052000: 是不是少了[心得]? 07/07 10:53
8F：→ x000032001: 加功能就容易伴隨著重構 不然經常會變得疊床架屋 07/07 11:05
9F：推 ian90911: 中肯 07/07 11:38
10F：推 jhengsiaomin: 推 07/07 12:02
11F：推 tbpfs: 中肯 07/07 12:37
12F：噓 iceman5566: 你的標題先重構一下吧 07/07 12:45
13F：推 Menderca: 中肯推 07/07 13:21
14F：→ longlyeagle: E 07/07 13:21
15F：→ shooter555: 加功能伴隨重構 那應該會想打寫架構的人 07/07 13:24
16F：推 TAKADO: 改扣之前先問問自己，這段程式這麼爛大家都有看到，為什麼 07/07 13:32
17F：→ TAKADO: 沒有前人去改它，凡事必有因果。 07/07 13:32
我自己改過很多次, 通常原因都是前人能力不足/膽量不足.
所以要問問的應該是自己有沒有比前人厲害.
18F：→ shooter555: 不過真的就是權限問題 權限夠想把所有功能改掉 不用說 07/07 13:33
19F：→ shooter555: 物件化 擬人化都可以 每個功能幫它取個名子 07/07 13:34
20F：推 jixiang: 中肯 07/07 13:39
21F：→ x000032001: 哪來的萬用架構都不用動就可以加功能 不需要dirty wo 07/07 13:39
22F：→ x000032001: rk 請務必讓我見識怎麼設計出來的 07/07 13:39
23F：→ KeyFSN: 就是有阿 看過才會讚嘆公司花大錢請 senior 不是沒有原因 07/07 13:54
24F：→ as30385438: 能搞出萬用架構應該也不是senior, 是神了 07/07 14:08
25F：→ as30385438: 不然就是一堆overdesign的garbage code自以為很神 07/07 14:08
※ 編輯: TonyQ (210.61.209.201 臺灣), 07/07/2020 14:11:04
26F：推 Darkword1987: 我覺得要refactor要有理由 醜不算 07/07 14:18
27F：→ shooter555: 架構總要保留可以擴充跟相容的空間吧 加一個功能就要 07/07 14:29
28F：→ shooter555: 這就是有問題的 07/07 14:30
29F：→ shooter555: 重構 07/07 14:30
30F：推 hichcock: 年輕人應該多鼓勵重構當練功, 但是請私下做 07/07 15:12
31F：→ hichcock: 不要影響到大家工作的環境, 重構下去你才會發現很多問題 07/07 15:12
32F：→ hichcock: 包含自己缺少的, 還有舊架構的涵義等等 07/07 15:13
33F：推 leveger0903: 如果該專案幾乎沒有後續需求的話 只是醜我可以 07/07 15:57
34F：→ leveger0903: 但是常常有後續一堆需求 加上前人刻意不照公司寫程 07/07 15:58
35F：→ leveger0903: 式規範走 留下很多坑 東西又在線上 不得不選這條路 07/07 15:58
36F：→ lazarus1121: 站在管理者的立場就是沒壞不要改，因為錯了要擔責任 07/07 17:37
37F：→ lazarus1121: 但站在開發者的角度，這東西不改我會很難維護跟除錯 07/07 17:38
我是管理者而且我會改，反正我扛責任沒人會說什麼。
這篇真正的立場叫做掂掂自己斤兩。
38F：→ lazarus1121: 這篇的立場只是偏前者 07/07 17:39
39F：推 joery: 推能跑沒問題再爛也不要動code,很難改不知道是否有地雷，可 07/07 18:42
40F：→ joery: 3會付出代價 07/07 18:42
41F：→ airtsubasa: 如果公司沒有自己的規範呢？ 呵呵… 07/07 20:35
42F：推 simpleplanya: 推唷 07/07 21:36
43F：推 ericjc: 推一個 07/07 22:24
44F：推 clamperni: 說真的我到現在還沒遇到真的懂重構的 2倒是一堆XD 07/08 00:03
45F：推 Csongs: 很多人只是看不慣別人寫得而已 07/08 00:05
※ 編輯: TonyQ (61.231.78.150 臺灣), 07/08/2020 05:24:43
46F：推 nenpow: 這篇整理得很清楚, 但多數人還是只會擷取自己想聽的 07/08 08:44
47F：推 LuyTe: premature optimization is the root of all evil 07/08 09:53
48F：→ LuyTe: 重構跟講英文很像，你會看到一堆英文很爛的人很有自信開口 07/08 09:57
49F：→ LuyTe: 講，也會看到英文很好卻不敢開口的人。你會看到一堆該重構 07/08 09:57
50F：→ LuyTe: 的人找理由不去重構，也會看到不該重構的人OCD發作改些沒意 07/08 09:57
51F：→ LuyTe: 義的東西 07/08 09:57
52F：推 smily134: 推 07/08 11:01
53F：→ zx1986: > 凡人登山要確保 超中肯！Testing 先起來再說重構！ 07/08 13:49
54F：推 overhead: 抱持code沒壞就不要改的，是爛主管。通常就是他自己cod 07/11 15:36
55F：→ overhead: ing能力差，不勤練，不好好寫test。還因為他是這種人， 07/11 15:36
56F：→ overhead: 整個團隊也慢慢變這種風格。 07/11 15:36