JProgressBar ohne Threads verwenden

[roddy]

Guten Morgen,

ich weiss dass es eine FAQ zu JProgressBars gibt, aber die verwendet Multithreading, das hab ich mir noch nicht angelesen bzw. verstehe ich noch nicht und deshalb wüsste ich gerne

a) ob es (in meinem Fall) nicht auch ohne Multithreading geht
b) wo mein Fehler ist

Es geht mir momentan darum, dass mehrere Dateien nacheinander in einer for-Schleife per StreamTokenizer eingeladen werden. Das ganze soll per JProgressBar dargestellt werden. Mein (entsprechend gekürzter) Code sieht wie folgt aus:

        final JFrame RRprogframe = new JFrame("RVstat05 - Einlesen der Erhebungsdaten");
        RRprogframe.setSize(320,240);
        RRprogframe.setLocationRelativeTo(null);      
        RRprogframe.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        
        JLabel proginfo = new JLabel("Daten werden eingelesen...");
        JProgressBar RRprogress = new JProgressBar(JProgressBar.HORIZONTAL,0,filename.length);
        RRprogress.setValue(0);
        RRprogress.setStringPainted(true);
        RRprogress.setString("");
        ActionListener proglistener = new ActionListener(){ public void actionPerformed(ActionEvent e){
            RRprogframe.dispose();
        }
        };
        JButton RRprogbutton = new JButton("OK");
        RRprogbutton.addActionListener(proglistener);
        RRprogbutton.setEnabled(false);
        
        GridBagLayout RRprogGBL = new GridBagLayout();
        
        RRprogGBL.addLayoutComponent(proginfo, new GridBagConstraints(0,0,1,1,1,1,GridBagConstraints.WEST,GridBagConstraints.NONE,new Insets(5,5,5,5),0,0));
        RRprogGBL.addLayoutComponent(RRprogress, new GridBagConstraints(0,1,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.HORIZONTAL,new Insets(0,5,0,5),0,0));
        RRprogGBL.addLayoutComponent(RRprogbutton, new GridBagConstraints(0,2,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.NONE,new Insets(5,5,5,5),0,0));
        
        JPanel progpanel = new JPanel(RRprogGBL);
        progpanel.add(proginfo);
        progpanel.add(RRprogress);
        progpanel.add(RRprogbutton);
        
        RRprogframe.getContentPane().add(progpanel);
        
        RRprogframe.setVisible(true);

        for (int file = 0; file<filename.length; file++){
        // StreamTokenizer liest eine Datei ein ...
        RRprogress.setValue(file+1);
        String purefilename = filename[file];
        int slash = purefilename.lastIndexOf(System.getProperty("file.separator"));
        purefilename = purefilename.substring(slash+1,purefilename.length());
        RRprogress.setString(purefilename + " eingelesen.");
        }
        RRprogbutton.setEnabled(true);

Der Gedanke (bzw. meine Hoffnung) ist, dass ich keinen Timer und nichts brauche und somit das ganze ja auch ohne Multithreading gehen könnte (denk ich mir mal so *g*). Mein Problem ist nun, dass zwar mein JFrame erscheint, aber das Label, der Button und die ProgressBar erst im Fenster erscheinen, wenn die For-Schleife komplett durchlaufen ist. Ich sehe also nur einen vollen Fortschrittsbalken mit dem String "LetzteDatei eingelesen" und vorher ein leeres Fenster. Kann mir da jemand helfen?

 

[Düark]

Das liegt wohl daran, daß deine Schleife die GUI blockiert, d.h. die Schleife wird abgearbeitet und danach findet die VM erst wieder Zeit für den Progress-Bar.
Deswegen wirst Du um Threads nicht herumkommen, aber so schlimm wie es anfangs scheint, sind sie gar nicht!

Gruss

Dirk

 

[Flash0r]

Ich würde mal sagen du hast vergessen deine Bar zu aktualisieren, versuchs mal damit:

Rectangle progressRect =RRprogress.getBounds();
progressRect.x = 0;
progressRect.y = 0;
RRprogress.paintImmediately( progressRect );

