SWT SWT Elemente aus anderen Klassen aufrufen - Invalid thread access

[DavidRei]

Hallo,
ich habe nun seit ca 2 Tagen das selbe Problem. Will mir für meine ganzen Programme nu GUI schreiben aber irgendwas stimmt nicht. Ich bekomme ständig folgende Fehlermeldung:

Exception in thread "main" org.eclipse.swt.SWTException: Invalid thread access
at org.eclipse.swt.SWT.error(Unknown Source)
at org.eclipse.swt.SWT.error(Unknown Source)
at org.eclipse.swt.SWT.error(Unknown Source)
at org.eclipse.swt.widgets.Display.checkDisplay(Unknown Source)
at org.eclipse.swt.widgets.Display.create(Unknown Source)
at org.eclipse.swt.graphics.Device.<init>(Unknown Source)
at org.eclipse.swt.widgets.Display.<init>(Unknown Source)
at org.eclipse.swt.widgets.Display.<init>(Unknown Source)
at org.adressverwaltung.start.ui.UI_Buttons.<init>(UI_Buttons.java:9)
at org.adressverwaltung.start.ui.UI_Shell.<init>(UI_Shell.java:10)
at org.adressverwaltung.start.Start.main(Start.java:8)

Möchte für alle grafischen Elemente eigene Klassen nutzen und dann von dort aus starten, da die UI Klasse irgendwann zu groß werden würde.

Es funktioniert alles wunderbar solange ich alles in eine Klasse stecke aber wie gesagt wird das recht unübersichtlich wenn die Programme etwas größer sind.

3 Klassen:
Start, UI_Shell, UI_Buttons

public class Start {

	public static void main(String[] args) {
		UI_Shell ui_shell = new UI_Shell();
		ui_shell.openShell();
	}

}

public class UI_Shell {
	private Display display = new Display();
	private Shell shell = new Shell(display);
	private UI_Buttons bt = new UI_Buttons();
	
	
	
	public void openShell(){
		
		shell.open();
		shell.setBounds(50, 50, 600, 750);
		bt.setButton();
		while (!shell.isDisposed()){
			if(!display.readAndDispatch()){
				display.sleep();
			}
		}
		display.dispose();
	}
}

public class UI_Buttons {
	private Display display = new Display();
	private Shell shell = new Shell(display);
	private Button bt = new Button(shell, SWT.Selection);
	
	public void setButton(){
		bt.setBounds(50 ,50, 20, 20);
		bt.setText("TEST");
	}
}

Es ist wahrscheinlich n absolut blöder Fehler aber ich finde ihn nicht.

Schon mal Danke im Voraus

 

[dzim]

Zum einen. Tutorial zum Thema "Event Loop".
SWT - Tutorial

Du machst hier zwei Fehler: 1) Der Event-Loop deiner Unterklasse wird nicht gestartet. Aber das ist eigentlich auch egal, denn du willst ja eigentlich nicht, dass deine Buttons in einem neuen Fenstern (Shell) angezeigt werden sollen, sondern im gleichen, wie das Hauptfenster. Also ist dein Fehler Nummer 2) der, dass du überhaupt mehrere Displays und Shells anlegst.

Dazu schau mal, wie in dem gleichen Tutorial von oben, Layouts gehandhabt werden.
SWT - Tutorial

Da sollte in etwa klar werden, wie du deine UI erweiterst.

Ich hab schon ewig (ca. 1.5 Jahre) nichts mehr mit SWT gemacht, kurz probiert und es so gelöst:

Main-Klasse:

import org.eclipse.swt.SWT;

public class Main {
	
	protected Shell shell;
	
	/**
	 * Launch the application.
	 * 
	 * @param args
	 */
	public static void main(String[] args) {
		try {
			Main window = new Main();
			window.open();
		} catch (Exception e) {
			e.printStackTrace();
		}
	}
	
