Swing GUI Process anzeigen

[Pommes9485]

Guten Tag,

ich tüflte jetzt schon seit einer Stunde daran, das der Prozess von der einen Klasse in der GUI angezeigt wird.

Jetzt weis ich nicht mehr weiter.
Ich habe die Arbeit-Klasse in einen thread verwandelt, in einen String den Process geschrieben und eine run methode geschrieben, in welcher ich die Methoden der Arbeit-Klasse nach und nach aufrufe. Das geht auch so laut Console. In der GUI starte ich den Thread(Also die Arbeit klasse) und frage dann per while schleife den Status vom String in der Arbeit Klasse ab und stecke ihn in ein Label.

Aber dennoch friert die GUI einfach ein.

Was kann ich tun ?

 

[eRaaaa]

Ein simples Beispielprogramm könnte helfen. Rufst du den Thread evtl. mit run() auf? Kam hier auch schon häufiger vor

 

[Pommes9485]

Ne, mit start() ;D

Ich hatte es auch schon einmal mit einem andern Programm hinbekommen.

Ich gebe euch mal den verkürzten Code :

 import java.io.*;
 import java.util.ArrayList;
  import java.util.Iterator;
  import org.apache.commons.net.ftp.FTPClient;
import java.io.FileInputStream;
import java.io.IOException;
public class Lesen extends Thread
{

String process="";
boolean lauf = true;
public Lesen(String pfad, String name, String titel)
{


}
public void lese() 
{
process="Beginne einlesen.";

process="Einlesen beendet. Starte einsetzen und bearbeiten des Textes.";
   
}
public void ersetzten()
process="Einsetzen und bearbeiten des Textes beendet. Starte löschen der alten Datei.";
   
}
public void löschen()
{

 process="Löschen beendet. Starte hochladen der neuen Datei.";
   
}
public void hochladen()
{

process="Vorgang erfolgreich abgeschlossen.";
   
}
public void run()
{
try{
lese();
ersetzen();
löschen();
hochladen();
lauf = false;
}catch(IOException i){System.out.println(i.getMessage());}
   

}
}

Und die GUI :

ls = new Lesen(jTextField1.getText(), name, titel);
    ls.start();
   while(ls.lauf)
   {
       System.out.println(ls.process);
   jLabel1.setText(ls.process);
}

Damit rufe ich es dann auf.

 

[XHelp]

Im 2. Schnipsel hast du ja in Zeile 3-7 ja auch eine Schleife die Blockiert. Die musst du entweder auch in ein Thread auslagern, oder in deinem Lesen-Thread sowas wie "ProcessStateChangeListener" implementieren

 

[Pommes9485]

Nein, da dies die letzte Aufgabe ist, welche ausgeführt wird...

 

[XHelp]

Doch, weil er dann solange in der Methode verweilt

 

[Pommes9485]

AH, ich verstehe ^^

Danke dir...

EDIT :

Habe das jetzt einfach noch in einen Extra Thread ausgelagert, die GUI friert auch nicht ein, aber es ändert sich einfach nichts...

Die neue Klasse sieht so aus :

public class Pro extends Thread
{
Lesen la;
GUI gu;
public Pro(Lesen ls, GUI gui)
{la=ls;gu = gui;}
public void run()
{
while(true)
{
gu.jLabel1.setText(la.process);
}
}
}

Sie wird von Lesen auch gestartet, aber das Label verändert sich erst am Ende...

 

[Empire Phoenix]

warum nicht den arbetenden Thread referenz auf textfeld übergeben , und dann dort von einfach setText machen. setText bei Lable ist threadsafe sogesehen sollte das auch keine probleme geben. (achtung am rande swing ist nicht überall threadsafe)

 

[Pommes9485]

Ich weis nicht genau, was du mit refenz meinst.
Ich habe das Label jetzt direkt weitergegeben. Aber das geht nicht, da ich ohne extends JFrame kein JLabel einbinden kann. Und bie der einen klasse erbe ich schon von Thread.

 

[XHelp]

Was gemeint war wohl:

public class MainGui extends JFrame {
  private JLabel procLablel;
  ....
  private void initComponents() {
    procLabel = new JLabel("test");
    ...
    add(procLabel);
    ...
  }
  private void startSomeProcess() {
    (new Thread(new MyWorker(procLabel))).start();
  }

  private class MyWorker implements Runnable {
    private JLabel procLabel;
    public MyWorker(JLabel procLabel) {
      this.procLabel = procLabel;
    }
    public void run() {
      while(true) {
        //doSmth
        procLabel.setText("new state");
      }
    }
  }
}

 

[Pommes9485]

Jetzt geht es. Ich hatte das von dir ja quasi schon, leider Blockierte noch eine Schleife die GUI...