Generic's und Vererbung

[Melfis]

Hallo an alle,
habe grade einen kleinen hänger
Ich habe eine Klasse "Test" und "AbstractTest",

Test soll..

-von ArrayList erben
-nur Objekte die von "AbstractTest" erben in die Liste aufnehmen
-Test soll kein generic's unterstützen

geht das überhaupt?
Ich habs mir so vorgestell:

public class Space2D extends ArrayList<? extends AbstractDrawObject>

funktioniert leider nicht

MFG Melfis

 

[bygones]

[c]public class Space2D extends ArrayList<AbstractDrawObject>[/c]
bei extends darf kein wildcard stehen.

und direkt von einer implementierung erben riecht stark nach designschwaeche

 

[Landei]

Wenn es "reicht", dass [c]Space2D[/c] nur [c]AbstractDrawObject[/c]-Objekte zurückgeliefert werden, kannst du einfach schreiben:

public class Space2D extends ArrayList<AbstractDrawObject>

Wenn du allerdings [c]Space2D[/c]-Objekte haben willst, die jeweils nur verschiedene Unterklassen von [c]AbstractDrawObject[/c] akzeptieren bzw. zurückliefern, musst du sowas schreiben:

public class Space2D<T extends AbstractDrawObject> extends ArrayList<T>

Es ist allerdings ziemlich fragwürdig, ein Domänen-Objekt wie [c]Space2D[/c] von einer Collection abzuleiten, dafür ist in der Regel Komposition besser geeignet.

 

[Melfis]

[c]public class Space2D extends ArrayList<AbstractDrawObject>[/c]
bei extends darf kein wildcard stehen.

und direkt von einer implementierung erben riecht stark nach designschwaeche

Kann sein das ich da noch designschwaechen drin habe,
zu meinen Konzept:

Ich moechte ein kleines "CAD-Programm" schreiben.
Dazu brauch ich eine funktionale Zeichenfläche (Space2D)
und passende Zeichenobjekte (? extends Drawobjects).

Space2D ist prinzipiel nur eine Liste mit den Drawobjects und eine Handvoll Eigenschaften.

Ich wollte das Space2D von ArrayList erbt um auf alle Methoden der Liste zugreifen zu können,
gleichzeitig beim zugriff von this.get() mir das nervige unboxing zu ersparen.

Bin für Verbesserungsvorschläge immer offen

MFG Melfis

 

[bygones]

siehe landeis post. statt vererbung komposition nehmen !

niemals vererbung aufgrund von gemuetlichkeit/faulheit einsetzen

 

[Melfis]

Jap, die frage wurde durch Landei beantwortet.
Ich will ja auch nicht aus Faulheit vererben,
sondern weil ich in den Methoden(bzw beim Überschreiben) einen nutzen sehe,
naja, hab mich vieleicht falsch ausgedrückt.

Danke an euch beide

 

[Landei]

Ich will ja auch nicht aus Faulheit vererben,
sondern weil ich in den Methoden(bzw beim Überschreiben) einen nutzen sehe,

Das reicht als Begründung nicht aus. Die Frage ist: Ist [c]Space2D[/c] eine Liste, oder benutzt es eine? Wenn du z.B. die Möglichkeit siehst, später einmal zusätzliche Daten (die nichts mit List zu tun haben) hinzufügen willst, benutzt [c]Space2D[/c] die Liste nur. Oder fügst du [c]ArrayList[/c] wirklich neue Listenfunktionialität hinzu, dann ist [c]Space2D[/c] tatsächlich eine Liste.

Was spricht gegen

public class Space2D {
   private List<AbstractDrawObject> elements = new ArrayList<AbstractDrawObject>();
   ...
   public List<AbstractDrawObject> getElements(){ return elements; } //oder eine schreibgeschützte Kopie
}

?

Dann kannst du alle Listen-Methoden an [c]Space2D.getElements()[/c] aufrufen.

Mit diesem Design werden spätere Änderungen einfacher. Hast du z.B. Threading-Probleme mit deiner [c]ArrayList[/c], kannst du sie durch eine [c]CopyOnWriteArrayList[/c] ersetzen. Willst du später benannte Elemente, kannst du sie durch eine [c]Map<String, AbstractDrawObject>[/c] ersetzen.