java swing progressBar

Question

私はswingListを使用してコードを記述しました。これはactionListenerを追加するとprogressBarを更新しません。

ボタンとアクションリスナーなしで、それは素晴らしい作品です。 progressBarの更新をできるだけ簡単かつきれいにする方法はありますか？添付されたコードは、私の問題を要約した分かりやすい例です。 ActionPerformedメソッドをコメントアウトしてmainからプログラムを実行すると、正常に動作します。

説明せずにコードを貼り付けるだけではありません。私はこれを見てきました。：

PSは：swing progressBar threading

public class Okno { 

    private JProgressBar progressBar = new JProgressBar(0,306); 
    JFrame f = new JFrame("JProgressBar Sample"); 
    JButton b = new JButton("start"); 
    ActionListener a = new ActionListener() { 

     @Override 
     public void actionPerformed(ActionEvent e) { 
      barupdate(); 
     } 
    }; 


    private void barupdate(){ 
     for(int p = 1; p<308;p=p+2){ 
      System.out.println(p); 

      progressBar.setValue(p); 

       try { 
         Thread.sleep(50); 
        } catch (InterruptedException e) { 
         e.printStackTrace(); 
        } 
     } 
} 

private Okno(){ 

    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
    progressBar.setStringPainted(true); 

    f.add(progressBar, BorderLayout.SOUTH); 
    f.add(b, BorderLayout.NORTH); 
    b.addActionListener(a); 
    f.setSize(300, 300); 
    f.setVisible(true); 

} 

public static void main(String[] args) { 
    Okno okno = new Okno(); 
} 
} 

Answer 1

問題は、あなたがアクションリスナーから呼び出されているプログレスバーの設定を調整しているループを持っています。問題は、リスナーが終了するまでバーが更新されないことです。そしてあなたは更新を得ません。それだけでなく、あなたがそのアクションリスナーにいる間にウィンドウがマウスクリックなどに反応することができないので、あなたはGUIを動かすことになります。

これを処理する最善の方法は、アクションリスナーにスイングタイマーを作成し、そこにボタンを更新するコードを入れ、アクションリスナーでタイマーを開始することです。

タイマーはバーを1回だけ更新する必要があります。スイングタイマーが複数回呼び出され、反復性の部分を演奏するという事実を許すべきです。だから、あなたのコードにループを持たせたくありません。

Answer 2

Thread.sleep(50); 

Thread.sleep（...）は使用しないでください。これにより、ループの実行が終了するまでGUIが再描画されなくなります。

代わりにSwingWorkerを使用できます。

Concurrency in SwingのSwingチュートリアルのセクションを参照してください。詳細については、SwingWorkerの動作例が含まれています。

また、チュートリアルの目次もご覧ください。 How to Use ProgressBarsには、実際の例も含まれています。このチュートリアルは、例を探す最初の場所です。