✔ Ersatz fuer instanceof

[MiMi]

Also ich hab nen Vector mit Objecten.
Menutyp1, Menutyp2 und Menutyp3
2 und 3 erben von 1
jetzt hab ich das mit instanceof geloest

Object o = menuTypen.elementAt(currentChooseNumber-1);
		if ( o instanceof MenuType2 ) {
			Display.getDisplay(main).setCurrent((MenuType2)menuTypen.elementAt(currentChooseNumber-1));
		} else if ( o instanceof MenuType1 ) {
			Display.getDisplay(main).setCurrent((MenuType1)menuTypen.elementAt(currentChooseNumber-1));
		}

Gibts da ne bessere Loesung? Weil ich mal gelesen hab, man sollte den Gebrauch von instanceof vermeiden.

 

[Oliver Gierke]

Also ich hab nen Vector mit Objecten.
Menutyp1, Menutyp2 und Menutyp3
2 und 3 erben von 1
jetzt hab ich das mit instanceof geloest

Object o = menuTypen.elementAt(currentChooseNumber-1);
		if ( o instanceof MenuType2 ) {
			Display.getDisplay(main).setCurrent((MenuType2)menuTypen.elementAt(currentChooseNumber-1));
		} else if ( o instanceof MenuType1 ) {
			Display.getDisplay(main).setCurrent((MenuType1)menuTypen.elementAt(currentChooseNumber-1));
		}

Gibts da ne bessere Loesung? Weil ich mal gelesen hab, man sollte den Gebrauch von instanceof vermeiden.

Hm.. deine setCurrent() akzeptiert welchen Typ? Ich versteh nicht, warum du da extra nen Cast machst?

Display.display(main).setCurrent(menuTypen.elementAt(currentChooseNumber-1));

sollte es doch tun, oder?

 

[MiMi]

setCurrent akzeptiert nur ein Displayable.
Und Menutyp1, 2 und 3 sind welche
Mach ich einfach

 Display.display(main).setCurrent(menuTypen.elementAt(currentChooseNumber-1));

sagt er das er keine methode findet, weil er das nur als Object ansieht.

 

[MiMi]

Hab jetzt einfach

 Display.getDisplay(main).setCurrent((MenuType1)menuTypen.elementAt(currentChooseNumber));

gemacht funktioniert. Trotzdem danke

 

[Oliver Gierke]

Wenn beide Klassen Displayable sind, dann caste doch einfach darauf...

Display.display(main).setCurrent((Displayable) menuTypen.elementAt(currentChooseNumber-1));