NO.57 代码优化初步 2.公共规则的抽取、可配置化

  有一天进行代码走查，发现这段代码出现在4个类中，一字不差——
public String changeCurrencyCode(String currencyCode){ String newCurrencyCode = currencyCode; if("CNY".equals(currencyCode)){//人民币 newCurrencyCode = "01"; } if("HKD".equals(currencyCode)){//港币 newCurrencyCode = "02"; } if("USD".equals(currencyCode)){//美元 newCurrencyCode = "03"; } …… if("NZD".equals(currencyCode)){//新西兰元 newCurrencyCode = "20"; } return newCurrencyCode; }
1.很明显：Ctrl+C，Ctrl+V的；类似的规则如果发生变化或者就是笔误哪个地方写错了，难道要去改4处？
2.同变量判断，却全用的if判断，把每种情况都变成最坏情况。

第1次改进：
Public Class CurrencyRules{ public static String changeCurrencyCode(String currencyCode){ String newCurrencyCode = currencyCode; if("CNY".equals(currencyCode)){//人民币 newCurrencyCode = "01"; }else if("HKD".equals(currencyCode)){//港币 newCurrencyCode = "02"; }else if("USD".equals(currencyCode)){//美元 newCurrencyCode = "03"; } …… }else if("NZD".equals(currencyCode)){//新西兰元 newCurrencyCode = "20"; } return newCurrencyCode; } }
1.使用了else if 做了一下小小的改进
2.方法改为static了，也把它放在一个规则类里了
但是不是感觉还是很笨拙？

第2次改进：
public final static Properties currencyProp = new Properties(); static { currencyProp.clear(); currencyProp.setProperty("CNY", "01"); currencyProp.setProperty("HKD", "02"); currencyProp.setProperty("USD", "03"); …… currencyProp.setProperty("NZD", "20"); } public static String changeCurrencyCode(String currencyCode){ String newCurrencyCode = currencyProp.getProperty(currencyCode); if(newCurrencyCode == null){//如果没有此映射 newCurrencyCode = currencyCode; } return newCurrencyCode; }
对于这种大量的常量映射，可以使用java.util.Properties
不过修改起来还是不方便...

第3次改进：  把这些配置信息放进一个CurrencyProp.properties文件中，文件和CurrencyRules.java放在同一个目录:
CNY=01 HKD=02 USD=03 …… NZD=20
然后那个CurrencyRules中的static块就变成这个样子——
public final static Properties currencyProp = new Properties(); static { currencyProp.load(CurrecnyRules.class.getResourceAsStream("CurrencyProp.properties")); } …
   至此，一段公共规则被抽取出来了，并且易于配置。
   不过如果希望规则修改后实时生效，可以放弃static方式而使用实例化方式，大家可以自己试着实现一下。