Menüauswahl per Enum

[sMuve]

Hallo liebe Java-Freunde,

ich hänge momentan bei einer Aufgabe fest. Ich weiß allerdings nicht ob ich grade einfach nicht verstehe, was genau verlangt wird oder ich einfach keine realisierbare Lösung finde.

Bevor ich mit meinem Problem beginne, möchte ich noch sagen, dass ich nur die Grundlagen der OOP kenne (Vererbung, Interfaces und co.), da ich mich grade am Anfang meines Studiums befinde.

Wir sollen für einen Shop ein Enum erstellen, dass quasi die bisherige Menüauswahl (List, Register, Adjust_stock, exit) ersetzt.

Das würde dann ja quasi wie folgt aussehen:

public enum MenuOption {
LIST, REGISTER, ADJUST_STOCK, EXIT;
}

Nun ist aber folgendes gefragt:

Implementieren Sie in MenuOption eine Methode optionForNumber, welche einem Parameter vom Typ int ein EnumObjekt zuordnet.

Wie ordne ich also einem Parameter vom Typen int ein EnumObjekt per Methode zu ?
Ich weiß zwar, dass man den Enums einen beliebigen Index zuweisen kann indem man hinter das Enum in () eine Zahl schreibt, jedoch ist hier ja gefordert, dass man eine Methode nutzen soll.

 

[JStein52]

So was:

static int optionForNumber(String name ){
  try{
    return MenuOption.valueOf( name ).ordinal();
  }
  catch(IllegalArgumentException e ){
    return–1;
  }}

 

[JStein52]

Oder so:

        public enum MenuOption {a, b;
            static public MenuOption optionForNumber(int id) {
                if (a.ordinal() == id)
                    return a;
                else if (b.ordinal() == id)
                    return b;
                else
                    return null;
            }
        }

Aber auch da verstehe ich den Sinn nicht.

Obwohl .... du kannst jetzt MenuOption.optionForNumber(2) aufrufen und erhätlst die zugehörige Option. Also das was verlangt war

 

[sMuve]

Naja als Parameter soll ja nen int-Wert übergeben werden und dem ein EnumObjekt zugewiesen werden, daher wäre der 2. Vorschlag näher dran denke ich. Das ganze soll so gemacht werden weil wir danach folgendes machen sollen

Passen Sie nun die main-Methode der Klasse ProductManager so an, dass intern
MenuOption-Objekte statt ints verwendet werden, die Funktionalität aber unverändert
bleibt. Verwenden Sie hierbei die optionsForNumber-Methode, um die eingegebenen
int-Werte umzuwandeln.

Vorher hatten wir im ProductManager:

public class ProductManager {

    public static final int LIST = 0;
    public static final int REGISTER = 1;
    public static final int ADJUST_STOCK = 2;
    public static final int EXIT = 3;

    /**
     * Provides a command line interface to interact with the product management
     * system for an online shop
     *
     * @param args
     */
    public static void main(String[] args) {
        Warehouse warehouse = new Warehouse();
        warehouse.registerProducts(Warehouse.preloadInitialDataSet());
        System.out.println("Welcome to the online shop product management");
        boolean running = true;
        while (running) {
            System.out.println("Please, enter one of the following commands:" + "\n0 to list all products"
                    + "\n1 to register a new product" + "\n2 to adjust stock levels" + "\n3 to exit.");
            int command = IOTools.readInt();
            switch (command) {
                case LIST :
                    warehouse.printInventory();
                    break;
                case REGISTER :
                    Product p;
                    switch (IOTools.readChar("Do you want to add a book (b) or a television (t)?")) {
                        case 'b' :
                        case 'B' :
                            p = Book.createBookFromUserinput();
                            break;
                        case 't' :
                        case 'T' :
                            p = Television.createTelevisionFromUserinput();
                            break;
                        default :
                            continue;
                    }
                    warehouse.registerProducts(p);
                    System.out.println("The following product has been added to the catalog:\n" + p.toString());
                    break;
                case ADJUST_STOCK :
                    int id = IOTools.readInt("Please, enter product id:");
                    int stock = IOTools.readInt("Please, enter new stock level:");
                    warehouse.setStockForProductById(id, stock);
                    break;
                case EXIT :
                    running = false;
                    break;
                default :
                    System.out.println("Unknown command: " + command);
                    break;
            }
        }
        System.out.println("Byebye!");
    }
}

Somit müssten wir dann ja

  int command = IOTools.readInt();

ändern und die Klassenvariablen, welche wir durch Enums ersetzt haben, weglassen

 

[JStein52]

