Remove from an AraayList

[Java-Problems]

Has been deleted!

 

[Ebenius]

Bei allen datenhaltenden Klassen solltest Du ─ bzw. Du musst, wenn Du sie vernünftig mit Collections verwenden können willst ─ equals() und hashCode() überschreiben. Außerdem sollte man auch immer toString() überschreiben, um vernünftige Ausgaben machen zu können.

 

[Java-Problems]

sind diese Methoden nicht schon vordefiniert?
Warum sollte ich die verändern?
Wie kann mir das bei meinem Problem mit dem Löschen eines Objektes helfen?

 

[Der Müde Joe]

>sind diese Methoden nicht schon vordefiniert?

werden von Object definiert, müssen von dir überschrieben werden

>Warum sollte ich die verändern?

damit du zB list.contains(rezept) schreiben kannst, oder list.remove(rezept), Der Rest erledigt dir die Liste: sie sucht anhand equals bzw hashcode das passende Objekt, welches identisch bzw den gleichen hashCode hat:
zB:

while(list.contains(rezept) { list.remove(rezept)}

Entferne alle vorkommen von rezept inder Liste

 

[Ebenius]

sind diese Methoden nicht schon vordefiniert?

Ja. Sie überprüfen auf Instanzgleichheit (also auf Identität). Sie sollen aber auf inhaltliche Gleichheit prüfen. Und das kann die Standard-Implementierung natürlich nicht gewährleisten. Steht auch in der API-Doc mit drinnen.

Warum sollte ich die verändern?

Dafür sind sie gedacht.

Wie kann mir das bei meinem Problem mit dem Löschen eines Objektes helfen?

Habe gerade nochmal in Deinem Code oben nachgelesen. In dem Beispiel aus der main()-Methode hilft es Dir nicht, weil Du tatsächlich die gleiche Instanz wieder löschst. :-( Trotzdem solltest Du's machen. Weil's besser ist.

Zum ganz konkreten Problem: Lass Dir doch in der Schleife (in remove) mal ausgeben, ob ein Rezept gefunden wurde...

Zum Programm: Da ergibt einiges keinen Sinn. Zum Beispiel die Methode remove(Recipe)... Wieso bekommt die ein Recipe übergeben? Du benutzt es gar nicht, sondern fragst den Nutzer irgendwelche Sachen. Das ergibt keinen Sinn. Selbst wenn Du den Recipe-Parameter wegnehmen würdest, wäre es hässlicher Stil, da eine datenhaltende Klasse keine Eingaben vom Nutzer verlangt. Das macht man außerhalb der Klasse.

 

[Java-Problems]

Entferne alle vorkommen von rezept inder Liste

Wo soll ich die entfernen?
Und wie kann ich die Methode überschreiben bzw. was genau muss dort rein in den Body der Methode?

 

[SlaterB]

in diesem Fall funktioniert also bereits das remove(),
nur gibts du (genau wie im letzten Thread) gar nicht den Inhalt der Liste aus, sondern irgendwas anderes, nämlich das Recipe selber,

davon, dass das Recipe erfolgreich aus einer Liste entfernt wurde, ändert sich doch nicht die Ausgabe das Recipe selber..,
du musst z.B. die Anzahl der Elemente der Liste ausgeben

 

[Java-Problems]

Zum Programm: Da ergibt einiges keinen Sinn. Zum Beispiel die Methode remove(Recipe)... Wieso bekommt die ein Recipe übergeben? Du benutzt es gar nicht, sondern fragst den Nutzer irgendwelche Sachen. Das ergibt keinen Sinn. Selbst wenn Du den Recipe-Parameter wegnehmen würdest, wäre es hässlicher Stil, da eine datenhaltende Klasse keine Eingaben vom Nutzer verlangt. Das macht man außerhalb der Klasse.

Sorry, aber so wurde die Methode vorgegeben und diese darf ich nicht verändern...
Die Eingaben etc. mach ich also in der main-Methode, oder wie?

 

[SlaterB]

bei add hattest du es doch früher auch so:
Parameter ignorieren, innerhalb von add() neues Recipe abfragen per IOTools,
das war auch komisch und inzwischen hast du es teilweise geändert:
nun wird in add() das übergebene Recipe genutzt, nur noch seltsamerweise an dieser Stelle nach den Zutaten gefragt,

warum machst du das bei remove() nicht genauso?
verwende das übergebe Recipe statt innerhalb von remove() den User per IOTools zu fragen

-----

die IOTools-Aufrufe innerhalb des Recipe-Konstruktors sind natürlich fast noch schlimmer,
besser wäre eine spezielle Methode, nicht direkt in der main, sondern eine Methode createReceipeAskingUser(),

allerdings möchtest du zum remove() eine andere Frage stellen, das wäre dann etwas aufwendig,
erstmal eine Methode
createReceipeToRemoveAskingUser()
dann das Recipen an remove() übergeben, dann dort wie bisher den Namen rausholen und manuell mit allen Recipes
in der Liste vergleichen oder doch besser equals() überschreiben (für Prinzipienreiter auch hashcode())

etwas zu aufwendig, besser wäre für remove() dann vielleicht eher, einen String name zu übergeben
oder den Namen doch in einer speziellen remove()-Methode abzufragen,

aber du hast ja, wie du sein 20 Threads erwähnst, deine vorgegeben Interface..
Vorgaben, deren Sinn man nicht versteht, sind wirklich hinderlich..