SWT unelegant? Problem mit Listener.eventHandle(Event e)

[pocketom]

Hallo,

nachdem ich nun schon ewig geooglet habe und es einfach nicht hinbekomme, versuche ich mein kniffliges problem mal hier zu schildern:

hier der Code(gibt natürlich Fehler)

public static boolean getResult() 
{		
		
	boolean result = false;
	
	final Shell dialog = new Shell (shell, SWT.DIALOG_TRIM | SWT.APPLICATION_MODAL);
					
	final Button ok = new Button (dialog, SWT.PUSH);
	ok.setText ("OK");
		
	Button cancel = new Button (dialog, SWT.PUSH);
	cancel.setText ("Cancel");
		
								
	Listener listener = new Listener () {
			
		// hier liegt das Problem.....
		public void handleEvent (Event event) {
			
			if(event.widget == ok)
				result = true;
					
			dialog.close ();
		}
	};
			
	ok.addListener (SWT.Selection, listener);
	cancel.addListener (SWT.Selection, listener);

	dialog.open ();
		
	while (!dialog.isDisposed ()) {
		if (!display.readAndDispatch ()) display.sleep ();
	}		
	return result;					
}

Was ich haben möchte ist eigentlich sehr simple: Die Funktion soll einfach wenn Sie aufgerufen wird, ein neues modales Fenster öffnen, mit 2 Buttons und je nachdem ob man "OK" oder "Cancel" geklickt hat, soll die funktion true oder false zurückliefern.
Leider kreige ich das aber nur sehr unelegant hin da:

- es gibt nur "void" handleEvent(Event e), ich bräuchte aber "boolean" handleEvent(Event e) um den jeweiligen Rückgabewert von meiner Hauptfunktion getResult() zurückliefern zu lassen
- in handleEvent(Event e) kann ich nicht auf meine Funktionsvariable "result" zugreifen, es sei den ich mache sie "final" (dann kann ich sie aber nicht mehr verändern)

Es bleibt mir also im Moment nur eine globale Variable zu verwenden, was mir aber absolut zuwider ist, zumal ich viele davon brauchen werde.... kann mir irgendwer helfen? Bin mit meinem Latein am Ende. Oder steh ich einfach nur auf dem Schlauch???

 

[byte]

Modale Dialoge brauchst Du Dir nicht händisch zusammenfrickeln.

Machs doch einfach so:

return new MessageBox(shell, SWT.OK | SWT.CANCEL).open() == SWT.OK;

 

[pocketom]

Ahh cool! Danke!

Aber wie sieht es aus wenn meine Dialoge etwas umfangreicher gestaltet werden sollen? Das mit "Ok", "Cancel" war jetzt nur als Beispiel gedacht... Normal hab ich noch diverse Eingabefelder, Radiobuttons, usw...

 

[byte]

Du kannst die MessageBox natürlich noch umfangreicher konfigurieren. Ein Blick in die API hilft da weiter:

http://help.eclipse.org/help31/nfto...e/api/org/eclipse/swt/widgets/MessageBox.html

Schau Dir im speziellen die Styles an. Du kannst somit unterschiedliche Buttons aktivieren oder ein Icon (z.B. Fragezeichen) setzen. Du kannst auch mit setText() einen Titel und mit setMessage() einen Nachrichtentext setzen.

 

[pocketom]

Danke für den Tipp, habs mir grade angesehen. Für einen einfachen Confirmation Dialog reicht das, aber für meine Zwecke leider nicht. Ich benötige Dialoge die dem User mehrere Eingabefelder zur Verfügung stellen, wie z.B. Textfelder, Radiobuttons, Checkboxes, usw... das muss auch alles individuell gestaltet werden. Die MessageBox gibt das leider nicht her.
Kennst du vielleicht noch einen anderen Ansatz für mein Problem? Ich muss einerseits verschiedene Eingaben in den Dialogen erfassen, andererseits brauche ich auf jeden Fall die Info ob der User abschliessend auf "OK" oder "Cancel" geklickt hat, weil das für die Weiterverarbeitung der soeben eingegebenen Daten genau in diesem Moment bekannt sein muss, und zwar ausserhalb von "void handleEvent(Event e)".... Ich möchte es auf jedenfall vermeiden für jeden Dialog globale boolsche Variablen anlegen zu müssen! Das muss doch irgendwie auch schick, elegant und vor allem sicherer gehen? Oder bin ich einfach nur zu doof? ???:L


