Unterframe aus ActionListener

[Seddy]

Hallo Zusammen, hier mal meine Situation:
Ich habe ein GUI welches im wesentlichen aus einem JFrame und einigen JButtons besteht, auf einem dieser JButtons liegt nun ein ActionListener:

jb_server.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent arg0) {
//Funktion
}});

In diesem ActionListener wird nun eine Funktion aufgerufen die sehr lange braucht um die Berechnungen durchzuführen, desewegen möchte ich gerne ein weiteres Fenster erstellen welches einen Prozentbalken anzeigt, das klappt als Trockenübung auch sehr gut, ich habe einfach eine neue Klasse erstellt die einen JFrame implementiert, allerdings kommt jetzt das Problem:
Wenn ich aus dem ActionListener herraus einen neuen JFrame erstelle wird dieser zwar erstellt, aber nie aktualisiert solange der ActionListener nicht durchgelaufen ist, sprich ich bekomme nur einen leeren Frame angezeigt, dessen Inhalt erst angezeigt wird, wenn die Funktion dessen Status ich visualisieren möchte schon beendet ist.
Wie kann ich nun dafür sorgen, dass der neue Frame sofort nach dem erstellen nutzbar ist?

 

[KrokoDiehl]

Startest du den neuen Frame über SwingUtilities.invokeLater()? Sprich

SwingUtilities.invokeLater(new Runnable()
{
    @Override
    public void run()
    {
        neuerFrame.setVisible(true);
    }
});

 

[Seddy]

Startest du den neuen Frame über SwingUtilities.invokeLater()?

Soweit ich weiß nicht, ich poste einfach mal den Code von dem Frame, der ist nicht so lang:

/* LoadScreen.java
** -zeigt ein Fenster mit Prozentanzeige
*/

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

public class LoadScreen extends JFrame{

	private JLabel proz1;
	private JLabel proz2;
	
	//Konstruktor zeigt das Fenster an
	public LoadScreen(String title){
		super(title);
		init(title);
	}

	//Konstruktor zeigt das Fenster an (default Titel)
	public LoadScreen(){
		super("laden...");
		init("laden...");
	}
	
	//initialisiert Fenster
	private void init(String title){
	
		//hellblaues Fenster ohne Umrandung
		setUndecorated(true);
		getContentPane().setLayout(null);
		setBounds(300,300,120,40);
		getContentPane().setBackground(new Color(168,203,233));
		
		//Text
		proz1=new JLabel(title);
		proz1.setBounds(40,5,100,15);
		getContentPane().add(proz1);
		
		//Balkenhintergrund
		proz1=new JLabel();
		proz1.setOpaque(true);
		proz1.setBackground(Color.white);
		proz1.setBounds(10,20,100,15);
		getContentPane().add(proz1);
		
		//Balken
		proz2=new JLabel();
		proz2.setOpaque(true);
		proz2.setBackground(Color.green);
		proz2.setBounds(10,20,0,15);
		getContentPane().add(proz2);
		setVisible(true);
		
	}
	
	//Destruktor schließt das Fenster
	public void close(){
		dispose();
	}
	
	//passt die Prozentanzeige an
	public void set(int p){
		proz1.setBounds(10+p,20,100-p,15);
		proz2.setBounds(10,20,p,15);
	}
	
}

Noch eine Anmerkung:
der erste Frame in dem sich der Button befindet "hängt" während der Berechnung auch.

 

[KrokoDiehl]

Ja weil der erste Frame noch im ActionListener hängt. Benutze mal den von mir genannten Code zum Starten des zweiten Frames in deinem ActionListener und ändere die Zeile mit

setVisible(true)

dort auf

new LoadScreen("MeinTitel");

.

 

[Seddy]

Gut, das habe ich nun gemacht, es ergeben sich aber folgende Probleme:

LoadScreen ls=new LoadScreen();
SwingUtilities.invokeLater(new Runnable(){
  public void run(){
    ls=new LoadScreen();
  }
});

Wird nicht angenommen:

local variable ls is accessed from within inner class; needs to be declared final

Wenn ich nun einfach zum testen nur

new LoadScreen();

reinschreibe geht das zwar, aber der Frame wird auch erst sichtbar, wenn die Berechnung beendet ist.

 