Btw, ProgressBars gehen auch ohne Threads.
Gruß
Flash

 

[roddy]

Wo müssten diese Befehle hin? Hab sie jetzt mal zur Initialisierung der ProgressBar geschrieben (Zeile 11), aber das ändert nichts am Problem.

 

[Flash0r]

In deine For Schleife, du willst ja immer deine Bar aktualisieren wenn ein File geladen wurde.

 

[roddy]

Ah, vielen Dank! Das macht genau was ich wollte

 

[roddy]

    public progPlausibility(JFrame daddy, Vehicle[] data){
        super(daddy,"Plausibilitätsprüfung");
        
        db = data;
        
        test0 = new JLabel("<html><font color=blue>[b]TEST 1:[/b]</html>");
        test0bar = new JProgressBar(JProgressBar.HORIZONTAL,0,data.length);
        test0bar.setValue(0);
        test0bar.setStringPainted(true);
        test0bar.setString("0 %");
        
        test1 = new JLabel("<html><font color=red>[b]TEST 2:[/b]</html>");
        test1.setMaximumSize(new Dimension(160,200));
        test1bar = new JProgressBar(JProgressBar.HORIZONTAL,0,data.length);
        test1bar.setValue(0);
        test1bar.setStringPainted(true);
        test1bar.setString("0 %");
        
        test2 = new JLabel("<html><font color=red>[b]TEST 3:[/b]</html>");
        test2.setMaximumSize(new Dimension(160,200));
        test2bar = new JProgressBar(JProgressBar.HORIZONTAL,0,data.length);
        test2bar.setValue(0);
        test2bar.setStringPainted(true);
        test2bar.setString("0 %");
        
        test3 = new JLabel("<html><font color=red>[b]TEST 4:[/b]</html>");
        test3.setMaximumSize(new Dimension(160,200));
        test3bar = new JProgressBar(JProgressBar.HORIZONTAL,0,data.length);
        test3bar.setValue(0);
        test3bar.setStringPainted(true);
        test3bar.setString("0 %");
        
        test4 = new JLabel("<html><font color=red>[b]TEST 5:[/b]</html>");
        test4.setMaximumSize(new Dimension(160,200));
        test4bar = new JProgressBar(JProgressBar.HORIZONTAL,0,data.length);
        test4bar.setValue(0);
        test4bar.setStringPainted(true);
        test4bar.setString("0 %");
        
        userGBL = new GridBagLayout();
        userspace = new JPanel(userGBL);
        
        backbutton = new JButton("Zurück");
        backbutton.addActionListener(this);
        
        forwardbutton = new JButton("Weiter");
        forwardbutton.addActionListener(this);
        forwardbutton.setEnabled(false);
        
        GridBagLayout myGBL = new GridBagLayout();
        setLayout(myGBL);
        
        myGBL.addLayoutComponent(test0, new GridBagConstraints(0,0,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.NONE,new Insets(5,0,2,0),0,0));
        myGBL.addLayoutComponent(test1, new GridBagConstraints(1,0,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.NONE,new Insets(5,0,2,0),0,0));
        myGBL.addLayoutComponent(test2, new GridBagConstraints(2,0,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.NONE,new Insets(5,0,2,0),0,0));
        myGBL.addLayoutComponent(test3, new GridBagConstraints(3,0,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.NONE,new Insets(5,0,2,0),0,0));
        myGBL.addLayoutComponent(test4, new GridBagConstraints(4,0,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.NONE,new Insets(5,0,2,0),0,0));
        myGBL.addLayoutComponent(test0bar, new GridBagConstraints(0,1,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.HORIZONTAL,new Insets(0,2,5,2),0,0));
        myGBL.addLayoutComponent(test1bar, new GridBagConstraints(1,1,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.HORIZONTAL,new Insets(0,2,5,2),0,0));
        myGBL.addLayoutComponent(test2bar, new GridBagConstraints(2,1,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.HORIZONTAL,new Insets(0,2,5,2),0,0));
        myGBL.addLayoutComponent(test3bar, new GridBagConstraints(3,1,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.HORIZONTAL,new Insets(0,2,5,2),0,0));
        myGBL.addLayoutComponent(test4bar, new GridBagConstraints(4,1,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.HORIZONTAL,new Insets(0,2,5,2),0,0));
        myGBL.addLayoutComponent(userspace, new GridBagConstraints(0,2,5,1,100,100,GridBagConstraints.CENTER,GridBagConstraints.BOTH,new Insets(0,2,5,2),0,0));
        myGBL.addLayoutComponent(backbutton, new GridBagConstraints(0,3,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.NONE,new Insets(0,0,5,0),0,0));
        myGBL.addLayoutComponent(forwardbutton, new GridBagConstraints(4,3,1,1,1,1,GridBagConstraints.CENTER,GridBagConstraints.NONE,new Insets(0,0,5,0),0,0));
        
        getContentPane().add(test0); getContentPane().add(test1); getContentPane().add(test2); getContentPane().add(test3); getContentPane().add(test4);
        getContentPane().add(test0bar); getContentPane().add(test1bar); getContentPane().add(test2bar); getContentPane().add(test3bar); getContentPane().add(test4bar);
        getContentPane().add(userspace);
        getContentPane().add(backbutton); getContentPane().add(forwardbutton);
        
        setSize(800,600);
        setResizable(false);
        setLocationRelativeTo(daddy);
        setDefaultCloseOperation(DO_NOTHING_ON_CLOSE);
        addWindowListener(this);
        setModal(false);
        
        setVisible(true);
        
        kickOffTest0();
    }
    
    private void kickOffTest0(){
        for (int i = 0; i<db.length; i++){
            if (i%10 == 0){
                double prozent = (100*i) / db.length;
                test0bar.setString(prozent + " %");
                test0bar.setValue(i);
                repaint(test0bar);
            }
            // TEST SOMETHING
        }
            JLabel showuser = new JLabel("TESTERGEBNIS");
            userspace.add(showuser);
            repaint(userspace);
            test0.setText("<html><font color=green>[b]TEST 1:[/b] 
[i](...abgeschlossen...)[/i]</font>");
            repaint(test0);
            test0bar.setValue(test0bar.getMaximum());
            test0bar.setString("100 %");
            repaint(test0bar);
            test1bar.setMaximum(res.size());
            repaint(test1bar);
    }
    
    private void repaint(JPanel topaint){
        Rectangle rect = topaint.getBounds();
        rect.x = 0;
        rect.y = 0;
        topaint.paintImmediately(rect);
        this.validate();
    }
    
    private void repaint(JProgressBar topaint){
        Rectangle rect = topaint.getBounds();
        rect.x = 0;
        rect.y = 0;
        topaint.paintImmediately(rect);
        this.validate();
    }
    
    private void repaint(JLabel topaint){
        Rectangle rect = topaint.getBounds();
        rect.x = 0;
        rect.y = 0;
        topaint.paintImmediately(rect);
        this.validate();
    }

In diesem Code hab ich das in diesem Thema hier aufgezeigte Prinzip angewandt. Allerdings habe ich Probleme mit der Darstellung. Es sollte so sein (bzw. so wünsche ich es mir), dass zuerst die Test-Labels und die leere ProgressBars gemalt werden, dann der erste Test losgeht und die erste ProgressBar aktualisiert wird. Ich hatte das auch mal so, habe keine Ahnung, was ich geändert hab setidem, jedenfalls jetzt ist es nicht mehr so. Ich sehe in meinem Fenster nur die erste ProgressBar, wie sie anwächst (und die Prozentzahlen korrekt aktualisiert werden) und wenn der erste Test fertig ist, werden erst die anderen ProgressBars, Labels, Buttons etc. angezeigt. Sieht jemand einen Fehler, den ich hierbei mache? Oder kann das aus irgendeinem Grund auf diese Art und Weise gar nicht funktionieren?