EDIT:

ich habs jetzt mal so "gelöst", leider komme ich einfach nicht um globale Variablen herum:

Listener listener = new Listener () {
			public void handleEvent (Event event) {
				if(event.widget == ok)
					regfile.printTags(mainlog,Integer.parseInt(temp[0].getText()));							
				
				dialog.close ();
			}
		};

temp[] ist ein globales Array vom Typ Text[]. temp[0] ist das Textfeld das ich innerhalb des modalen Dialogs verwende, der benutzer gibt hier eine Zahl ein. Die wird als Aufrufparameter für meine Funktion printTags(Text t, int n) benötigt. Es funktioniert so aber ich finde das ist einfach extrem unelegant und dürfte mit steigender Anzahl an Dialogen schnell unübersichtlich und fehleranfällig werden. mann, so hab ich das nicht gelernt, man hat mir beigebracht sowas tunlichst zu vermeiden, aber hier weiss ich echt nicht wie. Wie geb ich dem Listener zw. dem handleEvent etwas mit???? :cry:

 

[schalentier]

Mal wieder ein Fall fuer MVC (Model View Controller).

Dein Dialog ist der View, der schreibt Daten in ein Model (entspricht deinem komischen temp Array) und wird zusammengesetzt vom Controller (das ist deine Methode und handleEvent()).

Wenn du das ganze jetzt ein bisschen entkoppelst, sollte das alles auch funktionieren. Im handleEvent() brauchst du Zugriff auf eine Modellklassen-Instanz (im Konstruktor des Listeners uebergeben, dann brauchste ne eigne Klasse fuer den Listener/Controller - bietet sich an). Dieses Model kannst entweder der Methode uebergeben, oder von der Methode zurueckliefern lassen.

Bisschen Code:

public boolean getResult(final ResultModel resultModel) {
  // ...
  Listener listener = new Listener() {
    public void handleEvent(Event e){
      //..
      resultModel.setResult( true ); //[*]
      resultModel.setOkay( true );
    }
  }
  //...
  return resultModel.getOkay();
}

[*] resultModel is final Parameter, d.h. resultModel selbst, darf nicht auf eine neue Instanz gesetzt werden, Zugriffe funktionieren aber ganz gut - dennoch, pack das alles in eine eigne Klasse und uebergib das Model im Konstruktor

 

[pocketom]

Danke! Ich probier das mal bis morgen aus....

 

[byte]

Am sinnigsten schreibt man Custom Dialoge, indem man die Klasse Dialog erweitert. Wenn Du viele Informationen aus dem Dialog zurückgeben willst, dann schreib Dir einfach einen Datencontainer, in dem alle nötigen Daten gespeichert werden. Im Dialog werden dann die Daten in diesem Container gemäß den Angaben des Benutzers gesetzt und dieser am Ende als Rückgabewert zurückgegeben.

 

[KSG9|sebastian]

Nun ja, SWT und Vererbung ist auch so ne Sache..musst glaub die Methode boolean checkSubclass() überschreiben...

 

[byte]

Jein. Es gibt viele SWT-Klassen, von denen man nicht ableiten soll/ darf.

Dialog aber ist abstrakt und explizit zum Ableiten gedacht. Steht auch alles in der API...

 

[pocketom]

Also, von der Klasse Dialog ableiten tut es einwandfrei. Sieht ordentlich aus, lässt sich schön flexibel einsetzen und eröffnet neuen Raum für kreative Ideen. Dialoge sollte man wirklich so aufbauen. Danke für die Hilfe!