實戰|如何消除又臭又長的if...else判斷更優雅的編程

最近在作代碼重構,發現了不少代碼的爛味道。其餘的很少說,今天主要說說那些又臭又長的if...else要如何重構。java

在介紹更更優雅的編程以前,讓咱們一塊兒回顧一下,很差的if...else代碼spring

1、又臭又長的if...else

廢話很少說,先看看下面的代碼。編程

public interface IPay {

    void pay();
}


@Service
public class AliaPay implements IPay {


    @Override
    public void pay() {
        System.out.println("===發起支付寶支付===");
    }
}


@Service
public class WeixinPay implements IPay {

    @Override
    public void pay() {
        System.out.println("===發起微信支付===");
    }
}


@Service
public class JingDongPay implements IPay {

    @Override
    public void pay() {
        System.out.println("===發起京東支付===");
    }
}

@Service
public class PayService {

    @Autowired
    private AliaPay aliaPay;
    @Autowired
    private WeixinPay weixinPay;
    @Autowired
    private JingDongPay jingDongPay;


    public void toPay(String code) {
        if ("alia".equals(code)) {
            aliaPay.pay();
        } else if ("weixin".equals(code)) {
            weixinPay.pay();
        } else if ("jingdong".equals(code)) {
            jingDongPay.pay();
        } else {
            System.out.println("找不到支付方式");
        }
    }
}
複製代碼

PayService類的toPay方法主要是爲了發起支付,根據不一樣的code,決定調用用不一樣的支付類(好比:aliaPay)的pay方法進行支付。設計模式

這段代碼有什麼問題呢?也許有些人就是這麼幹的。微信

試想一下,若是支付方式愈來愈多,好比:又加了百度支付、美團支付、銀聯支付等等,就須要改toPay方法的代碼,增長新的else...if判斷,判斷多了就會致使邏輯愈來愈多?網絡

很明顯,這裏違法了設計模式六大原則的:開閉原則 和 單一職責原則。 ❝開閉原則:對擴展開放,對修改關閉。就是說增長新功能要儘可能少改動已有代碼。app

單一職責原則:顧名思義,要求邏輯儘可能單一,不要太複雜,便於複用。ide

那有什麼辦法能夠解決這個問題呢? post

2、使用註解

代碼中之因此要用code判斷使用哪一個支付類,是由於code和支付類沒有一個綁定關係,若是綁定關係存在了,就能夠不用判斷了。性能

咱們先定義一個註解。

@Retention(RetentionPolicy.RUNTIME)
@Target(ElementType.TYPE)
public @interface PayCode {

    String value();

    String name();
}
複製代碼

而後在全部的支付類上都加上註解

@PayCode(value = "alia", name = "支付寶支付")
@Service
public class AliaPay implements IPay {


    @Override
    public void pay() {
        System.out.println("===發起支付寶支付===");
    }
}



@PayCode(value = "weixin", name = "微信支付")
@Service
public class WeixinPay implements IPay {

    @Override
    public void pay() {
        System.out.println("===發起微信支付===");
    }
}


@PayCode(value = "jingdong", name = "京東支付")
@Service
public class JingDongPay implements IPay {

    @Override
    public void pay() {
        System.out.println("===發起京東支付===");
    }
}
複製代碼

而後增長最關鍵的類:

@Service
public class PayService2 implements ApplicationListener<ContextRefreshedEvent> {

    private static Map<String, IPay> payMap = null;

    @Override
    public void onApplicationEvent(ContextRefreshedEvent contextRefreshedEvent) {
        ApplicationContext applicationContext = contextRefreshedEvent.getApplicationContext();
        Map<String, Object> beansWithAnnotation = applicationContext.getBeansWithAnnotation(PayCode.class);

        if (beansWithAnnotation != null) {
            payMap = new HashMap<>();
            beansWithAnnotation.forEach((key, value) -> {
                String bizType = value.getClass().getAnnotation(PayCode.class).value();
                payMap.put(bizType, (IPay) value);
            });
        }
    }

    public void pay(String code) {
        payMap.get(code).pay();
    }
}
複製代碼

PayService2類實現了ApplicationListener接口,這樣在onApplicationEvent方法中,就能夠拿到ApplicationContext的實例。咱們再獲取打了PayCode註解的類,放到一個map中,map中的key就是PayCode註解中定義的value,跟code參數一致,value是支付類的實例。

