FileChooser Layout - Threads?

[ulr!ch]

Hi JavaGemeinde,

ich habe ein Dateiauswahlmenü, in dem man eine oder mehrere Dateien auswählen kann, die dann "ausgewertet" werden. Einige Daten werden in einer Tabelle angezeigt, wieder andere werden zwischengespeichert usw.
Während der "Auswertung", die eine gewisse, schwer abzuschätzende Zeit dauert, wird an der Stelle, an der zuvor der Kasten für die Dateiauswahl war, ein unerwünschter grauer Hintergrund angezeigt. Das sieht stümperhaft aus, was es ja auch ist.
Wie kann ich das verhindern, dass die GUI erst nach der Auswertung upgedatet wird?
Muss ich da mit Threads arbeiten?

By<e Ulrich

 

[Mag1c]

Jawoll,

du DARFST mit Threads arbeiten. Etwas mehr Freude bitte, Threads sind was feines

Gruß
Mag1c

 

[ulr!ch]

Jaja, was ganz Feines.
Also wenn ich das richtig verstanden habe, dann muss man die Berechnung/ Auswertung von der GUI trennen. Aber ich kann mir das ehrlich gesagt nicht ganz vorstellen. Sagen wir mal - der Einfachheit halber - ich implementiere eine JProgressBar über ( progressBar.setIndeterminate(true); ).
Also das Layout soll solange einen hin- und hersausenden Balken zeigen, bis

public class A extends JFrame implements runnable {
 public A() {
 }

 public void run() {
// solange Klasse B (wie Berechnung) nicht fertig ist, zeige die JProgressBar an
// hier also nur die GUI?
 }

 public B {
// eigentlichen Berechnungen
 }
}

Das stimmt so vom Aufbau nicht, oder?
Ich habe zwar den Thread zu JProgressBar in der FAQ gelesen, habe aber mit dem Aufbau Schwierigkeiten.
Vielleicht kann mir jemand die Struktur s. o. verdeutlichen.

Thx!

By<e Ulrich

 

[Mag1c]

Naja,

noch nicht so richtig:

public class A extends JFrame {
    ...
    // ProgressBar erzeugen
    // Thread erzeugen und starten
}

public class B implements Runnable {
    public B (JProgressBar pb) {
        ...
    }

    public void run () {
        // hier rechnen und ProgressBar aktualisieren
    }
}

Gruß
Mag1c

 

[ulr!ch]

Naja,

noch nicht so richtig:

public class A extends JFrame {
    ...
    // ProgressBar erzeugen
    // Thread erzeugen und starten
}

Super, danke schon einmal, aber ich habe natürlich noch Fragen.
Muss bei Klasse A nicht auch "implements Runnable" stehen, sonst kann ich doch keinen Thread erzeugen und starten? Hoffentlich klappt das. Aber das man GUI und Berechnungen trennt - das ist schon mal der richtige Weg?

By<e Ulrich

 

[Mag1c]

Moin,

es gibt viele Möglichkeiten, wie man das umsetzen kann. Das da oben ist nur ein Beispiel. Das Runnable mußt du dort implementieren, wo du etwas durch einen Thread ausführen lassen willst. Bei der Erzeugung des Threads mußt du dann eine Instanz des Runnable an den Thread übergeben:

new Thread(new B(progress)).start();

Und ja, natürlich ist das eine gute Idee, die GUI von der Berechnung zu trennen (vorausgesetzt, daß die beiden Dinge nicht zu stark miteinander verkoppelt sind).

Gruß
Mag1c

 

[ulr!ch]

Hi JavaGemeinde,

bin langsam am Ausrasten.
Habe nun das Beispiel in der FAQ versucht "zu adaptieren", aber selbst in seiner ursprünglichen Version bekomme ich immer die Fehlermeldung:

java.lang.Error: Cannot call invokeAndWait from the event dispatcher thread

Hier mal der eigentliche Quellcode:

package Auswertung;

import java.io.*;
import java.lang.reflect.*;

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

import org.jfree.ui.*;
import Methoden.*;
import MyComponents.*;

public class Exportieren extends JFrame implements Runnable {
 /** Der Algorithmus, der ausgeführt werden soll */
 private Algorithmus algorithmus;
/** Der Fortschrittsbalken */
 private JProgressBar progress;
/** Verschiedene Knöpfe um den Algorithmus zu steuern */
 private JButton buttonStart, buttonStop;

