Thread: Weird ActioListener problem

Hello,

I seem to have no syntax errors although when adding code it wont execute.

In the Action performed method i have:

if(e.getSource() == button1) { System.exit(0); }

the System.exit(0); will not execute a matter of fact nothing i put in there will... However if i add the code above the if statement it seems to execute. The problem being im having multiple buttons and dont want them all to do the same thing, Here is my code as ive said no errors but the code wont execute.

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.lang.*;
/**
 * This program will recieve the IP, PCI and client name of client computers and save them to a text file.
 * 
 * @author Marko Klesnik
 * @version 1.0
 */
 
public class guiDesign implements ActionListener
{
 
    // Variable Declarations
    // Declare JButtons
    public JButton btnNew, btnSave, btnFind, btnDelete, btnExit;
 
    // Declare JLabels
    public JLabel lblName, lblIP, lblPCI, lblDesc, lblDesc2, lblDesc3, lblDesc4;
 
    // Declare JTextFields
    public JTextField txtName, txtIP, txtPCI;
 
    // Create a string array
    public String [] clientList;
 
 
    public static void main(String[] args)
        {
            guiDesign test = new guiDesign();
            test.init();
 
        }
 
 
    public void init ()
    {
 
        // Create a string array
        clientList = new String [9];
        clientList [0] = "Please click Find Entry..";
 
        // Declare JList
        JList Selector = new JList ();
        Selector.setSize(90, 90);
        Selector.setBackground(Color.BLACK);
        Selector.setListData (clientList);
 
         // Scrollpane for JList
        JScrollPane ListScrollPane = new JScrollPane(Selector);
        ListScrollPane.setPreferredSize(new Dimension(550,200));
 
        // Define default JButton data
        JButton btnNew = new JButton("New Entry ");
        btnNew.setBackground(Color.BLACK);
        btnNew.setForeground(Color.GREEN);
        JButton btnSave = new JButton("Save Entry");
        btnSave.setBackground(Color.BLACK);
        btnSave.setForeground(Color.GREEN);
        JButton btnFind = new JButton("Find Entry");
        btnFind.setBackground(Color.BLACK);
        btnFind.setForeground(Color.GREEN);
        JButton btnDelete = new JButton("Delete Entry");
        btnDelete.setBackground(Color.BLACK);
        btnDelete.setForeground(Color.GREEN);
        JButton btnExit = new JButton("Exit System");
        btnExit.setBackground(Color.BLACK);
        btnExit.setForeground(Color.RED);
 
        // Define default JLabel data
        JLabel lblName = new JLabel("Client Name:");
        lblName.setForeground(Color.WHITE);
        JLabel lblIP = new JLabel("IP Address:");
        lblIP.setForeground(Color.WHITE);
        JLabel lblPCI = new JLabel("PC Identifier:");
        lblPCI.setForeground(Color.WHITE);
        JLabel lblDesc = new JLabel(" This program will allow the user to add ");
        lblDesc.setForeground(Color.WHITE);
        JLabel lblDesc2 = new JLabel(" edit or delete Users.their PC Identifiers ");
        lblDesc2.setForeground(Color.WHITE);
        JLabel lblDesc3 = new JLabel(" and their PC's IP Address. ");
        lblDesc3.setForeground(Color.WHITE);
        JLabel lblDesc4 = new JLabel(" Please click on the 'Find Entry' button to generate a client list... ");
        lblDesc4.setForeground(Color.WHITE);
 
        //Define default JTextField data
        JTextField txtName = new JTextField("Client1", 10);
        txtName.setBackground(Color.BLACK);
        txtName.setForeground(Color.GREEN);
        JTextField txtIP = new JTextField("", 10);
        txtIP.setBackground(Color.BLACK);
        txtIP.setForeground(Color.GREEN);
        JTextField txtPCI = new JTextField("", 10);
        txtPCI.setBackground(Color.BLACK);
        txtPCI.setForeground(Color.GREEN);
 
        //Create container + Layout
        SpringLayout scnLayout = new SpringLayout();
 
        // Create a frame and apply settings
        JFrame progFrame = new JFrame("[COMPANY NAME] Client Recording Software");
        progFrame.setVisible(true);
        progFrame.setSize(900, 800);
        progFrame.setResizable(false);
 
        // Create a JPanel and apply settings
        JPanel panel1 = new JPanel();
        panel1.setVisible(true);
        panel1.setSize(200, 300);
        panel1.setBackground(Color.black);
        panel1.setLayout(scnLayout);
 
        // Add panels to frame
        progFrame.add(panel1);
 
        // Add components to panels
        panel1.add(lblName);
        panel1.add(txtName);
        panel1.add(btnNew);
 
        panel1.add(lblPCI);
        panel1.add(txtPCI);
        panel1.add(lblIP);
        panel1.add(txtIP);
        panel1.add(btnSave);
        panel1.add(btnFind);
        panel1.add(btnDelete);
        panel1.add(btnExit);
        panel1.add(lblDesc);
        panel1.add(lblDesc2);
        panel1.add(lblDesc3);
        panel1.add(lblDesc4);
        panel1.add(ListScrollPane);
 
        // Set locations of panel1 components
        scnLayout.putConstraint(SpringLayout.WEST, lblName, 350, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, lblName, 200, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, txtName, 200, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, txtName, 35, SpringLayout.EAST, lblName);
        scnLayout.putConstraint(SpringLayout.NORTH, btnNew, 10, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, btnNew, 25, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, lblPCI, 250, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, lblPCI, 350, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, txtPCI, 250, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, txtPCI, 35, SpringLayout.EAST, lblPCI);
        scnLayout.putConstraint(SpringLayout.NORTH, lblIP, 300, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, lblIP, 350, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, txtIP, 300, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, txtIP, 45, SpringLayout.EAST, lblIP);
        scnLayout.putConstraint(SpringLayout.NORTH, btnSave, 10, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, btnSave, 150, SpringLayout.EAST, btnNew);
        scnLayout.putConstraint(SpringLayout.NORTH, btnFind, 10, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.EAST, btnFind, -150, SpringLayout.WEST, btnDelete);
        scnLayout.putConstraint(SpringLayout.NORTH, btnDelete, 10, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.EAST, btnDelete, -25, SpringLayout.EAST, panel1);
        scnLayout.putConstraint(SpringLayout.EAST, btnExit, -25, SpringLayout.EAST, panel1);
        scnLayout.putConstraint(SpringLayout.SOUTH, btnExit, -20, SpringLayout.SOUTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, ListScrollPane, 200, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, ListScrollPane, 150, SpringLayout.SOUTH, txtIP);
        scnLayout.putConstraint(SpringLayout.WEST, lblDesc, 350, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, lblDesc, 80, SpringLayout.NORTH, panel1);
        scnLayout.putConstraint(SpringLayout.WEST, lblDesc2, 350, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, lblDesc2, 10, SpringLayout.SOUTH, lblDesc);
        scnLayout.putConstraint(SpringLayout.WEST, lblDesc3, 385, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, lblDesc3, 10, SpringLayout.SOUTH, lblDesc2);
        scnLayout.putConstraint(SpringLayout.WEST, lblDesc4, 300, SpringLayout.WEST, panel1);
        scnLayout.putConstraint(SpringLayout.NORTH, lblDesc4, -16, SpringLayout.NORTH, ListScrollPane);
 
        btnExit.addActionListener((ActionListener) this.btnExit);
        btnNew.addActionListener(this);
 
    }
 
 
    public void actionPerformed(ActionEvent e)
    {
        //System.exit(0);
       if (e.getSource() == btnExit)  { System.exit(0); }
       //repaint();
       }
 
    }

Im puzzled lol..

Originally Posted by macko

if(e.getSource() == button1) { System.exit(0); }

There is no button1 in your code. There is a btnExit though. In future post correct code and information to avoid confusion.

You problem is a common one. You have 2 btnExit variables. An instance one and a local one in your constructor.

Oh i had figured it would be something tiny.. and sorry i should of just said the btnExit.

Thanks for your reply.