downcasting funktioniert nicht

[guni]

Hallo,
ich schreibe gerade einen Installer, der vom Konzept her so aufgebaut ist:

• * Installer.class implements Visitor
  * Step.class implements Visitable (hat eine Property: next)

der Visitor durchläuft die einzelnen Steps und führt pro Step eine gekapselte Operation durch.
Soweit ganz einfach; jetzt kommt die Schwierigkeit:
der Installer kann mit 2 Arten von Steps aufgerufen werden:

• * TextualStep
  * GraphicalStep

da sich beide Steps recht ähnlich sind, werden sie beide von AnyStep abgeleitet. Genau genommen unterscheiden sich die beiden Steps nur in der accept-Methode, die in einem Fall irgendeinen Text ausgibt; im anderen Fall ein neues Fenster öffnet!
Der Installer sieht ungefähr so aus:

public class Installer implements Visitor {

	public AnyStep start;
		
	public Installer() {
		// ermittle den Start Step
	}
	
	@Override
	public void visit(AnyStep s) {
		if (s.accept(this))
			this.visit(s.getNext());
	}

}

in der Main-Methode versuche ich den Installer wie folgt aufzurufen:

	public static void main(String args[]){
		Installer i = new Installer();
		i.visit((TextualStep)i.start);
	}

das Problem: Java akzeptiert mein Casting auf den TextualStep nicht!!
Wieso?
Gibt es eine Möglichkeit hier zu casten?
Oder muss ich das anders lösen?
Wenn ja, was wäre ein sinnvoller Ansatz?

danke.
lg, guni

 

[byte]

Wie lautet denn die Fehlermeldung?

 

[tfa]

Was für ein Objekt steht denn in "start"? (mal davon abgesehen, dass der Cast überflüssig ist)

 

[guni]

Hallo tfa,

Wie lautet denn die Fehlermeldung?

das Programm wirft eine java.lang.ClassCastException: AnyStep cannot be cast to TextualStep

Was für ein Objekt steht denn in "start"

Start ein ein Objekt der Klasse AnyStep (siehe Code im startpost)

public AnyStep start;

Ich denke schon, dass ich "start" typecasten muss; woher soll der Installer sonst wissen, ob es sich um einen TextualStep oder um einen GraphicalStep handelt?!

public class AnyStep implements Visitable {
	
	private String name;
	private String message;
	private Runner r;
	private String question;
	private String error;
	private AnyStep next;
	
	// some getters and setters here ...

	public AnyStep(String name, String message, Runner r, String question,
			String error, AnyStep next) {
		this.name = name;
		this.message = message;
		this.r = r;
		this.question = question;
		this.error = error;
		this.next = next;
	}

	@Override
	public boolean accept(Visitor v) {
		// is overriden in textual/graphical step
		return false;
	}	
}

... warum denkst du, dass der Cast überflüssig ist?!
lg, guni

 

[SlaterB]

> java.lang.ClassCastException: AnyStep cannot be cast to TextualStep

sagt eindeutig, dass AnyStep nicht nach TextualStep gecastet werden kann

da musst du nun nicht zig Zeilen Erklärungen und unwesentlichen Code posten,
was sollte denn das Visitor-Interface mit dem ganzen zu tun haben? (edit: wurde doch nicht gepostet, habe mich verschaut)

es zählt alleine die Frage, ob TextualStep von AnyStep erbt oder nicht,
und diese entscheidene Klasse TextualStep fehlt bisher

 

[tfa]

Die Variable "start" ist vom Typ AnyStep, d.h. dass dort auch Objekte von abgeleiteten Klassen (TextualStep) abgelegt sein können. Nur wenn das der Fall ist, kannst du auch nach TextualStep casten. Ein Objekt, das direkt von der Klasse AnyStep erzeugt wurde, ist eben kein TextualStep. Casting ist hier keine Typumwandlung, sondern nur eine Zusicherung an den Compiler, dass ein bestimmtes Objekt einen bestimmten Typ hat.

Ich denke schon, dass ich "start" typecasten muss; woher soll der Installer sonst wissen, ob es sich um einen TextualStep oder um einen GraphicalStep handelt?!

An der stelle wo du castest, wird eine Methode aufgerufen, die einen AnyStep-Parameter hat. Und da sowohl TextualStep als auch GraphicalStep den Typ AnyStep haben, ist ein Casting hier überflüssig. Die Methode wird sowie aufgerufen.

 

[Murray]

Zeig mal den Code, mit dem im Installer start initialisiert wird.

 

[guni]

Hallo Leute,

ich schick euch mal alle (relevanten) Klassen, die ich bis jetzt so hab ... danke derweil!

public class AnyStep implements Visitable {
	
	private String name;
	private String message;
	private Runner r;
	private String question;
	private String error;
	private AnyStep next;

	// some getters and setters here

	public AnyStep(String name) {
		this.name = name;
		this.message = message;
		this.r = r;
		this.question = question;
		this.error = error;
		this.next = next;
	}

	@Override
	public boolean accept(Visitor v) {
		// is overriden in textual / graphical step
		return false;
	}	
}

public class TextualStep extends AnyStep {

	public TextualStep(String name) {
		super(name);
	}

	@Override
	public boolean accept(Visitor v) {
		Runner r = this.getRunner();
		System.out.println(this.getMessage());
		if (r != null)
			r.run();
		if (this.getError() != null)
			System.out.println(this.getError());
		if (this.getQuestion() != null)
			System.out.println(this.getQuestion());
		// should only accept if next step is not null
		return (this.getNext() != null);
	}

}

public class Installer implements Visitor {

	public AnyStep start;

		AnyStep s1 = new AnyStep("Step 1");
		AnyStep s2 = new AnyStep("Step 2");
		
