Iterator + Methode

[Stargirlxo]

Hallo,
ich bräuchte bitte Hilfe zu einer Aufgabe, wo ich leider gar nicht weiter komme.
Sie lautet:
"Schreiben Sie für die Klasse Videospiel eine Methode figurenAktualisieren(), in der mit einem Iterator über alle Objekte in figuren iteriert wird, um jede Figur zu bewegen (s.o.). Wenn es sich bei dem Objekt um einen Gegner handelt, so soll vor dem Bewegen geprüft werden, ob der Gegner verletzt ist (Bei einem Gegner kann man mit der Methode boolean istVerletzt() abfragen, ob er durch deine Kugel verletzt worden ist).
Ist dies der Fall so soll das Objekt aus der ArrayList gelöscht werden."

Ich habe bis jetzt das aufgeschrieben gehabt:
public void figurenAktualisieren(){

Iterator <Spielfigur> it = figuren.iterator();
while(it.hasNext() ){

if(it.Next() instance of Gegner && boolean istVerletzt() = true){
it.remove();

}}}

Leider weiß ich nicht genau, wo ich die Methode istVerletzt() hinschreiben soll, weil ich vermute der Code wird so nicht funktionieren bzw. ich bin mir unsicher, da ich leider nicht so stark in Java bin.
Ich bitte um Feedback danke

Mfg.

 

[fhoffmann]

Bei einem Iterator solltest du dir das Objekt einmal holen, also:

while(it.hasNext() ) {
  Spielfigur spielfigur = it.next();
  // ...
}

Falls diese Spielfigur ein Gegner ist, kannst du dazu casten und darauf istVerletzt() aufrufen.

 

[Stargirlxo]

Bei einem Iterator solltest du dir das Objekt einmal holen, also:

while(it.hasNext() ) {
  Spielfigur spielfigur = it.next();
  // ...
}

Falls diese Spielfigur ein Gegner ist, kannst du dazu casten und darauf istVerletzt() aufrufen.

Danke also ich muss nur "Spielfigur spielfigur = it.next();" hinzufügen und davor schreiben und der Code ist richtig?
Und müsste dann nicht in meinem Code "gegner" durch spielfigur ersetzt werden?

 

[kneitzel]

Nein, Dein Code wird dadurch nicht wirklich besser. Statt weiterer next() Aufrufe, hast Du die Variable zu nutzen.
Und wenn die Spielfigur vom Typ Gegner ist, dann musst du da einen cast auf Gegner machen um dann die Methode auf dem Gegner aufzurufen.

 

[Stargirlxo]

Nein, Dein Code wird dadurch nicht wirklich besser. Statt weiterer next() Aufrufe, hast Du die Variable zu nutzen.
Und wenn die Spielfigur vom Typ Gegner ist, dann musst du da einen cast auf Gegner machen um dann die Methode auf dem Gegner aufzurufen.

Wird wohl doch schwieriger als ich gedacht habe.. dachte bin mit meinem Code schon halbwegs richtig und für n Anfänger habe ich mit Cast usw. noch immer meine Probleme
Spielfigur spielfigur = it.next();
Gegner spielfigur = (Gegner)spielfigur;

Komme jetzt aber auch nicht mehr weiter, wäre cool wenn hier irgendjemand ist der mir den korrekten Code zeigen könnte

 

[fhoffmann]

Spielfigur spielfigur = it.next();
if (spielfigur instanceof Gegner) {
    Gegner gegner = (Gegner)spielfigur;
    // ...
}

 

[Stargirlxo]

Spielfigur spielfigur = it.next();
if (spielfigur instanceof Gegner) {
    Gegner gegner = (Gegner)spielfigur;
    // ...
}

Also nochmal ich weiß ja dann ob er Gegner ist, würde ich dann im nächsten Schritt einfach danach dann schreiben können

if (Gegner.istVerletzt() ){
it.remove();
}

und der Code wäre somit komplett dann oder?

 

[kneitzel]

Und an der Stelle ist ja in der Aufgabe schon etwas vorgegeben:

(Bei einem Gegner kann man mit der Methode boolean istVerletzt() abfragen, ob er durch deine Kugel verletzt worden ist).

Ein Gegner hat also eine Methode istVerletzt() - eine Methode auf einer Instanz aufrufen hast Du ja schon gemacht ... z.B. next auf der Instanz in it...

Ist dies der Fall so soll das Objekt aus der ArrayList gelöscht werden.

Das wird dann ggf. noch problematisch. Denn eine ArrayList mag keine Veränderungen so lange ein Iterator aktiv ist. Daher solltest Du Dir einmal den Iterator in Ruhe ansehen, was der noch so an Methoden liefert.

Und da sich das nun überschnitten hat:

Also nochmal ich weiß ja dann ob er Gegner ist, würde ich dann im nächsten Schritt einfach danach dann schreiben können

if (Gegner.istVerletzt() ){
it.remove();
}

und der Code wäre somit komplett dann oder?

Fast. Gegner ist aber die Klasse - du willst aber doch die Instanz ...
Der zweite Punkt von mir ist aber damit schon erfüllt!

 

[Stargirlxo]

So das wäre jetzt mein fertiger Code.. diese Aufgabe zerbricht mir echt mein Kopf gerade, danke für die Hilfe alleine hätte ich es nicht geschafft.

public void figurenAktualisieren(){

Iterator <Spielfigur> it = figuren.iterator();
while(it.hasNext() ){

Spielfigur spielfigur = it.next();
if(spielfigur instanceof Gegner) {
Gegner gegner = (Gegner)spielfigur;

if(spielfigur.istVerletzt() )
it.remove();
}
}
}

 

[kneitzel]

Falls Spielfigur die Methode istVerletzt() nicht implementiert bekommst Du aber eine Fehlermeldung.

Wenn Du keine Fehlermeldung bekommst, kannst Du die Zeile
Gegner gegner = (Gegner)spielfigur;
auch löschen. Die wird dann nicht mehr benötigt. Die haben wir ja eingebracht, damit Du dann den Aufruf auf dieser Variable durchführen kannst.

 

[Stargirlxo]

Falls Spielfigur die Methode istVerletzt() nicht implementiert bekommst Du aber eine Fehlermeldung.

Wenn Du keine Fehlermeldung bekommst, kannst Du die Zeile
Gegner gegner = (Gegner)spielfigur;
auch löschen. Die wird dann nicht mehr benötigt. Die haben wir ja eingebracht, damit Du dann den Aufruf auf dieser Variable durchführen kannst.

Ach so gut zu wissen, danke für die Hilfe nochmal