Рефакторинг нескольких Else-if в Sonar Qube - PullRequest
Новая
       7

Рефакторинг нескольких Else-if в Sonar Qube

0 голосов
/ 08 января 2020

Ниже кода, который у меня есть, есть несколько других, если условия. Из-за этого, когда я запускаю код с использованием сонарного куба, он говорит: «Измените этот метод, чтобы уменьшить его когнитивную сложность с 31 до 15 разрешенных». Может ли кто-нибудь помочь мне уменьшить когнитивную сложность с 31 до 15. 

private final String method1(){

            if(!StringUtils.isEmpty(stringvariable)){
                if(stringvariable.equals("str1") || stringvariable.equals("str2") || stringvariable.equals("str3") ){
                    Stringvariable1 = "val1";
                }
                else if(stringvariable.equals("str4") || stringvariable.equals("str5") ){
                    Stringvariable1 = "val2";
                }
                else if(stringvariable.equals("str6") || stringvariable.equals("str7") || stringvariable.equals("str8")){
                    Stringvariable1 = "val3";
                }
                else if(stringvariable.equals("str9") || stringvariable.equals("str10") || stringvariable.equals("str11")){
                    Stringvariable1 = "val4";
                }
                else if(stringvariable.equals("str12") || stringvariable.equals("str13") || stringvariable.equals("str14")){
                    Stringvariable1 = "val5";
                }
                else if(stringvariable.equals("str15")){
                    Stringvariable1 = "val6";
                }
                else if(stringvariable.equals("str16") || stringvariable.equals("str17") || stringvariable.equals("str18") || stringvariable.equals("str19")){
                    Stringvariable1 = "val7";
                }
                else if(stringvariable.equals("str20") || stringvariable.equals("str21") ||stringvariable.equals("str22")){
                    Stringvariable1 = "val8";
                }
                else if(stringvariable.equals("str23") || stringvariable.equals("str24") || stringvariable.equals("str25") || stringvariable.equals("str26")){
                    Stringvariable1 = "val9";
                }
                else if(stringvariable.equals("str27") || stringvariable.equals("str28")){
                    Stringvariable1 = "val10";
                }
                else if(stringvariable.equals("str29") || stringvariable.equals("str30")){
                    Stringvariable1 = "val11";
                }
                else if(stringvariable.equals("str31")){
                    Stringvariable1 = "val12";
                }
                else if(stringvariable.equals("str32")){
                    Stringvariable1 = "val13";
                }
                else if(stringvariable.equals("str33") || stringvariable.equals("str34") || stringvariable.equals("str35") || stringvariable.equals("str36")){
                    Stringvariable1 = "val14";
                }
                else if(stringvariable.equals("str37")){
                    Stringvariable1 = "val15";
                }
                else if(StringUtils.isEmpty(stringvariable) ){
                    Stringvariable1 = "val16";
                }
                else {
                    Stringvariable1 = "val17";
                }
            }
            else{
                Stringvariable1 = "val18";
            }

        return Stringvariable1;
    }

1 Ответ

0 голосов
/ 08 января 2020

Если вы хотите сопоставить несколько наборов строк для каждого случая, вам, вероятно, будет лучше использовать какую-то функцию отображения, которая имеет Set<String> совпадающих значений, а затем соответствующий результат. 

Map<Set<String>, String> mappedOutcomes = new HashMap<>();
Set<String> matches = new HashSet<String>();
matches.add("match1");
mathces.add("match2);
mappedOutcomes.add(mySet, "str1");

String stringVariable = "somequery";
for(Map.Entry<Set, String> entry in mappedOutcomes.entrySet()){
  // Bear in mind case-sensitivity here for hashset, you may want to convert all strings to lower/upper case
  if(entry.getKey().contains(stringVariable)){
    return entry.getValue();
  }
}

// Else if you get here, no match was found

Вы могли бы go сделать еще один шаг и написать собственный класс-обертку, содержащий ваш набор совпадений и результат, а затем просто сохранить список того, чем является ваш объект. Это было бы лучше, чем перебирать набор записей Map.

...