JProgressBar und ihre Tücken

[Centurio]

Guten Morgen erstmal miteinander,

Zur Erklärung meines Problemes:
Ich habe eine boolean Funktion deren Ausführungszeit etwas länger dauern kann. Genauer gesagt ist dies ein Funktion, die sich auf einen RMI Server verbindet und sich dort anmeldet.
Soweit so gut, ich möchte dieses Verbinden mit einer JProgressBar im Indeterminate Modus visualisieren. Und zwar so, dass ein Dialog mit eben dieser JProgressBar angezeigt wird solange die Funktion nicht zurückgekommen ist. Das Probleme welches ich erfahren habe liegt darin, dass ich für diesen Dialog mit der JProgressBar einen eigenen Thread brauche, da sonst beides hintereinander ausgeführt werden würde (zuerst also der Dialog und dann die Funktion). Jetzt muss ich den Dialog aber nach Rückkehr der Funktion schliessen können, d.h. den Thread irgendwie auch beenden.

ich habe für den Dialog folgenden Ansatz:

public class VerbindungsDialog extends JDialog implements Runnable {
...
public void run() {
        this.setVisible(true);
    }
...
}

und für die Aufrufende Funktion:

VerbindungsDialog vDialog;
Thread vDialogThread;


public void doconnect() {
 if (vDialog == null) vDialog = new VerbindungsDialog();
 vDialogThread = new Thread(vDialog);
 vDialogThread.start();
 boolean done = // hier die Lange Funktion mit Rückgabewert boolean.
 vDialog.setVisible(false);
}

Dies funktioniert nur ein mal, nämlich beim Standardmäßigen Startaufruf der Funktion doconnect() aus dem Hauptthread heraus. Beim nächsten Aufruf per Menü von der Oberfläche (eigener Thread), bleibt der Dialog auch offen nachdem die Funktion zurückgekehrt ist.

Meine Frage also, warum ist dies so und wie kann ich einen Dialog schreiben, der das tut was ich möchte.


Mfg Centurio

 

[kleiner_held]

Nicht der Dialog muss in einen eigenen Thread sondern die lange Funktion. So wie ich dein Beispiel verstanden habe, blockierst du ja immer noch den GUI-Thread.

VerbindungsDialog vDialog;

private VerbindungsDialog getVDialog()
{
  return vDialog;
}

public void doconnect() 
{
   if (vDialog == null)
      vDialog = new VerbindungsDialog();
   Thread workerThread = new Thread(new Runnable() {
       public void run() {
          boolean done = // hier die Lange Funktion mit Rückgabewert boolean.
          SwingUtilities.invokeLater(
          new Runnable() {
             public void run() {
                getVDialog().setVisible(false);
          }});
        } 	
    });
    workerThread.start();
    vDialog.setVisible(true);
}

 

[Centurio]

Danke für die die Antwort, darüber hatte ich auch schon nachgedacht. Das Problem hierbei ist dass ich ja den boolean done zur Weiterverarbeitung wieder ausserhalb des Threads brauche, und mir kein schönes Konstrukt eingefallen ist, mit dem ich diesen wieder bekomme.

Mfg Centurio

 

[MasterEvil]

ich würde es in etwa so machen:

    private void foo() {
    	DialogThread th = new DialogThread();    	
    	th.start();
    	// Deine lange Anweisung
    	th.stopNow();
    }
    
    private class DialogThread extends Thread {
		private boolean goon = true;
		private JDialog dialog;
		
		public void run() {
			dialog = new JDialog();
			dialog.setVisible(true);
			while(goon) {
				try { sleep(200); }
				catch (InterruptedException e) { e.printStackTrace(); }
			}
		}
		public void stopNow() {
			goon = false;
			dialog.setVisible(false);
		}
    }

 

[kleiner_held]

Wenn dein Diaog mit dem Progressbar modal ist, bleibt die Methode doconnect() nach vDialog.setVisible(true); stehen und geht erst weiter, wenn der Dialog mit setVisible(false); wieder versteckt wurde. Du kannst also eine private Variable boolean result anlegen, die dann von dem, WorkerThread mittels setResult(boolean) gesetzt wird und in doconnect() nach dem setVisible() ausgewertet wird.

