Re: [討論] 前人的code 後人翻寫的機率高嗎?

作者: thefattiger (LT)   2018-09-26 20:15:37
這位前輩你好,小弟有些不同的意見
※ 引述《accessdenied (存取違規)》之銘言:
: 單元測試有時候反而破壞程式碼的易讀性和維護性
: 因為要做到單元測試,就得斷開所有的相依性
就算沒有單元測試,"高內聚,低耦合"的原則,也應該是適用的
所謂斷開相依性更準確地說應該是"相依於抽象而非具體類別"
Design Pattern有80%以上的pattern都是在做這件事
: 而對抗相依性,作法就是引入 DI。但是 DI 就會增加代碼閱讀和維護的複雜度。
: 舉例來說,如果代碼內有時間上的相依性,例如用了 DateTime 物件取得現在時間做某些
: 判斷,原本可以很簡單的寫出易於閱讀和維護的邏輯:
: If (DateTime.Now > 12:00:00) then return “PM” else return “AM”;
: 為了讓單元測試可以控制驗證條件,只能
: Interface IDateProvider { virtual GetNow(); }
: Class DateMock : IDateProvider { GetNow() { return 13:00 }
: IDateProvider dateProvider = container.Build(....);
: If (dateProvider.GetNow > 12:00:00) then return “PM” else return “AM”;
: 然後再搞個 config 想辦法讓程式吃到你寫的 DateMock 類別....
: 上面是sudo code 就不用討論語法細節
以您提的DateTime為例,若If那行出問題,沒有單元測試的情況下
我們可能沒法馬上知道錯的是DateTime,還是比較的邏輯(可能是>寫成<)
小弟以為單元測試的好處就是在測某個功能時,可以用Mock確保其他東西都是正常的
這樣有錯誤才好抓出來
多加一個IDateProvider有另一個好處是可以應付DateTime不只一種的狀況
例如TaiwanDateTime和USDateTime
: 這就是單元測試的代價!程式真的會比較容易閱讀嗎?
"理想上"的狀況是測試程式本身就能明確表達出需求
讓人能夠只看測試程式而不用去trace又臭又長的商業邏輯就能大概知道整隻程式在幹嘛
: 單元測試要花在切除相依性的條件花費的成本時間遠高於撰寫production code
: 而且你的 production code 能不能賺錢還不知道咧?
: 到底需不需要單元測試和 clean code ? 先搞清楚寫的用途和目的,你寫的東西有沒有
: 真正的商業價值再說吧。
商業價值跟有沒有單元測試或clean code,應該是沒有相關的
總不是說一個本來會賺錢的專案因為寫了單元測試就不賺錢了吧?
況且單元測試本身就是為了減少開發與維護成本所出現的東西
: 不要把 clean code 和 TDD 無限上綱了
: 工程師最容易自嗨就是這樣,還會自以爲「這是專業」?
: 乞丐的乞討專業比我們強也不會產生「價值」。
就我遇到的狀況都是"我們不要搞那些了,程式能動就好"
倒還沒聽說過因為用TDD,測試程式寫太多拖累整個專案的事情...
寫出易讀、易維護的程式碼也是軟體工程師的職責之一吧!
作者: senjor (哞哞)   2018-09-26 20:17:00
其實我大概知道為什麼這麼多人討厭clean code跟TDD了...因為他們不懂所以討厭,無法理解所以誤解,因為誤解而厭惡
作者: Killercat (殺人貓™)   2018-09-26 20:23:00
TDD其實不是一開始就走TDD的話 碰壁是天經地義的事這本來就是一個適合從零開始的方法
作者: accessdenied (存取違規)   2018-09-26 20:29:00
不是因為寫太多測試案例而拖累專案,而是維護太多測試案例而拖累專案。測試案例本身也是程式碼,也可能有bug,再來就是你的商業策略轉彎的時候,大量已存的測試案例要跟著全改就是包袱了但是那些因為商業邏輯變動而不適用的測試案例,要嘛一支支改,不然就是拋棄不用,未來也不會拿來再跑偏偏商業邏輯是變動最快的決策之一
作者: landlord (91)   2018-09-26 20:34:00
維護測試成本過高,要嘛測試品質很差,要嘛代碼設計爛
作者: sharku (明珠求瑕)   2018-09-26 20:35:00
測試個案也是需要clean code的好嗎
作者: Killercat (殺人貓™)   2018-09-26 20:36:00
這也是一種說法,所以才會有robot framework跟小黃瓜這種接近自然語言的東西 請QA/QE來寫這些壁其實前人都碰過 不然機器人小黃瓜不會紅起來
作者: sarafciel (Cattuz)   2018-09-26 20:40:00
如果改code要測試全翻 那可能是你的程式基礎架構拆的不
作者: accessdenied (存取違規)   2018-09-26 20:40:00
感謝樓上大大指點,這兩個關鍵字我去研究一下。我一直以為小黃瓜是女性用品,原來是工程師的良藥
作者: sarafciel (Cattuz)   2018-09-26 20:41:00
夠細 不過我是沒有碰過商業邏輯改到要幾乎全翻的情況XD
作者: alihue (wanda wanda)   2018-09-26 20:48:00
.net 也有 spec flow,不過 .net core 不知道有沒有要支援
作者: Killercat (殺人貓™)   2018-09-26 20:48:00
其實我覺得這發言情有可原 因為一開始就長歪的 要能CI跟test driven是的極度痛苦的流程,有偏見也難免這一開始就要寫的不要太歪 才不致於以後無法收拾
作者: accessdenied (存取違規)   2018-09-26 20:51:00
也許我沒見過樓上說的方法,以前看到的景象都讓我皺眉頭,既然時代有進步了,那我再多瞭解看看
作者: landlord (91)   2018-09-26 21:04:00
時代的眼淚跟可怕的測試,看了跟維護起來真的很嚇人的。@alihue,非驗收測試或跟需求方雙向交流,cucumber非必要
作者: clarkman (涼雨)   2018-09-26 21:22:00
以前滿常翻code的,但重寫自己就要有認知新的bug要自己吞下,不過重構後的code穩定很多,省了後期的debug時間
作者: rollr (衛生紙的心情)   2018-09-26 23:09:00
講的很好耶

Links booklink

Contact Us: admin [ a t ] ucptt.com