JProgressBar flackert

[jensr]

Hallo,

ich habe eine GUI Klasse in der ich die gesamte Oberfläche erzeuge.
Bei einem Klick auf einen bestimmten Button wird eine Funktion aus
einer anderen Klasse aufgerufen, welche einen Array zurückgibt.
In der Zeit wo der Array gefüllt wird, möchte ich eine JProgressBar anzeigen
lassen. Das funktioniert auch wunderbar, aber leider flackert die JProgressbar.
Woran kann das liegen?

In der von der GUI aufgerufenen Funktion wird die JProgressbar so gesetzt:

GUI.parseProgressBar.setMaximum(filenames.length);
for (i = 0; i < filenames.length; i++) {
   GUI.parseProgressBar.setValue(i+1);
   //enforce painting the progress bar immediatly
   GUI.parseProgressBar.paint(GUI.parseProgressBar.getGraphics());
}

 

[L-ectron-X]

Hast du AWT- und Swing-Komponenten gemischt?

 

[jensr]

Glaube nicht, woran erkenne ich AWT-Komponenten?
Ich habe bisschen geschummelt, habe den Eclipse Visual Editor verwendet..

 

[L-ectron-X]

Hast du das Package java.awt importiert? Benutzt du Klassen aus diesem Package, die von Component erben?

 

[Wildcard]

Ich hab mir den Titel durchgelesen, und mir war eigentlich schon klar das ich etwas in dieser Richtung finde:

GUI.parseProgressBar.paint(GUI.parseProgressBar.getGraphics());

Ein paar Swing Grundsätze die man beachten sollte solange man noch nicht genau weiß was man tut:
- nicht mit AWT mischen
- niemals getGraphics auf einer Komponente aufrufen
- niemals paint aufrufen oder überschreiben
- niemals ein Graphics Objekt speichern

 

[jensr]

Ja tue ich:

import java.awt.event.KeyEvent;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.awt.Color;
import java.awt.Event;
import java.awt.BorderLayout;
import java.awt.Font;

Aber ich komme doch nicht ohne diese aus oder?


Lasse ich diese Codezeile weg, wird die JProgressbar aber erst nach dem Funktionsauruf aktualisiert..

GUI.parseProgressBar.paint(GUI.parseProgressBar.getGraphics());

 

[L-ectron-X]

Keine der von dir verwendeten Klassen aus AWT erbt von Component, also scheidet diese Möglichkeit eines Fehlers schon aus. Bleiben noch die Ansätze vopn Wildcard zu verfolgen.

 

[jensr]

MMh, also halte ich die 3 Punkte ein:
- niemals getGraphics auf einer Komponente aufrufen
- niemals paint aufrufen oder überschreiben
- niemals ein Graphics Objekt speichern

wird der Bar ja erst am Schluss gezeichnet..

 

[Wildcard]

Wie meinen? ???:L

 

[jensr]

Ich benutze bisher nicht mehrere Threads, da ich es so einfach nicht zum laufen bekomme, im moment mache ich es so:

Eine Funktion in meiner GUI-Klasse:

private JScrollPane getParseResultsFrame() {
        
		parseResultsFrame = new JScrollPane();
		parseResultsFrame.setBounds(new Rectangle(-1, 80, jFrame.getSize().width-6, 400));
		
		parseButton.addActionListener(new java.awt.event.ActionListener() {
			public void actionPerformed(java.awt.event.ActionEvent e) {
				File checkdir = new File(getDirCText().getText());
				if (checkdir.exists() && checkdir.isDirectory()) {
					parseProgressBar.setVisible(true);
					try {
						rowData = MP3Parser.parse(getDirCText().getText());
					} catch (IOException e1) {
						// TODO Auto-generated catch block
						e1.printStackTrace();
					}
					parseResultsFrame.setViewportView(getParseResultsTable());
 				} else {
					JOptionPane.showMessageDialog(null, "This is not a directory!");
				}
			}
		});
		return parseResultsFrame;
	}