Ja, ok. Dann sollte es die zweite Methode sein !!

 

[sMuve]

okay, hab es so hinbekommen

Da ich allerdings 5 "if-Fälle" habe, wäre hier noch die Frage ob man das gescheit mit nem switchCase machen könnte.
Die ordinal von enum beginnen ja immer mit 0, also würde nur sowas wie

switch(selection){
case 0:
return LIST; break;
case 1:
return REGISTER; break;
etc.....}

möglich sein oder ?
Damit würde die Methode ja immer noch den übergebenen int-Wert in ein Enum-Objekt vom Typ MenuOption umwandeln oder ?

 

[JStein52]

Das mit dem Switch hatte ich auch überlegt aber nicht gewusst wie das zu bewerkstelligen ist. Du musst doch nacheinander die ordinal-Zahl der einzelnen Options abfragen ??!! Und ich weiss nicht ob man vorraussetzen kann dass die von 0 ... irgendwas lückenlos durchnummeriert sind ? Was ist denn selection bei dir ?

 

[sMuve]

    public static MenuOption operationForNumber(int selection)
    {
        switch(selection){
        case 0:
            return LIST; break;
        case 1:
            return REGISTER; break;
        case 2:
            return ADJUST_STOCK; break;
        case 3:
            return SHIP; break;
        case 4:
            return EXIT; break;
        }

selection ist also der int-Wert, den die Methode übergeben kriegt.

Die Methode funktioniert auch von 0-4 ohne Probleme, allerdings nur wenn ich die breaks weglasse, welche aber da sein sollten ^^

Enums hatte ich ja grad erst, aber so wie ich es der Vorlesung entnehmen konnte, beginnen die ordinal-Werte immer bei 0 und werden dann immer mit n+1 fortgesetzt, so dass man von einer lückenlosen Durchnummerierung ausgehen kann.

 

[JStein52]

Ja ok. Dann passt es so. Da vor dem break eh ein return steht ist es egal. Aber ein default mit return null solltest du schon einbauen. Was ist wenn jemand die Methode mit dem Wert 10 aufruft ?

 

[sMuve]

Was passiert, wenn man einen Wert außerhalb des Enum Bereiches aufruft wird nirgends beschrieben. Meine Idee war, dafür einfach ein SysO mit "Wert nicht vorhanden" oder ähnlichem einzubauen. Allerdings ist unser return Typ ja MenuOption und somit kein SysO möglich oder ?

EDIT:

    public static MenuOption operationForNumber(int selection)
    {
        switch(selection){
        case 0:
            return LIST; break;
        case 1:
            return REGISTER; break;
        case 2:
            return ADJUST_STOCK; break;
        case 3:
            return SHIP; break;
        case 4:
            return EXIT; break;
        default:
            return null;
        }

Alle break; werden als unreachable markiert von eclipse. Allerdings ist der Syntax so doch korrekt.

 

[JStein52]

Du kannst schon was mit System.out.... ausgeben. Aber der Aufrufer deiner Methode sollte ja auch in dem Fall einen definierten Wert zurückkriegen, deshalb return null;

 

[sMuve]

Es läuft nun alles Die breaks sind unreachable, weil man ja durch das return schon aus dem Block raus kommt ^^

Danke für die Hilfe!

 

[X5-599]

Noch als weitere Möglichkeit. Man könnte es auch so machen:

public enum MenuOption {
    LIST, REGISTER, ADJUST_STOCK, EXIT;
   
    public static MenuOption optionForNumber(int i)
    {
        MenuOption[] v = MenuOption.values();
        return ((i >= 0) && (i < v.length)) ? v[i] : null;
    }
}

Will heißen, man kann sich von einer Enum-Klasse alle Instanzen in einem Array geben lassen. (EnumKlasse.values())

System.out.println(MenuOption.optionForNumber(0));
System.out.println(MenuOption.optionForNumber(1));
System.out.println(MenuOption.optionForNumber(2));
System.out.println(MenuOption.optionForNumber(3));
System.out.println(MenuOption.optionForNumber(4));

Ausgabe:
LIST
REGISTER
ADJUST_STOCK
EXIT
null

Falls ihr noch keinen Ternary Operator hattet, oder du ihn nicht schön findest, kann man es natürlich auch so schreiben: (Der Ternary Operator ist die: BOOLEAN_AUSDRUCK ? DANN_DAS : SONST_DAS Anweisung)

public static MenuOption optionForNumber(int i) {
    MenuOption[] v = MenuOption.values();
    if((i >= 0) && (i < v.length))
    {
        return v[i];
    }
    return null;
}