Problem bei eshop -> Ausführen einer Methode über eine Shopverwaltung

[Fischi2k]

Hallo zusammen,

für die Uni müssen wir einen E-Shop erstellen. Hierbei nutzen wir:
Klasse CUI zur Erfassung der Nutzereingaben auf der Konsole
Klasse Shopmanager, die die Anfragen der CUI koordiniert an...
Klassen Artikelmanager, Kundenmanager, Mitarbeitermanager usw.

Folgendes Problem habe ich und kann es mir nicht erklären:

In der CUI folgende Abfrage (Auszug):

if (cmd.equalsIgnoreCase("abe"))
		{								
				System.out.println("Änderung des Bestands vornehmen: ");
				System.out.print("Artikelnummer eingeben: ");
				System.out.flush(); 
				String Artikelnummer = in.readLine();
				int aNr = Integer.parseInt(Artikelnummer);
			
				System.out.print("Menge eingeben (+ Bestand erhöhen / - Bestand verringern): ");
				System.out.flush(); 
				String betrag = in.readLine();
				int anzahl = Integer.parseInt(betrag);
			
				boolean ok = shopV.bestandAendern (aNr, anzahl);
				System.out.println("Änderung " +(ok? " erfolgreich":" gescheitert"));

		}

Oben: Weiterleitung der Eingaben die Methode bestandAendern in der Shopverwaltung.

Auszug aus der Shopverwaltung:

	public boolean bestandAendern(int aNr, int anz) {

		return artV.bestandAendern(aNr, anz);
		
	}

Oben: Einfach Weiterleitung an die Artikelverwaltung

In der Artikelverwaltung:

	public boolean bestandAendern(int aNr, int anz)
	{		
		Artikel a = map.get(aNr);		
		int tmp = a.getBst() + anz;
		a.setBst(tmp);
		return true;
	}

Der Bestand der Artikel wird nicht geändert. Rufe ich von der CUI die Artikelverwaltung direkt auf und wähle nicht den Weg über die Shopverwaltung, klappt es problemlos. Ich kann mir beim besten Willen nicht erklären, woran es liegen könnte.

Vielleicht habt Ihr einen Tipp für mich.

Vielen Dank vorab schon mal.

 

[Joose]

Greifst du möglicherweise auf das Falsche Objekt zu?
Mit diesen Codestücken lässt sich kein Fehler finden. Die Klassen ShopVerwaltung bzw. ArtikelVerwaltung wären hier noch interessant

 

[Fischi2k]

Hier die beiden Klassen.

Inwiefern auf das falsche Objekt zugreifen?
Ich bin mir sicher, ich stehe auf dem Schlauch, aber ich begreife es einfach nicht, wieso eine einfache Methode in der Shopverwaltung, die nur als Weiterleitung dient, solche Fehler macht.

Ich hoffe, Du kannst mir weiterhelfen.

Danke schon mal fürs Reinschauen.

 

[Joose]

Ich hoffe, Du kannst mir weiterhelfen.

Danke schon mal fürs Reinschauen.

Wirklichen Fehler sehe ich jetzt keinen.
Wie arbeitet denn die Klasse CUI? Wird hier jedes Mal ein neues Objekt der ShopVerwaltung erstellt?

Inwiefern auf das falsche Objekt zugreifen?

Viele Leute die mit OOP beginnen machen oft den Fehler das sie 2 Objekte einer Klasse erstellen.
Dann ändern sie im 1.Objekt etwas und wollen es sich dann anzeigen lassen. Zum Anzeigen greifen sie aber auf das zweiter (unveränderte) Objekt zu.

 

[Fischi2k]

Das seltsame ist ja, wenn ich hier

		if (cmd.equalsIgnoreCase("abe"))
		{
			
					
				System.out.println("Änderung des Bestands vornehmen: ");
				System.out.print("Artikelnummer eingeben: ");
				System.out.flush(); 
				String Artikelnummer = in.readLine();
				int aNr = Integer.parseInt(Artikelnummer);
			
				System.out.print("Menge eingeben (+ Bestand erhöhen / - Bestand verringern): ");
				System.out.flush(); 
				String betrag = in.readLine();
				int anzahl = Integer.parseInt(betrag);
			
				boolean ok = shopV.bestandAendern (aNr, anzahl);
				System.out.println("Änderung " +(ok? " erfolgreich":" gescheitert"));

		}

das Aufrufen der Methode so verändere:

boolean ok = artV.bestandAendern (aNr, anzahl);

Also direkt an die Artikelverwaltung funktioniert.

Als Anhang die NewCUI.java

 

[Joose]

Wie gedacht du hast 2 verschiedene Objekte vom Typ Artikelverwaltung. Bei dem 1. änderst du den Bestand von Artikel x, von dem 2. Objekt lässt du dir den Artikel (natürlich unverändert) anzeigen.

Das Problem ist im Konstruktor von NewCUI erstellst du ein Objekt der Klasse Artikelverwaltung, im Konstruktor von Shopverwaltung erstellst du ebenfalls ein Objekt der Klasse Artikelverwaltung. Änderungen an dem einen Objekt werden nicht auf das andere übertragen!

Lösung: Statt in NewCUI ein Objekt von Artikelverwaltung zu halten, hole dir die Artikelverwaltung per Getter von der Shopverwaltung.

 

[Fischi2k]

So, nach vielem Probieren und Probieren und Probieren mit dem Gettern und Settern habe ich es tatsächlich zum Laufen bekommen.

Du bist heute ganz klar mein Held. Selbst unser Prof. konnte uns vorhin nicht weiterhelfen.

Problem wurde gelöst!!!

Vielen, vielen Dank!!!