CODEWITHSUNDEEP
java
Brian Muhic
Brian Muhic

Reputation: 39

Button Click Action in Java

I am trying to make a game and right now and I need the buttons to have an action attached to the them when one is clicked but I keep getting an error saying "Syntax error, insert "}" to complete ClassBody" there is not an error of a missing "}" however so there must be a problem with the code itself. 

import java.awt.Button;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JFrame;

public class Menu extends Window implements ActionListener
{

  Button start = new Button("Start Game:");
  Button quit = new Button("Quit");
}


public void actionPerformed(ActionEvent e)
{
  if (e.getActionCommand().equals("Start Game: "))
{
  System.out.print("Button one clicked");
}
else if (e.getActionCommand().equals("Quit"))
{
  System.out.print("Button two clicked");
}


}

Upvotes: 0

Views: 6434

Answers (2)

CaffeinatedCod3r
CaffeinatedCod3r

Reputation: 882

You forgot to register an instance of the event handler class as a listener on the button.

Add this inside your class 

start.addActionListener(this);
quit.addActionListener(this);

Also you have to move the actionPerformed function inside class body. Your class will look like

public class Menu extends Window implements ActionListener
{

  Button start = new Button("Start Game:");
  Button quit = new Button("Quit");
  start.addActionListener(this);
  quit.addActionListener(this);

 public void actionPerformed(ActionEvent e)
{
  if (e.getActionCommand().equals("Start Game: "))
  {
    System.out.print("Button one clicked");
  }
  else if (e.getActionCommand().equals("Quit"))
  {
    System.out.print("Button two clicked");
  }    
}
}

Upvotes: 0

George Z.
George Z.

Reputation: 6818

Method onActionPerformed is not in a class block:

public class Menu extends Window implements ActionListener
{ //Class block starts

  Button start = new Button("Start Game:");
  Button quit = new Button("Quit");
} //Class block ends

public void actionPerformed(ActionEvent e)
....

All ethods of a class must be inside the block of the class:

public class Menu extends Window implements ActionListener {

    Button start = new Button("Start Game:");
    Button quit = new Button("Quit");

    @Override
    public void actionPerformed(ActionEvent e) {
        if (e.getActionCommand().equals("Start Game: ")) {
            System.out.print("Button one clicked");
        } else if (e.getActionCommand().equals("Quit")) {
            System.out.print("Button two clicked");
        }

    }
}

Also, you might have to decide if you are going to use AWT widgets (java.awt.*) or Swing Components (javax.swing.*).

For instance, when you say Button button = new Button();, its an AWT widget. In order to use the Swing one, you must JButton jButton = new JButton();.

Upvotes: 1

Related Questions