Czy da się uniknąć dużej liczby if-ów bądź switch-ów w wyborze poziomu?

Question

Witam wszystkich, chcę poprawić swój kod dotyczący wyboru poziomów, gdyż mam gigantycznego switcha, który składa się z 18 case-ów. Aktualnie działa to tak, że na każdy poziomie czyszczona jest lista w wrogami, dodaje wrogów do listy i broń jaka jest na danym poziomie, po zabiciu wszystkich wrogów, wywołuje funkcję klasy i przechodzę na case 2 i tak do case 18. Nie wiem, czy intuicja mi dobrze mówi, ale niezbyt dobrze taki switch wygląda. Dla poglądu sytuacji wrzucę kilka case-ów, żeby nie wrzucić 250 linii kodu

 case 8:
                    {
                        Enemies.Clear();
                        Enemies.Add(new Wizard(this, GetRandomLocation(random)));
                        if(CheckPlayerInventory("Bow"))
                            WeaponInRoom = new Quiver(this, GetRandomLocation(random));
                        break;
                    }
                case 9:
                    {
                        Enemies.Clear();
                        Enemies.Add(new Ghost(this, GetRandomLocation(random)));
                        Enemies.Add(new Ghoul(this, GetRandomLocation(random)));
                        if (CheckPlayerInventory("Bow"))
                            WeaponInRoom = new Shield(this, GetRandomLocation(random), 5);
                        else
                            WeaponInRoom = new Bow(this, GetRandomLocation(random));
                        break;
                    }
                case 10:
                    {
                        Enemies.Clear();
                        Enemies.Add(new Wizard(this, GetRandomLocation(random)));
                        Enemies.Add(new Bat(this, GetRandomLocation(random)));
                        if (CheckPlayerInventory("Red potion"))
                            WeaponInRoom = new BattleAxe(this, GetRandomLocation(random));
                        else
                            WeaponInRoom = new RedPotion(this, GetRandomLocation(random));
                        break;
                    }
                case 11:
                    {
                        Enemies.Clear();
                        Enemies.Add(new Wizard(this, GetRandomLocation(random)));
                        Enemies.Add(new Ghost(this, GetRandomLocation(random)));
                        if (CheckPlayerInventory("Battle axe"))
                            ;
                        else
                            WeaponInRoom = new BattleAxe(this, GetRandomLocation(random));
                        break;
                    }
                case 12:
                    {
                        Enemies.Clear();
                        Enemies.Add(new Bat(this, GetRandomLocation(random)));
                        Enemies.Add(new Ghost(this, GetRandomLocation(random)));
                        Enemies.Add(new Ghoul(this, GetRandomLocation(random)));
                        if (CheckPlayerInventory("Bow"))
                        {
                            if (CheckPlayerInventory("Quiver"))
                                ;
                            else
                                WeaponInRoom = new Quiver(this, GetRandomLocation(random));
                        }
                        break;
                    }

Chciałbym uprościć nieco logikę i zastanawiałem się nad funkcjami dla każdego poziomiu typu void, ale jestem zbyt cienki w uszach i jeszcze i być może czegoś nie widzę, dlatego chciałbym się spytać, co o tym sądzicie, czy może są jakieś lepsze rozwiązania na takie gigantyczne switche? A może utworzyć oddzielną klasę dla każdego z poziomu? Z góry dziękuję za odpowiedź.

 

Comments

Tak, możesz to rozbić, tworząc 2 metody - jedna do tworzenia przeciwników w zależności od poziomu, a druga do tworzenia ekwipunku w zależności od poziomu. Nie wiem, jak wygląda całość twojej gry, ale jeśli np. łuk jest na 2 poziomie, a potion na 3. To może warto stworzyć sobie enuma z ekwipunkiem na dany poziom.

Answer 1

Napewno możesz wyciągnąć Enemies.Clear() jeżeli i tak robisz to za każdym razem.

Co do przeciwników to ja bym zrobił tablicę vector'ów z x vector'ami, gdzie x to liczba poziomów.

Każdy vector przechowywałby wskaźniki na konstruktory i wtedy możnaby przed switch'a wyciągnąć coś takiego:

for(int i=0;i<EnemiesVectors[level].Length;i++){
    Enemies.Add(EnemiesVectors[level][i](this,GetRandomLocation(random)));
}

Co do broni to napewno dałoby się wymyśleć jakąś zależność.

Answer 2

Hej! Oczywiście, że da się to wszystko ładnie posprzątać, i przede wszystkim uniknąć takiej złożonej struktury ifów. Jednak jest to temat złożony i trzeba by przeanalizować całą aplikację, być może problem zaczyna się gdzieś wcześniej ze złą organizacja. Nie jestem w stanie dać Ci konkretnego rozwiązania "zrób to i to", tak jak pisałem to może być bardzo złożone.

Jest taka naprawdę bardzo dobra książka "Clean Code", jest też Polskie wydanie Czysty Kod (polecam jednak angielską wersję), która właśnie mówi o tym jak refakturować kod aby pozbywać się takich struktur.