有没有更好的方法在 for 循环中为 JButtons 创建一个 actionListener？

Question

我想创建一些代表特定时间间隔的JButton 。 所有JButton基本上共享相同的actionListener ：它们将特定的间隔标记为“免费”（“Frei”）或“不免费”（“Belegt”）。 为此，他们不仅需要更改文本，而且还必须调用另一个 class 的方法，该方法使用他们的区间作为@parameter 。 我创建JButton并命名JLabel ，如下所示：

for (int i = 1; i <= numberOfIntervalls; i++) {
            String toolTextString;
            if(controller.getCalendar(calendarName).isFree(i, controller.getDayString(year, month, day))) {
                buttonText = "Frei";
                toolTextString = "Belegt";
            }else {
                buttonText = "Belegt";
                toolTextString = "Frei";
            }
            
            if(i%5 == 0) {
                gridx = 1;
                buttonGridy += 2;
                textGridy += 2;
            }else if(i < 5){
                gridx = (i%5);
            }else {
                gridx = (i%5) + 1;
            }
            intervallButtons[i-1] = new JButton(buttonText);
            intervallButtons[i-1].addActionListener((ActionListener) this);
            intervallLabels[i-1] = new JLabel(intervallLabelBuilder(i));
            intervallButtons[i-1].setToolTipText(String.format("als %s markieren", toolTextString));
            
            GridBagConstraints gbc_btnNewButton = new GridBagConstraints();
            gbc_btnNewButton.insets = new Insets(0, 0, 0, 5);
            gbc_btnNewButton.gridx = gridx;
            gbc_btnNewButton.gridy = buttonGridy;       
            contentPane.add(intervallButtons[i-1], gbc_btnNewButton);
            
            GridBagConstraints gbc_labels = new GridBagConstraints();
            //gbc_lblNewLabel.insets = new Insets(0, 0, 5, 5);
            gbc_labels.gridx = gridx;
            gbc_labels.gridy = textGridy;
            contentPane.add(intervallLabels[i-1], gbc_labels); 
            
        }
    }

我的actionListener方法是这样的，因为我知道numberOfIntervalls = 12 ，对于快速调整， numberOfIntervalls应该是一个变量，但它s static and final`：

public void actionPerformed (ActionEvent e){
         System.out.println("-----------");
            if(e.getSource() == intervallButtons[0]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                        controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(1, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(1, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[0].setToolTipText("Als Frei markieren");
                    
                }else {
                    controller.getCalendar(calendarName).setFree(1, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[0].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[0].setText(buttonText);
                intervallButtons[0].setEnabled(false);
                intervallButtons[0].setEnabled(true);

            }else if(e.getSource() == intervallButtons[1]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                        controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                
                if(controller.getCalendar(calendarName).isFree(2, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(2, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[1].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(2, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[1].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[1].setText(buttonText);
                intervallButtons[1].setEnabled(false);
                intervallButtons[1].setEnabled(true);
            }
            else if (e.getSource() == intervallButtons[2]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(3, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(3, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[2].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(3, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[2].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[2].setText(buttonText);
                intervallButtons[2].setEnabled(false);
                intervallButtons[2].setEnabled(true);
                
            }else if (e.getSource() == intervallButtons[3]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(4, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(4, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[3].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(4, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[3].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[3].setText(buttonText);
                intervallButtons[3].setEnabled(false);
                intervallButtons[3].setEnabled(true);
                
            }else if (e.getSource() == intervallButtons[4]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(5, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(5, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[4].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(5, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[4].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[4].setText(buttonText);
                intervallButtons[4].setEnabled(false);
                intervallButtons[4].setEnabled(true);
                
            }else if (e.getSource() == intervallButtons[5]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(6, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(6, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[5].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(6, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[5].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[5].setText(buttonText);
                intervallButtons[5].setEnabled(false);
                intervallButtons[5].setEnabled(true);
                
            }else if (e.getSource() == intervallButtons[6]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(7, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(7, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[6].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(7, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[6].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[6].setText(buttonText);
                intervallButtons[5].setEnabled(false);
                intervallButtons[5].setEnabled(true);
                
            }else if (e.getSource() == intervallButtons[7]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(8, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(8, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[7].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(8, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[7].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[7].setText(buttonText);
                intervallButtons[7].setEnabled(false);
                intervallButtons[7].setEnabled(true);
                
            }else if (e.getSource() == intervallButtons[8]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(9, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(9, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[8].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(9, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[8].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[8].setText(buttonText);
                intervallButtons[8].setEnabled(false);
                intervallButtons[8].setEnabled(true);
                
            }else if (e.getSource() == intervallButtons[9]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                        controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(10, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(10, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[9].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(10, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[9].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[9].setText(buttonText);
                intervallButtons[9].setEnabled(false);
                intervallButtons[9].setEnabled(true);
                
            }else  if (e.getSource() == intervallButtons[10]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(11, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(11, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[10].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(11, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[10].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[10].setText(buttonText);
                intervallButtons[10].setEnabled(false);
                intervallButtons[10].setEnabled(true);
                
            }else if (e.getSource() == intervallButtons[11]){
                if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                    controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(12, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(12, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[11].setToolTipText("Als Frei markieren");
                }else {
                    controller.getCalendar(calendarName).setFree(12, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[11].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[11].setText(buttonText);
                intervallButtons[11].setEnabled(false);
                intervallButtons[11].setEnabled(true);
                
            }
        }

它非常长，而且只有一个变量几乎不能调整，所以我想要的只是遵循创建for循环中的代码：

 if(!controller.getCalendar(calendarName).isLoaded(controller.getDayString(year, month, day)))
                        controller.getCalendar(calendarName).summonDay(controller.getDayString(year, month, day));
                if(controller.getCalendar(calendarName).isFree(1, controller.getDayString(year, month, day))) {
                    controller.getCalendar(calendarName).setFree(i, false, controller.getDayString(year, month, day));
                    buttonText = "Belegt";
                    intervallButtons[i-1].setToolTipText("Als Frei markieren");
                    
                }else {
                    controller.getCalendar(calendarName).setFree(i, true, controller.getDayString(year, month, day));
                    buttonText = "Frei";
                    intervallButtons[i-1].setToolTipText("Als Belegt markieren");
                }
                intervallButtons[i-1].setText(buttonText);
                intervallButtons[i-1].setEnabled(false);
                intervallButtons[i-1].setEnabled(true);

但它不起作用，因为actionListener只能处理非局部变量......你有什么想法吗？

Answer 1

并且 int i 对于每个 JButton 都是不同的，因为它需要是来自该 for 循环的 int i。

一种方法是为按钮设置“动作命令”。

当您创建您使用的按钮时：

JButton button = new JButton(...);
button.setActionCommand("" + i);

然后在您使用的 ActionListener 中：

int index = Integer.parseInt( event.getActionCommand() );