2017-01-02 12 views
-1

私は私のactionPerformedメソッドに問題があります。ボタンを押すたびにボタンの画像を変更したい。私は、単一のボタン(例えばbutton[0][0])のためにこれを行う場合、それは動作しますが、すべてのために、私は配列を使用してボタンを押したときのJButtonイメージの変更

java.lang.ArrayIndexOutOfBoundsException

を取得

私はここに捕まってしまったと私は方法がわかりません問題を修正するには...

public class ExButtons { 


JButton[][] button = new JButton[4][5]; 
ImageIcon[][] img = new ImageIcon[4][5]; 
ImageIcon cardTurned = new ImageIcon(); 

private final JPanel panel = new JPanel(); 
private final JFrame frame = new JFrame(); 
private int i, j; 

public ExButtons() { 

    img[0][0] = new ImageIcon("..."); 
    img[0][1] = new ImageIcon("..."); 
    img[0][2] = new ImageIcon("..."); 
    img[0][3] = new ImageIcon("..."); 
    img[0][4] = new ImageIcon("..."); 
    img[1][0] = new ImageIcon("..."); 
    img[1][1] = new ImageIcon("..."); 
    img[1][2] = new ImageIcon("..."); 
    img[1][3] = new ImageIcon("..."); 
    img[1][4] = new ImageIcon("..."); 
    img[2][0] = new ImageIcon("..."); 
    img[2][1] = new ImageIcon("..."); 
    img[2][2] = new ImageIcon("..."); 
    img[2][3] = new ImageIcon("..."); 
    img[2][4] = new ImageIcon("..."); 
    img[3][0] = new ImageIcon("..."); 
    img[3][1] = new ImageIcon("..."); 
    img[3][2] = new ImageIcon("..."); 
    img[3][3] = new ImageIcon("..."); 
    img[3][4] = new ImageIcon("..."); 
    cardTurned = new ImageIcon("..."); 

    for (i = 0; i < button.length; i++) { 
     for (j = 0; j < button[0].length; j++) { 
      button[i][j] = new JButton(img[i][j]); 

     } 
    } 

    int x = 100, y = 100; 

    for (i = 0; i < button.length; i++) { 
     for (j = 0; j < button[0].length; j++) { 

      button[i][j].setBounds(x, y, 60, 60); 
      panel.add(button[i][j]); 

      x += 80; 

      if (j == 4) { 

       x = 100; 
       y += 80; 
      } 
     } 
    } 

    for (i = 0; i < button.length; i++) { 
     for (j = 0; j < button[0].length; j++) { 

      button[i][j].addActionListener(new ActionListener() { 

       @Override 
       public void actionPerformed(ActionEvent e) { 

        System.out.println("Button[" + i + "][" + j + "] was pressed"); 
        button[i][j].setIcon(button[i][j].getIcon() == img[i][j] ? cardTurned : img[i][j]); 

       } 
      }); 
     } 
    } 

    frame.setSize(600, 600); 
    frame.setLocationRelativeTo(null); 
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); 
    panel.setLayout(null); 
    for (i = 0; i < button.length; i++) { 
     for (j = 0; j < button[0].length; j++) { 
      panel.add(button[i][j]); 
     } 
    } 

    frame.add(panel); 
    frame.setVisible(true); 

} 

public static void main(String[] args) { 
    new ExButtons().frame.setVisible(true); 
} 
} 

答えて

1

ループの最後では、iとjの両方が範囲外です。したがって、すべてのリスナーは、iとjに対してこれらの範囲外の値を使用します。

iとjはフィールドであってはなりません。それらはローカル変数でなければなりません。あなたはまた、間違いなくあなたのボタンの境界を設定する考えを放棄する必要があります

for (int i = 0; i < button.length; i++) { 
    for (int j = 0; j < button[0].length; j++) { 

     final JButton btn = button[i][j]; 
     final ImageIcon image = img[i][j]; 

     btn.addActionListener(new ActionListener() { 

      @Override 
      public void actionPerformed(ActionEvent e) { 

       btn.setIcon(btn.getIcon() == image ? cardTurned : image); 

      } 
     }); 
    } 
} 

for (int i = 0; i < button.length; i++) { 
    for (int j = 0; j < button[0].length; j++) { 

     final int i2 = i; 
     final int j2 = j; 

     button[i][j].addActionListener(new ActionListener() { 

      @Override 
      public void actionPerformed(ActionEvent e) { 

       System.out.println("Button[" + i2 + "][" + j2 + "] was pressed"); 
       button[i2][j2].setIcon(button[i2][j2].getIcon() == img[i2][j2] ? cardTurned : img[i2][j2]); 

      } 
     }); 
    } 
} 

あるいは、単純:リスナーが、最終的な変数を必要とするので、あなたはこれらの変数の最終コピーが必要です。代わりにレイアウトマネージャを使用してください。それはその仕事です。

+0

ありがとう@JBニジェット! – Nico

関連する問題