私はランダムな色のプログラムを持っていると私はそれが急速に私は私の「瘤モード」ボタンのActionListenerを持っているが、私はそれを押したときに、それは何もしない色

Question

を通じて切り替える必要があります...

はのためのコードですActionListener：

private class AneurysmMode implements ActionListener { 


      AneurysmMode() {} 

      @Override 
      public void actionPerformed(ActionEvent e) { 
       while (true) { 

        try { 
         Thread.sleep(100); 
         frame.getContentPane().setBackground(colours[(int)(Math.random()*(9)+0)]); 
        } 
       catch (InterruptedException ex) { 

       } 
      } 
     }   
    } 

Answer 1

あなたがそのようなあなたがすべてのレンダリングの上に多くの制御を必要とするゲームとして洗練された何かを作っている場合を除き。そうでない場合は、独自のレンダリングループを実装する代わりに、javax.swing.timerをフルに使用できます。

EDTでレンダリングループまたはスリープを適用しないでください。 actionPerformed内のアクションは、通常、「1回限り」のアクションです。

この場合はボタンであなたのactionPerformedが故にあなたがする必要があるすべては、それがクリックされたときに（アニメーションを制御）、タイマーをオン/オフすることで、単に「オン/オフ」ボタンのように動作します：

class DrawingSpace extends JPanel{ 

    private JButton button; 
    private Timer timer; 
    private int idx; 
    private Color[] colors;  

    public DrawingSpace(){ 
     setPreferredSize(new Dimension(200, 200)); 
     initComponents(); 
     add(button);  
    } 

    public void initComponents(){ 
     idx = 0; 
     colors = new Color[]{Color.RED, Color.YELLOW, Color.ORANGE, Color.MAGENTA, Color.BLUE, Color.CYAN, Color.GREEN}; 
     button = new JButton("Let the colors rock!"); 
     timer = new Timer(100, new ActionListener(){ 
      @Override 
      public void actionPerformed(ActionEvent e){ 
       //Change color every (approx) 100 milliseconds 
       idx = (idx + 1) % colors.length; //cycle through the colours 
       setBackground(colors[idx]); 
      }    
     });  

     button.addActionListener(new ActionListener(){ 
      @Override 
      public void actionPerformed(ActionEvent e){ 
       if(!timer.isRunning()) 
        timer.start(); 
       else 
        timer.stop(); 
      } 
     }); 
    } 
} 

コードを実行するランナークラス：

class RainbowRunner{ 
    public static void main(String[] args){  
     // Run the GUI code on the EDT 
     SwingUtilities.invokeLater(new Runnable() {  
      @Override 
      public void run() { 
       JFrame frame = new JFrame("Rainbow Frame"); 
       frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
       frame.add(new DrawingSpace()); 
       frame.pack(); 
       frame.setLocationRelativeTo(null); 
       frame.setVisible(true);    
      } 
     }); 
    } 
} 

Answer 2

あなたはイベントのディスパッチスレッドを眠っている、それは悪い考えです。新しい色をイベントキューに追加しますが、ボタンアクションリスナーが戻ってこないため、アクションキューに入れません。 whileループをボタンハンドラから取り除くと、ボタンが押されるたびに変更されるはずです。

より良いオプションは、タイマーアクションイベントの色を変更し、ボタンを押してタイマーを開始することです。

Answer 3

actionPerformedメソッドにゲームループ全体（またはそれを呼び出すもの）を投げたくないでしょう。それが開始されていない場合で何ができ、すべてのボタンのクリックで、タイマーをトリガー：

button.addActionListener(new ActionListener(){ 
     @Override 
     public void actionPerformed(ActionEvent e){ 
      if(!timer.isRunning()) 
       timer.start(); 
      else 
       timer.stop(); 
     } 
    }); 

は、あなた自身のループを書くのではなく、javax.swing.timerを使用して検討することができます。あなたのjavax.swing.timerで

は、あなたがこのようなものを持つことができます。

timer = new Timer(100, new ActionListener(){ 
     @Override 
     public void actionPerformed(ActionEvent e){ 
      //Change color every (approx) 100 milliseconds 
      idx = (idx + 1) % colours.length; //cycle through the colours 
      frame.setBackground(colours[idx]); 
     }    
    });