Datentypen Array-Konstante

[Cybran]

Meine Klassen sollen dynamisch zu laden sein und dafür ein Component-Interface einbinden, was zum Beispiel Methoden integriert, um benötigte Klassen herauszufinden. Lässt sich die folgende Methode, bei der die benötigten Klassen zurückgegeben werden die bei jeder einzelnen Klasse konstant sind, irgendwie vereinfachen (bzw. ist das so, wie ich mir das gedacht habe überhaupt richtig)?

	@SuppressWarnings("unchecked")
	@Override
	public Class<? extends Component>[] getComponentsNeeded() {
		Class<?>[] neededComponentClasses = {TaskManager.class};
		return (Class<? extends Component>[]) neededComponentClasses;
	}

 

[SlaterB]

wenn die Methode konstant ist, wozu sie dann im Interface aufnehmen?
damit alle Klassen sie haben bietet sich eine gemeinsame Superklasse an,
anderenfalls gibts keinen gemeinsamen Code

und einer meiner Lieblingsmerksätze:
niemals generische <>-Klammern innerhalb eines Casts, hat im Programm keine Funktion, höchstens als eine Art Doku für den Leser,

[c]return (Class[]) neededComponentClasses;[/c]
wäre dasselbe, vielleicht nichtmal dieser Cast nötig, Unchecked hast du ja eh alles

 

[Landei]

Du kannst keine Arrays von parametrisierten Typen erzeugen, und ein nachträglicher Cast ist nicht typsicherer. Wenn du (trotz Slaters berechtigten Kritikpunkten) unbedingt soetwas haben willst, musst du auf Collections zurückgreifen.

    public List<Class<? extends Component>> getComponentsNeeded() {
        List<Class<? extends Component>> result = new ArrayList<Class<? extends Component>>();
        result.add(TaskManager.class);
        return result;
    }

 

[Cybran]

Danke für die Hinweise!
Beim Nachträglichen Lesen fällt mir auf, dass ich das arg blöd formuliert habe - die Methode soll innerhalb einer Klasse ein konstantes Array zurückgeben (also unabhängig von anderen Variablen), aber zwischen verschiedenen Klassen gibt es schon Unterschiede. Dass ich keine Klasse ableite, sondern ein Interface implementiere, hängt auch damit zusammen, dass meine Klassen Methoden von ganz anderen Klassen erben sollen (je nachdem, was sie eben machen sollen).

- Wieso sollte man keine generischen <>-Klammern verwenden? Schließlich will ich, dass nur Klassen übergeben werden dürfen, die eben DIESES Interface implementieren!
- Egal wie ich es mache, gibt mir eclipse ohne "unchecked" Warnungen zur Typsicherheit aus. Gibt es keine "saubere" Art, das Problem ohne Warnungen zu lösen?
- Collections will ich eigentlich vermeiden. Das Array, dass ich zurückgeben will, ist ja konstant, man kann sich also den Mehraufwand (auch an Speicher) sparen.

Vielen Dank für eure Hilfe!

 

[SlaterB]

> Wieso sollte man keine generischen <>-Klammern verwenden? Schließlich will ich, dass nur Klassen übergeben werden dürfen, die eben DIESES Interface implementieren!

ich sage ja nicht dass du sie überall streichen sollst, sondern dass sie in diesem Cast keine Auswirkung haben
(zwischenzeitlich habe ich woanders doch eine gesehen, generell muss ich mich von meinem Merksatz verabschieden )
streiche im Cast die Generics und es kompiliert genauso oder? also ohne Wirkung, zur Laufzeit eh nicht mehr vorhanden

> Egal wie ich es mache, gibt mir eclipse ohne "unchecked" Warnungen zur Typsicherheit aus. Gibt es keine "saubere" Art, das Problem ohne Warnungen zu lösen?

Posting von Landei? für Arrays gibts nichts, nein, und Speicher spielt bei sowas normalerweise keine Rolle