[KrokoDiehl]

Es geht darum, dass der neue Frame in einem eigenen Displaythread gestartet wird, genau das macht das

 invokeLater()

. Wenn man

new LoadScreen()

macht, wird dort bereits über die

init()

-Methode der Frame angezeigt, d.h. der Konstruktoraufruf muss in das

invokeLater()

rein.
Wenn du sonst noch Zugriff auf das Frame-Objekt haben willst, könntest du es so machen:

public void actionPerformed(ActionEvent event)
{
    final LoadScreen frame = null;
    SwingUtilities.invokeLater(new Runnable()
    {
        @Override
        public void run()
        {
            // Erstellen und Anzeigen
            frame = new LoadScreen("meinTitel");
        }
    }
}

 

[Seddy]

Der Code funktioniert (bei mir) nicht:

cannot assign a value to final variable frame

 

[KrokoDiehl]

Ach so ja ... mein Fehler final kann man ja nur einmal belegen.

public void actionPerformed(ActionEvent event)
{
    //A    
    final LoadScreen frame;

    //B (Deklaration weglassen)

    SwingUtilities.invokeLater(new Runnable()
    {
        @Override
        public void run()
        {
            //A Erstellen und Anzeigen
            frame = new LoadScreen("meinTitel");

            //B oder nur
            new LoadScreen("meinTitel");
        }
    }
}

 

[Seddy]

Das bringt mich immer noch nicht weiter, denn final Variablen kann man nur bei der Deklaration belegen , das gibt also immer noch den selben Fehler, und wie ich schon sagte wird das ohne Referenz erstellte Fenster trotz invokeLater erst nach der Beendigung der Funktion aufgerufen.

 

[KrokoDiehl]

An welcher Stelle wird denn die Berechnung gestartet, bzw. wer startet sie?

 

[Seddy]

Ok, hier mal die wichtigsten Codeauschnitte:

//Der Aufruf:
jb_suche.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent arg0) {
				alleLieder.refresh(jtf_suche.getText());
			}
		});

//die Funktion refresh:
public void refresh(String search) {
	
		Vector<Vector<String>> inhalt=null;
		switch(which){
			case 1:
				inhalt=client.alleLieder(search);
				break;

//die Funktion alleLieder ruft die Funktion getSonginfoList auf, Auschnitt daraus:
//LoadScreen ls=new LoadScreen();
  		SwingUtilities.invokeLater(new Runnable(){
				public void run(){
					new LoadScreen();
				}
			});

Das ist aber alles nicht so wichtig, denn es reicht schon dieses einfache Beispiel:

jb_suche.addActionListener(new ActionListener() {
  public void actionPerformed(ActionEvent arg0) {
    SwingUtilities.invokeLater(new Runnable(){
      public void run(){
        new LoadScreen();
      }
    });	
    try{
      Thread.sleep(5000);
    }catch(Exception e2){}
  }
});

Auch hier erscheint der LoadScreen erst wenn das sleep beendet ist.

 

[KrokoDiehl]

Ok.
Das

 invokeLater()

schiebt die Anzeige des Frames in die Ereigniskette und führt sie sobald aus (deswegen invokeLATER), wenn aber das aktuelle Ereignis (das

actionPerformed()

) lange braucht, kommt der Frame natürlich noch nicht dran.
Was du machen musst, ist die lange Bearbeitung in einen Thread auslagern. Entweder du schaffst es, es in deinen zweiten Frame zu machen, oder du machst einen eigenen Thread (bzw. einen SwingWorker), der seine Fortschritte meldet.
Letztere Methode ist wohl die saubere und du müsstest wahrscheinlich nicht einmal einen neuen Frame erstellen, der einen Fortschrittsbalken anzeigt, sondern könntest ihn im gleichen Fenster anzeigen lassen. Z.B. wie bei einigen Browsern rechts unten in der Ecke.

 

[Seddy]

Sehr gut, so geht es:

jb_suche.addActionListener(new ActionListener() {
  public void actionPerformed(ActionEvent arg0) {
    new Thread() {
      public void run(){
        alleLieder.refresh(jtf_suche.getText());
      }
    }.start();
  }
});