Die Funktion in der MP3Parser-Klasse:

public static String[][] parse(String parseDir) throws IOException {
		File[] filenames = null;
		filenames = getFiles(parseDir);
		String[][] Tags = new String[filenames.length][7];
		GUI.parseProgressBar.setMaximum(filenames.length);

		for (i = 0; i < filenames.length; i++) {
	           GUI.parseProgressBar.setValue(i+1);
		   //enforce painting the progress bar immediatly
	           GUI.parseProgressBar.paint(GUI.parseProgressBar.getGraphics());
                   
                   /*
                   / ziemlich rechenaufwendiges zeugs
                   */
		}

		return Tags;
	}

Lasse ich nun diese Zeile weg:

GUI.parseProgressBar.paint(GUI.parseProgressBar.getGraphics());

läuft das Programm zwar wieder schneller und die Bar flackert nicht, aber die Bar wird erst gezeichnet, also die 100%, wenn die Funktion parse() durchgelaufen ist, da ja die Swing-Oberfläche solange blockiert ist.

Wenn ihr mir helfen könnt, das mit mehreren Threads aufzubauen, gibt zwar viele Tutorials im Netz, aber mein Hauptprogramm ist mittlerweile ziemlich komplex und irgendwie blicke ich das nicht mehr auf mein Programm anzuwenden!

Gruß Jens

 

[Wildcard]

Jau, du hast dir ja selbst schon das Stichwort gegeben: Threads.
Alles was zeitintensiv ist darf nicht im EventDispatcher Thread ausgeführt werden.

 

[jensr]

Ich werd verrückt, ich sitze da jetzt seid 10 Stunden dran und auf einmal gehts!

private JScrollPane getParseResultsFrame() {
       
      parseResultsFrame = new JScrollPane();
      parseResultsFrame.setBounds(new Rectangle(-1, 80, jFrame.getSize().width-6, 400));
      
      parseButton.addActionListener(new java.awt.event.ActionListener() {
         public void actionPerformed(java.awt.event.ActionEvent e) {
            File checkdir = new File(getDirCText().getText());
            if (checkdir.exists() && checkdir.isDirectory()) {
               parseProgressBar.setVisible(true);
               new Thread() {
                  @Override
                     public void run() {
		        try {
			   rowData = MP3Parser.parse(getDirCText().getText());
		        } catch (IOException e1) {
			   e1.printStackTrace();
			}
			parseResultsFrame.setViewportView(getParseResultsTable());
		     }
		 }.start();
            } else {
               JOptionPane.showMessageDialog(null, "This is not a directory!");
            }
         }
      });
      return parseResultsFrame;
   }

public static String[][] parse(String parseDir) throws IOException {
      File[] filenames = null;
      filenames = getFiles(parseDir);
      String[][] Tags = new String[filenames.length][7];
      GUI.parseProgressBar.setMaximum(filenames.length);

      for (i = 0; i < filenames.length; i++) {
              SwingUtilities.invokeLater( new Runnable() {
				public void run() {
					GUI.parseProgressBar.setValue(i+1);
				}
			});
                   
                   /*
                   / ziemlich rechenaufwendiges zeugs
                   */
      }
      return Tags;
   }

Vielen Dank
Gruß Jens

 

[jensr]

Jetzt nur noch eins, wie mache ich das, dass die JProgressBar wieder verschwindet, sobald der Thread abgearbeitet ist?


Gruß Jens

 

[Larouso]

in der Methode in der du den Thread "startest" mit join() warten bis er beendet ist und dann dispose() für deinen ProgressBar-Frame aufrufen? nur so ne idee...

 

[HardNoise]

Hi,

hier noch ein Vorschlag:
How to Use Progress Bars

Ich schreibe meine ProgressBars nur noch so. Außenrum ein Wrapper, der die Werte entgegennimmt und innen ein TimerThread, der die Werte, bei mir meistens im 100ms Intervall, in die eigentliche Progressbar schreibt.

Christian