DRY код для реализации action listenener (JAVA)

Question

Теперь я реализую игру в лоток, в которой я использую JButton для представления лотка. Но лоток составляет 7x7, поэтому для реализации прослушивателя действий это не так весело. У меня есть такой код:

public void actionPerformed(ActionEvent ae) 
    { 
     if (ae.getSource() == Bouton11) 
      { 
       this.PosePion(1, 1, Bouton11); 
      } 
      else if (ae.getSource() == Bouton21) 
      { 
       this.PosePion(2, 1, Bouton21); 
      } 
      else if (ae.getSource() == Bouton31) 
      { 
       this.PosePion(3, 1, Bouton31); 
      } 
      ...... 
    } 

Как я могу уменьшить этот код? :/

Спасибо :)

Answer 1

Сделайте свой собственный тип слушателя. Ваш тип должен реализовывать ActionListener (и, следовательно, метод actionPerformed) и быть сконструирован с тремя параметрами: кнопкой и двумя целыми числами. Причина, по которой вам нужны эти три параметра, заключается в том, что вы можете передать их методу PosePion (который, кстати, должен быть капитализирован posePion).

Например:

class PoseActionListener implements ActionListener { 

    private JButton button; 
    private int a, b; 

    public PoseActionListener(JButton btn, int a, int b) { 
     this.button = btn; 
     this.a = a; 
     this.b = b; 
    } 

    @Override 
    public void actionPerformed(ActionEvent e) { 
     posePion(a, b, btn); 
    } 
} 

Тогда:

button11.addActionListener(new PoseActionListener(button11, 1, 1); 
button12.addActionListener(new PoseActionListener(button12, 1, 2); 

Или, еще лучше, создать все кнопки сразу:

for (int i=1; i<=7; i++) { 
    for (int j=1; j<=7; j++) { 
     JButton btn = new JButton("Button " + i + ", " + j); 
     // store the button in an array if you want 
     btn.addActionListener(new PoseActionListener(btn, i, j); 
    } 
} 

Answer 2

Поместите ваши JButtons в 2D, массив 7x7, когда они созданы.

Затем в методе слушателя проведите цикл по массиву, чтобы определить, какой из JButton был нажат. Ваши индексы цикла помогут вам определить, что нужно передать PosePion.

Answer 3

Я предлагаю вам изучить соглашения о кодировании Sun Java. Твой код трудно прочитать.

Я бы подумал о том, что для каждого экземпляра есть отдельный ActionListener, а не один для всех.

Я также рекомендую структуру данных и реализацию команд, чтобы сократить бессмысленное повторение.