		s1.setNext(s2);
		
		this.start = s1;
	}
	
	@Override
	public void visit(AnyStep s) {
		if (s.accept(this))
			this.visit(s.getNext());
	}

}

so ... ich hab euch den Code wieder ein bisschen vereinfacht und nur die Dinge gepostet, von denen ich denke, dass sie notwendig sind (ich weiß, wie es ist, sich in fremden Code einzulesen) ...

also dann. ich hoffe, dass ihr mir damit weiterhelfen könnt.

lg, guni

 

[guni]

... ach ja.
und hier nochmal der Aufruf:

	public static void main(String args[]) throws ClassNotFoundException, InstantiationException, IllegalAccessException {
		
		Installer i = new Installer();
		i.visit((TextualStep)i.start);
		
	}

lg

 

[Murray]

Im Installer müssen die beiden AnySteps s1 und s2 entweder TextualSteps oder GraphicalSteps sein (AnyStep könnte abstratc sein, dann würde der Compiler das schon erzwingen).

 

[guni]

das ist es ja! der konstruktor vom installer ist in wirklichkeit etwas komplexer aufgebaut, im prinzip aber für den grafischen Installer / den textbasierten Installer genau gleich! (ausser, dass eben verschiedene Step-Typen aufgerufen werden)

muss ich jetzt allen ernstes meinen installer 2x schreiben und in den beiden Versionen dann AnyStep einmal durch TextualStep und einmal durch GraphicalStep ersetzen?!
das macht doch keinen Sinn, oder?!

lg, guni

 

[SlaterB]

> java.lang.ClassCastException: AnyStep cannot be cast to TextualStep

sagt eindeutig, dass AnyStep nicht nach TextualStep gecastet werden kann
[..]
es zählt alleine die Frage, ob TextualStep von AnyStep erbt oder nicht

verdammt, da habe ich dann ja eine Laufzeitexception mit einer Compiler-Fehlermeldung verwechselt,

sorry für Spam, manchmal muss ich aber meine falschen Aussagen revidieren

 

[guni]

kein problem :lol:
aber was heißt das jetzt für mich?
ich kann ja nicht 2 genau gleiche klassen schreiben, die sich nur im aufruf einer Variablen unterscheiden, oder?!

lg

 

[Murray]

muss ich jetzt allen ernstes meinen installer 2x schreiben und in den beiden Versionen dann AnyStep einmal durch TextualStep und einmal durch GraphicalStep ersetzen?!

Du kannst sicher auch in einer Installer-Klasse eine Fallunterscheidung einbauen; es wird aber kein Weg darum herumführen, bei der Erzeugung der Step-Instanzen deren konkreten Typ anzugeben.

Wovon soll es denn abhängen, welche Art von Steps der Installer verwenden soll?

 

[SlaterB]

und nochmal der Hinweis von tfa:

An der stelle wo du castest, wird eine Methode aufgerufen, die einen AnyStep-Parameter hat.
Und da sowohl TextualStep als auch GraphicalStep den Typ AnyStep haben, ist ein Casting hier überflüssig.
Die Methode wird sowie aufgerufen.

wieso überhaupt eine Fallunterscheidung? welchen Zweck hat das?

 

[guni]

hmm ... eigentlich schwebt mir vor, den Installer abstrakt zu schreiben und dann einen Textual / einen Graphical-Installer zu schreiben.
Mein Problem is wie gesagt: Installer selbst weiß nicht, ob er mit einem Textual / einem Graphical-Step aufgerufen wird. Vielleicht schreibe ich das mal so, dass je nach Kommandozeilenparameter der Textual oder der Graphical-Installer aufgerufen wird. So genau weiß ich das noch nicht!
Gibt es da jetzt echt keine Lösung, dass ich den gleichen Konstruktor mit unterschiedlichen Steps aufrufe (obwohl beide Steps ja die gleiche ParentClass haben) :bahnhof:

 

[guni]

@SlaterB

ok. ich probier die Situation nochmal zu beschreiben:

Ich habe einen Installer nach dem Visitor-Pattern; er besucht Steps.

Ich habe eine Klasse AnyStep die Visitable ist.
Ich habe 2 Subklassen, die sich derzeit nur in der accept Methode unterscheiden (unterschiedliche Reaktion auf den "Besuch" des Installers)

der Installer soll entweder TextualSteps oder GraphicalSteps besuchen können. Je nachdem was für eine Art Step in Main gewählt wird.

vielleicht muss ich wirklich zwei unterschiedliche Initialisierungsmethoden schreiben. :?

oder hat noch irgendwer eine bessere Idee?!

 

[Murray]

eigentlich schwebt mir vor, den Installer abstrakt zu schreiben und dann einen Textual / einen Graphical-Installer zu schreiben.

Dann könnte die konkrete Instanziierung der Steps ja durch eine (abstrakte) Methode createStep passieren, die dann in den konkreten Ausprägungen TerxtualInstaller und GraphicalInstaller entsprechend implementiert wird.
Alternativ könntest Du die Instanziierung auch auslagern:

public interface StepFactory {
  public AnyStep createStep( String name);
}


public class Installer {
  public Installer( StepFactory sf) {
     AnyStep s1 = sf.createStep( "Step 1");
     AnyStep s2 = sf.createStep( "Step 2");
  }
}

Dann muss der Code, der den Installer aufruft, eben eine passende Implementierung des StepFactory-Interface mitgeben.

 

[guni]

hi murray,

danke für deinen hinweis.
auch, wenn ich etwas enttäuscht bin, das ganze nicht über vererbung realisieren zu können, habe ich mir deinen hint mit dem factory-pattern mal durch den kopf gehen lassen und mein programm dann dahingehend modifiziert!
danke für die hilfestellung; so funktionierts.

lg, guni