這樣,每次就能夠每次直接經過code獲取支付類實例,而不用if...else判斷了。若是要加新的支付方法,只需在支付類上面打上PayCode註解定義一個新的code便可。

注意:這種方式的code能夠沒有業務含義,能夠是純數字,只有不重複就行。

3、動態拼接名稱

再看看這種方法,主要針對code是有業務含義的場景。

@Service
public class PayService3 implements ApplicationContextAware {

    private ApplicationContext applicationContext;

    private static final String SUFFIX = "Pay";

    @Override
    public void setApplicationContext(ApplicationContext applicationContext) throws BeansException {
        this.applicationContext = applicationContext;
    }

    public void toPay(String payCode) {
        ((IPay) applicationContext.getBean(getBeanName(payCode))).pay();
    }

    public String getBeanName(String payCode) {
        return payCode + SUFFIX;
    }

}
複製代碼

咱們能夠看到,支付類bean的名稱是由code和後綴拼接而成,好比:aliaPay、weixinPay和jingDongPay。這就要求支付類取名的時候要特別注意,前面的一段要和code保持一致。調用的支付類的實例是直接從ApplicationContext實例中獲取的,默認狀況下bean是單例的,放在內存的一個map中,因此不會有性能問題。

特別說明一下,這種方法實現了ApplicationContextAware接口跟上面的ApplicationListener接口不同,是想告訴你們獲取ApplicationContext實例的方法不僅一種。

4、模板方法判斷

固然除了上面介紹的兩種方法以外,spring的源碼實現中也告訴咱們另一種思路,解決if...else問題。

咱們先一塊兒看看spring AOP的部分源碼,看一下DefaultAdvisorAdapterRegistry的wrap方法

public Advisor wrap(Object adviceObject) throws UnknownAdviceTypeException {
    if (adviceObject instanceof Advisor) {
      return (Advisor) adviceObject;
    }
    if (!(adviceObject instanceof Advice)) {
      throw new UnknownAdviceTypeException(adviceObject);
    }
    Advice advice = (Advice) adviceObject;
    if (advice instanceof MethodInterceptor) {
      // So well-known it doesn't even need an adapter.
      return new DefaultPointcutAdvisor(advice);
    }
    for (AdvisorAdapter adapter : this.adapters) {
      // Check that it is supported.
      if (adapter.supportsAdvice(advice)) {
        return new DefaultPointcutAdvisor(advice);
      }
    }
    throw new UnknownAdviceTypeException(advice);
  }
複製代碼

重點看看supportAdvice方法,有三個類實現了這個方法。咱們隨便抽一個類看看

class AfterReturningAdviceAdapter implements AdvisorAdapter, Serializable {

  @Override
  public boolean supportsAdvice(Advice advice) {
    return (advice instanceof AfterReturningAdvice);
  }

  @Override
  public MethodInterceptor getInterceptor(Advisor advisor) {
    AfterReturningAdvice advice = (AfterReturningAdvice) advisor.getAdvice();
    return new AfterReturningAdviceInterceptor(advice);
  }

}
複製代碼

該類的supportsAdvice方法很是簡單,只是判斷了一下advice的類型是否是AfterReturningAdvice。

咱們看到這裏應該有所啓發。

其實,咱們能夠這樣作,定義一個接口或者抽象類,裏面有個support方法判斷參數傳的code是否本身能夠處理,若是能夠處理則走支付邏輯。

public interface IPay {

    boolean support(String code);

    void pay();
}


@Service
public class AliaPay implements IPay {

    @Override
    public boolean support(String code) {
        return "alia".equals(code);
    }

    @Override
    public void pay() {
        System.out.println("===發起支付寶支付===");
    }
}


@Service
public class WeixinPay implements IPay {


    @Override
    public boolean support(String code) {
        return "weixin".equals(code);
    }

    @Override
    public void pay() {
        System.out.println("===發起微信支付===");
    }
}


@Service
public class JingDongPay implements IPay {
    @Override
    public boolean support(String code) {
        return "jingdong".equals(code);
    }

    @Override
    public void pay() {
        System.out.println("===發起京東支付===");
    }
}
複製代碼

每一個支付類都有一個support方法,判斷傳過來的code是否和本身定義的相等。

@Service
public class PayService4 implements ApplicationContextAware, InitializingBean {

    private ApplicationContext applicationContext;

    private List<IPay> payList = null;

    @Override
    public void afterPropertiesSet() throws Exception {
        if (payList == null) {
            payList = new ArrayList<>();
            Map<String, IPay> beansOfType = applicationContext.getBeansOfType(IPay.class);

            beansOfType.forEach((key, value) -> payList.add(value));
        }
    }