  public Exportieren(int t, int azm, FileManagement ergebnis,
		     JPanel HG, JPanel inner_layer) {
    this.t = t;
    this.azm = azm;
    this.ergebnis = ergebnis;
    this.HG = HG;
    this.innerLayer = inner_layer;

    {
      try {
	jbInit();
      }
      catch (Exception e) {
	e.printStackTrace();
      }
    }
  }

  public void jbInit() throws Exception {
    innerLayer.setVisible(false);
    innerLayer.removeAll();
    innerLayer.setVisible(true);
    innerLayer.setBackground(new MyColor("weiss").getMyColor());

    progress = new JProgressBar();
    buttonStart = new JButton( "Start" );
    buttonStop = new JButton( "Stop" );
    algorithmus = new Algorithmus();

// Wenn der Algo nicht läuft, kann man ihn auch nicht stoppen
    buttonStop.setEnabled( false );

// Der folgende Teil ist nur für das Auge, eine Benutzeroberfläche die
// etwas dynamisch ist, kommt immer gut an.
   JPanel panel = new JPanel( new GridLayout( 1, 2 ));
   panel.add( buttonStart );
   panel.add( buttonStop );

   innerLayer.add( progress, new GridBagConstraints( 0, 0, 1, 1, 1.0, 1.0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets( 2, 2, 2, 2 ), 0, 0) );
   innerLayer.add( panel, new GridBagConstraints( 0, 1, 1, 1, 1.0, 1.0, GridBagConstraints.SOUTHEAST, GridBagConstraints.NONE, new Insets( 2, 2, 2, 2 ), 0, 0) );      
   HG.add(innerLayer, new GridBagConstraints(0, 1, 1, 1, 0.0, 0.0, GridBagConstraints.CENTER, GridBagConstraints.NONE, new Insets(0, 0, 0, 0), 0, 0));
   HG.updateUI();

// Die Buttons sollen ja auch reagieren können
   buttonStart.addActionListener( new ActionListener(){
   public void actionPerformed(ActionEvent e) {
      start();
   }
});

buttonStop.addActionListener( new ActionListener(){
   public void actionPerformed(ActionEvent e) {
      stop();
   }
});

   setDefaultCloseOperation( EXIT_ON_CLOSE );
  }

 /**
     * Startet den Algorithmus, das Verhalten ist nicht definiert, wenn der
     * Algorithmus bereits aktiviert wurde.
     */
    protected void start(){
       buttonStart.setEnabled( false );
 
       // Einen neuen Thread herstellen, und starten.
       new Thread( this ).start();
 
       buttonStop.setEnabled( true );
    }
 
    /**
     * Stoppt den Algorithmus, immer unter der Annahme, dass er zurzeit
     * läuft.
     */
    protected void stop(){
       buttonStop.setEnabled( false );
       algorithmus.setInterrupted( true );
    }
 
    /**
     * Die Methode wird aus einem Zusatzthread aufgerufen. Sie startet
     * den Algorithmus, und sorgt nach seiner Beendigung dafür, dass das
     * Fenster weiterhin schön aussieht
     */
    public void run(){
       // Zurzeit ist der Algorithmus nicht unterbrochen
       algorithmus.setInterrupted( false );
 
       // Den Algorithmus starten. Die Methode kehrt erst zurück, wenn der
       // Algo abgearbeitet oder unterbrochen wurde.
       algorithmus.run( progress );
 
       if( algorithmus.isInterrupted() ){
	  /*
	   * Sollte der Algorithmus abgebrochen worden sein, wird die
	   * Progressbar zurück auf den Startwert gesetzt.
	   *
	   * Die Methode "invokeLater" garantiert, dass dass der Code des
	   * übergebenen Runnable's synchronisiert zum Dispatch-Thread
	   * ausgeführt wird. Dies ist notwendig, weil Swing nicht
	   * threadsicher ist.
	   */
	  SwingUtilities.invokeLater( new Runnable(){
	     public void run(){
		progress.setMinimum( 0 );
		progress.setMaximum( 1 );
		progress.setValue( 0 );
		buttonStart.setEnabled( true );
	     }
	  });
       }
       else{
	  /*
	   * Der Algorithmus wurde korrekt beendet, ein Dialog soll den
	   * Benutzer unterrichten, dass das Programm nun geschlossen wird.
	   *
	   * Die Methode "invokeAndWait" garantiert, dass der Code des
	   * Runnable-Objektes, welches übergeben wird, synchron zum
	   * Dispath-Thread ausgeführt wird. Ausserdem kehrt die Methode
	   * erst dann zurück, wenn eben dieser Code ausgeführt wurde.
	   */
	  try {
	     SwingUtilities.invokeAndWait( new Runnable(){
		public void run(){
		   JOptionPane.showMessageDialog( Exportieren.this,
			 "Algorithmus erfolgreich beendet.", "Beendet",
			 JOptionPane.INFORMATION_MESSAGE );
		}
	     });
	  } catch (InterruptedException e) {
	     // Kann ja sein, das hier was schiefgeht...
	     e.printStackTrace();
	  } catch (InvocationTargetException e) {
	     e.printStackTrace();
	  }
 
	  // Programm beenden, nachdem der Dialog geschlossen wurde
	  System.exit(0);
       }
   }
}

