В чём ошибка ActionListener?

Question

[Алексей Р]

Заранее извиняюсь, вернулся к изучению Java, и сразу застрял на простом примере и не как не могу разобраться в чём данная ошибка заключается. Понимаю что не красиво написать код и просить в нём разобраться но в данной ситуации как мне кажется нет другого выхода. Подскажите что почитать по этому поводу и как избежать подобной ошибки.

public class OOP {
	public static void main(String[] args) {
		
		
		JFrame frame = new JFrame();
		frame.setSize(400, 500);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setLocationRelativeTo(null);
		frame.setLayout(new GridBagLayout());

		JTextField textf = new JTextField();
		JTextField textf2 = new JTextField();
		JButton Mybutton = new JButton("Start");
		
		
		
		frame.add(textf, new GridBagConstraints(1, 1, 1, 1, 0.0, 0.9,
				GridBagConstraints.NORTH, GridBagConstraints.HORIZONTAL,
				new Insets(2, 2, 2, 2), 0, 0));

		frame.add(textf2, new GridBagConstraints(1, 2, 1, 1, 0.0, 0.9,
				GridBagConstraints.NORTH, GridBagConstraints.HORIZONTAL,
				new Insets(2, 2, 2, 2), 0, 0));

		frame.add(Mybutton, new GridBagConstraints(1, 3, 1, 1, 0.0, 0.9,
				GridBagConstraints.NORTH, GridBagConstraints.HORIZONTAL,
				new Insets(1, 2, 2, 2), 0, 0));
		
		Mybutton.addActionListener(new ButtonActionListener()); // Ошибка

		frame.setVisible(true);
		frame.pack();
		
		
	}
	
	
	public class ButtonActionListener implements ActionListener{

		@Override
		public void actionPerformed(ActionEvent e) {
          JOptionPane.showMessageDialog(null, "Messege box");
			
		}
		
	}
}

Как следствие слушатель не работает.

Видимо я делаю ошибки в концепции ООП но понять какую именно не могу. Пример беру из видео уроков, и как раз там то всё прекрасно

Answer 1

[Юрий Витковский]

Проблема в том что static код который выполняется в методе public static void main не может получит доступ к не статичному внутреннему классу. Либо сделайте класс public static class ButtonActionListener implements ActionListener, либо перенесите основной код в нестатичный метод

Comments

[Алексей Р]

Я пытался вынести код в отдельное место но это не увенчалось успехом. Попробую ещё раз. А вот статик действительно помог. Спасибо Вам. Самое обидное что читал читал про статические методы и всё забыл. Ещё раз спасибо

[Юрий Витковский]

Для использования не статичного кода необходимо создать экземпляр класса. К примеру:

public class OOP {

public static void main(String... args) {
new OOP().start();
}

void start() {
//Ваш код
}

}

[Алексей Р]

То есть Мы создаём метод void start() пишем в нём обработчик кнопки, а далее создаём экземпляр класса ООР и вызываем метод start()?

[Юрий Витковский]

Если в двух словах то да. В Вашем случае надо просто весь код и метода main перенести в метод start. И тогда можно будет использовать не статичный ButtonActionListener

[Алексей Р]

Юрий Витковский: Спасибо

[Алексей Р]

Юрий Витковский: Здравствуйте Юрий не могли бы Вы мне подсказать почему возникает ошибка в коде когда нажимаю сложение "package calculatortest;

import java.awt.Color;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JTextField;