    @Override
    public void setApplicationContext(ApplicationContext applicationContext) throws BeansException {
        this.applicationContext = applicationContext;
    }

    public void toPay(String code) {
        for (IPay iPay : payList) {
            if (iPay.support(code)) {
                iPay.pay();
            }
        }
    }

}
複製代碼

這段代碼中先把實現了IPay接口的支付類實例初始化到一個list集合中,返回在調用支付接口時循環遍歷這個list集合,若是code跟本身定義的同樣,則調用當前的支付類實例的pay方法。

5、其餘的消除if...else的方法

固然實際項目開發中使用if...else判斷的場景很是多,上面只是其中幾種場景。下面再列舉一下,其餘常見的場景。

1.根據不一樣的數字返回不一樣的字符串

public String getMessage(int code) {
    if (code == 1) {
        return "成功";
    } else if (code == -1) {
        return "失敗";
    } else if (code == -2) {
        return "網絡超時";
    } else if (code == -3) {
        return "參數錯誤";
    }
    throw new RuntimeException("code錯誤");
}
複製代碼

其實,這種判斷沒有必要,用一個枚舉就能夠搞定。

public enum MessageEnum {

    SUCCESS(1, "成功"),
    FAIL(-1, "失敗"),
    TIME_OUT(-2, "網絡超時"),
    PARAM_ERROR(-3, "參數錯誤");

    private int code;
    private String message;

    MessageEnum(int code, String message) {
        this.code = code;
        this.message = message;
    }

    public int getCode() {
        return this.code;
    }

    public String getMessage() {
        return this.message;
    }

    public static MessageEnum getMessageEnum(int code) {
        return Arrays.stream(MessageEnum.values()).filter(x -> x.code == code).findFirst().orElse(null);
    }
}
複製代碼

再把調用方法稍微調整一下

public String getMessage(int code) {
    MessageEnum messageEnum = MessageEnum.getMessageEnum(code);
    return messageEnum.getMessage();
}
複製代碼

完美。

2.集合中的判斷

上面的枚舉MessageEnum中的getMessageEnum方法,若是不用java8的語法的話,可能要這樣寫

public static MessageEnum getMessageEnum(int code) {
    for (MessageEnum messageEnum : MessageEnum.values()) {
        if (code == messageEnum.code) {
            return messageEnum;
        }
    }
    return null;
}
複製代碼

對於集合中過濾數據,或者查找方法,java8有更簡單的方法消除if...else判斷。

public static MessageEnum getMessageEnum(int code) {
    return Arrays.stream(MessageEnum.values()).filter(x -> x.code == code).findFirst().orElse(null);
}
複製代碼

3.簡單的判斷

其實有些簡單的if...else徹底沒有必要寫,能夠用三目運算符代替,好比這種狀況:

public String getMessage2(int code) {
    if(code == 1) {
        return  "成功";
    }
    return "失敗";
}
複製代碼

改爲三目運算符:

public String getMessage2(int code) {
    return code == 1 ? "成功" : "失敗";
}
複製代碼

修改以後代碼更簡潔一些。

4.判斷是否爲null

java中自從有了null以後,不少地方都要判斷實例是否爲null,否則可能會出現NPE的異常。

public String getMessage2(int code) {
      return code == 1 ? "成功" : "失敗";
  }

  public String getMessage3(int code) {
      Test test = null;
          return test.getMessage2(1);
  }
複製代碼

這裏若是不判斷異常的話,就會出現NPE異常。咱們只能老老實實加上判斷。

public String getMessage3(int code) {
    Test test = null;
    if (test != null) {
        return test.getMessage2(1);
    }
    return null;
}
複製代碼

有沒有其餘更優雅的處理方式呢?

public String getMessage3(int code) {
    Test test = null;
    Optional<Test> testOptional = Optional.of(test);
    return testOptional.isPresent() ? testOptional.get().getMessage2(1) : null;
}
複製代碼

答案是使用Optional

固然,還有不少其餘的場景能夠優化if...else,我再這裏就不一一介紹了,感興趣的朋友能夠給我留言,一塊兒探討和研究一下。

做者:蘇三說技術 連接:https://juejin.cn/post/6903430159567847432 來源:掘金 著做權歸做者全部。商業轉載請聯繫做者得到受權,非商業轉載請註明出處。
相關文章
相關標籤/搜索