Threads PrograssBar update während thread

[JAMarlonVA]

Hallo zusammen,

ich habe heute meinen ersten Thread erstellt und der funktioniert auch soweit.

public class MyRunnable implements Runnable
    {
        public void run()
        {     
            cGuid0 myGuid = new cGuid0();
         
            resetCells();
     
            List<String> guids = new ArrayList<String>();     
     
            jLabel1.setText("Creating records");
         
            for(int i = 0; i < data.length-1; i++)
            {
                guids.add(myGuid.createGuid0());
                mySQL.insertQuery("INSERT INTO Words (Guid0) VALUES ('" + guids.get(i) + "')");
                                       
                jProgressBar1.setValue(Math.round(5 / data.length-1 * (i / 20)));
            }
            for(int i = 0; i < data[0].length; i++) 
            {
                String row = mySQL.getRecent(data[0][i]).get(0)[0];
             
                jLabel1.setText("Updating -" + row + "- (" + data.length + ") records");
             
                jProgressBar1.setValue(Math.round(5 + (95 / data[0].length * (i / 100/95))));
             
                for(int i2 = 0; i2 < guids.size(); i2++)
                {
                    mySQL.updateQuery("UPDATE Words SET "+ row +"=N'" + data[i2+1][i].replace("'", "''") + "' WHERE Guid0='"+ guids.get(i2) +"'");
                }
            }
        }
    }

Folgendes Problem, welches für die meisten Leser wahrscheinlich klar sein wird:
Ich versuche in einem Thread eine Komponente meines Frames zu beeinflussen.
Mir ist bewusst, dass das nicht funktioniert aber ich denke jeder weiß, was ich versuche zu machen wenn er den Code liest.

Wie kann ich eine Variable, in meinem MainFrame "überwachen", mit einem Listner o.ä., sodass ich die Prograssbar während des Prozesses updaten kann?

Komischer Weise funktioniert das Label sogar.

Freue mich über jede Antwort.

PS: Habe schon einiges dazu im Internet gefunden aber komme an dem Punkt nicht weiter. Also wäre eine Situationsbezogene Antwort sehr hilfreich.

 

[Robat]

Ich denke das was du suchst ist ein SwingWorker. Damit können im Hintergrund "zeitfressende" Operationen bearbeitet werden und die GUI geupdatet werden.

 

[JAMarlonVA]

Kannst du mir evtl. einen Link oder Erklärung geben, wie man einen SwingWorker nutzt? Ich hab schon einige Lösungen mit ihm gesehen aber werde einfach nicht schlau draus.

 

[Robat]

Schau mal hier für ein Beispiel.
Generelle Erklärung für SwingWorker gibt es bspw. direkt bei Oracle

 

[JAMarlonVA]

Ich habe versucht dein Beispiel leicht abgeändert umzusetzen. Allerdings ohne Erfolg.

Ich werde hier Morgen weiter machen.
Danke für deine Hilfe

PS: Ich war leider eine Minute zu spät mit der Änderung meines BB-Codes zu"Code=Java" wäre super, wenn das jmd mit den nötigen Rechten noch übernehmen könnte.

 

[JAMarlonVA]

Habe es mir nochmal angeschaut aber ich versteh wirklich überhaupt nicht in welcher Reihenfolge hier die Methoden wie aufgerufen werden.

Hast du zufällig ein wirklich simples Beispiel, an dem ich erkennen kann wie die Methoden aufgerufen werden?

 

[JAMarlonVA]

Noch eine Sache die ich nicht verstehe, ist wie ich meiner doInBackground irgendwelche Informationen mitgeben kann. Wenn hier der Aufwändige Teil geschehen soll, wie soll ich dann ohne Zugriff auf meine abgespeicherten Informationen irgendetwas mit ihnen machen in dieser Methode?

 

[Robat]

Das hier sollte eigentlich verdeutlichen, wann welche Methode aufgerufen wird

public class Example {

    public static void main(String[] args) {
        new Example();
    }

    public Example() {
        buildAndShowUI();
    }

    private void buildAndShowUI() {
        JFrame frame = new JFrame();
        frame.setSize(500,500);
        frame.setLocationRelativeTo(null);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

        JTextArea statusArea = new JTextArea();
        frame.add(new JScrollPane(statusArea));

        frame.setVisible(true);

        SimpleTask task = new SimpleTask(statusArea);
        task.execute();
    }

    class SimpleTask extends SwingWorker<Void, Integer> {
        private JTextArea area;
        public SimpleTask(JTextArea area) {
            this.area = area;
        }
        @Override
        protected Void doInBackground() throws Exception {
            area.append("[BACKGROUND]   in doInBackground() ... \n\n");

            for (int i = 0; i < 10; i++) {
                Thread.sleep(350);
                area.append("[BACKGROUND]   Working with " + i + "\n");
                publish(i);
            }
            return null;
        }

        @Override
        protected void process(List<Integer> chunks) {
            int lastValue = chunks.get(chunks.size() - 1);
            area.append("[PROCESS]   Publishing " + lastValue + "\n\n");
        }