	/**
	 * Open the window.
	 */
	public void open() {
		Display display = Display.getDefault();
		createContents();
		shell.open();
		shell.layout();
		while (!shell.isDisposed()) {
			if (!display.readAndDispatch()) {
				display.sleep();
			}
		}
	}
	
	/**
	 * Create contents of the window.
	 */
	protected void createContents() {
		shell = new Shell();
		shell.setSize(450, 300);
		shell.setText("SWT Application");
		shell.setLayout(new GridLayout(1, false));
		Test test = new Test(shell, SWT.NONE);
		GridData gd_test = new GridData(SWT.FILL, SWT.CENTER, true, true, 1, 1);
		test.setLayoutData(gd_test);
	}
}

Die Klasse "Test":

import org.eclipse.swt.SWT;
import org.eclipse.swt.layout.GridData;
import org.eclipse.swt.layout.GridLayout;
import org.eclipse.swt.widgets.Button;
import org.eclipse.swt.widgets.Composite;
import org.eclipse.swt.widgets.Label;
import org.eclipse.swt.events.SelectionAdapter;
import org.eclipse.swt.events.SelectionEvent;

public class Test extends Composite {
	
	/**
	 * Create the composite.
	 * 
	 * @param parent
	 * @param style
	 */
	public Test(Composite parent, int style) {
		super(parent, style);
		setLayout(new GridLayout(2, false));
		
		Label lblTest = new Label(this, SWT.NONE);
		GridData gd_lblTest = new GridData(SWT.FILL, SWT.CENTER, false, false, 1, 1);
		gd_lblTest.grabExcessHorizontalSpace = true;
		lblTest.setLayoutData(gd_lblTest);
		lblTest.setText("Test");
		
		Button btnNewButton = new Button(this, SWT.NONE);
		btnNewButton.addSelectionListener(new SelectionAdapter() {
			@Override
			public void widgetSelected(SelectionEvent e) {
			}
		});
		btnNewButton.setText("Do something");
		
	}
	
	@Override
	protected void checkSubclass() {
		// Disable the check that prevents subclassing of SWT components
	}
}

Wenn du Eclipse nutzt (machst du das?), dann verwende den WindowBuilder (findest du in "Help" > "Install New Software"). Ich hab den früher immer für Prototypen verwendet, die Controller und das Feintuning hab ich immer "von Hand" gemacht.

 

[DavidRei]

Schon mal Danke im Voraus, werd mir alles gleich mal anschauen. Ja ich nutze Eclipse. Hab auch WindowBuilder aber um solche Builder richtig zu checken find ichs sinnvoll mal alles von Hand zu machen. So weiss man was der Builder so macht

 

[DavidRei]

Vielen Dank für die Hilfe. Läuft jetzt alles einwandfrei. Hab mich allerdings für FormLayout entschieden.

;-)

 

[dzim]

Ich hab erst gestern oder so gelesen, das die Performance von FormLayout nicht so toll sein soll. War früher auch mein Favorit, bis ich gelernt habe, mit dem GridLayout richtig umzugehen. Das FormLayout ist für manche Spezialfälle unabdingbar, aber meist reicht das GridLayout.

Der WindowBuilder ist insofern praktisch, dass man schnell sieht, wie man etwas zusammenstellen muss. Perfekt ist er aber nicht und es lohnt sich daher, immer von Hand noch etwas zu verbessern (habe ich selbst in dem kleinen Beispiel gestern gemacht).
Und man kann extrem schnell Prototypen basteln. die Arbeit, dass die dann auch was machen, hat man hinterher sowieso, daher will ich immer den Design-Teil schnell hinter mit haben.

 

[dzim]

Schau mal, ob das dir hier helfen kann:

Slim Down SWT FormLayout Usage | Code Affine

 

[DavidRei]

Ja ok, hab gestern gemerkt dass das Positionieren alles Elemente schon einige Zeit in Anspruch nimmt.

Danke für den Link ;-)