Kann man das verkürzen/vereinfachen?

[oldshoe]

public void handleEvent(Event event) {
		ContentObject conObj;
		
		if (event instanceof PageCreateEvent){
			PageCreateEvent pageCrtEv = (PageCreateEvent)event;
			conObj= pageCrtEv.getContent();
			analyzeContentObj(conObj);
		}else{
			if(event instanceof PageUpdateEvent) {
				PageUpdateEvent pageUpdEv=(PageUpdateEvent) event;
				conObj= pageUpdEv.getContent();
				analyzeContentObj(conObj);
		     }else{
			        if (event instanceof BlogPostUpdateEvent){
				BlogPostUpdateEvent blogUpdEv= (BlogPostUpdateEvent)event;
				conObj= blogUpdEv.getContent();				
				analyzeContentObj(conObj);
				
			}else{
				if (event instanceof BlogPostCreateEvent){
					BlogPostCreateEvent blogCrtEv = (BlogPostCreateEvent)event;
					conObj= blogCrtEv.getContent();
					analyzeContentObj(conObj);
				}
			}
		}
				
		
		}
	}

Ideen? Danke...

 

[SlaterB]

public void handleEvent(Event event) {
		if (event instanceof ContentHolder){
			ContentHolder ch= (ContentHolder)event;
			ContentObject conObj= ch.getContent();
			analyzeContentObj(conObj);
		}
	}

ContentHolder/ ContentEvent wäre ein Interface/ Basisklasse mit der Methode getContent(), von allen zugehörigen Klassen implementiert/ geerbt,
wenn du weißt dass nur derartige Events kommen, kann das if wegfallen

 

[oldshoe]

Hmmm..danke erstmal nur leider wird sich das wohl nicht umsetzen lassen, da ich die EventKlassen wie PageCreateEvent nicht bearbeiten und somit keine "implements" hinzufügen kann.

 

[SlaterB]

ohne extreme Tricks wie Reflection musst du dann wirklich jede Klasse einzeln bearbeiten,

das immer gleiche
analyzeContentObj(conObj);
könnte noch raus und nur einmal am Ende stehen

-------

BlogPostCreateEvent blogCrtEv = (BlogPostCreateEvent)event;
 conObj= blogCrtEv.getContent();
->
conObj= ((BlogPostCreateEvent)event).getContent();

--------

und benutze sauberes

  if {

  } else if {

  } else if {

statt deiner wilden Verschachtelung

 

[faetzminator]

Sprichst du von [c]com.atlassian.confluence.*[/c]? Da gibt es [c]ContentEvent[/c], welches [c]public abstract ContentEntityObject getContent()[/c] definiert. Da drunter sind alle PageEvents, BlogEvents etc.

Edit: D.h. für dich, dass du SlaterB's Code 1:1 verwenden kannst, mit der Ausnahme, dass du einfach auf [c]ContentEvent[/c] überprüfen und casten musst an Stelle von [c]ContentHolder[/c].

 

[FArt]

da ich die EventKlassen wie PageCreateEvent nicht bearbeiten und somit keine "implements" hinzufügen kann.

Muss man gar nicht. Ein Wrapper tut es auch, z.B.
Dynamic Proxy Classes
Oder die Verwendung von Mustern:
pattern instanceof - Google-Suche

 

[Landei]

Oder eine java-kompatible Sprache benutzen, die Pattern Matching unterstützt *hüstl*

 

[oldshoe]

Sprichst du von [c]com.atlassian.confluence.*[/c]? Da gibt es [c]ContentEvent[/c], welches [c]public abstract ContentEntityObject getContent()[/c] definiert. Da drunter sind alle PageEvents, BlogEvents etc.

Edit: D.h. für dich, dass du SlaterB's Code 1:1 verwenden kannst, mit der Ausnahme, dass du einfach auf [c]ContentEvent[/c] überprüfen und casten musst an Stelle von [c]ContentHolder[/c].

ja super so funktioniert es...wie von SlaterB vorgeschlagen reicht dann auch:

ContentEvent ch= (ContentEvent)event;
ContentEntityObject conObj= ch.getContent();
analyzeContentObj(conObj);

Danke