Screen nur 1x öffnen können

[internet]

Hallo,

ich habe den folgenden Code und möchte gerne, dass der Display / Screen nur 1x geöffnet werden kann.
D.h. ich rufe die Methode auf und ein Display / Screen öffnet sich (was es Stand jetzt auch macht).
Nun rufe ich noch einmal die Methode auf, aber es soll der verwendete Screen nur aktualisiert werden.
Aktuell ist es aber so, dass ich dann in der Task - Leiste einen zweiten Prozess (und ein weiteres Window) erhalten.

So sieht der Code aktuell aus:
Display display speichere ich aktuell auch in einem Layer drüber, sodass es nicht ständig neu erzeugt wird...

public static void openScreen(Display display, String url, int width, int height, String titleText,
            boolean fullscreen, boolean exitWithEsc, int exitKey) {

        LOGGER.info("START openScreen");

        display.asyncExec(new Runnable() {
            public void run() {

                final Shell shell = new Shell(display);

                if (exitWithEsc) {
                    shell.addListener(SWT.Traverse, new Listener() {

                        @Override
                        public void handleEvent(Event event) {
                            if (event.character == exitKey) {

                                event.doit = false;
                                shell.close();
                               
                                display.close();
                            }

                        }
                    });
                }

                shell.setSize(width, height);
                shell.setFullScreen(fullscreen);
                shell.setText(titleText);

                shell.setLocation(0, 0);
                GridLayout gridLayout = new GridLayout();
                gridLayout.numColumns = 3;
                gridLayout.marginWidth = 0;
                gridLayout.marginHeight = 0;

                shell.setLayout(gridLayout);

                final Browser browser = new Browser(shell, SWT.NONE);
                browser.setJavascriptEnabled(true);

                GridData data = new GridData();
                data.horizontalAlignment = GridData.FILL;
                data.verticalAlignment = GridData.FILL;
                data.horizontalSpan = 3;
                data.grabExcessHorizontalSpace = true;
                data.grabExcessVerticalSpace = true;
                data.widthHint = width;
                data.heightHint = height;
                browser.setLayoutData(data);

                shell.open();

                browser.setUrl(url);

            }
        });

        LOGGER.info("END openScreen");
        return;
    }

Ich schätze das Problem ist dadurch, dass ich mehrere Threads erstelle?

display.asyncExec(new Runnable() {

Lasse ich das weg, bekomme ich eine Exception (invalid Thread Access)...
Was muss ich hier ändern?

 

[KonradN]

Wie wäre es, einfach einmal den Code sauber zu strukturieren statt alles in eine Methode zu werfen?

Dann wird es auch deutlich einfacher. sowas umzusetzen. Denn dann hast Du im Programm eine Instanz und die kennt den aktuellen Zustand und kann aktiv werden.

Die weiss dann z,B.: Es muss ein neues Fenster geöffnet werden bzw. es hat das offene Fenster und kann es aktualisieren.

Oder in anderen Worten:

Deine Probleme sind:
a) Du baust einfach alles in einer Methode mit lokalen Variablen auf
b) Du hast selbst eine Komplexität erreicht, die Dich offensichtlich den Überblick verlieren lässt.

Anzuwendende "Pattern":
a) Du willst Code sauber in Klassen aufteilen., Code gehört dann immer an die entsprechend richtige Stelle. (Bei Oberflächen haben sich Pattern wie MVC, MVVM, MVP, ... recht gut bewährt.)
b) Teile und Herrsche - Du willst den Code so unterteilen, dass Du kleine, übersichtliche Teile hast, die Du einfach und unproblematisch beliebig zusammen fügen kannst.

 

[internet]

Wie wäre es, einfach einmal den Code sauber zu strukturieren statt alles in eine Methode zu werfen?

Dann wird es auch deutlich einfacher. sowas umzusetzen. Denn dann hast Du im Programm eine Instanz und die kennt den aktuellen Zustand und kann aktiv werden.

Die weiss dann z,B.: Es muss ein neues Fenster geöffnet werden bzw. es hat das offene Fenster und kann es aktualisieren.

Oder in anderen Worten:

Deine Probleme sind:
a) Du baust einfach alles in einer Methode mit lokalen Variablen auf
b) Du hast selbst eine Komplexität erreicht, die Dich offensichtlich den Überblick verlieren lässt.

Anzuwendende "Pattern":
a) Du willst Code sauber in Klassen aufteilen., Code gehört dann immer an die entsprechend richtige Stelle. (Bei Oberflächen haben sich Pattern wie MVC, MVVM, MVP, ... recht gut bewährt.)
b) Teile und Herrsche - Du willst den Code so unterteilen, dass Du kleine, übersichtliche Teile hast, die Du einfach und unproblematisch beliebig zusammen fügen kannst.

