Appropriate use of JProgressBar

Question

/* Is this the correct way of using a JProgressBar, this is my first time using this class.
Please give your comments and correct me.
By the way, can you tell me when to use this method, addChangeListener(ChangeListener l). Is it possible for me to use when
the value increases.*/
public void check()
     cPane.remove(but);
     progressBar=new javax.swing.JProgressBar(javax.swing.JProgressBar.HORIZONTAL, 0, 100);
     cPane.add(progressBar, java.awt.BorderLayout.CENTER);
     setVisible(true);     
     Thread t=new Thread(new MonitorProgressBar());
     t.start();
     class MonitorProgressBar implements Runnable
          public MonitorProgressBar()
          public void run()
          int value=95;
               while(value <= progressBar.getMaximum())
               progressBar.setValue(value);
               progressBar.setString("Loading.... "+(progressBar.getPercentComplete()*100)+"%");
               progressBar.setStringPainted(true);
                    try
                    Thread.sleep(1000);
                    value++;
                    catch(Exception e)
                    { System.err.println(e.toString()); }
          System.out.println("Hello");
          //dispose();
     }

Answer 1

Hi,
/* Is this the correct way of using a JProgressBar,
this is my first time using this class.
Please give your comments and correct me.
By the way, can you tell me when to use this method,
addChangeListener(ChangeListener l). Is it possibleTry it. It should work.
for me to use when
the value increases.*/
public void check()
     cPane.remove(but);
progressBar=new
javax.swing.JProgressBar(javax.swing.JProgressBar.HORI
ONTAL, 0, 100);
cPane.add(progressBar,
java.awt.BorderLayout.CENTER);
     setVisible(true);     
     Thread t=new Thread(new MonitorProgressBar());
     t.start();
     }If you use Threads with a Progreesbar they should have a priority less than 4, cause the repainting thread has 4.
even the sleep doesn't garantues that the painting-progres
will do his work.
t.setPriority(Thread.Min_Priority) If you don't need a thread, have a look at SwingUtilities.invokeLater.
if setVisible(true);     is invoked by a modal dialog
the Thread will be created after closing the dialog.
     class MonitorProgressBar implements Runnable
          public MonitorProgressBar()
          public void run()
          int value=95;
               while(value <= progressBar.getMaximum())
               progressBar.setValue(value);
progressBar.setString("Loading....
.. "+(progressBar.getPercentComplete()*100)+"%");
               progressBar.setStringPainted(true);
                    try
                    Thread.sleep(1000);
                    value++;
                    catch(Exception e)
                    { System.err.println(e.toString()); }
          System.out.println("Hello");
          //dispose();
     }Hope this helps.
Greetings Michael