Re: [討論] 重構跟kpi的考量

看板Soft_Job作者 (NickLin)時間2年前 (2022/02/26 08:57), 2年前編輯推噓13(13推 0噓 13→)
留言26則, 15人參與, 2年前最新討論串2/4 (看更多)
※ 引述《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因為Unit test來不及寫，那你新寫的C就不 用unit test了？ 然後你又說三個code一模一樣，假設你幫C寫完unit test了，那你不就也把AB搞好了嗎？ 再退一萬步來講，AB沒有unit test大家用的那麼爽你還硬要去動也只是吃飽太閒，不如 好好寫你C的unit test，寫完大家就用C就好啦 -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 98.207.136.165 (美國) ※ 文章網址: https://www.ptt.cc/bbs/Soft_Job/M.1645837075.A.310.html
推
02/26 09:11, 2年前 , 1F
推
02/26 09:11, 1F
→
02/26 10:07, 2年前 , 2F
他是說ABC其中各有一小塊重複的,變abc..
02/26 10:07, 2F
如果是不同東西的話也完全沒必要動 ※ 編輯: handsomeLin (98.207.136.165 美國), 02/26/2022 10:13:05
推
02/26 10:44, 2年前 , 3F
這篇正解... 所謂的專業不是自己想怎麼搞就怎麼搞...
02/26 10:44, 3F
→
02/26 11:34, 2年前 , 4F
評論要不要重構不能只看程式碼的邏，原原po應該是菜鳥
02/26 11:34, 4F
→
02/26 11:35, 2年前 , 5F
.......................................邏輯
02/26 11:35, 5F
推
02/26 14:09, 2年前 , 6F
菜鳥最喜歡重構了，以為很行
02/26 14:09, 6F
推
02/26 14:41, 2年前 , 7F
推樓上，一堆沒經驗的整天在那邊重構，搞出一堆問題
02/26 14:41, 7F
推
02/26 14:44, 2年前 , 8F
不過就是要這樣 才會有經驗啊
02/26 14:44, 8F
推
02/26 15:01, 2年前 , 9F
怎麼推文說得像是重構像是一種錯誤行為一樣...
02/26 15:01, 9F
推
02/26 15:24, 2年前 , 10F
不好說
02/26 15:24, 10F
→
02/26 15:30, 2年前 , 11F
重構只在有完善測試的情況下才能安全進行，測試越少
02/26 15:30, 11F
→
02/26 15:30, 2年前 , 12F
風險越大，故重構並不是在每個context下都是好的行為
02/26 15:30, 12F
→
02/26 15:34, 2年前 , 13F
有些人重構跟亂寫沒兩樣
02/26 15:34, 13F
推
02/26 17:16, 2年前 , 14F
現有測試再來重構
02/26 17:16, 14F
推
02/26 21:34, 2年前 , 15F
重構跟砍掉重練不一樣,一次改太多又沒測試沒bug才奇怪
02/26 21:34, 15F
推
02/26 22:49, 2年前 , 16F
Code的價值不在優不優雅 在於有沒有帶來新利潤
02/26 22:49, 16F
推
02/26 22:51, 2年前 , 17F
請問你重構的目的是為了節省記體？為了執行更快？為了
02/26 22:51, 17F
→
02/26 22:51, 2年前 , 18F
未來擴展性？還是為了規模化？
02/26 22:51, 18F
推
02/26 22:53, 2年前 , 19F
不要只為了code不合"優雅"的定義而重構
02/26 22:53, 19F
推
02/26 23:54, 2年前 , 20F
重構有時比砍掉重練難多了= =" debug 不如 delete
02/26 23:54, 20F
→
02/26 23:54, 2年前 , 21F
code 優不優雅這回事 可以加上註解 最終還是看出來
02/26 23:54, 21F
→
02/26 23:55, 2年前 , 22F
的東西是什麼鬼 沒利潤真的懶 除非是要改架構再說
02/26 23:55, 22F
→
02/26 23:55, 2年前 , 23F
不然光去找人協調就幾個月過去了
02/26 23:55, 23F
→
02/28 20:23, 2年前 , 24F
重構要看你有沒有這個決定權以及是誰想要重構 有時候
02/28 20:23, 24F
→
02/28 20:23, 2年前 , 25F
你想要重構上頭不准 有時候你不想重構上頭想 XD
02/28 20:23, 25F
→
02/28 20:25, 2年前 , 26F
通常上頭想的不是上頭自己也操刀
02/28 20:25, 26F
‣ 更多 handsomeLin 的文章
文章代碼(AID): #1Y6NiJCG (Soft_Job)
‣ 更多 handsomeLin 的文章
文章代碼(AID): #1Y6NiJCG (Soft_Job)