danke...
Aber ich brauche quasi zwei Instanzen...

Ich habe ein Window, das geöffnet bleibt.
Nun möchte ich von diesem Window einen weiteren Window öffnen.
Hier ist aber eben das Problem, dass ich das Window nicht nur 1x öffnen kann, sondern x mal...

Der Code soll mir nur ein zweites Window öffnen oder falls eines vorhanden ist bereits dies aktualisieren.
Die Instanz (Display) davon erstelle ich bereits in der Präsentationsschicht und schleife diese weiter (Parameter Display in der Methode...)

 

[KonradN]

Du hast ein Programm. Und in dem musst Du Deine Daten halt vernünftig halten.

Und da ist das, was Du da haben willst doch unproblematisch. Du kannst dann eine Instanz einer Klasse haben, die die Verantwortung hat für dieses spezielle Fenster. Und von dieser Klasse hast Du eine Instanz. Von mir aus als Singleton (Wobei ich das kritisch sehe, aber am Anfang mag es das Handling vereinfachen. Nur eben gewöhnt man sich damit etwas in meinen Augen Schlechtes an). Dann kann Dein Programm von beliebig vielen Stellen aus darauf zugreifen und das Öffnen des Fensters anfordern.

Und diese Instanz schaut dann: Gibt es das besagte Fenster noch nicht? Dann wird es geöffnet. Danach werden die Daten gesetzt / aktualisiert.

Das kann man sogar in Deinen Code mit einbauen - was soll man auch Dinge sauber trennen:

• Du fügst eine Klassenvariable ein, die die Shell speichert.
• Da Du eine final Variable brauchst, wird das zu final Shell shell = getShell();
• In der neuen Methode getShell prüfst Du, ob in der Klassenvariable schon etwas in einem gültigen Zustand ist. Wenn ja, dann wird das zurück gegeben, wenn nicht, dann wird eine neue Shell erzeugt mit allem, was zur Initialisierung notwendig ist.

Vermutlich wirst Du etwas in der Art mit dem GridData machen wollen, damit Du das anpassen kannst statt es nu zu erzeugen.

Aber wie gesagt: Sowas wird schlecht überschaubar. Das würde ich sauber aufteilen. Dann hast Du auch nicht so Probleme wie diese final (oder quasi final) Variablen. Du hättest dann auch nicht diese static Elemente.

 

[internet]

Ich habe bereits ein Singleton, welches ich in einer Variable speichere (Display display). Dies übergebe ich dann der oben erstellten Methode..
Der oben erstellte Code ist nur eine Art Helper Klasse, damit ich andere Fenster vllt später mal öffnen kann..

Ich vermute aber dass diese Code stelle das Problem ist:

display.asyncExec(new Runnable() {

Daher die Frage, was muss hier geändert werden, damit ich einen neuen Thread hier erstelle (falls nicht vorhanden) oder eben den aktuellen Thread nehme?

 

[KonradN]

Ich vermute aber dass diese Code stelle das Problem ist

Nein, das ist nicht das Problem!

Damit führst Du doch nur etwas im User Interface Thread aus.

Display (Eclipse Platform API Specification)

 

[internet]

Ok, dann verstehe ich es nicht...

Ich habe das. Das heißt, dass ich doch immer erstmal prüfe, ob Display bereits instanziert ist.
Wenn nein, erstellt es ein neues Objekt. Falls doch, nimmt es das Objekt.
Jedoch wird trotzdem immer ein neues Fenster erzeugt?

@Singleton
public class MyExecutionServiceBean {

    private Display display;

    public void loadScreen(){
    
        if(display == null) {
            display = Display.getDefault();
        }
        
        // Screen öffnen
        SWTScreenHelper.openScreen(display, "https://google.com", 300, 400, "Test",
                true, true, SWT.ESC);
        
    }

 

[KonradN]

Ich habe das. Das heißt, dass ich doch immer erstmal prüfe, ob Display bereits instanziert ist.
Wenn nein, erstellt es ein neues Objekt. Falls doch, nimmt es das Objekt.

Display wird nie von Dir erzeugt! Du hast in der Klasse eine Referenz auf ein Display und wenn diese noch nicht gesetzt wird, wird die Variable mit dem default Display gesetzt.

Jedoch wird trotzdem immer ein neues Fenster erzeugt?

Display ist doch auch kein Fenster.

Dokumentation von Display:

Instances of this class are responsible for managing the connection between SWT and the underlying operating system. Their most important function is to implement the SWT event loop in terms of the platform event model. They also provide various methods for accessing information about the operating system, and have overall control over the operating system resources which SWT allocates.

Applications which are built with SWT will almost always require only a single display. In particular, some platforms which SWT supports will not allow more than one active display. In other words, some platforms do not support creating a new display if one already exists that has not been sent the dispose() message.

Das, was das Fenster erzeugt, ist im letzten Code das openScreen und davor halt die Erstellung einer neuen Shell Instanz (+ Anzeige dieser natürlich)
Dokumentation von Shell:

Instances of this class represent the "windows" which the desktop or "window manager" is managing. Instances that do not have a parent (that is, they are built using the constructor, which takes a Display as the argument) are described as top level shells. Instances that do have a parent are described as secondary or dialog shells.

Evtl. wäre es hilfreich, sich die Grundlagen von SWT etwas anzusehen, z.B. unter

Javabeginners - SWT Grundlagen

javabeginners.de

 

[internet]

ich habe es nun hinbekommen, nachdem ich Shell entsprechend verwendet habe...
Nun habe ich aber noch ein Problem:
- In der Seite, welche ich über den SWT Browser aufrufe, starte ich Musik (Primefaces nutze ich hierfür):

<p:audio
value="http://download.lisztonian.com/music/download/Larghetto+in+F-62.mp3"
player="mp3" autoplay="true" controls="false" />

Anhand der ESC - Taste habe ich definiert, dass das Fenster geschlossen werden soll, wenn diese betätigt wird:
Sie Code Block, den ich bereits gepostet habe...

                if (exitWithEsc) {
                    currentShell.addListener(SWT.Traverse, new Listener() {

                        @Override
                        public void handleEvent(Event event) {
                            if (event.character == exitKey) {

                                event.doit = false;
                                currentShell.close();
                                                        
                                displayShell = null;
                            }

                        }
                    });
                }

Nun ist das Problem aber, dass die Musik weiter abspielt.
Es scheint mir also, dass die Seite irgendwie im Hintergrund doch noch aktiv ist oder ähnliches.

Hier nun mein kompletter Code:

private Shell displayShell;

private void openScreen(String url, int width, int height, String titleText, boolean fullscreen,
            boolean exitWithEsc, int exitKey) throws ScreenAlreadyOpenException{

        LOGGER.info("START openScreen");

        Display display = Display.getDefault();
        
        if(displayShell != null) {
            throw new ScreenAlreadyOpenException();
        }

        display.asyncExec(new Runnable() {
            public void run() {

                final Shell currentShell;

                if (displayShell == null) {
                    currentShell = new Shell(display);
                } else {
                    currentShell = displayShell;
                }
                
                displayShell = currentShell;

                if (exitWithEsc) {
                    currentShell.addListener(SWT.Traverse, new Listener() {

                        @Override
                        public void handleEvent(Event event) {
                            if (event.character == exitKey) {

                                event.doit = false;
                                currentShell.close();
                                                        
                                displayShell = null;
                            }

                        }
                    });
                }

                currentShell.setSize(width, height);
                currentShell.setFullScreen(fullscreen);
                currentShell.setText(titleText);

                currentShell.setLocation(0, 0);
                GridLayout gridLayout = new GridLayout();
                gridLayout.numColumns = 3;
                gridLayout.marginWidth = 0;
                gridLayout.marginHeight = 0;

                currentShell.setLayout(gridLayout);

                final Browser browser = new Browser(currentShell, SWT.NONE);
                browser.setJavascriptEnabled(true);

                GridData data = new GridData();
                data.horizontalAlignment = GridData.FILL;
                data.verticalAlignment = GridData.FILL;
                data.horizontalSpan = 3;
                data.grabExcessHorizontalSpace = true;
                data.grabExcessVerticalSpace = true;
                data.widthHint = width;
                data.heightHint = height;
                browser.setLayoutData(data);

                currentShell.open();

                browser.setUrl(url);

            }
        });

        LOGGER.info("END openScreen");
        return;
    }

Eine Idee, wie ich das Problem lösen kann?

 

[KonradN]

Wenn Du es über die Bowser Instanz abspielst: Wie wäre es, denn Du die Instanz merkst um da dann zu schauen, was da evtl. Abhilfe schaffen kann. Da gibt es ja viele Ideen:

• Evtl. reicht es, da dispose() aufzurufen?
• Evtl. eine neue URL setzen?
• Evtl. ein Script ausführen lassen, dass das Abspielen von Audio stoppt. (Siehe z.B. https://stackoverflow.com/questions/14834520/html5-audio-stop-function)

Es gibt also generell viele Ideen und Möglichkeiten, die man testen könnte.

 

[internet]

Wenn Du es über die Bowser Instanz abspielst: Wie wäre es, denn Du die Instanz merkst um da dann zu schauen, was da evtl. Abhilfe schaffen kann. Da gibt es ja viele Ideen:

• Evtl. reicht es, da dispose() aufzurufen?
• Evtl. eine neue URL setzen?
• Evtl. ein Script ausführen lassen, dass das Abspielen von Audio stoppt. (Siehe z.B. https://stackoverflow.com/questions/14834520/html5-audio-stop-function)

Es gibt also generell viele Ideen und Möglichkeiten, die man testen könnte.

dispose() aufzurufen hat schon gereicht.
Vielen lieben Dank!

 

[internet]

ich habe doch noch eine Frage bzgl. Fullscreen....

Ich hätte gerne einen kompletten Fullscreen. Beim Mac zeigt es mir dennoch die obere Leiste an.
Wie kann ich das verhindern?

                final Shell currentShell = new Shell(display);
                currentShell.setSize(width, height);
                currentShell.setFullScreen(fullscreen);
                currentShell.setText(titleText);

                currentShell.setLocation(0, 0);
                GridLayout gridLayout = new GridLayout();
                gridLayout.numColumns = 3;
                gridLayout.marginWidth = 0;
                gridLayout.marginHeight = 0;

                currentShell.setLayout(gridLayout);

                browser = new Browser(currentShell, SWT.NONE);
                browser.setJavascriptEnabled(true);

                GridData data = new GridData();
                data.horizontalAlignment = GridData.FILL;
                data.verticalAlignment = GridData.FILL;
                data.horizontalSpan = 3;
                data.grabExcessHorizontalSpace = true;
                data.grabExcessVerticalSpace = true;
                data.widthHint = width;
                data.heightHint = height;
                browser.setLayoutData(data);

                browser.setUrl(url);

                currentShell.pack();
                currentShell.open();
                
                while (!currentShell.isDisposed()) {
                    if (!display.readAndDispatch()) {
                        display.sleep();
                    }
                }

 

[internet]

Ok, klappt wohl mit diesem Hack:

Java SWT application - Bring To Front

I am currently developing an SWT java application on Windows 7. Usually the application will be minimized, and when there is an event on the serial port the application should maximizes itself. The

stackoverflow.com

 

[samsoo]

Hallo

new Runnable() {
            public void run() {

                final Shell shell = new Shell(display);

                if (exitWithEsc) {
                    shell.addListener(SWT.Traverse, new Listener() {

                        @Override
                        public void handleEvent(Event event) {
                            if (event.character == exitKey) {

                                event.doit = false;
                                shell.close();
                              
                                display.close();
                            }

                        }
                    });
                }

                shell.setSize(width, height);
                shell.setFullScreen(fullscreen);
                shell.setText(titleText);

                shell.setLocation(0, 0);
                GridLayout gridLayout = new GridLayout();
                gridLayout.numColumns = 3;
                gridLayout.marginWidth = 0;
                gridLayout.marginHeight = 0;

                shell.setLayout(gridLayout);

                final Browser browser = new Browser(shell, SWT.NONE);
                browser.setJavascriptEnabled(true);

                GridData data = new GridData();
                data.horizontalAlignment = GridData.FILL;
                data.verticalAlignment = GridData.FILL;
                data.horizontalSpan = 3;
                data.grabExcessHorizontalSpace = true;
                data.grabExcessVerticalSpace = true;
                data.widthHint = width;
                data.heightHint = height;
                browser.setLayoutData(data);

                shell.open();

                browser.setUrl(url);

            }.run();

sollte klappen

(nicht start aufrufen!!!)

 

[mrBrown]

Hallo

…

sollte klappen

(nicht start aufrufen!!!)

Auf runnable kann man dann aber auch verzichten und den Code direkt ausführen.

 

[samsoo]

Auf runnable kann man dann aber auch verzichten und den Code direkt ausführen.

Kommt auf die abgeschlossenheit an.

 

[mrBrown]

Kommt auf die abgeschlossenheit an.

Keine Ahnung was „Abgeschlossenheit“ ist, aber das Runnable ist trotzdem überflüssig.

 

[KonradN]

Das display.asyncExec ist ja dazu da, um auf den user-interface thread zu wechseln. UI Libraries mögen es in der Regel weniger, wenn diese von außerhalb des user-interface threads aufgerufen werden.

Daher wird da auch nirgends start aufgerufen - es ist ja kein Thread. Es wird nur Runnable als Interface genutzt damit das eben übergeben werden kann.