Den zweiten Teil des Codes habe ich in eine externe Klasse gelegt.
Wieso gibt es diese Fehlermeldung? Ich habe die Threads zu diesem Thema im Forum gelesen, jedoch ohne Erfolg.
Ich habe doch nicht verändert. :cry:

Vielleicht kann mir jemand helfen, mag1c?

By<e Ulrich

 

[Mag1c]

Hmm,

sehr komisch :roll: Wie gehst du vor bzw. wann kommt der Fehler ? Wie rufst du die Klasse auf ? Was passiert im Algorithmus (insbesondere beim Unterbrechen) ? Ist das exakt der Code, den du ausgeführt hast ?

Gruß
Mag1c

 

[ulr!ch]

sehr komisch :roll:
a) Wie gehst du vor bzw. wann kommt der Fehler ?
b) Wie rufst du die Klasse auf ?
c) Was passiert im Algorithmus (insbesondere beim Unterbrechen) ?
d) Ist das exakt der Code, den du ausgeführt hast ?

Gruß
Mag1c

(Aufzählung eingefügt)

Also, da sind wohl eine ganze Menge Fragen noch nicht beantwortet worden.
zu b) Ich habe eine Menüleiste, wenn ich dort den Menüpunkt "Exportieren" anklicke, dann rufe ich die Klasse so auf:

Exportieren exportieren = new Exportieren(t, azm, ergebnis, HG, inner_layer);
exportieren.run();

a) Direkt beim Aufrufen der Klasse; ich brauch also nicht noch auf den Start-Knopf drücken.

c) Den Code aus dem FAQ-Bsp habe ich einfach in eine neue Klasse gelegt.
Den entsprechende Stelle ist wohl, weiß nicht, ob du dies meinst.

while( current <= end && !isInterrupted() ){
	 try{
	    Thread.sleep( 100 ); // Kurz warten
	 }
	 catch( InterruptedException e ){
	 }
	 setValue( current++ ); // Hey! Es ist was passiert!
      }

PS: Funktioniert auch nicht, wenn ich eine Instanz von Thread aufrufe.

d) Ja, es steht sonst nur auskommentiertes Zeug drin.

:roll: Wir stehen selbst enttäuscht und sehr betroffen,
der Vorhang zu und viele Fragen offen.

By<e Ulrich

 

[Mag1c]

Hi,

Exportieren exportieren = new Exportieren(t, azm, ergebnis, HG, inner_layer);
exportieren.run();

