Enum w Javie - switch case

Question

Cześć.

Potrzebuje zrobić coś takiego. Mam klasę enum w której są różne opcje:

public enum differentOptions {
    FIRST_OPTION(1),SECOND_OPTION(2);
     differentOptions(int option) {
    }
}

w drugiej klasie w metodzie chciałbym użyć tego w swithc case, czyli:

private void testMethod(int option)
{
      switch(option)
             case FIRST_OPTION:
                      jakieś instrukcje;
             case SECOND_OPTION:
                      jakieś instrukcje;
}

Chodzi mi własnie o to, żeby nie było magic numbers. 

Wie ktoś może jak to zrobić?

 

PS: Nie wiem czy ten enum jest poprawny. Ale ogólnie chodzi o to, że jak ktoś w menu wpisze 1, to żeby się odpaliła FIRST_OPTION ze switch case'a

Answer 1

differentOptions.values()[option]

to to, czego szukasz.

private void testMethod(int option)
{
      switch(differentOptions.values()[option])
             case FIRST_OPTION:
                      jakieś instrukcje;
             case SECOND_OPTION:
                      jakieś instrukcje;
}

Comments

IMO wygodniej i ładniej byłoby użycie stałych zamiast enuma.

---

Cos koło tego. Tylko mam rozumieć, że to jest tablica, a ona się zaczyna od 0, w wyniku czego opcja 0 jest opcją 1, opcja 1 opcją 2 etc.... Oczywiście mógłbym zrobić testmethod(3+1), jeśli chciałbym 4 opcję, ale czy jest jakiś sens tego?

Da się to obejść?

---

No właśnie dlatego sugeruję użycie stałych. Ładniej, wygodniej, bezpieczniej i  tak dalej

---

public class switchz {

    private static final int FIRST_OPTION =1 ;
    private static final int SECOND_OPTION =2 ;

    public void chooseOPption(int option) {
        switch (option) {
            case FIRST_OPTION:
                System.out.println("1 opcja");
                break;
            case SECOND_OPTION:
                System.out.println("2 opcja");
                break;
        }
    }
}

o coś takiego Ci chodzi?

---

Swoją drogą, takich opcji bedzie 11.

Powineinem więc stworzyć osobną klasę do tego?

---

dokładnie o coś takiego. I myślę, że można to wyabstrahować do osobnej klasy finalnej albo lepiej interfejsu, bo przecież nie chcesz instancji tego tworzyć. Nie jestem javovcem więc jak gdzieś tu walnąłem bzdurę to sory :P

---

W jakim sensie do interfejsu? Do klasy to jeszcze umiałbym zrobić, tylko własnie muszę instancję tworzyć.

Interfejs mogę zaimplementować, ale nie wiem jak mam zaimplementować ten interfejs.

---

http://www.javapractices.com/topic/TopicAction.do?Id=3

Możesz też zrobić tak:

http://www.javapractices.com/topic/TopicAction.do?Id=2

---

Nie wiem która opcja jest lepsza.

Czy importować je statycznie z tej klasy, czy pisać Options.FIRSTOPTION. Tych importów faktycznie będzie z 11, dlatego mam dylemat.