        @Override
        protected void done() {
            try {
                get();
                area.append("[DONE] Done...\n");
            } catch (InterruptedException | ExecutionException e) {
                e.printStackTrace();
            }
        }
    }
}

Dem Task kannst du "Informationen" über den Konstruktor mitgeben. Hier wird bspw nur die JTextArea mitgegeben, auf die die Informationen geschrieben werden sollen. Du kannst aber natürlich auch eine Liste von Dateien, Zahlen oder irgendwelche anderen Daten im Konstruktor übergeben und diese in den Methoden verwenden.

 

[JAMarlonVA]

Okay so langsam macht es klick^^

werde meine Resultate hier rein posten. Danke für den Support

 

[JAMarlonVA]

Müsste ich mich in der background Methode neu mit meiner DB verbinden oder kann ich eine bestehende con verwenden?

Was kannst du mir da empfehlen?

 

[JAMarlonVA]

Ich bin soweit zufrieden mit der Funktion.

Optimal wäre gewesen, wenn das Label auch wie die ProgressBar gehandhabt worden wäre und ich eine neue Verbindung zur Datenbank erstellt hätte. Jedoch funktioniert es auch so und macht das was ich will.

Danke Robat für die Unterstützung

   class importTask extends SwingWorker<Void, Integer>
    {   
        private JProgressBar bar;
        private JLabel label;
        private String[][] data;
     
        public importTask(JProgressBar bar, JLabel label, String[][] myCSV) {
            this.bar = bar;
            this.label = label;
            this.data =myCSV;
        }
     
        @Override
        protected Void doInBackground() throws Exception
        {
            cGuid0 myGuid = new cGuid0();
     
            List<String> guids = new ArrayList<String>();     
         
            label.setText("Creating " + String.valueOf(this.data.length-1 + " records"));
         
            for(int i = 0; i < this.data.length-1; i++)
            {
                guids.add(myGuid.createGuid0());
                mySQL.insertQuery("INSERT INTO Words (Guid0) VALUES ('" + guids.get(i) + "')");
                                       
                publish(Math.round(i * 100 / data.length));
            }
         
            publish(0);
         
            for(int i = 0; i < this.data[0].length; i++) 
            {
                String row = mySQL.getRecent(this.data[0][i]).get(0)[0];
             
                label.setText("Adding cotents --> " + row);
             
                for(int i2 = 0; i2 < guids.size(); i2++)
                {
                    mySQL.updateQuery("UPDATE Words SET "+ row +"=N'" + this.data[i2+1][i].replace("'", "''") + "' WHERE Guid0='"+ guids.get(i2) +"'");
                }
             
                publish(i * 100 / data[0].length);
            }         
            return null;
        }

        @Override
        protected void process(List<Integer> chunks) {
            int lastValue = chunks.get(chunks.size() - 1);
            bar.setValue(lastValue);
        }

        @Override
        protected void done() {
            try {
                get();
                label.setText("finished");
            } catch (InterruptedException | ExecutionException e) {
                e.printStackTrace();
            }
        }
    }

 

[Robat]

Optimal wäre gewesen, wenn das Label auch wie die ProgressBar gehandhabt worden wäre

Das versteh ich nicht ganz. Was meinst du damit?

und ich eine neue Verbindung zur Datenbank erstellt hätte.

Könntest du ja im Konstruktor machen, wenn du wölltest.

BTW:
- Gewöhn dir am Besten an, Klassennamen immer in UpperCammelCase zu schreiben, das ist beim lesen nicht so verwirrend
- SQL-Queries solltest du nicht mit String-Konkatenation zusammen bauen (Sitchwort: SQL-Injection) - schau dir lieber mal PreparedStatements an

 

[JAMarlonVA]

Das versteh ich nicht ganz. Was meinst du damit?

In meiner "doInBackground" Methode setze ich den Lable - Text.
Die Value von der PGB setze ich über den "publish" - Befehl in der "Process" - Methode.
Optimal wäre es doch gewesen wenn ich den Wert der in dem Label stehen soll auch über den "publish" - Befehl an die "process" - Methode übergeben hätte oder irre ich mich da?

Guter Tipp. Ich habe aber auch schon festgestellt, dass meine Klasse für die SQL Abfragen etwas misslungen ist. Aber es sind auch meine ersten Abfragen mit Java.

 

[Robat]

Ach so. Ja aber auch das würde gehen. Du musst dir einfach eine eigene Wrapper-Klasse schreiben die Prozentzahl + Text in ein Objekt mapped.

class ResultChunk {
    private int percentage;
    private String info;

    public ResultChunk(int percentage, String info) {
        this.percentage = percentage;
        this.info = info;
    }
}

Dann kannst du beim SwingWorker eben Instanzen von ResultChunk übergeben, anstatt nur Zahlen

class Task extends SwingWorker<Void, ResultChunk> {
   @Override
    protected Void doInBackground() {
        ... 
        publish(new ResultChunk(0, "Initial Statement");
        ...
        return null;
    }
 
    @Override
    protected void process(List<ResultChunk> chunks) {
        ResultChunk lastValue = chunks.get(chunks.size() - 1);
        lbl.setText(lastValue.info);
        progressbar.(lastValue.percentage);
    }
   ...
}