摘要:面對這樣的已經上線的代碼,我并沒有想去重構他因為成本太高,只好鞭策自己不要寫出這種代碼面對的問題有時候,我們可能面對這樣的業務邏輯。一坨一坨的看著非常不舒服,并且難以維護。如果不滿足條件返回調用職責鏈的下一個節點。
前言
新入職的公司,前人留下來一個項目,里面充斥著大量的if...else...,則倒是其次,主要連注釋寫的都很少。面對這樣的已經上線的代碼,我并沒有想去重構他因為成本太高,只好鞭策自己不要寫出這種代碼
面對的問題?有時候,我們可能面對這樣的業務邏輯。(公司項目的業務邏輯),如果是回答過題目通過,如果回答過題目沒有通過,如果沒有回答過題目。如果不使用特定的模式,可能會寫出下面這樣的代碼。一坨一坨的if...else看著非常不舒服,并且難以維護。
/** * 初始化函數 * if...else if... 的情況較為簡單 * @return undefined */ function init () { // 是否回答過題目 1-回答過, 通過 2-回答過, 沒有通過 3-沒有回答過 let isAnswer // 是否是老用戶 1-老用戶 2-新用戶 let isOldUser if (isAnswer === 1) { // ... } else if (isAnswer === 2) { // ... } else if (isAnswer === 3) { // ... } if (isOldUser === 1) { // ... } else if (isOldUser === 2) { // ... } }
/** * 初始化函數 * if...else if... 嵌套的情況 * @return undefined */ function init () { if (isAnswer === 1) { if (isOldUser === 1) { // ... } else if (isOldUser === 2) { // ... } } else if (isAnswer === 2) { if (isOldUser === 1) { // ... } else if (isOldUser === 2) { // ... } } else if (isAnswer === 3) { if (isOldUser === 1) { // ... } else if (isOldUser === 2) { // ... } } }解決辦法1: 查找表, 職責鏈查找表
雖然可能看著是治標不治本,其實不然,init函數的復雜度大大的降低了。我們已經把控制流程的復雜邏輯,拆分到determineAction函數中
// 可以解決if...else if...簡單的問題 const rules = { isAnswer1 () { return code }, isAnswer2 () { return code }, isAnswer3 () { return code } } function determineAction (type) { if (isAnswer === 1) { return "isAnswer1" } else if (isAnswer === 2) { return "isAnswer2" } else if (isAnswer === 3) { return "isAnswer3" } } function init () { let key = determineAction(isAnswer) return rules[key] }
// 面對if...else if...else 嵌套的復雜情況 const rules = [ { match (an, old) { if (an === 1) { return true } }, action (an, old) { if (old === 1) { // ... } else if (old === 2) { // ... } } }, { match (an, old) { if (an === 2) { return true } }, action (an, old) { if (old === 1) { // ... } else if (old === 2) { // ... } } }, { match (an, old) { if (an === 3) { return true } }, action (an, old) { if (old === 1) { // ... } else if (old === 2) { // ... } } } ] function init (an, old) { for (let i = 0; i < rules.length; i++) { // 如果返回true if (rules[i].match(an, old)) { rules[i].action(an, old) } } } init(isAnswer, isOldUser)
??上面復雜的情況,也可以吧action的判斷抽離出來但是可能要寫出三個抽離的函數,因為an值有三種不同的情況
解決辦法2: 面向切面的編程(AOP)為Function的原型鏈,擴展after語法,如果滿足要求直接在函數內運算并返回結果。如果不滿足條件返回"next"調用職責鏈的下一個節點。所謂的Function.prototype.after就是在本函數執行前執行after添加的函數
// 可以解決if...else if...簡單的問題 Function.prototype.after = function (nextFn) { let self = this return function (...rest) { let code = self(...rest) if (code === "next") { return nextFn(...rest) } return code } } // 重構原函數 function isAnswer1 (type) { if (type === 1) { return code } return "next" } function isAnswer2 () { if (type === 2) { return code } return "next" } function isAnswer3 () { if (type === 3) { return code } return "next" } let isAnswerFn = isAnswer1.after(isAnswer2).after(isAnswer3) isAnswerFn(isAnswer)
// 面對if...else if...else 嵌套的復雜情況 function isAnswer1 (an, old) { if (an === 1) { return isOldUserFn1(an, old) } return "next" } function isAnswer2 (an, old) { if (an === 2) { return isOldUserFn2(an, old) } return "next" } function isAnswer3 (an, old) { if (an === 3) { return isOldUserFn3(an, old) } return "next" } /** * isAnswer == 1 isOldUser == 1 的情況 */ function isAnswer1IsOldUser1 (an, old) { if (old === 1) { return code } return "next" } /** * isAnswer == 1 isOldUser == 2 的情況 */ function isAnswer1IsOldUser2 (an, old) { if (old === 2) { return code } return "next" } /** * isAnswer == 2 isOldUser == 1 的情況 */ function isAnswer2IsOldUser1 (an, old) { if (old === 1) { return code } return "next" } /** * isAnswer == 2 isOldUser == 2 的情況 */ function isAnswer2IsOldUser2 (an, old) { if (old === 2) { return code } return "next" } /** * isAnswer == 3 isOldUser == 1 的情況 */ function isAnswer3IsOldUser1 (an, old) { if (old === 1) { return code } return "next" } /** * isAnswer == 3 isOldUser == 2 的情況 */ function isAnswer3IsOldUser2 (an, old) { if (old === 2) { return code } return "next" } let isAnswerFn = isAnswer1.after(isAnswer2).after(isAnswer3) // 三條職責鏈 let isOldUserFn1 = isAnswer1IsOldUser1.after(isAnswer1IsOldUser2) let isOldUserFn2 = isAnswer2IsOldUser1.after(isAnswer2IsOldUser2) let isOldUserFn3 = isAnswer3IsOldUser1.after(isAnswer3IsOldUser2) isAnswerFn(isAnswer, isOldUser)解決辦法3: 函數式編程
利用ramda等函數式編程庫解決這種問題,
文章版權歸作者所有,未經允許請勿轉載,若此文章存在違規行為,您可以聯系管理員刪除。
轉載請注明本文地址:http://specialneedsforspecialkids.com/yun/104541.html
摘要:重構定義重構是對軟件內部結構的調整,目的是在不改變軟件可觀察行為的前提下,提高其可理解性,降低其修改成本。重構節奏小步前進,頻繁測試。 1.重構定義: 重構是對軟件內部結構的調整,目的是在不改變軟件可觀察行為的前提下,提高其可理解性,降低其修改成本。 2.重構節奏: 小步前進,頻繁測試。 3.重構意義: 1.改進軟件設計 2.使軟件更容易被理解 3.幫助找到bug 4.提高編程速度 惡...
摘要:重構在不改變代碼的外在的行為的前提下對代碼進行修改最大限度的減少錯誤的幾率本質上,就是代碼寫好之后修改它的設計。重構可以深入理解代碼并且幫助找到。同時重構可以減少引入的機率,方便日后擴展。平行繼承目的在于消除類之間的重復代碼。 重構 (refactoring) 在不改變代碼的外在的行為的前提下 對代碼進行修改最大限度的減少錯誤的幾率 本質上, 就是代碼寫好之后 修改它的設計。 1,書中...
摘要:通常情況下,面向對象編程讓我們得以避免條件式,并代之以繼承和多態。同時,使用條件式簡寫來表示值。因此,對于以這種方式編寫的代碼,你需要使用進行編譯。 原文地址:Tips and Tricks for Better JavaScript Conditionals and Match Criteria 原文作者:Milos Protic 介紹 如果你像我一樣樂于見到整潔的代碼,那么你...
摘要:在開發的過程中相信你也會寫很多的語句吧,此篇主要來講講如何在日常開發的過程中盡量少的使用語句。策略一單例模式這種單例模式在類一加載的時候就將單例對象創建完畢,總是這個對象存在內存中,避免了通過線程同步來生成對象,線程安全的創建方式。 在開發的過程中相信你也會寫很多的if else語句吧,此篇主要來講講如何在日常開發的過程中盡量少的使用if else語句。 0x01 為什么要去if el...
閱讀 1518·2021-11-18 10:02
閱讀 1657·2021-09-04 16:40
閱讀 3171·2021-09-01 10:48
閱讀 874·2019-08-30 15:55
閱讀 1853·2019-08-30 15:55
閱讀 1365·2019-08-30 13:05
閱讀 3013·2019-08-30 12:52
閱讀 1624·2019-08-30 11:24