Threads jProgressBar update

[Kyb2010]

Hallo,
ich bin ein ziemlich unerfahren im Programmieren, ich sitze jetzt schon ewig an dieser Sache.
Ich will "eigentlich" nur einen einfachen Countdowntimer erstellen, desse Fortschritt durch ein jProgressBar angezeigt wird. Ich hab jezt schon viel gelesen aber bekomme es einfach nicht hin.
Erst wenn die schleife fertig ist springt die Progressbar auf 100.

public void countdown()
    {
       
        long ctime;
        ctime = System.currentTimeMillis();
        
        while(System.currentTimeMillis()-ctime <=5000)
        {
            float x =(float)(System.currentTimeMillis()-ctime);
            float count = x/5000*100;
           
            final int percent = (int) count +1 ;
            
            System.out.println(percent);
            
            SwingUtilities.invokeLater(new Runnable()
                    {
                        @Override
                        public void run()
                        {
                            jProgressBar2.setValue(percent);
                            
                        }
                    });  
        }
    }

Grüße

 

[kaetzacoatl]

Vielleicht hilft dir dass: http://www.java-forum.org/awt-swing-swt/139450-keylistener-gui-thread-repaint.html
ohne einen repaint() aufruf geht nichts!

 

[Gast2]

Ein repaint() ist hier unnötig.

@TO:
Du rufst die countdown() vermutlich vom EDT aus auf. Verpacke den Aufruf in einen separaten Thread.

 

[Kyb2010]

Hey,
also ich hab die ganzen sachen die mir dazu eingefallen sind wie rapaint, revalidate etc... schon versucht.
Nach der Lektüre vieler Posts ist mir mehr oder weniger klar, dass ich den Timer in einem anderen thread laufen lassen muss oder so ähnlich. Ich hab aber keine Ahnung davon und bekomm das einfach nicht hin.

 

[kaetzacoatl]

public void run(){
		for(int i = 0;i <= 100;i++){
			jpb.setValue(i);
			long end = System.currentTimeMillis() + 1000;
			while(end > System.currentTimeMillis());
		}
	}

 

[Kyb2010]

Also irgendiwe komme ich nicht voran.
Ich habe ja in meinem code das mit der Runnable gemacht. der Timer soll starten indem ich auf einen Knopf drücke. Ich verstehe aber nicht, wo ich wann wie einen neuen Thread erzeuge und ihm die Runnable übergebe.

 

[Gast2]

Dann poste mal nen KSKB der dein Problem zeigt. Den zusätzlichen Thread kriegen wir dann schon da rein

 

[kaetzacoatl]

new Thread(new Runnable(){
public void run(){
        for(int i = 0;i <= 100;i++){
            jpb.setValue(i);
            long end = System.currentTimeMillis() + 1000;
            while(end > System.currentTimeMillis());
        }
    }
).start();

Geht natürlich auch schöner, aber so dürfte es gehen.

 

[Crian]

Vielleicht ist SwingWorker, was du suchst.

 

[Kyb2010]

Also eigentlich hab ich einen button:

private void jButton7ActionPerformed(java.awt.event.ActionEvent evt) {                                         
    

 }

Der soll die methode

 public void countdown()
    {
       
        long ctime;
        ctime = System.currentTimeMillis();
        
        while(System.currentTimeMillis()-ctime <=5000)
        {
            float x =(float)(System.currentTimeMillis()-ctime);
            float count = x/5000*100;
           
            final int percent = (int) count +1 ;
            this.percent = percent;
             System.out.println(percent);
             Thread thread = new Thread(new Runnable()
                  {
                      
                        public void run()
                       {
                          jProgressBar2.setValue(percent);
                        }
                    }); 
             
          thread.start();
        }
    }

Starten.

 

[kaetzacoatl]

public void run()
{
jProgressBar2.setValue(percent);
}

Durch:

public void run(){
        for(int i = 0;i <= 100;i++){
            jpb.setValue(i);
            long end = System.currentTimeMillis() + 1000;
            while(end > System.currentTimeMillis());
        }
    }

ersetzen!

Und das davor weglassen!
Denn das wird dann im EDT ausgeführt!

 

[xehpuk]

kaetzacoatl, ich möchte dir nicht zu nahe treten, aber deine Beiträge zeigen, dass du es nicht wirklich besser weißt als der TE.

private void jButton7ActionPerformed(java.awt.event.ActionEvent evt) {                                         
    jButton7.setEnabled(false);
    new Thread(new Runnable() {
        @Override
        public void run() {
            try {
                countdown();
            } finally {
                SwingUtilities.invokeLater(new Runnable() {
                    @Override
                    public void run() {
                        jButton7.setEnabled(true):
                    }
                }
            }
        }
    }).start();
}

public void countdown() {
    long ctime = System.currentTimeMillis();
    
    while(System.currentTimeMillis() - ctime <= 5000) {
        float x = (float) (System.currentTimeMillis() - ctime);
        float count = x/5000*100;
        
        final int percent = (int) count +1 ;
        this.percent = percent;
        System.out.println(percent);
        SwingUtilities.invokeLater(new Runnable() {
            @Override
            public void run() {
                jProgressBar2.setValue(percent);
            }
        }
    }
}

In diese Richtung würde es gehen. Habe das jetzt in keiner IDE abgetippt oder genauer überprüft, also habt Erbarmen.

countdown()

würde ich bei diesem Code lieber

burndown()

nennen. Sie frisst nämlich für fünf Sekunden die maximale CPU-Zeit. Da gehört ein

Thread.sleep()

rein.

 

[Kyb2010]

vielen Dank, das funktioniert jetzt.
Eine Frage habe ich noch:

ist es möglich die Zeit der Methode als parameter zu übergeben, um sie variabel zu machen?
habs versucht aber kann von "innen" nicht auf die übergebenen attribute zugreifen.

.

 

[Kyb2010]

Ich habs hinbekommen, vielen Dank nochmal an alle.

Grüße

 

[Rubinum]

Versuche es wirklich einfach mal mit dem SwingWorker. Dieser sorgt 100%ig dafür, dass dein herunterzählen in einem anderem Thread abläuft und du kannst zusätzlich noch Dinge angeben, die am Ende des Threads passieren sollen (.done()).

Hier nochmal die Doku:
How to Use Progress Bars (The Java™ Tutorials > Creating a GUI With JFC/Swing > Using Swing Components)
und hier ist ein Quellcode, wo die dort beschriebene ProgressBar mit diesem SwingWorker benutzt wird: Java Tutorials Sample Code

Ich hab vieles auch rumprobiert, aber mit dem SwingWorker funktioniert es 100%ig, sofern du es ordentlich implementierst .