public class CalculatorTest {

public static JPanel panelGeneral;
public static JLabel numL1, numL2, numL3;
public static JTextField numEdit1, numEdit2, numEdit3;
public static JButton buttonSum, buttonSub, buttonMulti, buttonDiv, buttonEqually;
public static JPanel panel1, panel2, panel3, panel4;

public static void main(String[] args) {


JFrame frame = new JFrame("Калькулятор");
frame.setSize(250, 360);
frame.setVisible(true);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.setLocationRelativeTo(null);

panelGeneral = new JPanel();
frame.add(panelGeneral);
BoxLayout bxLayout = new BoxLayout(panelGeneral, BoxLayout.PAGE_AXIS);
panelGeneral.setLayout(bxLayout);

numL1 = new JLabel("Введите первое число");
numL2 = new JLabel("Введите второе число");
numL3 = new JLabel("Результат");

numEdit1 = new JTextField(20);
numEdit2 = new JTextField(20);
numEdit3 = new JTextField(5);

buttonSum = new JButton("+");
buttonSum.setPreferredSize(new Dimension(50, 50));

buttonSub = new JButton("-");
buttonSub.setPreferredSize(new Dimension(50, 50));

buttonMulti = new JButton("x");
buttonMulti.setPreferredSize(new Dimension(50, 50));

buttonDiv = new JButton(":");
buttonDiv.setPreferredSize(new Dimension(50, 50));

buttonEqually = new JButton("=");
buttonEqually.setPreferredSize(new Dimension(80, 50));

panel1 = new JPanel(new FlowLayout(FlowLayout.LEFT));
panel1.setPreferredSize(new Dimension(250, 60));
panel1.add(numL1);
panel1.add(numEdit1);
panel1.add(numL2);
panel1.add(numEdit2);
// panel1.setBackground(Color.red);

panel2 = new JPanel(new FlowLayout(FlowLayout.LEFT));
panel2.setPreferredSize(new Dimension(250, 20));
panel2.add(numL3);
panel2.add(numEdit3);
// panel2.setBackground(Color.blue);

panel3 = new JPanel(new FlowLayout(FlowLayout.CENTER));
panel3.setPreferredSize(new Dimension(250, 20));
panel3.add(buttonSum);
panel3.add(buttonSub);
panel3.add(buttonMulti);
panel3.add(buttonDiv);
// panel3.setBackground(Color.yellow);

panel4 = new JPanel(new FlowLayout(FlowLayout.CENTER));
panel4.setPreferredSize(new Dimension(250, 20));
panel4.add(buttonEqually);
// panel4.setBackground(Color.cyan);


panelGeneral.add(panel1);
panelGeneral.add(panel2);
panelGeneral.add(panel3);
panelGeneral.add(panel4);

buttonEqually.addActionListener(new ButtonListener());
buttonSum.addActionListener(new SumButtonListener());


/*buttonEqually.addActionListener(new ActionListener() {
@Override
public void actionPerformed(ActionEvent e) {
JOptionPane.showConfirmDialog(null, "Всем привет");
}
});*/



}

public static class ButtonListener implements ActionListener{
@Override
public void actionPerformed(ActionEvent event) {
JOptionPane.showMessageDialog(null, "Это кнопка сработала");
}

}

public static class SumButtonListener implements ActionListener{
@Override
public void actionPerformed(ActionEvent e) {
int pole1 = Integer.parseInt(numEdit1.toString());
int pole2 = Integer.parseInt(numEdit2.toString());

Integer sumPole = pole1 + pole2;
numEdit3.setText(sumPole.toString());
}

}


}
" Пытаюсь реализовать простой калькулятор, мне нужно понять как он работает. Пока просто без выкрутасов. И не знаю где прочитать про парсинг более полноценнее

Answer 2

[asd111]

Как вариант можно сделать ButtonActionListener package-private классом

public class OOP {
  public static void main(String[] args) {
    
    JFrame frame = new JFrame();
    frame.setSize(400, 500);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setLocationRelativeTo(null);
    frame.setLayout(new GridBagLayout());

    JTextField textf = new JTextField();
    JTextField textf2 = new JTextField();
    JButton Mybutton = new JButton("Start");
    
    frame.add(textf, new GridBagConstraints(1, 1, 1, 1, 0.0, 0.9,
        GridBagConstraints.NORTH, GridBagConstraints.HORIZONTAL,
        new Insets(2, 2, 2, 2), 0, 0));

    frame.add(textf2, new GridBagConstraints(1, 2, 1, 1, 0.0, 0.9,
        GridBagConstraints.NORTH, GridBagConstraints.HORIZONTAL,
        new Insets(2, 2, 2, 2), 0, 0));

    frame.add(Mybutton, new GridBagConstraints(1, 3, 1, 1, 0.0, 0.9,
        GridBagConstraints.NORTH, GridBagConstraints.HORIZONTAL,
        new Insets(1, 2, 2, 2), 0, 0));
    
    Mybutton.addActionListener(new ButtonActionListener()); // Ошибка

    frame.setVisible(true);
    frame.pack();   
  } 
}

class ButtonActionListener implements ActionListener{
    @Override
    public void actionPerformed(ActionEvent e) {
          JOptionPane.showMessageDialog(null, "Messege box");      
    }
}