Refaktoryzacja metody [Java]

Question

Da się to jakoś ładniej zapisać?

public AbstractLevel getLevel(Game game, int n){

        switch (n){
            case 1: return new Level1(game);

            case 2: return new Level2(game);

            case 3: return new Level3(game);

            case 4: return new Level4(game);

            case 5: return new Level5(game);

            case 6: return new Level6(game);

            case 7: return new Level7(game);

            case 8: return new Level8(game);

            case 9: return new Level9(game);

            case 10: return new Level10(game);

            case 11: return new Level11(game);

            case 12: return new Level12(game);

            case 13: return new Level13(game);

            case 14: return new Level14(game);

            case 15: return new Level15(game);

            //itd tak jakoś do 60 levela
        }
        return new Level1(game);
    }

 

Answer 1

Ja bym umieścił to w tablicy, zbiorze - > z elementem klucza tutaj skorzystaj z mapy.
Gdzie elementem kluczem-> będzie iterator :) o wartościach SWITCH(N). Rozwinę to.

Np. stworzysz osobną klase, zeby nadac w konstruktorze domyślnym -> inicjalizacje dla wszystkich mozliwosci tego zbioru i po prostu  w klasie implementujesz metode, która korzysta z tego zbioru, a w swoim kodzie tutaj wprowadzasz wywoływanie tej metody dla parametru który dostałeś :)

 

Comments

Dzięki za odpowiedź. Pierwszy raz mam do czynienia z tymi mapami. Jeżeli dobrze zrozumiałem to i tak żeby dodać to do mapy to muszę zrobić to tak:

map.put(1, new Level1(game));

map.put(2, new Level2(game));

itd.

co chyba jakimś lepszym rozwiązaniem nie jest?

Answer 2

Czy tworzenie osobnej klasy do każdego poziomu ma jakiś sens? Nie lepiej trzymać poziomy w plikach?

Comments

a kto mówi o tworzeniu klasy do każdego poziomu :D. Chyba że to nie retoryczne pytanie. Trzymanie w plikach, zależy jak duży mamy projekt, jak często wymieniamy dane na tych plikach - szanujmy pamięć ... i to zdecydowanie zwalnia  program i wydłuża jego czas pracy

---

return new Level14(game);

Javy nie znam, ale Level14 mi wygląda na klasę.

Uważasz więc że lepiej zaśmiecać kod milionami klas, każda dla jednego poziomu, niż po prostu trzymać informacje na temat każdego poziomu w plikach? Taki plik nie byłby duży i byłby wczytywany podczas wczytywania danego poziomu - nie wiem co tutaj ma aż tak znacznie wydłużać czas pracy.

---

Owszem każdy level to osobna klasa. Aplikacja śmiga na razie nawet na słabych smarfonach więc chyba nie widzę przeszkód. Dla mnie to wygodne że każdy level ma swoją klasę. Dużo więcej kodu to też nie jest bo praktycznie wszystko jest w klasie abstrakcyjnej, tu tylko ustawiam różne parametry i dodaje obiekty.

---

To moze dekorator:

https://pl.wikipedia.org/wiki/Dekorator_(wzorzec_projektowy)