EventListenerList in Java 1.5

[dArignac]

Hallo,

ich habe im Rahmen eines Frameworks eine Klasse ListenerHelper die von javax.swing.event.EventListenerList erbt. Sie ist quasi die Hilfsmethode um bei selbstdefinierten Datenklassen entsprechende Events zu der entsprechenden List hinzuzufügen.
In Java 1.4.2 sahen die add/remove Methoden, die die von EventListenerList überschrieben haben, so aus:

public synchronized void add(Class t, EventListener l) { .... }

public synchronized void remove(Class t, EventListener l) { .... }

Der Aufruf aus den Datenklassen sah zb so aus:

protected ListenerHelper m_lhListeners = new ListenerHelper();

...

    public void addDataChangeListener(DataChangeListener scl) {
        m_lhListeners.add(DataChangeListener.class, scl);
    }

Hierbei gibt es eben für die Datenklassen immer auch entsprechende Listener.

Jetzt ist halt meine Aufgabe, dass ganze in Java 1.5 zum Laufen zu bringen.
Ohne irgendwas zu verändern bekomme ich die Fehlermeldung:

The method add(Class, EventListener) is ambiguous for the type ListenerHelper

Habe ich in der javadoc nachgeschaut, steht nun bei EventListenerList:

public <T extends EventListener> void add(Class<T> t, T l)

Also eine "generische" Parameterliste (oder wie man das auch immer genau bezeichnet).

Ändere ich jetzt meine Klasse in ListenerHelper<T> und passe die add/remove Methoden entsprechend der Signatur aus 1.5 an sind erstmal alle Fehler weg.
Leider aber bis auf einen. Denn innerhalb von ListenerHelper gibt es eine Methode die aus einer Datei die gespeicherten Werte ausliest. Das sind dann alles Objects. Mein Problem ist nun, die entsprechend im add-Methodenaufruf zu casten.

Die alte (1.4.2) Methode sieht so aus:

    private void readObject(ObjectInputStream s) throws IOException, ClassNotFoundException {

        s.defaultReadObject();
        Object listenerTypeOrNull;

        while (null != (listenerTypeOrNull = s.readObject())) {
            EventListener l = (EventListener)s.readObject();
            add(Class.forName((String)listenerTypeOrNull), l);
        }
    }

Der Fehler lautet da

Bound mismatch: The generic method add(Class<T>, T) of type EventListenerList is not applicable for the arguments (Class<capture-of ?>, EventListener). The inferred type capture-of ? is not a valid substitute for the bounded parameter <T extends EventListener>

Den EventListener bekomme ich ja hin, ein ein T anstelle des EventListener.
Aber wie caste ich jetzt das ausgelesene Object richtig? Ich weiss ja dass es entweder null ist oder irgendeine Klasse (entsprechend der Datenklassen) die von EventListener erbt. Ich probier da jetzt schon ne Weile rum, komme aber auf gar keinen grünen Zweig. Ist denn der Ansatz, die EventListenerList-Klasse als Generic zu machen sinnvoll/richtig? Vielleicht liegt der Fehler ja auch irgendwo anders.... thx

 

[Guest]

Hast du schon den Cast versucht?

add((Class<EventListener>)Class.forName((String)listenerTypeOrNull), l);

 

[dArignac]

Wenn ich das mache ist zunächst erstmal der Fehler weg.
Es wird aber eine Warnung gebracht:

Type safety: The Cast from Class<capture-of ?> to Class<EventListener> is actually checking against the erased type Class

Was soll mir diese Fehlermeldung sagen? Irgendwie sehe ich bei den Generics nur zur Hälfte durch...

 

[Guest]

Wenn ich das mache ist zunächst erstmal der Fehler weg.
Es wird aber eine Warnung gebracht:

Type safety: The Cast from Class<capture-of ?> to Class<EventListener> is actually checking against the erased type Class

Was soll mir diese Fehlermeldung sagen?

Das kannst du ignorieren, da EventListener ein Interface ist. Eine Klasse, die das Interface implementiert,
ist immer eine "Erweiterung" von EventListener. Gleiches gilt für abstrakte Klassen.

Erst in der kommenden Version von Java wird man es mit Annotation "@SuppressWarnings" ausschalten
können. Es ist bereits als Bug gemeldet. http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4986256

Irgendwie sehe ich bei den Generics nur zur Hälfte durch...

Geht mir genau so.

 

[dArignac]

Gut danke, das hat mir erstmal weiter geholfen.
Nun habe ich bei jeder compareTo()-Methode wieder Hinweise:

public int compareTo(Object o) {
        return m_stSource.compareTo(o);
    }

m_stSource ist dabei eine eigene Datenklasse.
Der Hinweis lautet:

Type safety: The method compareTo(Object) belongs to the raw type Comparable. References to generic type Comparable<T> should be parameterized

Wie soll ich das nun parametisieren? Als Argument braucht es ein Object. Ja und sonst?
Das erscheint mir völlig unlogisch daran was ändern zu wollen...

 

[Beni]

Comparable vergleicht nicht mehr Objects, sondern neu T's. Du solltest jeweils angeben, was T ist, wenn du das Interface implementierst.

public class X implements Comparable<X>{
  public int compareTo( X x ){
     ...
  }
}

 

[dArignac]

Aha, danke, wieder was gelernt. Hatte ich schon erwähnt dass ich Java 1.5 nicht mag wenn ich 1.4er Projekte dran anpassen muss? narf

 

[Lim_Dul]

erung" von EventListener. Gleiches gilt für abstrakte Klassen.

Erst in der kommenden Version von Java wird man es mit Annotation "@SuppressWarnings" ausschalten
können. Es ist bereits als Bug gemeldet. http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=4986256

In Java 1.5.0_06, als dem aktuellsten Update, wird @SupressWarnings unterstützt.