Threads

[Danny Stephan]

Hallo Zusammen,

wenn ich einen weiteren Thread starte wird dieser nicht gleich ausgeführt. Es macht mich langsam wahnsinnig! Habe schon in meheren Büchern nachgesehen und das Internet ebenfalls durchkemmt. Ich hoffe ihr könnt mir weiter helfen.

Ich schreibe im Normalfall nicht wegen jeder Kleinigkeit aber an dem Thema sitze ich schon mehere Tage.

Hintergrund:
Im Run-Zweig werden bei sa.save(profil, dst); Dateien in ein anderes Verzeichnis kopiert(ca. 400 MB). parallel soll in der progressBar der Fortschritt angezeigt werden.

Die GUI friert quasi ein, nachdem der startListener aufgerufen. Erst nachdem der Kopiervorgang abgeschlossen ist, wird die GUI mit den ganzen removes aktualisiert und ist wieder bedienbar.

Hier das Code-Schnipsel:

class startListener extends Thread implements ActionListener{
            
            saveData sa = new saveData();
            String pa = path.getText();
            File dst = new File(pa);
            final String[] KEYS = { "user.name", "user.language", "user.home","os.name", "java.version" };
            String username = System.getProperty(KEYS[0]);
            String prof = "C:\\Dokumente und Einstellungen\\";
            File profil = new File(prof+""+username);
            int soValue;
            int isValue;

            public void actionPerformed(ActionEvent e){

                //Code von hier bis zu t1.start() wird erst nach
                //Ablauf der run-Methode ausgeführt!? Warum???
                process.setVisible(true);
                back.setEnabled(false);
                path.setVisible(false);
                checkPath.setVisible(false);
                choice.setVisible(false);
                remove(T1);
                remove(T2);
                remove(T3);
                remove(I2);
                remove(I1);
                repaint();
                start.setEnabled(false);
                startListener sl = new startListener();
                Thread t1 = new Thread(sl);
                t1.start();
                sl.calcSize();
                             

            }
            
            public void run(){
                 sa.save(profil, dst);//hier findet ein Kopiervorgang statt
                 exit.setVisible(true);
                 start.setVisible(false);
            }
            
       
            void calcSize(){  //hier soll die parallel progressBar aktualisiert.

                long istWert = sa.getFolderSize(profil, pa)/1000;
                isValue = (int)istWert;

                while(isValue>soValue){
                    try{
                        long sollWert = sa.getFolderSize(dst, pa)/1000;
                        soValue = (int)sollWert;
                    }catch(ArrayIndexOutOfBoundsException ae){}

                    System.out.println(isValue+ " " +soValue);
                    process.setStringPainted(true);
                    process.setMaximum(isValue);
                    process.setValue(soValue);
                }
            }
    }

 

[OliverKroll]

Dein Code-Schnipsel ist nicht compilierbar. Schick mal das komplette Programm, oder einen verkürzten Teil, in dem der Fehler auftritt.

 

[SlaterB]

das Kopieren findet nebenläufig statt,
aber dann hast du noch die lange while-Schleife, die genauso beschäftigt ist, bis das Kopieren beendet wurde,
die wird derzeit ganz normal vom ActionListener ausgeführt, also vom AWT-Thread,
und solange der damit beschäftigt ist, wird die GUI nicht neu gezeichnet

entweder macht der parallele Kopier-Thread ab und zu Pausen um die GUI zu aktualisieren oder du brauchst noch einen zweiten separaten Thread für die while-Schleife,
diese dürfte derzeit auch ein paar Millionen mal pro Sekunde aktualisieren und 50% der CPU wegfressen,
baue ein Thread.sleep(50) in die Schleife, reicht doch, 20x pro Sekunde den aktuellen Stand zu bestimmen

 

[MQue]

Bevor du da lowlevel- Threads programmierst, schau dir lieber ExecutorService an,

Gib das zeichnen der GUI und das kopieren der Dateien je in ein eigenes Runnable und übergib diese Runnable's dann dem Executor (Java Concurrency in Practice -> Task Execution)

 

[MQue]

So wie unten würd ich das machen, die GUI wird immer im EDT (Event Dispatcher Thread) gezeichnet, schau dir auch mal SwingWorker an, der ist auch ganz brauchbar.

package startlistener;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;

public class StartListener extends Thread implements ActionListener{

    private final ExecutorService es = Executors.newFixedThreadPool(3);
    
    public StartListener() {
        final Runnable r1 = new Runnable() {
            public void run() {
                while(true) {
                    System.out.println("Mach was 1");
                    try {
                        Thread.sleep(1000);
                        }
                    catch(Exception e){
                        System.out.println(e);
                        }
                    }
                }
            };
            
        final Runnable r2 = new Runnable() {
            public void run() {
                while(true) {
                    System.out.println("Mach was 2");
                    try {
                        Thread.sleep(1000);
                        }
                    catch(Exception e){
                        System.out.println(e);
                        }
                    }
                }
            };
            
        es.submit(r1);
        es.submit(r2);
        }

    public void actionPerformed(ActionEvent e) {
        
        }
    
    public static void main(String[] args) {
        new StartListener();
        }
}

 

[Danny Stephan]

Hallo SlaterB,

VIELEN HERZLICHEN DANK!! Es funzt!!!

Eigentlich nicht so schwer. Warum ich nicht selber darauf gekommen bin???

@MQue: Das sieht nach einer Lösung mit besserer Performance aus! Werde mich mit ExecutorService mal näher beschäftigen um die Lösung später auch unterzubringen.

Also Vielen Dank @All für eure Zeit, Mühe und Ideen. Viele Stunden habe daran verbracht u. hier werden die Lösungen innerhalb einer Stunde präsentiert.