Thread legt GUI lahm

[Eddie Irvine]

Hallo,

ich habe eine Swing-GUI, in der ein über einen Button ein Thread gestartet wird.

Nachdem der Thread beendet wurde, soll das Resultat ausgegeben werden.

Frage: Wie überprüfe ich, ob der Thread beendet wurde? Wenn ich t.join() nehme, wird ja leider die ganze GUI lahngelegt...

Hier mal ein Auszug des Codes:

StartButton.addActionListener(new ActionListener() {
	public void actionPerformed(ActionEvent evt) {
		StartButton.setEnabled(false);
		statusText.setText("Das Programm läuft.");
		        
		TextMining tm = new TextMining();
		Thread t = new Thread(tm);
		t.start();

		try {
			t.join();
			Result r = new Result();
			r.setVisible(true);
			StartButton.setEnabled(true);
		} catch (InterruptedException e) {
			e.printStackTrace();
		}
	}
});

DANKE.

 

[bygones]

in der FAQ gibts z.b. ein Bsp für Multithreading... such mal auch nach invokeLater.

 

[L-ectron-X]

Oder du lässt den Thread eine eine Nachricht schicken, wenn er fertig ist.

Ein Fenster in dem eine Animation laufen soll:

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

public class MyFrame extends JFrame {
  private JPanel iconPanel, buttonPanel;
  private JButton button;
  private IconDraw iconDraw; //ein Thread
  private int numberOfIcons = 10; //Anzahl der Punkte, die gezeichnet werden sollen

  public MyFrame(String title) {
    super(title);
    setSize(400, 300);
    setLocationRelativeTo(null);
    setDefaultCloseOperation(EXIT_ON_CLOSE);

    iconDraw = new IconDraw(this, numberOfIcons); //Thread initialisieren
    
    iconPanel = new JPanel(new GridLayout(0, numberOfIcons));
    
    buttonPanel = new JPanel();
    button = new JButton("Play");
    button.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent e) {
        if(e.getActionCommand().equals("Play")) {
          createThread();
            
          if(iconPanel.getComponentCount() > 0) {
            iconPanel.removeAll();
            iconPanel.repaint();
          }
            
          iconDraw.start();
          button.setText("Stop");
        }
        else if(e.getActionCommand().equals("Stop")) {
          iconDraw.interrupt();
          ready();
        }
      }
    });
    
    buttonPanel.add(button);
    
    this.add(iconPanel, BorderLayout.CENTER);
    this.add(buttonPanel, BorderLayout.SOUTH);
  }
  
  public void addLabel() {
    MyLabel label = new MyLabel();
    iconPanel.add(label);
    iconPanel.validate();
  }
  
  private void createThread() {
    if(iconDraw == null)
      iconDraw = new IconDraw(this, numberOfIcons);
  }
  
  public void ready() {
    iconDraw = null;
    button.setText("Play");
  }
  
  
  //eine eigenes Label auf dem die Punkte gezeichnet werden sollen
  class MyLabel extends JLabel {
    int iconSize = 20;
    public void paintComponent(Graphics g) {
      g.setColor(Color.RED);
      g.fillOval((int)(this.getSize().getWidth()-iconSize)/2,
        (int)(this.getSize().getHeight()-iconSize)/2, iconSize, iconSize);
    }
  }
  
  public static void main(String[] args) {
    new MyFrame("Punkte").setVisible(true);
  }
}

Der Thread für die Animation:

public class IconDraw extends Thread {
  private int counter;
  private MyFrame starter;

  public IconDraw(MyFrame starter, int counter) {
    this.starter = starter;
    this.counter = counter;
  }

  public void run() {
    while(!isInterrupted()) {
      for(int i = 0; i < counter; i++) {
        try {
          Thread.sleep(1000);
          starter.addLabel();
        }
        catch(InterruptedException e) {
          interrupt();
        }
      }
      interrupt();
      starter.ready(); //Nachricht schicken: "Fertig!"
    }
  }
}

 

[Eddie Irvine]

Danke. Ich hab's jetzt so gemacht...

Ein Problem besteht aber leider noch:
Das Ergebnis-Fenster wird zwar erstellt und aufgerufen, aber leider wird das JTextField statusText in der Haupt-GUI nicht aktualisiert.
Muss der JFrame evtl. aktualiert werden?
Hier meine ready()-Methode (in der Haupt-GUI-Klasse):

public void ready() {
	statusText.setText("Das Clustering ist abgeschlossen.");
	StartButton.setEnabled(true);
        
	Result r = new Result();
	Thread t = new Thread(r);
		
	t.start();
}

 

[L-ectron-X]

Muss der JFrame evtl. aktualiert werden?

Nein, denn du hast in der Methode ready() den direkten Zugriff auf dein Textfeld.
Dann ist da noch was anderes faul.

 

[Eddie Irvine]

Komisch, ich habe einfach keinen Zugriff auf die Komponenten des GUI...
Auch einen Button kann ich bspw. nicht verändern.

Woran kann das liegen?

Hier nochmal der Aufruf:

StartButton.addActionListener(new ActionListener() {
	public void actionPerformed(ActionEvent evt) {
	
		statusText.setText("Das Programm wurde gestartet.");
		StartButton.setEnabled(false);

		TextMining tm = new TextMining();
		Thread t = new Thread(tm);
					
		t.start();	
	}
});

Das steht in der Methode run() in der Klasse TextMining:

public void run() {
	try {
		startClustering(myGUI.choice, myGUI.k, myGUI.num_runs, myGUI.tf_factor, 
			myGUI.idf_factor, myGUI.file);
	} catch (ClassNotFoundException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (SQLException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	} catch (IOException e) {
		// TODO Auto-generated catch block
		e.printStackTrace();
	}
		
	myGUI mG = new myGUI();
	mG.ready();
}

Liegt das daran, dass ich die Funktion ready() für ein anderes Objekt (mG) aufrufe??
Wenn ja: Wie kann ich das anders hinbekommen??

DANKE.

 

[Eddie Irvine]

So, hab's hinbekommen.

Mann, ganz schön kompliziert, das Ganze!