JProgressBar不更新，找不到线索

Question

不错的工作，现在我只想知道为什么如果我加入while循环指令System.out.println下面的进度显示在这两个cmd和Pgbar在桂？ ：

while(progress < 99){ 
    System.out.println("into while of PBar Thread progress = "+progress); 
    if(progress != Path.operationProgress){ 
    operationProgressBar.setValue(progress); 
    progress = Path.operationProgress; 
    operationProgressBar.repaint(); } } 

需要一些帮助身边，我不能得到的JProgressBar更新，我 不能使用SwingWorker的，我必须解决这个问题离不开它。变量 Path.operationProgress是一个来自“Path”类的实例的静态变量，它从另一个线程更新，所以我认为PBar 和Path实例都在用户线程中执行，而不是在EDT中执行 EDT。这里是进度条的代码：

import javax.swing.*; 
    public class Pbar extends Thread { 
     JProgressBar operationProgressBar; 
     public Pbar(JProgressBar operationProgressBar) { 
      this.operationProgressBar = operationProgressBar; 
     } 

     @Override 
     public void run() { 
      int progress = Path.operationProgress; 
      while(progress < 99) { 
       if(progress != Path.operationProgress) { 
        operationProgressBar.setValue(progress); 
        progress = Path.operationProgress; 
        operationProgressBar.repaint(); 
       }}} 
    } 

这是启动线程的操作：

private javax.swing.JProgressBar operationProgressBar; 
private javax.swing.JLabel pathImage; 
private javax.swing.JButton simulatedAnnelingButton; 

public class TSPGUI extends javax.swing.JFrame { 

    TSPMG tspInstance; 
    Path p, result; 
    String filename = ""; 
    int neighborHood_Type = 1, i = 0; 
    // ......Constructor Stuff and init() 

private void simulatedAnnelingButtonActionPerformed(java.awt.event.ActionEvent evt) 

{
主题SA =新的Thread（）{ @覆盖 公共无效的run （）{ result = p.SimulatedAnnealing（neighborHood_Type）; String lastCostString = result.Cost（）+“”; lastCostLabel.setText（lastCostString）; }}; sa.start（）; Pbar pb = new Pbar（operationProgressBar）; pb.start（）; } // 一些其他的东西...... }

Answer 1

我会使用PropertyChangeListener来让退火进度值成为类的“绑定”属性。如果需要的话，任何观察者都可以关注这个属性。例如：

import java.awt.GridLayout; 
import java.awt.event.ActionEvent; 
import java.awt.event.ActionListener; 
import java.beans.PropertyChangeEvent; 
import java.beans.PropertyChangeListener; 

import javax.swing.*; 
import javax.swing.event.SwingPropertyChangeSupport; 

@SuppressWarnings("serial") 
public class TspGui2 extends JPanel { 
    private static final String ANNEALING_PROGRESS = "Annealing Progress"; 
    private JProgressBar progBar = new JProgressBar(0, 100); 
    private JLabel valueLabel = new JLabel(); 
    private JButton beginAnnealingBtn = new JButton("Begin Annealing"); 
    private MyAnnealing myAnnealing = new MyAnnealing(this); 

    public TspGui2() { 
     beginAnnealingBtn.addActionListener(new ActionListener() { 

     @Override 
     public void actionPerformed(ActionEvent e) { 
      beginAnnealing(); 
     } 
     }); 
     myAnnealing.addPropertyChangeListener(new PropertyChangeListener() { 

     @Override 
     public void propertyChange(PropertyChangeEvent evt) { 
      if (evt.getPropertyName().equals(MyAnnealing.ANNEALING)) { 
       // be sure this is done on the EDT 
       SwingUtilities.invokeLater(new Runnable() { 
        public void run() { 
        int annealedValue = myAnnealing.getAnnealedValue(); 
        setValue(annealedValue); 
        if (annealedValue >= MyAnnealing.MAX_ANNEALED_VALUE) { 
         beginAnnealingBtn.setEnabled(true); 
        } 
        } 
       }); 
      } 
     } 
     }); 
     progBar.setString(ANNEALING_PROGRESS); 
     progBar.setStringPainted(true); 

     JPanel northPanel = new JPanel(new GridLayout(1, 0)); 
     northPanel.add(beginAnnealingBtn); 
     northPanel.add(valueLabel); 

     setLayout(new BoxLayout(this, BoxLayout.PAGE_AXIS)); 
     add(northPanel); 
     add(progBar); 
    } 

    public void setValue(int value) { 
     valueLabel.setText("Value:" + value); 
     progBar.setValue(value); 
    } 

    public void beginAnnealing() { 
     beginAnnealingBtn.setEnabled(false); 
     setValue(0); 
     myAnnealing.reset(); 
     new Thread(new Runnable() { 
     public void run() { 
      myAnnealing.beginAnnealing(); 
     } 
     }).start(); 
    } 

    private static void createAndShowGui() { 
     TspGui2 mainPanel = new TspGui2(); 

     JFrame frame = new JFrame("TspGui2"); 
     frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
     frame.getContentPane().add(mainPanel); 
     frame.pack(); 
     frame.setLocationByPlatform(true); 
     frame.setVisible(true); 
    } 

    public static void main(String[] args) { 
     SwingUtilities.invokeLater(new Runnable() { 
     public void run() { 
      createAndShowGui(); 
     } 
     }); 
    } 
} 

class MyAnnealing { 
    public static final String ANNEALING = "Annealing"; 
    public static final int MAX_ANNEALED_VALUE = 100; 
    private SwingPropertyChangeSupport propChangeSupport = 
     new SwingPropertyChangeSupport(this); 
    private TspGui2 gui; 
    private int annealedValue; 

    public MyAnnealing(TspGui2 gui) { 
     this.gui = gui; 
    } 

    public void addPropertyChangeListener(
     PropertyChangeListener listener) { 
     propChangeSupport.addPropertyChangeListener(listener); 
    } 

    public void removePropertyChangeListener(
     PropertyChangeListener listener) { 
     propChangeSupport.removePropertyChangeListener(listener); 
    } 

    public void reset() { 
     setAnnealedValue(0); 
    } 

    // simulate some long process... 
    public void beginAnnealing() { 
     long sleepDelay = 100; 
     while (annealedValue < MAX_ANNEALED_VALUE) { 
     setAnnealedValue(annealedValue + 1); 
     try { 
      Thread.sleep(sleepDelay); 
     } catch (InterruptedException e) { 
      e.printStackTrace(); 
     } 
     } 
    } 

    public int getAnnealedValue() { 
     return annealedValue; 
    } 

    private void setAnnealedValue(int value) { 
     final int oldValue = this.annealedValue; 
     this.annealedValue = value; 
     propChangeSupport.firePropertyChange(ANNEALING, oldValue, annealedValue); 
    } 
} 

Answer 2

如果您不能使用SwingWorker然后用SwingUtilities.invokeLater，如：

if (progress != Path.operationProgress) { 
    final int progressCopy = progress; // Probably not final so copy is needed 
    SwingUtilities.invokeLater(new Runnable() { 
     @Override 
     void run() { 
      operationsProgressBar.setValue(progressCopy); 
     } 
    }); 
} 

注：执行此操作时，在run使用任何事物都有是最终的，或者必须有其他措施来访问这些变量。这个代码在这方面是象征性的。

您需要对事件派发线程外的Swing组件执行操作，但没有办法解决此问题。