Re: [討論] 重構跟kpi的考量
※ 引述《VScode (VSisBestIDEinTheWorld)》之銘言:
: 假設以下情境
: 有個功能A、B都會用到相同邏輯,且有兩份重覆的code
: (沒有unit test保護,而且年久失修 要加入unit test會需要更多時程)
: 現在要加入C,也會用到相同邏輯
: 身為合格的工程師 應該會把ABC重覆的部份提取出來
: 而不是讓這邏輯重覆三次
: 但以公司營運的角度來看 這次專案就只會測試C的部份
: 不應該動到A、B
: 這時就要冒著A、B壞掉風險重構,或是因為來不及加入unit test
: 就乾脆讓相同邏輯存在三個地方
: 身為專業工程師,我很想選擇重構
: 但過去的經驗告訴我
: 絕對要以kpi為最優先考量
: 於是程式充滿了註解、重覆片段
: 雖然靠著筆記、git log,能還原當時寫code的思路
: 但這些髒code就會永遠留存在程式裡
: 想問大家遇到這情況會怎麼做?
如果 A, B 都沒有任何 tests,建議不要動他。
幫 C 實做這個功能的時候,把 unit test 寫好寫滿,確保 C 是對的
行有餘力,針對 A, B 的使用情境也加上 test case,確保未來在 A, B 確實能重用
(這點很重要,否則很容易程式長得很像你以為可以重用,實際上根本不能)
就先做到這樣就好,確保 C 的品質,同時你獲得了高品質的 reusable 模組
隨著時間推演,有幾種情況:
1. A, B 的生命週期已經結束,直接淘汰,不用 refactor (這超常發生)
2. A, B 只是在維護修 bug,不會再有新功能,那通常也不值得花時間去弄
但若經常造成 bugs 的地方,正是跟 C 共用的那段程式,
那 refactor 就很有商業價值,長官也不會反對你做。
這種時候 refactor A, B 會變成重要的工作,你就不會沒時間做。
Refactor 完畢之後改善多少,就可以變成 Kpi,做起來名正言順
3. 如果真的發展到有機會 refactor A, B,這時就先幫最主要商業邏輯加上
"大範圍" 的 "integration tests" (不用花時間補寫 A, B 的 unit tests)
接著把 A, B 重複的邏輯抽換成 C 的 (之前開發 C 已 unit test 過,確保正確)
抽換完畢後,大範圍整合測試確認整體行為沒改變,就可以收工了
上線後持續監測,萬一遇到問題,直接 rollback 回上一版
專業的工程師,在開發的時候也會考慮實務面,以及這些 code 的商業價值,
來決定事情的先後順序,幫助產品獲得成功。
好的外科醫師,手術開刀,目標是病人要治好,而不是手術成功,病人死亡。
--
Sent from PCMan on PCMan's PC
--
※ 編輯: HZYSoft (111.249.169.220 臺灣), 02/26/2022 23:46:42
講得太好!
推這篇
正確觀念!推PCMan
有道理
就是因為東西常常不上線 才狂重構啊
推
推
推
推
推這篇
商業價值真的是重點
推pcman!!
推
推這篇!
推 謝謝分享
推
推
推
推
推
偷問哪種成功的手術是病人死亡的?
推
推
4
我覺得有個盲點就是 重複程式碼的邏輯 我的經驗是在需求還沒穩定前 一樣的程式碼複製到不同地方才是最佳解 你根本不知道什麼時候 某個地方要用的邏輯不同 一但要改寫的邏輯不通 你就會被共用的程式碼卡住13
感覺這個標題就是個假議題,你說不重構A、B因為Unit test來不及寫,那你新寫的C就不 用unit test了? 然後你又說三個code一模一樣,假設你幫C寫完unit test了,那你不就也把AB搞好了嗎? 再退一萬步來講,AB沒有unit test大家用的那麼爽你還硬要去動也只是吃飽太閒,不如 好好寫你C的unit test,寫完大家就用C就好啦35
首Po假設以下情境 有個功能A、B都會用到相同邏輯,且有兩份重覆的code (沒有unit test保護,而且年久失修 要加入unit test會需要更多時程) 現在要加入C,也會用到相同邏輯 身為合格的工程師 應該會把ABC重覆的部份提取出來
爆
Re: [問卦] 身邊不相信演化論的人多嗎?1.為什麼動物園裡ㄉ猩猩就不會變成人 演化需要幾十代到幾萬代的累積,不會突然間一隻狗生出一窩貓出來 2.人類的智慧要怎麼演化? 一群猩猩在一起,比較聰明加上運氣好的比較讓下一代安全長大 長期累積下來,過幾百代之後的下一代會比原始第一代聰明爆
[問卦] 有當男生不追了,反而女生檢討男生是渣男有當男生不追了,反而女生檢討男生是渣男呢? 聽朋友說 他的朋友a在追一個女生b一年多 line不回 也約不出去44
[問卦] 年收100 買一A二B會不會太硬朋友目前年收一百左右 北市已有房無貸款 原本想買golf但好像有人勸他都這錢了 再加點就能買一A二B了 所以他現在想買一A二B初階款,會不會太硬?39
[問卦] 大家會因為討厭B所以忍痛支持A嗎現在好像是個邏輯教授的世代 處處有二分法 如果你不支持我的言論 那你一定是挺我的敵人 ??????????????????????????????? OK13
Re: [問卦] C++可以啟發孩子的程式天份嗎?那要看你怎麼定義程式天份 如果是講演算法跟資料結構 用C或Python下去操比較快 如果是講計算機結構 一樣也是用C跟OS下去操 國中時 C++是我第一門接觸的語言 那個時候要做遊戲的難度不低5
[問卦] 二手雙B跟全新神A哪個好?如題 最近瘋狂看車 還有車商YouTube 預算應該70-80 全新神A2
[問卦] 本田翼是趨近於B的C嗎?宅偶像本田翼 以前在日劇裡說過 自己是C 有時會變B 所以是趨近於B的C3
[問卦] B肝疫苗有補打的必要嗎?小妹體檢發現B肝疫苗已經沒抗體了 醫師建議補打疫苗 但小妹打一劑他跟我收600... 還有兩劑要打...干 這麼貴接下來兩劑小妹就不想打了