Fehlersuche

[DaSt]

Hallo,
ich habe hier eine Prüfung aus dem letzten Jahr und in folgendem Code ist (noch) ein Fehler, den ich nicht finde.. könnt ihr mir helfen.

import java.util.ArrayList;
import java.util.List;

public class Main {
   public static void main(String[] args) {
     List<Integer> list = new ArrayList<>();

     for(Integer element :list){
       if(element>10){
         list.remove(element);
         System.out.println("Removed");
       } 
     }
   }
}

Also ich würde sagen, die if-Abfrage stimmt so nicht, da die Variable "element" ja ein Objekt ist und ich das mit einer int Zahl vergleiche.

und bei list.remove() muss man doch die Stelle übergeben, die man löschen will oder?

Danke

 

[Flown]

Also die Fehler die ich hier sehe sind:
- Liste ist leer
- ConcurrentModificationException, weil du während der Iteration ein Element löscht (würde man mit einem Iterator vermeiden)

Deine Aussagen sind allesamt falsch:
- if passt (da Autoboxing)
- list.remove(int) würde an der Position löschen list.remove(Object) resp. list.remove(Integer) löscht das Objekt, aber nicht an der aktuellen Stelle, sondern an der Ersten.
- Exception würde fliegen, wegen siehe oben!

Aber was war die eigentliche Prüfungsfrage?

 

[DaSt]

Aber was war die eigentliche Prüfungsfrage?

"in Folgender Klasse sind zwei Fehler enthalten. Nennen Sie die beiden Fehler und begründen Sie ihre Wahl"

der Originalcode lautet:

import java.util.List;

public class Main {
   public static void main(String[] args) {
     List<Integer> list =new Int[10];
     for(Integer element :list){
       if(element>10){
         list.remove(element);
         System.out.println("Removed");
       }
     }
   }
}

 

[Flown]

Ja das Erste ist klar, das es erstens kein Typ Int gibt und zweitens, dass diese nicht einer List zugeordnet werden können.

Das andere wie schon gesagt würde dir eine ConcurrentModificationException werfen (nicht in allen Fällen, aber in den meisten!).

Beispiel mit Exception:

List<Integer> list = new ArrayList<>(Arrays.asList(1, 10, 20, 30, 100));

for (Integer element : list) {
  if (element > 10) {
    list.remove(element);
    System.out.println("Removed");
  }
}

Beispiel ohne Exception:

List<Integer> list = new ArrayList<>(Arrays.asList(1, 10, 20, 30));

for (Integer element : list) {
  if (element > 10) {
    list.remove(element);
    System.out.println("Removed");
  }
}

EDIT: Das letzte Beispiel liefert dir zwar keinen Fehler, aber hat unvorhersehbare Ergebnisse, die auch falsch sind

 

[Thallius]

Ja das Erste ist klar, das es erstens kein Typ Int gibt und zweitens, dass diese nicht einer List zugeordnet werden können.

Das andere wie schon gesagt würde dir eine ConcurrentModificationException werfen (nicht in allen Fällen, aber in den meisten!).

Beispiel mit Exception:

List<Integer> list = new ArrayList<>(Arrays.asList(1, 10, 20, 30, 100));

for (Integer element : list) {
  if (element > 10) {
    list.remove(element);
    System.out.println("Removed");
  }
}

Beispiel ohne Exception:

List<Integer> list = new ArrayList<>(Arrays.asList(1, 10, 20, 30));

for (Integer element : list) {
  if (element > 10) {
    list.remove(element);
    System.out.println("Removed");
  }
}

Erklärst Du mir gerade mal den Unterschied zwischen dem ersten und dem zweiten Beispiel? Kann sein das es an dem fehlenden Kaffee liegt aber ich sehe da einfach keinen.

Gruß

Claus

 

[Elenteria]

- list.remove(int) würde an der Position löschen list.remove(Object) resp. list.remove(Integer) löscht das Objekt, aber nicht an der aktuellen Stelle, sondern an der Ersten.

hier muss ich Flown wiedersprechen. list.remove(Integer) ist aufgrund des Autoboxings mit list.remove(int) gleichzusetzten und löscht das Element am Index welcher im Integer steht. Dadurch wäre in deinem Code auch noch eine potenzielle ArrayIndexOutOfBoundsException vorhanden.

 

[Flown]

Erklärst Du mir gerade mal den Unterschied zwischen dem ersten und dem zweiten Beispiel? Kann sein das es an dem fehlenden Kaffee liegt aber ich sehe da einfach keinen.

Hat nur was mit den Values zu tun. Bei der einen fliegt die CME bei der anderen nicht.

hier muss ich Flown wiedersprechen. list.remove(Integer) ist aufgrund des Autoboxings mit list.remove(int) gleichzusetzten und löscht das Element am Index welcher im Integer steht. Dadurch wäre in deinem Code auch noch eine potenzielle ArrayIndexOutOfBoundsException vorhanden.

Nope es wird definitiv list.remove(Object) aufgerufen!

Es würde das remove mit dem Index aufgerufen werden, wenn du in der foreach Autoboxing hättest: for(int element : list) {...}. Aber nachdem Integer ein Object ist, wird die andere Methode bedient!

 

[Thallius]

Hat nur was mit den Values zu tun. Bei der einen fliegt die CME bei der anderen nicht.

Soll heissen ich kann dem Iterator sein aktuelles Objekt unter dem Arsch wegziehen wenn es nicht das letzte ist?

 

[Joose]

Soll heissen ich kann dem Iterator sein aktuelles Objekt unter dem Arsch wegziehen wenn es nicht das letzte ist?

Dachte auch nicht das es geht, habe es aber gerade ausprobiert und es stimmt wirklich.

Hat nur was mit den Values zu tun. Bei der einen fliegt die CME bei der anderen nicht.

Meiner Meinung nach funktioniert es eben nur wenn der vorletzte Value entfernt wird. Daher finde ich es ungeeigent zu sagen das man einfach "remove" aufrufen kann in einer foreach Schleife.

 

[Flown]

Es funktioniert auch nur, wenn das vorletzte Element gelöscht wird!
Eigentlich wollte ich damit ausdrücken, dass man ohne Iterator unvorhergesehene Ergebnisse (auch ohne Fehlermeldung) erhalten kann und wollte verdeutlichen, dass das ein Fehler ist!

Soll heissen ich kann dem Iterator sein aktuelles Objekt unter dem Arsch wegziehen wenn es nicht das letzte ist?

Sein letztes Element, nicht das aktuelle, aber ja!