Альтернатива вложенным операторам в Java

Итак, сегодня я написал метод, который включал использование вложенных операторов switch, и код выглядел довольно чистым и сжатым для меня, но мне сказали, что вложенные операторы switch обычно не лучший способ, поскольку они могут запутать с более активными инструкциями переключателей. Вот пример моего кода:

EnumOne enumOne;
EnumTwo enumTwo = null;
EnumTwo enumThree = null;

switch (enumOne) {
   case CASE_ONE:

      switch (enumTwo){
         case A: enumTwo = EnumTwo.B; break;
         case C: enumTwo = EnumTwo.D; break;
         default: break;
      }

      switch (enumThree) {
         case AA: enumThree = EnumTwo.BB; break;
         case CC: enumThree = EnumTwo.DD; break;
         default: break;
      }

      break;

   case CASE_TWO:
   case CASE_THREE:

      switch(EnumTwo) {
         default: break;
      }

      switch (enumThree) {
         case AA: enumThree = EnumTwo.XX; break;
         case CC: enumThree = EnumTwo.YY; break;
         default: break;
      }

      break;

   default:
      break;
}

Итак, мой вопрос, по сути, был бы подходящей альтернативой этим операторам switch?

Ответ 1

Я рекомендую заменить каждый вложенный оператор switch вызовом процедуры, которая затем выполняет вложенный код коммутатора.

Вместо этого напишите что-то вроде этого:

    EnumOne enumOne;
    EnumTwo enumTwo = null;
    EnumTwo enumThree = null;

    switch (enumOne)
    {
       case CASE_ONE:

          nested_switch1();

       case CASE_TWO:
       case CASE_THREE:

          nested_switch2();

          break;

       default:
          break;
    }

    nested_switch1() {
          switch (enumTwo)
          {
             case A:
                enumTwo = EnumTwo.B;
                break;
             case C:
                enumTwo = EnumTwo.D;
                break;
             default:
                break;
          }

          switch (enumThree)
          {
             case AA:
                enumTwo = EnumTwo.BB;
                break;
             case CC:
                enumTwo = EnumTwo.DD;
                break;
             default:
                break;
          }

          break;
    }

nested_switch2() {
          switch(EnumTwo)
          {
             default:
                break;
          }

          switch (enumThree)
          {
             case AA:
                enumTwo = EnumTwo.XX;
                break;
             case CC:
                enumTwo = EnumTwo.YY;
                break;
             default:
                break;
          }
}

Ответ 2

Поскольку использование большого количества переключателей становится довольно трудным для чтения.
И в любое время, когда возникает новый случай, мы должны изменить код и добавить CASE

мы можем рассмотреть использование polymorphism в таких случаях

Я собираюсь дать простой класс, чтобы вы поняли. Предположим, что ранее класс с корпусом коммутатора

class Test
{ 
    Animal a;
    public Test(Animal a)
    { 
        this.a=a;
    }

    public moveThisAnimal()
    {
        switch(this.a)
        {
            case fish:
            System.out.println("swim");
            break;

            case dog:
            System.out.println("walk");
            break;

            case bird:
            System.out.println("fly");
            break;
        }
    }
}

теперь мы заменим этот коммутатор нашей логикой полиморфизма

Interface Animal
{
    String move();
} 

Class Dog implements Animal
{
    public String move()
    {
        return "walk";
    }
}


Class Bird implements Animal
{
    public String move()
    {
        return "fly";
    }
}


Class Fish implements Animal
{
    public String move()
    {
        return "swim";
    }
}

теперь у нас есть класс Test без случая переключения

class Test
{ 
    Animal a;
    public Test(Animal a)
    { 
        this.a=a;
    }
    public moveThisAnimal()
    {
        System.out.println(this.a.move()); // all switch case statements removed 
    }
}

и даже если нам нужно добавить дополнительные случаи, нам нужно просто добавить реализации без изменений здесь

Посмотрите ваш полный код и посмотрите, можно ли сделать

Ответ 3

Если у вас есть целые числа X и Y, и вам нужно включить их, вы можете объединить их недвусмысленным образом и включить комбинацию. Например, если y < 10:

switch (x*10+y)
{
case 0: // x == y == 0
case 1: // x ==0, y == 1
///
case 10: // x == 1, y == 0
case 11: // x == y == 1
//
}