Collections Frage Beispielprogrammierung Iterator

[mdey86]

Hallo zusammen,

ich habe eine Frage bezüglich eines Programmierbeispiel im Rahmen meines Fernstudiums. Gegeben ist folgender Beispieltext

public void bereinigeAlleKunden(Collection<Kunde> c) {
  Kunde k = null;
  for (Iterator<Kunde> it = c.iterator(); it.hasNext(); k = it.next()) {
    if (!k.getGeschlecht().equals("")) {
      it.remove();
    }
  }
}

Meine Frage: Wieso ist in der If-Schleife vor k.getGeschlecht ein Ausrufezeichen? Normalerweise bedeutet das doch eine Negation, was in diesem Zusammenhang keinen Sinn machen würde... Oder stehe ich auf dem Schlauch?

 

[tommysenf]

k.getGeschlecht().equals("") --> Wenn das Geschlecht ein leerer String ist
!k.getGeschlecht().equals("") --> Wenn das Geschlecht KEIN leerer String ist

 

[VfL_Freak]

Moin,

also ich würde auf den Schlauch tippen

'!' steht für not
ergo: wenn das Geschlecht von k nicht leer ist, dann .....

Wobei mir der Code auch nicht vollständig zu sein scheint, da die Zuweisung zu k fehlt !

BTW: Code-Tags sind nicht zum Spaß erfunden worden .....

Gruß Klaus

 

[tommysenf]

Wobei mir der Code auch nicht vollständig zu sein scheint, da die Zuweisung zu k fehlt !

for (Iterator<Kunde> it = c.iterator(); it.hasNext(); k = it.next())

 

[VfL_Freak]

oops
doch wohl zu warm heute ..... mea maxima culpa !

 

[Flown]

Sowas sieht normalerweise so aus:

public void bereinigeAlleKunden(Collection<Kunde> c) {
  for (Iterator<Kunde> it = c.iterator(); it.hasNext();) {
    Kunde k = it.next();
    if (k.getGeschlecht().isEmpty()) {
      it.remove();
    }
  }
}

Der erste Kunde wird übersprungen, da der letzte Teil einer for-Schleife immer am Ende einer Iteration ausgeführt wird.

 

[mdey86]

Aber das macht doch dann keinen Sinn, oder? Das würde ja bedeuten, dass wenn das Geschlecht leer ist, das Objekt gelöscht wird, oder liege ich da falsch. Eigentlich würde bei einer Bereinigung ja eher Sinn machen, das Objekt zu löschen wenn das Geschlecht leer ist und das ist das was mich verwirrt...

Oder ist es wirklich nur das Wetter

 

[mdey86]

Also ich meinte jetzt die Posts vor Flown

 

[JStein52]

Der erste Kunde wird übersprungen

Liegt es alles nur am Wetter, ich versuchs auch mal: eigentlich sollte doch gleich im ersten Durchlauf eine NullPointerException kommen weil k dann null ist oder ?

 

[JStein52]

Eigentlich würde bei einer Bereinigung ja eher Sinn machen, das Objekt zu löschen wenn das Geschlecht leer ist

Wie denn nun ? bei @Flown wird gelöscht wenn das Geschlecht leer ist, in deinem Beispielcode wurde gelöscht wenn das Geschlecht nicht leer ist. Was soll denn passieren ?

 

[flopalko]

Liegt es alles nur am Wetter, ich versuchs auch mal: eigentlich sollte doch gleich im ersten Durchlauf eine NullPointerException kommen weil k dann null ist oder ?

Sehe ich auch so, das increment der Schleife wird ja erst NACH dem 1. Durchlauf angewandt.

Wie denn nun ? bei @Flown wird gelöscht wenn das Geschlecht leer ist, in deinem Beispielcode wurde gelöscht wenn das Geschlecht nicht leer ist. Was soll denn passieren ?

Keine Ahnung, das erwäht der TE nicht. Sinnvoller wäre es, Kunden ohne Geschlecht (wobei heutzutage wer weiß^^) zu löschen, als Kunden mit Geschlecht. Im Originalcode werden aber alle Kunden mit Geschlecht gelöscht.

 

[Flown]

Hängt ihr euch jetzt wirklich alle auf, ob eine Bedingung verneint wird oder nicht? Ich wollte eigentlich zeigen, wie man das aktuelle Element aus dem Iterator erhält und wie man es verarbeitet.

 

[flopalko]

Nein ist mir vollkommen egal, ich wollte es nur für den TE erklären bzw. wissen was er will.
Eine Frage hätte ich auch noch an den TE, @mdey86: Ist dieser Code der eine NPE wirft tatsächlich von deiner Fernuni gegeben? Falls ja, auweh wer unterrichtet denn da bitte?^^

 

[JStein52]

Hängt ihr euch jetzt wirklich alle auf, ob eine Bedingung verneint wird oder nicht

Eigentlich ist es mir auch egal aber das mit der Verneinung war nun mal die ursprüngliche Frage von @mdey86