循環的複雑さを減らすために、複数のif文java

Question

私が含まれていないこれらのelse-if条件をもっとたくさん持っています。循環器系の複雑さを減らすために、これをどのようにリファクタリングすることができますか？私はJavaでこの関数の循環的複雑度を軽減することができますどのように

if (ONE.equalsIgnoreCase(eachTag.getNodeName())) 
{ 
    myclassDto.setOne(formatter 
      .getElementValueAfterNullCheckWithTrim((Element) eachTag)); 
} 
else if (TWO.equalsIgnoreCase(eachTag.getNodeName())) 
{ 
    myclassDto.setTwo(formatter 
      .getElementValueAfterNullCheckWithTrim((Element) eachTag)); 
} 
else if (THREE.equalsIgnoreCase(eachTag.getNodeName())) 
{ 
    myclassDto.setThree(formatter 
      .getElementValueAfterNullCheckWithTrim((Element) eachTag)); 
} 
else if (FOUR.equalsIgnoreCase(eachTag.getNodeName())) 
{ 
    myclassDto.setFour(formatter 
      .getElementValueAfterNullCheckWithTrim((Element) eachTag)); 
} 
else if (FIVE.equalsIgnoreCase(eachTag.getNodeName())) 
{ 
    myclassDto.setFive(formatter 
      .getElementValueAfterNullCheckWithTrim((Element) eachTag)); 
} 
else if (SIX.equalsIgnoreCase(eachTag.getNodeName())) 
{ 
    myclassDto.setSix(formatter 
      .getElementValueAfterNullCheckWithTrim((Element) eachTag)); 
} 

？

Answer 1

それはあなたが（それは本当に複雑ではないですが）、あれば同じように「複雑」だながら、あなたのコードは、読みやすく、次のようになります。

• は（GETNODENAMEの値を抽出し、switch文
• を使用します）

• 予めgetElementValueAfterNullCheckWithTrim（によって返される値を抽出する）

  String value = formatter.getElementValueAfterNullCheckWithTrim((Element) eachTag); 
  String nodeName = eachTag.getNodeName(); 
  
  switch (nodeName) { 
      case ONE: 
       myclassDto.setOne(value); 
       break; 
      case TWO: 
       myclassDto.setTwo(value); 
       break; 
      ... 
  } 
  

予め

編集： あなたのDTOをリファクタリングする場合がありますので、それは

myclassDto.setValue(nodeName, value) 

Answer 2

のように、私は彼らが対応関数に文字列のマッピングを定義したい、使用する方が簡単です。これは静的にすることができます。

次に、マッピングをループして、適切なもの（filter）を見つけて機能を適用することができます。

private static final Map<String, BiConsumer<MyClassDto, Element>> MAPPING = new HashMap(); 

static 
{ 
    mapping.put(ONE, MyClassDto::setOne); 
    mapping.put(TWO, MyClassDto::setTwo); 
    //... 
    mapping.put(SIX, MyClassDto::setSix); 
} 


//... 

MAPPING.entrySet().stream() 
    .filter(pair -> pair.getKey().equalsIgnoreCase(eachTag.getNodeName())) 
    .map(Map.Entry::getValue) 
    .forEach(func -> func.accept(
     myClassDto, 
     formatter.getElementValueAfterNullCheckWithTrim((Element) eachTag) 
    )); 

あなたはMAPPINGがequalsIgnoreCase（例えば「AAA」と「AAA」）によって平等に扱われるの異なる鍵を含めることができる場合を検討する必要があります。

forEach（daniuのように）の代わりにfindFirst().ifPresent()を使用するのが1つの解決策ですが、これはいくつかのエラーケースをマスクする可能性がありますので注意して使用してください。