ArrayList<Element> - "unchecked or unsafe operati

[SpammerSlammer]

Hallo!

Beim untenstehenden Code handelt es sich um einen Auszug aus einem JDOM-basierten XML-Parser. Mein Problem bezieht sich auf Zeile 5, in der mehrere Elemente zu einer ArrayList vom Typ <Element> zugewiesen werden. Der Code wird zwar kompiliert und scheint zu funktionieren, aber ich erhalte die ebenfalls unten aufgeführte Warnung. Was heißt das, und wie kriege ich es weg? Scheint was mit dem Element-Typ zu tun zu haben, oder?

Falls ein vollständiges Code-Beispiel gewünscht ist, bitte melden!

ArrayList<Element> artikelliste = new ArrayList<Element>(root.getChildren("article"));
ArrayList<Element> statementliste = new ArrayList<Element>();
for (i = 0; i < artikelliste.size(); i = i + 1) {
    Element artikel = (Element) artikelliste.get(i);
    statementliste.addAll(artikel.getChildren("statement"));
}

Note: Dateioperationen.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

 

[Gast2]

In artikel.getChildren("statement") können auch andere Objekte als Elemente vorhanden sein d.h. es könnte eine Exception geworfen werden, wenn andere Objekte drin sind... das heißt artikel.getChildren("statement") ist nicht typsicher

 

[Eminent]

Edit: SirWayne war schneller

 

[SpammerSlammer]

OK, und wie ändere ich das? Eine Exception abfangen? Oder irgendwie den Typ vorschreiben? Ich habe es damit versucht, <Element> an diversen Stellen in Zeile 5 unterzubringen, aber das hat nicht geklappt.

 

[Landei]

Geht das?

List<Element> artikelliste = new ArrayList<Element>(root.getChildren("article")); 
List<Element> statementliste = new ArrayList<Element>(); 
for (Element artikel : artikelliste) { 
    statementliste.addAll(artikel.getChildren("statement")); 
}

 

[SlaterB]

einfache Regel: in Cast-Klammern () brauchen niemals eckige Generics-Klammen <> auftauchen

 

[Landei]

Richtig. Hab's oben korrigiert...

 

[Gast2]

Edit: sicher das es geht???

 

[Landei]

Was stört dich?

 

[SpammerSlammer]

Geht das?

Leider nicht. Das ganze bringt keine Änderung, d.h. das Problem mit der Warnung habe ich immer noch. Was tun?

 

[Landei]

Wenn getChildren() einfach nur eine ungetypte ("raw-type") Liste zurückgibt, wirst du die Warnung nicht wegbekommen, den theoretisch *ist* die Konvertierung dann auch unsicher. Da du allerdings davon ausgehen kannst, dass es sich *immer* um eine Liste von Elementen handelt, kannst du die Warning ruhig mit @SuppressWarnings("unchecked") unterdrücken. Derartige Warnungen sind prinzipiell kaum zu vermeiden, wenn du mit Bibliotheken vor Java 1.5 arbeitest.

Anstelle der @SuppressWarnings gibt es noch eine andere Möglichkeit, die Warnung zu umgehen, nämlich Class.cast():

    Object o = new Integer(23);
    Integer i = Integer.class.cast(o); //keine Warnung

 

[SpammerSlammer]

Da du allerdings davon ausgehen kannst, dass es sich *immer* um eine Liste von Elementen handelt, kannst du die Warning ruhig mit @SuppressWarnings("unchecked") unterdrücken.

Danke, das klappt.

Anstelle der @SuppressWarnings gibt es noch eine andere Möglichkeit, die Warnung zu umgehen, nämlich Class.cast():

    Object o = new Integer(23);
    Integer i = Integer.class.cast(o); //keine Warnung

Bei Class.cast() erhalte ich die Meldung: "non-static method cast(java.lang.Object) cannot be referenced from a static context."

Welche Möglichkeit habe ich, wenn ich die Warnung nicht einfach unterdrücken möchte, sondern die Ursache beseitigen möchte?

 

[Landei]

Welche Möglichkeit habe ich, wenn ich die Warnung nicht einfach unterdrücken möchte, sondern die Ursache beseitigen möchte?

Keine. Wen getChildren keine Liste mit Typ liefert, ist der cast zu einer Liste mit Typ prinzipiell unsicher.

Das läuft bei mir übrigens ohne Probleme:

public class CastTest {
  public static void main(String... args) {
    Object o = new Integer(23);
    Integer i = Integer.class.cast(o);
    System.out.println(i);
  }
}

 

[Saxony]

Hiho,

genau dazu hatte ich sogar mal einen Beitrag geschrieben: Java Blog Buch : 07.06 Legacy Code

bye Saxony