過多if-else分支的優(yōu)化
我想談一談這個話題是因為我的上一篇博客有一些朋友回復(fù),說if-else過多的分支可以使用switch或者責(zé)任鏈模式等等方式來優(yōu)化。確實,這是一個小問題,不過我們還是可以整理一下這個小問題的重構(gòu)方式。
為什么要優(yōu)化?
你沒有看錯。這是要放在第一條談?wù)摰摹?/p>
有許多人會說,疊起來一堆if-else分支,代碼就不優(yōu)雅了??墒牵鯓尤ザx“優(yōu)雅”的概念呢?再退一步說,即便不“優(yōu)雅”,又有什么問題?
對于這樣一段再普通不過的代碼:
- int code;
- if("Name".equals(str))
- code = 0;
- else if("Age".equals(str))
- code = 1;
- else if("Address".equals(str))
- code = 2;
- ...
可以有好多種重構(gòu)方式,但是使用這樣的代碼,雖然簡陋,但在大多數(shù)情況下,并不會影響什么,比如,對可維護(hù)性沒有影響。當(dāng)然,如果你發(fā)現(xiàn)其中確有不好的一面,那就要考慮重構(gòu)它。換言之,通常你首先要說出某段代碼的問題(比如,你覺得這段代碼不符合開閉原則,因為你希望保持這段代碼閉合穩(wěn)定),那么才去存在重構(gòu)的必要,而不要總是使用“優(yōu)雅”和“簡潔”搪塞疑問。幾乎所有的書上都說要寫出優(yōu)雅的、簡潔的代碼,這本身無可厚非,但是事物需要使用自己的判斷,可不要被習(xí)慣性地洗了腦。
在我前一家公司,是典型的通訊和傳統(tǒng)軟件的公司,代碼質(zhì)量普遍不錯,但是很多時候,會看到許許多多不夠優(yōu)雅的代碼——也許你覺得不夠簡潔、美觀,但是下代碼嚴(yán)謹(jǐn)、清晰,我覺得這就很好。反之,某一些精巧的設(shè)計,可能會帶來可閱讀性和可理解性下降的問題。
尋找代替分支判斷的方式
接下去我們再來考慮怎么樣去重構(gòu)優(yōu)化過多的if-else分支。
程序邏輯最基本的組成就是分支、判斷和循環(huán)。而過多if-else正是由于在某一個變化的點(diǎn)上,有許多判斷條件和結(jié)果分支造成的。所以最基本的解決辦法就是把多個判斷條件合成一個,也就是把若干個分支合成一個。
但是在大多數(shù)情況下,條件判斷的分支都是無法合并的。所以,我們需要把這個變化點(diǎn)通過別的途徑封裝起來,而不是采用if-else。
1. 用一個Map可以做到,if-else的變化點(diǎn)使用Map的get方法來代替:
- Map typeCodeMap = new HashMap();
- typeCodeMap.put("Name", 0);
- typeCodeMap.put("Age", 1);
- typeCodeMap.put("Address", 2);
- ...
- int code = typeCode.get(type);
2. 枚舉:
- public enum Codes {
- Name(0), Age(1), Address(2);
- public int code;
- Codes(int code){
- this.code = code;
- }
- }
- //使用:
- int code = Codes.valueOf(str).code;
3. 多態(tài):
- ICode iCode = (ICode)Class.forName("com.xxx." + str).newInstance();
- int code = iCode.getCode();
當(dāng)然,如果僅考慮從String轉(zhuǎn)向int這樣的轉(zhuǎn)換,用這樣的方式來簡化分支判斷邏輯,這個方式、這個例子不是很恰當(dāng)。當(dāng)然,這樣的方式經(jīng)常被用來做從字符串到具體對象的轉(zhuǎn)換。
還有一些朋友說的這個模式那個模式來解決多if-else的問題,這些都是正確的,當(dāng)然本質(zhì)上也無一例外基于多態(tài)來實現(xiàn)的,所以我就不提及了。這些都不錯,至少比那些老說用switch來代替if-else的有價值多了 :)
最后,對于如此小的一個問題,我要補(bǔ)充說明的一點(diǎn)是,看不得大片if-else和看不得大片new關(guān)鍵字一樣,我覺得這是許多Java程序員的既有觀念或者說習(xí)慣,甚至通病——這并不好。Java最有價值的地方不是它的語義語法也不是它的虛擬機(jī)跨平臺和有多高性能,而在于它的社區(qū)它的無比豐富的類庫,在于使用它的人可以從設(shè)計上和宏觀上去思考問題。但是Java程序員,也包括我在內(nèi),很容易把這條路走得過于極端,比如遍地的Factory,比如漫山遍野的配置,比如永遠(yuǎn)也不會被復(fù)用的可復(fù)用代碼,比如永遠(yuǎn)也不會被擴(kuò)展的可擴(kuò)展代碼,還比如從前到后由內(nèi)到外的分層,一層又一層。相對于這些方面無止境的追求,我們還是專注于要解決的問題,多寫一些清晰可用的代碼吧。