PS: Ich sehe gerade, das dein eigentliches Problem (ProgressBar verschwindet nicht) auch darin liegen koennte, dass deine lange Funktion beim zweiten mal doch nicht solange dauert wie gedacht Dann koennte passieren dass setVisible(false) vor setVisible(true) aufgerufen wird und der Dialog bleibt ewig sichtbar.
Da muesste man quasi ueber synchronized sicherstellen, dass der Dialog in diesem Fall garnicht erst angezeigt wird.

 

[kleiner_held]

@MasterEvil
das gleiche Problem - wenn foo() im AWT Event Thread aufgerufen wird, also zB durch einen ActionListener an einem Button, dann haengt die ganze GUI waehrend die "lange Anweisung" abgearbeitet wird.
D.h. nicht nur keine Eventverarbeitung, sondern vor allem kein Neuzeichnen, falls das Fenster mal ueberdeckt wird.

Ich hab deinen Code mal als Beispiel ausgebaut um das zu veranschaulichen:

public class DialogTest
{	
	public static void main(String[] args)
	{
		JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
		frame.getContentPane().add(new JButton(new AbstractAction("Test") {
			public void actionPerformed(ActionEvent e)
			{
				foo();
			}
		}));
		frame.pack();
		frame.setVisible(true);
	}
	
	private static void foo() {
		DialogThread th = new DialogThread();       
		th.start();
		// simulierte lange Anweisung
		try 
		{ 
			Thread.currentThread().sleep(10000); 
		}
		catch (InterruptedException e) { e.printStackTrace(); }
		th.stopNow();
	}

	private static class DialogThread extends Thread {
		private boolean goon = true;
		private JDialog dialog;

		public void run() {
			dialog = new JDialog();
			dialog.getContentPane().add(new JButton("Test"));
			dialog.pack();
			dialog.setVisible(true);
			while(goon) {
				try { sleep(200); }
				catch (InterruptedException e) { e.printStackTrace(); }
			}
		}
		public void stopNow() {
			goon = false;
			dialog.setVisible(false);
		}
 }
}

 

[MasterEvil]

Hmm .. stimmt :/

 

[kleiner_held]

vielleicht noch mal ein Beispiel wie ich sowas angehen wuerde:

public class DialogTest
{
	
	public static void main(String[] args)
	{
		final JFrame frame = new JFrame();
		frame.setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
		frame.getContentPane().add(new JButton(new AbstractAction("Test") {
			public void actionPerformed(ActionEvent e)
			{
				System.out.println(new SampleWorker(frame).doconnect());
			}
		}));
		frame.pack();
		frame.setVisible(true);
	}

	private static class SampleWorker
	{
		JDialog vDialog;
		private boolean result;
		
		public SampleWorker (JFrame frame)
		{
			vDialog = new JDialog(frame, "5 Sekunden warten", true);
			JProgressBar bar = new JProgressBar();
			bar.setIndeterminate(true);
			vDialog.getContentPane().add(bar);
			vDialog.setDefaultCloseOperation(WindowConstants.DO_NOTHING_ON_CLOSE);
			vDialog.pack();
		}
		
		private void finish()
		{
			vDialog.setVisible(false);
		}
		
		public boolean doconnect()
		{
		   Thread workerThread = new Thread(new Runnable() {
		       public void run() {
		    	   // hier die Lange Funktion mit Rückgabewert boolean.
		           boolean done = true;
		           try
		           {
		              Thread.currentThread().sleep(5000);
		           }
		           catch (InterruptedException e) { e.printStackTrace(); }
		           result = done;
		           SwingUtilities.invokeLater(
		        		   new Runnable() {
		        			   public void run() {
		        				   finish();
		        			   }});
		        }    
		    });
		    workerThread.start();
		    vDialog.setVisible(true);
		    return result;
		} 
	}
}

 

[Centurio]

Wunderbar danke euch, damit ist das Problem gelöst.