スイッチコードの共通コードのリファクタリング

Question

switch (customerPaymentInfo.getPtType()) { 
     case CASH: 
      test(); 
      otherMethod1(); 
      break; 
     case CARD: 
      test(); 
      otherMethod2(); 
      break; 
     default: 
      throw new IllegalArgumentException("Payment Type is Not Correct."); 
    } 

上記のコードでは、CASHまたはCARDの両方の場合に1つの共通の方法を実行しています。

それを単一の時間を使用するようにスイッチケース内の任意の可能性はありますか？私たちは、コードを書くことができるブロックの場合の場合のeaxmpleについては

以下のもの：これにアプローチする1つの方法は、今switch

switch (customerPaymentInfo.getPtType()) { 
    case CASH: 
    case CARD: 
     test(); 
     break; 
    default: 
     throw new IllegalArgumentException("Payment Type is Not Correct."); 
} 

switch (customerPaymentInfo.getPtType()) { 
    case CASH: 
     otherMethod1(); 
     break; 
    case CARD: 
     otherMethod2(); 
     break; 
    default: 
     throw new IllegalArgumentException("Payment Type is Not Correct."); 
} 

のカスケードプロパティを使用することです

if (customerPaymentInfo.getPtType().equals("CASH") || customerPaymentInfo.getPtType().equals("CARD")) { 
    test(); 
} 

Answer 1

case CASH: 
    case CARD: 
     test(); 
     if(customerPaymentInfo.getPtType().equals("CASH")) 
      otherMethod1(); 
     else 
      otherMethod2(); 
     break; 

Answer 2

ただし、switchステートメントには冗長性があります。あなたは（多くのそのような例を持っている場合、最後に

if (!customerPaymentInfo.getPtType().equals(CASH) || !customerPaymentInfo.getPtType().equals(CARD)) 
    throw new IllegalArgumentException("Payment Type is Not Correct."); 
test(); 
switch(customerPaymentInfo.getPtType()) { 
    //switch code 
} 

、Java8のみsoltion、彼らは共通があります

もう一つの方法は、switch文を行う、その後test()を実行し、IllegalArgumentExceptionの最初のチェックになりますコールと特定のコール）をグループ化することができます。特定の部分については、EnumsとFunctionsのマップを使用できます。

switch (customerPaymentInfo.getPtType()) { 
    case CASH: 
    case CARD: 
     test(); 
     functionMap.get(customerPaymentInfo.getPtType()).apply(holderOfOtherMethod); 
     break; 
    default: 
     throw new IllegalArgumentException("Payment Type is Not Correct."); 
} 

Answer 3

おそらく、これを別の角度から見るとよいでしょう。なぜスイッチはありますか？ otherMethod1()とotherMethod2()のように、支払方法に依存してさまざまな方法で同じことをしています。

IイメージothMethod1()は、processPaymentByCash()とprocessPaymentByCard()のようになります。そして、実装の違いではなく、これらの支払タイプに対して異なるクラスによって処理されなければならない：

class PaymentCash extends Payment { 
    processPayment() { 
     test(); 
     // Code from othermethod1 
    } 
} 

class PaymentCard extends Payment { 
    processPayment() { 
     test(); 
     // Code from othermethod2 
    } 
} 

class PaymentWhatever extends Payment { 
    processPayment() { 
     throw new IllegalArgumentException("Payment Type is Not Correct."); 
    } 
} 

スイッチの上、単にこの1つのライナーに置き換えられます：今

customerPaymentInfo.getPtType().processPayment(); 

、あなたはまだ2を持っていますあなたのコードでtest()を呼び出しますが、これは本当にすべてがあなたのコードのより大きなコンテキストに依存しています。それはまた別の支払タイプのように見えます

ではなくenum値として実装する必要があります。