Probleme mit HashMap und Klassenobjekte

[Korkzapfen]

Hallo zusammen

Als blutiger Anfänger besuche ich einen Java-Kurs an der Universität. Im Rahmen der Vorlesung müssen wir verschiedene kleine Projekte programmieren. Genau an einem solchen Projekt bin ich nun stehengeblieben. Hier kurz mein Problem:

Ich habe zwei Klassen (Verwaltung und Lagerposition). Die Klasse Lagerposition dient zur Erstellung von Lagerpositionen, jede Lagerposition besitzt Informationen über Artikel, Bestand, Hersteller und besitzt eine Positionsnummer, welche als Schlüsselwert in einer HashMap dienen soll. Die Klasse Verwaltung soll u.a. eine Methode haben, mit welcher man alle Lagerpositionen und die dazugehörigen Attribute auf der Konsole ausgeben kann. Ich habe eine solche Methode "positionenAuflisten" entworfen, jedoch bekomme ich auf der Konsole nur eine Lagerposition und deren Attribute (und zwar mehrfach, also bei 3 Lagerpositionen 3mal) obwohl ich mehr als eine Lagerposition angelegt habe.

Kann mir jemand weiterhelfen? Im Anhang findet Ihr die zwei txt-files Verwaltung und Lagerpositon. Die Methode "positionenAuflisten" befindet sich in der Klasse "Verwaltung".

Besten Dank im voraus & freundliche Grüsse
Alex

    public void positionenAuflisten()
    {
        // Auflisten der Einträge mithilfe von entryset()
        Set Inhalte = Sammlung.entrySet();
        Iterator it = Inhalte.iterator();
        while(it.hasNext()) {
            System.out.println(lager.positionsnummer + " | " + lager.bezeichnung + " | " + lager.artikelnummer + " | " + lager.hersteller);
            it.next();
        }
    }

 

[Marco13]

EDIT: Sorry, bin mit den Typen durcheinader gekommen ... gibt gleich ein update...

EDIT2: Also.. die HashMap bildet von einem "int" auf ein "Lagerposition" ab... Dann müßte der Code jetzt stimmen ... so solltest dir aber mal "Generics" ansehen..

Das "lager" das dort verwendet wird ist wohl eine Variable, die oben in der Klasse steht (eine Instanzvariable)?

 public void positionenAuflisten()
    {
        // Auflisten der Einträge mithilfe von entryset()
        Set Inhalte = Sammlung.entrySet();
        Iterator it = Inhalte.iterator();

        Lagerposition lagerposition = null; // Lokale Variable

        while(it.hasNext()) 
        {
            // Den nächsten Eintrag aus dem Iterator holen
            Map.Entry entry = (Map.Entry)it.next();

            // Der lokalen Variablen den Wert des Eintrags zuwiesen
            lagerposition = (Lagerposition)entry.getValue();

            System.out.println(lagerposition.positionsnummer + " | " + lagerposition.bezeichnung + " | " + lagerposition.artikelnummer + " | " + lagerposition.hersteller);
        }
    }

 

[Korkzapfen]

Hallo Marco

Ich habe deine Code übernommen und lagerposition mit lager ersetzt (lagerposition existiert nicht). Noch eine kleine letzte Frage:
Wieso setzt du lagerposition (also lager) = null?

Besten Dank für deine Hilfe!
Alex

 

[SlaterB]

macht hier keinen Unterschied, allgemein kann es aber helfen

String x;
if (y) {
  x = "z";
}
System.out.println(x); // Compilerfehler, Variable möglicherweise nicht initialisiert

----

String x = null;
if (y) {
  x = "z";
}
System.out.println(x); // Ausgabe null oder "z"

 

[Marco13]

Wobei man in diesem Fall auch einfach hätte schreiben können

 public void positionenAuflisten()
    {
        // Auflisten der Einträge mithilfe von entryset()
        Set Inhalte = Sammlung.entrySet();
        Iterator it = Inhalte.iterator();
 
        while(it.hasNext()) 
        {
            // Den nächsten Eintrag aus dem Iterator holen
            Map.Entry entry = (Map.Entry)it.next();
 
            // Der lokalen Variablen den Wert des Eintrags zuwiesen
            Lagerposition lagerposition = (Lagerposition)entry.getValue();
 
            System.out.println(lagerposition.positionsnummer + " | " + lagerposition.bezeichnung + " | " + lagerposition.artikelnummer + " | " + lagerposition.hersteller);
        }
    }