Eintrag aus ArrayList entfernen

[Herr Kaiser]

Hallo Forum,

ich müsste noch'mal was erklärt haben. Ich habe folgende zweidimensionale ArrayList:

ArrayList<ArrayList<String>> al

ich möchte den Eintrag mit den Indizes x und y (beides int) entfernen und schreibe daher:

al.get(x).remove(y)

wenn ich nach al.get(x) den Punkt eingebe, bietet mir die IDE remove(Object o) und remove(int index) an. Ich wähle natürlich die letztere Variante, da ich nicht über den Inhalt sondern über den Index auf den zu löschenden Eintrag zugreifen will und y ein int ist.

Trotzdem beanstandet NetBeans:

"Suspicious call to java.util.Collection.remove: Given object cannot contain instances of int (expected String)"

Ist das logisch???

 

[Haave]

Merkwürdige Warnung. Kannst du eventuell mehr Code posten?

Abseits davon ist es egal (zumindest), ob du bei der Codevervollständigung der IDE remove(Object o) oder remove(int index) auswählst, da ja der Compiler entscheidet, ob es klappt oder nicht, und bei einem int als Parameter sollte sich der Compiler eigentlich für remove(int) entscheiden.

 

[Sym]

Das ist nur ein Warnung, oder?

Scheint jedenfalls ein Netbeans-Feature zu sein: Java Hints - NetBeans Wiki

 

[Herr Kaiser]

@ Haave: Mein eigener Code ist vermutlich zu lang und unübersichtlich, man kann die Warnung allerdings schon mit folgendem simplen Code provozieren:

package test;

import java.util.ArrayList;

public class Main {
    static ArrayList<ArrayList<String>> al;

    public static void main(String[] args) {
        al.add(new ArrayList<java.lang.String>());
        al.get(0).add("Hallo");
        al.get(0).remove(0);
    }
}

@ Sym: Ja, es ist nur eine Warnung, die die betreffende Stelle im Quellcode markiert. Eine Exception wird nicht geschmissen. Heißt das, dass ich den Hinweise ignorieren kann, oder könnte ich damit Probleme an anderer Stelle provozieren?

Danke schonmal für eure Hinweise

 

[eRaaaa]

In deinem Fall nicht, aufpassen musst du nur wenn du Integers in deinen Listen hältst beispielsweise
Die Meldung kannst du aber auch in Netbeans ausstellen wenn sie dich stört, einfach auf das Warnschild klicken ;D

 

[Herr Kaiser]

Ah, I see!

dann würde in folgendem Code:

int a=0;
int b=1;
Integer c=1;
al.add(new ArrayList<java.lang.Integer>());
...
al.get(a).remove(b);
al.get(a).remove(c);

remove(b) den Eintrag mit dem Index "1" entfernen, während remove(c) den ersten Eintrag mit dem Wert "1" entfernt (oder?)

Dann verstehe ich die Warnung also nur als Aufforderung, noch einmal bewusst zu überlegen, ob man wirklich den Index oder eher das Objekt meinte? Oder hat sie sonst noch einen tieferen Sinn?

 

[eRaaaa]

remove(b) den Eintrag mit dem Index "1" entfernen, während remove(c) den ersten Eintrag mit dem Wert "1" entfernt (oder?)

richtig

Dann verstehe ich die Warnung also nur als Aufforderung, noch einmal bewusst zu überlegen, ob man wirklich den Index oder eher das Objekt meinte? Oder hat sie sonst noch einen tieferen Sinn?

Naja, da die Methode(n) ein Object annehmen, könntest du theoretisch auch [c]al.get(a).remove(new Person());[/c] o.ä. schreiben, die Meldung würde dir dann eben sagen, hey, die Liste kann gar keine Personen enthalten (der Compiler würde es aber durchgehen lassen und es wird dir false zurück liefern während der Laufzeit (oder in deinen Tests *g*)).

 

[Herr Kaiser]

Ja, das klingt sehr logisch. Langsam blicke ich durch ...

1000 Dank!