Na da haben wir doch gleich den Übeltäter Du rufst das run() aus dem EventDispatch-Thread auf, was normalerweise von einem anderen Thread ausgeführt werden soll. Warum rufst du nicht stattdessen start() auf ? Dann würden auch die Buttons korrekt behandelt ? (Und ich würde start() und stop() noch umbenennen in startAlgo() und stopAlgo() oder so, sonst kommst du noch durcheinander mit Thread#start und Thread#stop).

Gruß
Mag1c

 

[ulr!ch]

Hi Mag1c,

also ich habe start nach startAlgo und stop nach stopAlgo umbenannt, und rufe nun die Klasse mit:

    else if (action.equals("Exportieren")) {
      Exportieren exportieren = new Exportieren(t, azm, ergebnis, HG, inner_layer);
      exportieren.startAlgo();;
    }

auf. Dadurch verschwindet - wirklich die Fehlermeldung garndios! Auch öffnet sich nach ein gewissen Zeit ein OptionPane und bestätigt, dass der Algorithmus "durchgelaufen" ist, ABER die JProgressBar verändert ihr aussehen leider nicht.
Why?

By<e Ulrich

 

[ulr!ch]

Hi Mag1c,

kann es damit etwas zu tun haben, dass ich den Algorithmus in eine externe Klasse gelegt habe?

By<e Ulrich

 

[Mag1c]

Hi,

dann müsstest du doch mal zeigen, was du im Algorithmus mit dem ProgressBar anstellst.

Gruß
Mag1c

 

[ulr!ch]

Hi,

dann müsstest du doch mal zeigen, was du im Algorithmus mit dem ProgressBar anstellst.

Gruß
Mag1c

OK

package Auswertung;

import javax.swing.JProgressBar;
import javax.swing.SwingUtilities;

/**
* Der Algorithmus stellt einige langwierige Berechnungen an, für deren
* Erklärung hier leider keinen Platz gefunden werden kann.
*/
class Algorithmus{
   /**
    * Dieser Fortschrittsbalken wird benutzt, um den Fortschritt des Algos
    * anzuzeigen
    */
   private JProgressBar status;

   /**
    * Dieser Boolean wird auf "true" gesetzt, sollte der Algorithmus
    * abgebrochen werden.
    */
   private boolean interrupted = false;

   public Algorithmus(){
   }

   /**
    * Gibt an, ob der Algorithmus abgebrochen wird, oder wurde.
    * @return true, falls der Algorithmus stoppen soll
    */
   public boolean isInterrupted() {
      return interrupted;
   }

   /**
    * Gibt an, ob der Algorithmus abgebrochen werden soll.
    * @param interrupt true, falls der Algorithmus abgebrochen werden soll,
    *            andernfalls false.
    */
   public void setInterrupted( boolean interrupt ) {
      interrupted = interrupt;
   }

   /**
    * Die Hauptmethode des Algorithmuses, hier geschehen alle wichtigen
    * Berechnungen. Leider konnte die Weltformel hier nicht Platz finden,
    * deshalb beschränkt sich die Methode auf stupides hochzählen eines
    * Integers :-)
    * @param progress Der Fortschrittsbalken, welcher den Fortschritt des Algos
    *            anzeigen wird.
    */
   public void run( JProgressBar progress ) {
      // Den Fortschrittsbalken speichern wir als Instanzvariable. Das
      // hat den Vorteil, dass jede der 1000 Methoden des Algos Zugriff
      // auf die Progressbar hat.
      status = progress;

      int begin = 13;
      int end = 65;
      int current = begin;

      // Minimum und Maximum des Fortschrittbalkens setzen
      setMinMax( begin, end );

      while( current <= end && !isInterrupted() ){
	 try{
	    Thread.sleep( 100 ); // Kurz warten
	 }
	 catch( InterruptedException e ){
	 }
	 setValue( current++ ); // Hey! Es ist was passiert!
      }

      // Instanzvariable auf null setzen, damit der Garbagecollector
      // allenfalls Müll entfernen kann
      status = null;
   }

   /**
    * Setzt die minimalen und die maximalen Werte des Fortschrittbalkens. Die
    * Methode kehrt erst zurück, wenn die Werte tatsächlich gesetzt wurden.
    * @param min Das Minimum
    * @param max Das Maximum
    */
   public void setMinMax( final int min, final int max ) {
      try{
	 /*
	  * Diese Methode wird nicht aus dem Dispatcher-Thread aufgerufen. Da
	  * Swing nicht threadsicher ist, müssen wir für eine Synchronisation
	  * sorgen, deshalb wird hier "invokeAndWait" verwendet.
	  */
	 SwingUtilities.invokeAndWait(new Runnable(){
	    public void run() {
	       status.setMinimum( min );
	       status.setValue( min );
	       status.setMaximum( max );
	    }
	 });
      }
      catch( Exception e ){
      }
   }

   private void setValue( final int value ) {
      /*
       * Der Algorithmus sollte nicht unnötig angehalten werden, nur damit die
       * Progressbar schön aussieht, deshalb "invokeLater". Es kann nun aber
       * passieren, dass der Algorithmus abbricht, nachdem er "setValue"
       * aufgerufen hat (und damit die Variable "status" auf null setzt), das
       * übergebene Runnable aber erst später ausgeführt wird. Aus diesem
       * Grund wird ein try-catch-Block benutzt, der die mögliche
       * NullPointerException abfängt. Alternativ könnte man mit dem Keyword
       * "synchronized" einen Mechanismus bauen, der verhindert dass die
       * Variable "status" verändert wird, während wir uns in dieser Methode
       * befinden (und dann eine einfache if-Abfrage machen).
       */
      SwingUtilities.invokeLater( new Runnable(){
	 public void run() {
	    try{
	       status.setValue( value );
	    }
	    catch( NullPointerException ex ){
	       // silent exception
	    }
	 }
      });
   }
}

[Das ist genau das Bsp. aus der FAQ]
Ich hatte daran gedacht, dass ich vielleicht den Thread an den Algorithmus übergeben müsste, und dann halt in der Exporiteren-Klasse:

algorithmus = new Algorithmus(Thread.currentThread);

aufrufe, aber ohne Erfolg.
Irgendwie verstehe ich das mit diesen Threads immer noch nicht ganz.

By<e Ulrich

 

[Mag1c]

Hi Ulrich,

tja, was soll ich sagen ... bei mir funktioniert dein Code

Könnte evtl. an deiner GUI liegen. Ich musste an der Stelle etwas umbauen, damit ich etwas sehe. Wo kommen denn die Parameter für den Exportieren(...)-Konstruktor her ?

Gruß
Mag1c

 

[ulr!ch]

Ah, isnichwahroder?

Die Parameter für den Konstruktor kommen von der eigentlich Hauptklasse.
Habe nun auch rumexperimentiert, hier ein HG.updateUI(); dort ein anderen LayoutManager eingfügt, aber kein Erfolg.
Was hast du denn noch verändert, damit etwas bei dir angezeigt wird?
Muss ich irgendwo die GUI updaten, eigentich nicht, dafür haben wir ja die Threads programmiert, oder?
Ich finde einfach den verdammten Fehler nicht. :?

By<e Ulrich

 

[Mag1c]

Moin,

    public void jbInit() throws Exception {
          JPanel innerLayer = new JPanel();
          innerLayer.setLayout(new GridBagLayout());
        innerLayer.setBackground(Color.WHITE/*new MyColor("weiss").getMyColor()*/);

        progress = new JProgressBar();
        buttonStart = new JButton( "Start" );
        buttonStop = new JButton( "Stop" );
        algorithmus = new Algorithmus();

//     Wenn der Algo nicht läuft, kann man ihn auch nicht stoppen
        buttonStop.setEnabled( false );

//     Der folgende Teil ist nur für das Auge, eine Benutzeroberfläche die
//     etwas dynamisch ist, kommt immer gut an.
       JPanel panel = new JPanel( new GridLayout( 1, 2 ));
       panel.add( buttonStart );
       panel.add( buttonStop );

       innerLayer.add( progress, new GridBagConstraints( 0, 0, 1, 1, 1.0, 1.0, GridBagConstraints.CENTER, GridBagConstraints.HORIZONTAL, new Insets( 2, 2, 2, 2 ), 0, 0) );
       innerLayer.add( panel, new GridBagConstraints( 0, 1, 1, 1, 1.0, 1.0, GridBagConstraints.SOUTHEAST, GridBagConstraints.NONE, new Insets( 2, 2, 2, 2 ), 0, 0) );
//       HG.add(innerLayer, new GridBagConstraints(0, 1, 1, 1, 0.0, 0.0, GridBagConstraints.CENTER, GridBagConstraints.NONE, new Insets(0, 0, 0, 0), 0, 0));
//       HG.updateUI();
       getContentPane().add(innerLayer);
       pack();

die Konstruktor-Parameter habe ich einfach mal ignoriert. Das jbInit habe ich so modifiziert, daß
die GUI dem Frame hinzugefügt wird. Der Aufruf sieht dann ganz simpel so aus:

       Exportiern exportieren = new Exportiern(0, 0, null, null, null);
        exportieren.setVisible(true);
        exportieren.startAlgo();

Gruß
Mag1c

 

[ulr!ch]

Hi Mag1c,

vielen Dank für deine Hilfe, ohne dich hätte ich den Fehler nie gefunden.
Und toll, dass du den Code bei dir ausprobiert hast und mir so Hinweise liefern konntest.
Also es war eine ganz unglückliche Verkettung von "Spezialeinstellungen":
a) Zum einen benutze ich ein anderes laf ("com.jgoodies.looks.plastic.PlasticXPLookAndFeel").
b) Zum anderen hatte ich Vista Inspirat ausprobiert, um zu schauen, wie der Nachfolger von XP aussieht.

Und dann hat sich dieser Balken hat nicht bewegt.
Zu viel Spielerei wird halt bestraft :!:

Thx Mag1c!
By<e Ulrich