Wieder einmal Event-Dispatch-Thread

[AMiGA]

Hallo,

ich habe leider immer noch ein Verständnisproblem bzgl. des Event-Dispatch-Threads. Angenommen, ich möchte bei einem Knopfdruck Daten aus einer Datenbank lesen und dem Benutzer dies vorher mitteilen, dann würde der Code folgendermaßen aussehen:

public void actionPerformed(ActionEvent evt) {
    textField.setText("Es werden nun Daten aus der DB gelesen");
    Database.execQuery("...");
    textField.setText("Anfrage erfolgreich");
}

Nun würde die Ausgabe "Es werden Daten gelesen" aber erst *nach* Beendigung der DB-Anfrage erscheinen. Daher müsste man wohl mit Hilfe eines zusätzlichen Threads die DB-Anfrage entkoppeln:

public void actionPerformed(ActionEvent evt) {
    textField.setText("Es werden nun Daten aus der DB gelesen");
    new Thread() {
        public void run() {
            Database.execQuery("...");

            SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                    textField.setText("Anfrage erfolgreich");
                }
            }
    }.start();
}

Dies ist für meine Begriffe aber äußerst unschön. Gibt es keine andere (="sauberere") Möglichkeit, das GUI aus dem Event-Dispatch-Thread zu aktualisieren?

Gruß,
AMiGA

 

[Mag1c]

Hi,

müsste auch so gehen:

public void actionPerformed(ActionEvent evt) {
    new Thread() {
        public void run() {
            textField.setText("Es werden nun Daten aus der DB gelesen");
            Database.execQuery("...");
            textField.setText("Anfrage erfolgreich");
            }
    }.start();
}

was ist daran unschön ?

Gruß
Mag1c

 

[AMiGA]

was ist daran unschön

Mir gefällt das Multithreading und das Asynchrone dabei nicht wirklich. Nun muss ich mich darum kümmern, dass sämtliche Komponenten gesperrt werden, damit der Benutzer so lange nichts anstellt, bis die DB-Anfrage beendet ist. Bislang versuche ich eigentlich, Threads wenn irgendwie möglich zu vermeiden, um die Fehlerträchtigkeit gering zu halten.

Gruß,
AMiGA

 

[Mag1c]

Hi,

damit geht es:

textField.paintImmediately(textField.getBounds());

Gruß
Mag1c

 

[Beni]

Um Multithreading, wie du es am Anfang gepostet hast, wirst du nicht herumkommen (es sei denn, du baust gerne potentielle Fehlerquellen ein ... wie Magic1 krampfhaft versucht vorzuführen :bae.

Du musst das ganze ja nicht so offensichtlich machen (und in jeder Methode extra einen Thread starten...), schau dir mal das Command-Pattern an. Da machst du ein Interface "Command" mit einem Methode "work", und gibst diese Commands in irgendeine Liste (die von einem anderen Thread abgearbeitet wird). Die Commands können natürlich auch direkt vom EventDispatcher aufgerufen werden, mit dem invokeAndWait...

actionPerformed( ... ){
  commandListe.execute( new Command(){
     public void work(){
        textField.setText( "Warten" );
     }, true // true: im EventDispatcher
  }

  commandListe.execute( new Command(){
     public void work(){
        ladeDBZeugsUndSpeichereEs();
     }, false // false: in eigenem Thread
  }

  commandListe.execute( new Command(){
     public void work(){
        textField.setText( "Fertig" );
     }, true
  }
}

 

[Mag1c]

Hi,

kann mich Beni nur anschließen. Wenn nämlich jemand während der DB-Abfrage schnell mal ein Spielchen macht und anschließend zu der Anwendung zurückgeht, dann sieht er dort (falls die DB-Abfrage noch läuft) nur ein leeres Fenster ... sehr unschön !

Gruß
Mag1c