[Sayang]

Hallöchen nochmal...

ich habe eine UI, in der man u. a. eine Datei per JFileChooser auswählt. Wenn der Benutzer das macht, soll die ausgewählte Datei durchlaufen werden, um schonmal bestimmte Informationen anzuzeigen. Im Prinzip passiert hier nicht viel mehr als das Zählen der Zeilen dieser Textdatei.
Was mich nur stört ist, dass diese Zählmethode die UI blockiert und diese erst wieder "freigibt", wenn zuende gezählt wurde (das Zählen könnte bis zu 10 Sek. dauern - große Dateien ^^). Ich hätte es aber nun gerne, dass diese Methode die UI nicht blockiert, so dass der Benutzer weitere Einstellungen auf der UI vornehmen kann, während im Hintergrund diese Datei durchgezählt wird.

Erstmal ein paar wichtige Infos:

FileSelector - kombiniert ein JTextField mit einem Browse-Button, der den JFileChooser aufruft. Besitzt ebenfalls ein optionales JLabel, um auf diesem irgendwelche Infos darzustellen. Dieses JLabel möchte ich dazu verwenden, dass der Benutzer mitbekommt, dass die selektierte Datei a) noch gezählt wird oder b) xyz Zeilen enthält. Das JLabel ist per Methode public JLabel getDescriptionLabel() zugreifbar.

GeneCountThread - Zählt die Zeilen der Datei, die Genexpressionsprofile definieren. Hab mir schon gedacht, dass man das in einen eigenen Thread packen muss, damit es nebenher läuft, aber ich bin Thread-Noob ^^ und irgendwas mach ich hier falsch. Code-Auszug:

public class GeneCountThread implements Runnable {
  //...
   public GeneCountThread(File f, JLabel display) { //Die Datei und das JLabel des FileSelectors
     //...
   }
 
   public void run() {
     genes = 0;
     try {
       StanfordReader sp = new StanfordReader(f);
       Gene g = sp.readGene();
       while(g!=null) {
         genes++;
         //das JLabel 'display' soll nicht für jedes einzeln gezählte Gen aktualisiert werden
         if(genes%10==0) this.display.setText(genes+" genes. Still counting...");
          //ein Thread.sleep() scheint hier auch nicht viel zu bringen
         g = sp.readGene();
       }
       sp.close();
       // während des Zählens hat das JLabel rote Schrift.
       // Wenn fertig gezählt wurde, soll Schrift schwarz sein
       this.display.setForeground(Color.BLACK);
       this.display.setText("File contains "+genes+" genes");
     }
     catch(Exception e) {
           // ...
     }
   }
 }

Das müsste eigentlich ausreichen... Was muss ich denn tun, damit der Benutzer während dieser run()-Methode weiter auf dem UI rumklicken kann?
Hoffe es gibt da was, was mir meine Architektur nicht über den Haufen wirft ^^

Thx in advance
sayang

[beny_mcde]

die endscheidenden zeilen code hast du nicht gepostet, aber ich vermute das du die run methode von GeneCountThread direkt aufrufst.

GeneCountThread t = new GeneCountThread();
t.run();

das ist natürlich nicht sinn der sache.
Klassen die Runnable implementieren sind noch keine threads, sie können von threads aufgerufen werden.
entwerder du änders die klasse GeneCountThread um, das sie Thread erweitert,
oder du änderst den aufrufenden code, dann solltest du GeneCountThread in GeneCountRunnable umbenennen.

Runnable run = new GeneCountThread();
Thread thread = new Thread(run);
thread.start();

und die run methode wird nebenläufig aufgerufen...

[Sayang]

Perfekt... manchmal sind die einfachsten Fehler doch diejenigen, auf die man einfach nicht kommt ^^

dankeschön...
sayang

PS: kann man hier die Themen eigentlich auch als erledigt markieren? ich hab sone option jedenfalls noch nicht gefunden... vll. bin ich hier ja auch so blind wie bei meinem problem ^^

[Ben]

Zitat von Sayang

PS: kann man hier die Themen eigentlich auch als erledigt markieren? ich hab sone option jedenfalls noch nicht gefunden... vll. bin ich hier ja auch so blind wie bei meinem problem ^^

Yeah. Find ich auch äußerst nützlich. Steht auf der ToDo-Liste, da es leider nicht von Haus aus möglich ist.

Aber gut, dass Du es angemerkt hast, dann muss ich mich da langsam mal mit beschäftigen .
Diskussion dazu kann bei Bedarf und Interesse hier fortgeführt werden.

[pago]

Sei froh, dass es das nicht gibt...
Dein Problem fängt nämlich gerade erst an.

Du darfst nicht aus einem anderen Thread auf Swing-Komponenten zugreifen.

d.h. aus:

if(genes%10==0) this.display.setText(genes+" genes. Still counting...");

Wird sowas:

if(genes%10==0) {
    SwingUtilities.invokeLater(new Runnable() {
        public void run() {
            this.display.setText(genes+" genes. Still counting...");
        }
    });
}

Das gleiche gilt auch für die andere Zugriffe auf Swing-Komponenten...


Als kleinen Tipp möchte ich zwei Namen in den Raum werfen, die du ergooglen könntest: Foxtrot und SwingWorker.

[Xean]

Warum darf man das nicht??? ist es nur nicht schön geschrieben, oder bekommt man dann probleme???

[Ben]

[GEFÄHRLICHES HALBWISSEN]
Afaik ist es doch so, dass Swing mit einem eigenen Thread arbeitet und das es aus diesem Grunde dann zu Komplikationen kommen kann.

[Gottzilla]

Zitat von Xean

Warum darf man das nicht??? ist es nur nicht schön geschrieben, oder bekommt man dann probleme???

Swing ist nicht Thread Sicher. => Kommt dann zu Darstellungsfehlern u. ä. . Deswegen immer wenn Swing in Verbindung mit einem anderen Thread kommt invokeLater verwenden.

Was die Begründung angeht dürfte Ben gar nicht ma so falsch liegen

[Ben]

Man lese auch:
15.33.1 Swing ist nicht Thread-sicher, Java ist auch eine Insel

[Sayang]

Ähh ja... danke für den Hinweis. War ja klar, dass sowas kommt ^^
Versuche mich dann mal da durchzuwurschteln.

Lg
sayang

[Ben]

Hab ich aber auch erst rausgesucht, als ich mit meinem Halbwissen wenigstens so teilweise bestätigt wurde .

[Xean]

ok, werd es dann mal beachten.... hatte bis jetzt noch keine probleme, hab auch nicht so oft irgend ein prog geschrieben, was in diese richtung geht...