Обработчики событий Java

Question

Я пишу небольшое приложение Java (в Windows, следовательно, _on_vista прилагается к моему имени).

У меня есть 3 кнопки, каждая из которых будет реагировать на событие клика, но делать разные вещи.

Является ли следующий код принятым способом или есть более чистый способ, о котором я не знаю? На одной половине, он работает, на другой половине, что-то не кажется правильным ...

Благодаря

cool_button_1.addActionListener(new ActionListener() { 
    public void actionPerformed(ActionEvent e) { 
     coolfunction1(); 
    } 
}); 

cool_button_2.addActionListener(new ActionListener() { 
    public void actionPerformed(ActionEvent e) { 
     coolfunction2(); 
    } 
}); 

// etc ... 

Функции, которые вызываются породит от темы, по мере необходимости, так далее и так вперед.

ОБНОВЛЕНИЕ - Оба были хорошими (почти одинаковыми) ответами. Я согласился с тем, кто имеет низшую репутацию, чтобы разделить богатство. Еще раз спасибо ребятам.

Answer 1

Да, это правильный способ сделать это. Это немного неуклюжий (нужно написать пять длинных строк кода, чтобы можно было вызвать метод), но это Java :(

Answer 2

Я бы сказал, что код в порядке, так как код Java идет. Это хороший пример .. для часто критикуемого многословия в Java - в этом случае из-за отсутствие замыканий

Answer 3

Это не так уж плохо, я предпочитаю использовать Actions и создать JButtons из них:

Action fooAction = new AbstractAction() { ... }; 
JButton fooButton = new JButton(fooAction); 

Answer 4

Его один из правильных способов для этого.

Вы используете анонс МЫ слушатели, но есть Alternatif

хорошо объяснен на http://java.sun.com/docs/books/tutorial/uiswing/events/intro.html

где вы пишете класс, который реализует ActionListener и он использует логику для вас

Answer 5

я склонен сделать что-то вроде этого:

public class ButtonPanel implements ActionListener { 
    JButton button1, button2, button3; 
    public ButtonPanel() { 
     button1 = new JButton(this); 
     button2 = new JButton(this); 
     button3 = new JButton(this): 
     ... 
    } 

    public void ActionPerformed(ActionEvent e) { 
     if(e.getSource() == button1) action1(); 
     else if(e.getSouce() == button2 ) action2(); 
     else if(e.getSource() == button3) action3(); 
    } 
}