Problem beim Visitorpattern

[Rudolf]

Hallo,

ich möchte für ein MVC Konzept den Visitorpattern einsetzen und wunder mich über eine Fehlermeldung. Ich habe einen kurzen Code geschrieben, der das Problem schildert.

class ConcreteEvent1 implements Event
{

	@Override
	public void accept(final Visitor1 visitor)
	{
		// TODO Auto-generated method stub

	}

}

class ConcreteEvent2 implements Event
{

	@Override
	public void accept(final Visitor2 visitor)
	{
		// TODO Auto-generated method stub

	}

}

interface Event
{
	void accept(EventVisitor visitor);
}

interface EventVisitor
{

}

interface Visitor1 extends EventVisitor
{

}

interface Visitor2 extends EventVisitor
{

}

wie man sieht meckert er bei ConcreteEvent1 und ConcreteEvent2, dass die Methode "void accept(EventVisitor visitor);" nicht überschrieben wird. Die Alternative wäre natürlich für jeden konkreten Visitor eine neue Operation in Event reinzuschreiben, aber das hat den Nachteil, dass jede neue Eventklasse zwei accept-Methoden implementieren muss. Hat jemand eine Idee, wie man dieses Problem geschickt löst?

Dann würde der Code zum besseren Verständnis so aussehen:

class ConcreteEvent1 implements Event
{

	@Override
	public void accept(final Visitor1 visitor)
	{
		// TODO Auto-generated method stub

	}

	@Override
	public void accept(final Visitor2 visitor)
	{
		// TODO Auto-generated method stub

	}

}

class ConcreteEvent2 implements Event
{

	@Override
	public void accept(final Visitor1 visitor)
	{
		// TODO Auto-generated method stub

	}

	@Override
	public void accept(final Visitor2 visitor)
	{
		// TODO Auto-generated method stub

	}

}

interface Event
{
	void accept(Visitor1 visitor);

	void accept(Visitor2 visitor);
}

interface EventVisitor
{

}

interface Visitor1 extends EventVisitor
{

}

interface Visitor2 extends EventVisitor
{

}

Wäre es evtl möglich eine Operation so zu schreiben wie in folgender Art:

void accept(<? extends IEventVisitor> v);

Damit jeder, der IEventVisitor erweitert als Parametertyp akzeptiert wird?

 

[SlaterB]

wenn der Parameter EventVisitor ist, dann musst jede das Interface implementierende Klasse auch EventVisitor als Parameter akzeptieren, alle möglichen Visitor, nicht nur eine Unterart,
zwei Methode helfen da nicht, eine Methode mit dem allgemeinen Parameter

dein letzter Vorschlag kann hinkommen:

interface  X<T extends Visitor> { accept(T) }
+
class Y implements X<V1> {accept(V1) } // V1 als Unterklasse von Visitor

 

[mohrenkopf]

Ich verstehe nicht genau, was du im Endeffekt erreichen willst, aber vielleicht so:

interface EventVisitor{
  public void accept(EventVisitor visitor)
}

interface EventVisitor{}
interface Visitor1 extends EventVisitor{}
interface Visitor2 extends EventVisitor{}


class ConcreteEvent1 implements Event
{
         
	@Override
	public void accept(final EventVisitor visitor)
	{
		if( visitor instanceof Visitor1) 
                  acceptVisitor1(visitor);
                //optional
                else if(visitor instanceof Visitor2) {...}

	}

        
        private void acceptVisitor1(Visitor1 v)
        {
              //eigentlicher Code hier
        }

}