Soft_Job 板


LINE

繼續占版面分享少許心得。 在Python中也有類似直接指定參數的寫法,如: book_preservation = BookPreservation(user_name="Jack",...) 如果這些參數只是單純指定進去當作fields的話, 的確這樣的寫法就夠用了:) 下面分享一下這個重構手法的第3個好處: 3. 容易因此將相關功能移入新造的class中,改善程式碼分工 假設在我們的租書店程式中,在caller端有以下程式碼: class PreservationChecker{ void checkPreservation( String userName, String userId, String startTime, String endTime){ BookPreservation bookPreservation = new BookPreservation( "Jack", "1433717", "2016/5/8", "2016/8/8"); if(!isInTime( bookPreservation.getStartTime(), bookPreservation.getEndTime())){ return; } // 其他動作 } boolean isInTime(String startTime, String endTime){ Calendar calendar = Calendar.getInstance(); // 檢查過程 return result; } // 其他 methods } 透過上一篇文章中提到的重構手法,我們可以得到以下程式碼: class PreservationChecker{ void checkPreservation( String userName, String userId, String startTime, String endTime){ TimePeriod timePeriod = new TimePeriod("2016/5/8", "2016/8/8"); BookPreservation bookPreservation = new BookPreservation( new User("Jack", "1433717"), timePeriod); if(!isInTime( timePeriod.getStartTime(), timePeriod.getEndTime())){ return; } // 其他動作 } boolean isInTime(String startTime, String endTime){ Calendar calendar = Calendar.getInstance(); // 檢查過程 return result; } // 其他methods } 以及2個根據參數關係所分離出來的class: User與TimePeriod。 接下來,因為caller端的程式碼可能已經太多了, 或是在其他地方也有使用到TimePeriod的時間檢查, 好比說、可能另外有個租書店用來舉辦活動的程式碼, 用來檢查今天是不是應該給客人30元優惠: class FesCoupon{ private String startTime; private String endTime; double getCouponAmount(){ if(!isInTime( this.getStartTime(), this.getEndTime())){ return 0; } return 30; } // 以下是重複碼 boolean isInTime(String startTime, String endTime){ Calendar calendar = Calendar.getInstance(); // 檢查過程 return result; } } 因為我們先前把BookPreservation的2個時間相關參數, 透過Extract Class,得到了TimePeriod這個class, 我們現在就可以把重複碼isInTime()放進去,得到: class TimePeriod{ private String startTime; private String endTIme; boolean isInTime(){ Calendar calendar = Calendar.getInstance(); // 檢查過程 return result; } } 然後、上述兩個引用點就可以精簡為: class PreservationChecker{ void checkPreservation( String userName, String userId, String startTime, String endTime){ TimePeriod timePeriod = new TimePeriod("2016/5/8", "2016/8/8"); BookPreservation bookPreservation = new BookPreservation( new User("Jack", "1433717"), timePeriod); if(!timePeriod.isInTime()){ return; } // 其他動作 } // 其他methods } class FesCoupon{ private TimePeriod timePeriod; double getCouponAmount(){ if(!timePeriod.isInTime()){ return 0; } return 30; } } 如此, ① 不僅消除了2個caller class當中的重複碼, ② 也將isInTime()這個methods放到了與最相關的class當中(也就是TimePeriod)。 即使在支援直接指定參數寫法的語言,如Python中, 也能因為這個重構手法,而獲得以上2個改善的好處。 ※ 引述《leicheong (睡魔)》之銘言: : ※ 引述《ADYex (寵物狼音樹)》之銘言: : : 例如,假設在一個租書店的程式中有以下程式碼: : : BookPreservation bookPreservation = new BookPreservation( : : "Jack", "1433717", "2016/5/8", "2016/8/8"); : : 其中4個參數分別為 userName, userId, startTime, endTime, : : 比較好的作法是將各自相關聯的參數各自包裝,變成: : : BookPreservation bookPreservation = new BookPreservation( : : new User("Jack", "1433717"), new TimePeriod("2016/5/8", "2016/8/8")); : : 這個重構手法能帶來的好處如下: : : 1. 提升可讀性 : : 2. 未來維護簡單 : : 3. 容易因此將相關功能移入新造的class中,改善程式碼分工 : : 試著像這樣將原作法的壞處與新作法的好處跟主管說看看吧。或是塊陶。 : 這個的話還需要看在用甚麼程式語言吧. : 像在VB和C# v4+上也可以這樣寫: : BookPreservation bookPreservation = new BookPreservation( : userName: "Jack", : userId: "1433717", : startTime: "2016/05/08", : endTime: "2016/08/08"); : 這樣寫比分拆成用property設定更好. 也是你之前說的「在初始化時設定」 : 和「先全部初始化成null, 在建構完成後再設定」的差別. --



※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 61.231.166.119
※ 文章網址: https://webptt.com/m.aspx?n=bbs/Soft_Job/M.1462703532.A.A73.html
1F:→ ADYex: 雖然是用Java舉例,希望這樣看得懂在其他語言中的適用性 :P 05/08 18:35
2F:推 ian90911: 推 05/09 00:49







like.gif 您可能會有興趣的文章
icon.png[問題/行為] 貓晚上進房間會不會有憋尿問題
icon.pngRe: [閒聊] 選了錯誤的女孩成為魔法少女 XDDDDDDDDDD
icon.png[正妹] 瑞典 一張
icon.png[心得] EMS高領長版毛衣.墨小樓MC1002
icon.png[分享] 丹龍隔熱紙GE55+33+22
icon.png[問題] 清洗洗衣機
icon.png[尋物] 窗台下的空間
icon.png[閒聊] 双極の女神1 木魔爵
icon.png[售車] 新竹 1997 march 1297cc 白色 四門
icon.png[討論] 能從照片感受到攝影者心情嗎
icon.png[狂賀] 賀賀賀賀 賀!島村卯月!總選舉NO.1
icon.png[難過] 羨慕白皮膚的女生
icon.png閱讀文章
icon.png[黑特]
icon.png[問題] SBK S1安裝於安全帽位置
icon.png[分享] 舊woo100絕版開箱!!
icon.pngRe: [無言] 關於小包衛生紙
icon.png[開箱] E5-2683V3 RX480Strix 快睿C1 簡單測試
icon.png[心得] 蒼の海賊龍 地獄 執行者16PT
icon.png[售車] 1999年Virage iO 1.8EXi
icon.png[心得] 挑戰33 LV10 獅子座pt solo
icon.png[閒聊] 手把手教你不被桶之新手主購教學
icon.png[分享] Civic Type R 量產版官方照無預警流出
icon.png[售車] Golf 4 2.0 銀色 自排
icon.png[出售] Graco提籃汽座(有底座)2000元誠可議
icon.png[問題] 請問補牙材質掉了還能再補嗎?(台中半年內
icon.png[問題] 44th 單曲 生寫竟然都給重複的啊啊!
icon.png[心得] 華南紅卡/icash 核卡
icon.png[問題] 拔牙矯正這樣正常嗎
icon.png[贈送] 老莫高業 初業 102年版
icon.png[情報] 三大行動支付 本季掀戰火
icon.png[寶寶] 博客來Amos水蠟筆5/1特價五折
icon.pngRe: [心得] 新鮮人一些面試分享
icon.png[心得] 蒼の海賊龍 地獄 麒麟25PT
icon.pngRe: [閒聊] (君の名は。雷慎入) 君名二創漫畫翻譯
icon.pngRe: [閒聊] OGN中場影片:失蹤人口局 (英文字幕)
icon.png[問題] 台灣大哥大4G訊號差
icon.png[出售] [全國]全新千尋侘草LED燈, 水草

請輸入看板名稱,例如:iOS站內搜尋

TOP