Cannot find symbol...

[John Pisci]

Hi Folks,

Got a problem that is really doing my head in.

I have a class that builds the GUI for my app in one file and in another file I have an action listener class.
In the action listener class, I want to get the text from each field and assign it to a variable as so:

    
String inputName = tfName.getText();

The problem I'm having is that I get the 'cannot find symbol' error.

It's all in the same package and has package level access, so I'm really confused!

Thanks in advance!

[Henry Wong]

There are numerous reasons why the compiler can't resolve a symbol -- you need to provide more info here. Maybe you could show us the exact error message. Or even show some of the code.

Henry

[John Pisci]

Hi Henry,

The action listener is called in my buildingGUI class:

    
btnSave = new JButton("Save");
btnSave.setForeground(Color.darkGray);
btnSave.setVisible(true);
btnSave.addActionListener(new btnSaveListener());

The action listener class itself is:

    
package guestList;

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import net.miginfocom.swing.*;
import java.sql.*;

public class btnSaveListener implements ActionListener 
{
  public void actionPerformed(ActionEvent event)
  {
    //Get the name and address into a string
    String inputName = tfName.getText();
    String inputAddress = taAddress.getText();
    //Get the number and convert it to an int
    String num = tfNumber.getText();
    int number = Integer.parseInt( num );
    //Get the no. of guests from a combobox and conver to an int
    String guest = (String)guestNoCombo.getSelectedItem();
    int guestNo = Integer.parseInt( guest );
    //Convert kids checkbox result into a string
    //Get no. of kids into an int if applicable
    if(kidBox.isSelected())
    {
      String kidsYN = "Y";
      String kids = (String)kidNoCombo.getSelectedItem();
      int kidsNo = Integer.parseInt( kids );
    }
    else
    {
      String kidsYN = "N";
      int kidsNo = '0';
    }
    //Get reply result into a string
    if(replyBox.isSelected())
    {
      String replyYN = "Y";
    }
    else
    {
      String replyYN = "N";
    }
    //Get the attendance
    String attend = (String)attendCombo.getSelectedItem();
    String invite = (String)inviteCombo.getSelectedItem();
    if(invite == "All Day")
    {
      if(dietBox.isSelected())
      {
      String dietRes = "Y";
      String dietInfo = tfDiet.getText();
      }
      else
      {
        String dietRes = "N";
      }
    }
    else
    {
      String dietRes = "N";
    }
  }
  connect c = new connect();
  c.dbConnect();
  SQLShizzle sql = new SQLShizzle();
  sql.insert();
}      

And the error message:

    
cannot find symbol
symbol  : variable tfName
location: class guestList.btnSaveListener
String dietInfo = tfName.getText();

Hope this helps!

[Henry Wong]

Basically, you code references the symbol tfName at various places...

String inputName = tfName.getText();   

And the compiler can't resolve it. It isn't a local variable. It isn't an instance or class variable. It isn't a classname. etc. There is nothing in scope that the compiler can find that is tfName.

Henry

[John Pisci]

Should I use getter and setter methods?

[Ernest Friedman-Hill]

John Pisci wrote:Should I use getter and setter methods?

Maybe, maybe not. But the most important thing you need to do is declare the variables someplace. Maybe you're already doing that. Are they member variables declared in your "buildingGUI" class, or some other class? Then your btnSaveListener needs to have a reference to the object that contains the variables; i.e.,

btnSave = new JButton("Save");
btnSave.setForeground(Color.darkGray);
btnSave.setVisible(true);
btnSave.addActionListener(new btnSaveListener(whateverObjectContainsThoseVariables));

and

public class btnSaveListener implements ActionListener   
{
    private SomeClass theObjectWithTheVariablesInIt;
    public btnSaveListener(SomeClass theObject) {
        theObjectWithTheVariablesInIt = theObject; 
    }

    public void actionPerformed(ActionEvent event) {
        //Get the name and address into a string
        String inputName = theObjectWithTheVariablesInIt.tfName.getText(); // if tfName is not private
        String inputName = theObjectWithTheVariablesInIt.getTFName().getText(); // if tfName is  private and you provide a getter method
       

[John Pisci]

The variables are defined in the class gList (which contains the main method and buildingGUI method).
Should I reference the the object 'g' when creating btnSaveListener as that is the object of the class 'gList' where all the variables are initially declared?
As in:

    
btnSave.addActionListener(new btnSaveListener(g));

Also, in your example, you've mentioned the following, but I don't quite understand it (sorry- I know I'm being a bit thick!):

    
private SomeClass theObjectWithTheVariablesInIt;   
public btnSaveListener(SomeClass theObject) {   
theObjectWithTheVariablesInIt = theObject;

Here's my gList class, with my main method and buildingGUI method:

    
package guestList;

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import net.miginfocom.swing.*;
import javax.swing.event.DocumentListener;
import java.sql.*;

class gList
{
  // Define the components-------------------------------------------------
  JFrame frame;
  JPanel background;
  JPanel foreground1;
  JPanel foreground2;
  
  JLabel lblName;
  JTextField tfName;
  
  JLabel lblAddress;
  JTextArea taAddress;
  
  JLabel lblNumber;
  JTextField tfNumber;
  
  JLabel lblOther;
  JTextArea taOther;
  
  JLabel lblGuestNo;
  JComboBox guestNoCombo;
  
  JLabel lblKidNo;
  JComboBox kidNoCombo;
  
  JLabel lblInvite;
  JComboBox inviteCombo;
  
  JLabel lblAttend;
  JComboBox attendCombo;
  
  JLabel lblKidBox;
  JCheckBox kidBox;
  
  JLabel lblReply;
  JCheckBox replyBox;
  
  JLabel lblDiet;
  JCheckBox dietBox;
  JLabel lblDietReq;
  JTextField tfDiet;
  
  JButton btnSave;
  JButton btnDelete;
  JButton btnClear;
  
  JMenuBar menuBar;
  JLabel lblInfo;
  
  String[] inviteData = {" ", "All Day", "Evening Only"};
  String[] numData = {"0","1","2","3","4","5","6","7","8","9"};
  String[] ynData = {" ", "Y", "N"};
  
  public static void main(String[] args) throws Exception
  {
    gList g = new gList();
    g.buildingGUI();
  }
  void buildingGUI()
  {
      //Set the frame details
      frame = new JFrame();
      frame.setTitle("Guest List");
      frame.setSize(600, 400);
      frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      frame.setBackground(Color.darkGray);
      //Get the menu bar inserted
      menuBar = new JMenuBar();
      menuBar.setPreferredSize(new Dimension(20, 25));
      menuBar.setBackground(Color.darkGray);
      
      lblInfo = new JLabel();
      lblInfo.setText("");
      lblInfo.setForeground(Color.darkGray);
      //lblInfo.setBackground(Color.darkGray);
      //Insert the background panel
      background = new JPanel(new MigLayout());
      background.setBackground(Color.darkGray);
      background.setPreferredSize(new Dimension(600,400));
      
      //Customise the components
      lblName = new JLabel("Name:   ");
      lblName.setForeground(Color.white);
      
      tfName = new JTextField(20);
      lblName.setLabelFor(tfName);
      //tfName.addActionListener(new tfNameListener());
            
      lblAddress = new JLabel("Address:");
      lblAddress.setForeground(Color.white);
      
      taAddress = new JTextArea(5, 20);
      taAddress.setLineWrap(true);
      lblAddress.setLabelFor(taAddress);
      //taAddress.getDocument().addDocumentListener(new taAddressListener());
      
      lblNumber = new JLabel("Number: ");
      lblNumber.setForeground(Color.white);
      
      tfNumber = new JTextField(20);
      lblNumber.setLabelFor(tfNumber);
      
      lblGuestNo = new JLabel("No. of Adults:");
      lblGuestNo.setForeground(Color.white);
      //Will need to use parseInt to convert
      //guestNoCombo to int
      guestNoCombo = new JComboBox(numData);
      //guestNoCombo.setVisibleRowCount(1);
      lblGuestNo.setLabelFor(guestNoCombo);
      
      lblKidBox = new JLabel("Children: ");
      lblKidBox.setForeground(Color.white);
      
      kidBox = new JCheckBox();
      kidBox.setForeground(Color.darkGray);
      kidBox.setBackground(Color.darkGray);
      lblKidBox.setLabelFor(kidBox);
      kidBox.addItemListener(new kidBoxListener());
      
      lblKidNo = new JLabel("No. of Children: ");
      lblKidNo.setForeground(Color.white);
      lblKidNo.setVisible(false);
      //Will need to use parseInt to convert
      //kid.2NoCombo to int
      kidNoCombo = new JComboBox(numData);
      lblKidNo.setLabelFor(kidNoCombo);
      kidNoCombo.setVisible(false);
      
      lblReply = new JLabel("Replied: ");
      lblReply.setForeground(Color.white);
      
      replyBox = new JCheckBox();
      replyBox.setBackground(Color.darkGray);
      replyBox.addItemListener(new replyBoxListener());
      
      lblAttend = new JLabel("Attending: ");
      lblAttend.setForeground(Color.white);
      lblAttend.setVisible(false);
      
      attendCombo = new JComboBox(ynData);
      lblAttend.setLabelFor(attendCombo);
      attendCombo.setVisible(false);
      
      lblInvite = new JLabel("Invite Type: ");
      lblInvite.setForeground(Color.white);
      
      inviteCombo = new JComboBox(inviteData);
      //inviteList.setVisibleRowCount(1);
      lblInvite.setLabelFor(inviteCombo);
      inviteCombo.addItemListener(new inviteComboListener());
      
      lblDiet = new JLabel("Special Diet Needed: ");
      lblDiet.setForeground(Color.white);
      lblDiet.setVisible(false);
      
      dietBox = new JCheckBox();
      dietBox.setBackground(Color.darkGray);
      lblDiet.setLabelFor(dietBox);
      dietBox.setVisible(false);
      dietBox.addItemListener(new dietBoxListener());
      
      lblDietReq = new JLabel("Dietary Requirement: ");
      lblDietReq.setForeground(Color.white);
      lblDietReq.setVisible(false);
      
      tfDiet = new JTextField(20);
      lblDietReq.setLabelFor(tfDiet);
      tfDiet.setVisible(false);
      
      btnSave = new JButton("Save");
      btnSave.setForeground(Color.darkGray);
      btnSave.setVisible(true);
      btnSave.addActionListener(new btnSaveListener());
      
      btnClear = new JButton("Clear");
      btnClear.setForeground(Color.darkGray);
      btnClear.setVisible(true);
      //btnClear.addActionListener(new btnClearListener());
      
      btnDelete = new JButton("Delete");
      btnDelete.setForeground(Color.darkGray);
      btnDelete.setVisible(true);
      //btnDelete.addActionListener(new btnDeleteListener());
      
      // Position the components
      background.add(lblName);
      background.add(tfName, "wrap");
      background.add(lblAddress);
      background.add(taAddress, "wrap");
      background.add(lblNumber);
      background.add(tfNumber, "wrap");
      background.add(lblGuestNo);
      background.add(guestNoCombo, "wrap");
      background.add(lblKidBox);
      background.add(kidBox, "split 3");
      background.add(lblKidNo, "gapleft 80");
      background.add(kidNoCombo, "wrap");
      background.add(lblReply);
      background.add(replyBox, "split 3");
      background.add(lblAttend, "gapleft 80");
      background.add(attendCombo, "gapleft 31");
      background.add(lblInvite, "newline");
      background.add(inviteCombo, "wrap");
      background.add(lblDiet);
      background.add(dietBox, "wrap");
      background.add(lblDietReq);
      background.add(tfDiet, "wrap");
      background.add(btnSave, "gapleft 80");
      background.add(btnClear, "split 2");
      background.add(btnDelete);
           
      frame.getContentPane().add(menuBar, BorderLayout.NORTH);
      frame.getContentPane().add(background, BorderLayout.CENTER);
      frame.getContentPane().add(lblInfo, BorderLayout.SOUTH);
      frame.setVisible(true);
  } // Close buildGUI()
} // Close gList

[John Pisci]

Right, think I've got it sorted, thanks for all of your help!

Added to the buildingGUI method:

    
      btnSave = new JButton("Save");
      btnSave.setForeground(Color.darkGray);
      btnSave.setVisible(true);
      btnSave.addActionListener(new btnSaveListener(this));

Added to the btnSaveListener class

    
package guestList;

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import net.miginfocom.swing.*;
import java.sql.*;

public class btnSaveListener implements ActionListener 
{
  private gList glist;
  public btnSaveListener(gList glist)
  {
    this.glist = glist;
  }
  public void actionPerformed(ActionEvent event)
  {
    //Get the name and address into a string
    String inputName = glist.tfName.getText();
    String inputAddress = glist.taAddress.getText();
    //Get the number and convert it to an int
    String num = glist.tfNumber.getText();
    int number = Integer.parseInt( num );
    //Get the no. of guests from a combobox and conver to an int
    String guest = (String)glist.guestNoCombo.getSelectedItem();
    int guestNo = Integer.parseInt( guest );
    //Convert kids checkbox result into a string
    //Get no. of kids into an int if applicable
    if(glist.kidBox.isSelected())
    {
      String kidsYN = "Y";
      String kids = (String)glist.kidNoCombo.getSelectedItem();
      int kidsNo = Integer.parseInt( kids );
    }
    else
    {
      String kidsYN = "N";
      int kidsNo = '0';
    }
    //Get reply result into a string
    if(glist.replyBox.isSelected())
    {
      String replyYN = "Y";
    }
    else
    {
      String replyYN = "N";
    }
    //Get the attendance
    String attend = (String)glist.attendCombo.getSelectedItem();
    String invite = (String)glist.inviteCombo.getSelectedItem();
    if(invite == "All Day")
    {
      if(glist.dietBox.isSelected())
      {
      String dietRes = "Y";
      String dietInfo = glist.tfDiet.getText();
      }
      else
      {
        String dietRes = "N";
      }
    }
    else
    {
      String dietRes = "N";
    }
  }
  connect c = new connect();
  c.dbConnect();
  SQLShizzle sql = new SQLShizzle();
  sql.insert();
}      
   

[John Pisci]

Thought I'd may as well just bump this thread instead of creating a new one as it is a similar problem.

Anyway, I have an action listener for a button that gets all of the info in the fields of the GUI and assigns them to variable. This then calls an SQL method which performs the SQL insert.

I am having trouble with getting the variables from the action listener into the SQL insert method.

Thanks in advance!

I get the following error for each and everyone of the variables:

    
L:\My Projects\MyJavaApps\guestList\SQLShizzle.java:44: cannot find symbol
symbol  : variable inputName
location: class guestList.btnSaveListener + bsl.inputName +"','"

action listener:

    
public class btnSaveListener implements ActionListener 
{
  private gList glist;
  public btnSaveListener(gList glist)
  {
    this.glist = glist;
  }
  public void actionPerformed(ActionEvent event)
  {
    //Get the name and address into a string
    String inputName = glist.tfName.getText();
    String inputAddress = glist.taAddress.getText();
    //Get the number and convert it to an int
    String num = glist.tfNumber.getText();
    int number = Integer.parseInt( num );
    //Get the no. of guests from a combobox and conver to an int
    String guest = (String)glist.guestNoCombo.getSelectedItem();
    int guestNo = Integer.parseInt( guest );
    String kidsYN;
    int kidsNo;
    String replyYN;
    String dietRes;
    String dietInfo;
    //Convert kids checkbox result into a string
    //Get no. of kids into an int if applicable
    if(glist.kidBox.isSelected())
    {
      kidsYN = "Y";
      String kids = (String)glist.kidNoCombo.getSelectedItem();
      kidsNo = Integer.parseInt( kids );
    }
    else
    {
      kidsYN = "N";
      kidsNo = '0';
    }
    //Get reply result into a string
    if(glist.replyBox.isSelected())
    {
      replyYN = "Y";
    }
    else
    {
      replyYN = "N";
    }
    //Get the attendance
    String attend = (String)glist.attendCombo.getSelectedItem();
    String invite = (String)glist.inviteCombo.getSelectedItem();
    if(invite == "All Day")
    {
      if(glist.dietBox.isSelected())
      {
        dietRes = "Y";
        dietInfo = glist.tfDiet.getText();
      }
      else
      {
        dietRes = "N";
      }
    }
    else
    {
      dietRes = "N";
    }
  connect c = new connect();
  c.dbConnect();
  SQLShizzle sql = new SQLShizzle();
  sql.insert(inputName,inputAddress,number,guestNo,kidsYN,kidsNo,
              replyYN,attend,invite,dietRes,dietInfo);
  }
}

SQL class:

    
class SQLShizzle
{
	private connect connect;
	private btnSaveListener bsl;
  public SQLShizzle(btnSaveListener bsl, connect connect)
  {
    this.bsl = bsl;
    this.connect = connect;
  }
	Statement stmt = connect.con.createStatement();
	//dialogBox db = new dialogBox();
	//Chances are I'll have to pass in the input values.
	public void insert(String inputName, String inputAddress,
              int number, int guestNo, String kidsYN, int kidsNo,
              String replyYN, String attend, String invite,
              String dietRes, String dietInfo) throws Exception
	{
		//Statement stmt = con.createStatement();
		try 
		{
			//Need to get the correct column names
			stmt.executeUpdate( 
			"INSERT INTO guestList "
			+ " ( name,"
				+"address,"
				+"number,"
				+"guestNo,"
				+"children,"
				+"kidsNo,"
				+"reply,"
				+"attending,"
				+"inviteType,"
				+"diet,"
				+"dietInfo )" 
					+ "VALUES ( '"
					+ bsl.inputName +"','"
					+ bsl.inputAddress +"','"
					+ bsl.number +"','"
					+ bsl.guestNo +"','"
					+ bsl.kidsYN +"', '"
					+ bsl.kidsNo +"','"
					+ bsl.replyYN +"','"
					+ bsl.attend +"','"
					+ bsl.invite +"','"
					+ bsl.dietRes +"','"
					+ bsl.dietInfo +
					"'  ) " );
			//Need to add in JOptionPane from dialogBox.java to act as a confirmation box
			//db.dialogueBox("Are you sure you want to insert this record?",YES_NO_OPTION, QUESTION_MESSAGE));
			}
			catch (SQLException e)
			{
			System.err.println("SQLException: " + e.getMessage());
			//db.dialogueBox(e.getMessage(),
				//						OK_OPTION, ERROR_MESSAGE);
			}
		finally
		{
		stmt.close();
		connect.con.close();
		}
	}
	public void delete() throws Exception
	{
		Statement stmt = con.createStatement();
		try
		{
      stmt.executeUpdate(
        "DELETE FROM guestList WHERE name = '"
        + bsl.inputName + "'");
    }
    catch (SQLException e)
    {
      System.err.println("SQLException: " + e.getMessage());
      //db.dialogueBox(e.getMessage(),
        //          OK_OPTION, ERROR_MESSAGE);
    }
    finally
		{
      stmt.close();
      connect.con.close();
		}
  }
}

[Henry Wong]

I get the following error for each and everyone of the variables:

Each and every one of those variables are *local* variables of the actionPerformed() method. They are not in scope outside of the method.

Henry

[Fred Hamilton]

Henry Wong wrote:

I get the following error for each and everyone of the variables:

Each and every one of those variables are *local* variables of the actionPerformed() method. They are not in scope outside of the method.

Henry

It seems to me there some questions of programming style here also?

We have separate class for just an action listener and little else. (btnSaveListener) And that class is doing a lot of work with a lot of different variables. It seems that is a scenario that invites trouble. Sure you can make it work, but...

Maybe there are reasons for doing it this way, I don't know. There's always different ways of doing things. But I like to keep my action listeners much cleaner than what is shown here, and rely more on appropriate method calls.

[John Pisci]

Henry Wong wrote:

I get the following error for each and everyone of the variables:

Each and every one of those variables are *local* variables of the actionPerformed() method. They are not in scope outside of the method.

Henry

As soon as I read this, I realised what I was doing wrong! Thanks!

I have now declared the variables outside of the actionPerformed() method.

Now on to the next set of errors...

[John Pisci]

Fred Hamilton wrote:

Henry Wong wrote:

I get the following error for each and everyone of the variables:

Each and every one of those variables are *local* variables of the actionPerformed() method. They are not in scope outside of the method.

Henry

It seems to me there some questions of programming style here also?

We have separate class for just an action listener and little else. (btnSaveListener) And that class is doing a lot of work with a lot of different variables. It seems that is a scenario that invites trouble. Sure you can make it work, but...

Maybe there are reasons for doing it this way, I don't know. There's always different ways of doing things. But I like to keep my action listeners much cleaner than what is shown here, and rely more on appropriate method calls.

I hadn't thought about just using the action listener to perform method calls.

Will the performance be affected doing it the way I am? Or is it just a personal preference to have clean action listeners?

As you can tell, I am learning, but I appreciate all of the advice I can get!

[David Newton]

It's not an issue of performance, it's an issue of readability and maintainability.

[Fred Hamilton]

exactly. Readability and maintainability. And in addition to considerations about how much work you want your event handler (actionPerformed) to do, also one can think about the degree to which the application is "classified".

Just seeing this one class which is little more than an event handler leads to believe that you might end up with a real proliferation of classes. If you wanted to be able to use this particular listener/event handler with other applications, then your idea is pretty good. If the listener has no use outside of the current application, then maybe it's not so good.

In my programs, I usually have a main GUI class that includes most if not all of my listeners and event handlers, and the event handlers themselves are very minimal, often just a method call, and the method is in another class. This approach serves to separate functionality from presentation.

Again, this sort of stuff is more a question of style, readability, maintainability. You know your program best, so it's for you to decide these things, I'm not saying what is best, just pointing out a few design decisions you need to make. I haven't done much database work with java, so maybe that influences general application design.

[John Pisci]

Fred Hamilton wrote:exactly. Readability and maintainability. And in addition to considerations about how much work you want your event handler (actionPerformed) to do, also one can think about the degree to which the application is "classified".

Just seeing this one class which is little more than an event handler leads to believe that you might end up with a real proliferation of classes. If you wanted to be able to use this particular listener/event handler with other applications, then your idea is pretty good. If the listener has no use outside of the current application, then maybe it's not so good.

In my programs, I usually have a main GUI class that includes most if not all of my listeners and event handlers, and the event handlers themselves are very minimal, often just a method call, and the method is in another class. This approach serves to separate functionality from presentation.

Again, this sort of stuff is more a question of style, readability, maintainability. You know your program best, so it's for you to decide these things, I'm not saying what is best, just pointing out a few design decisions you need to make. I haven't done much database work with java, so maybe that influences general application design.

That makes a lot of sense, I'd not really thought about doing it that way.
This is my first java app. with a GUI and a database and to be honest, I think I've made quite a few design mistakes. Saying that though, it is the best way to learn.

Have another problem know though; one that just randomly appeared. I don't know why, but it has been working.

When I compile, I get the error:

    
gList.java:183: cannot find symbol
symbol  : constructor btnSaveListener(guestList.gList)
location: class guestList.btnSaveListener
btnSave.addActionListener(new btnSaveListener(this));

gList class:

    
btnSave = new JButton("Save");
btnSave.setForeground(Color.darkGray);
btnSave.setVisible(true);
btnSave.addActionListener(new btnSaveListener(this));

btnSaveListener class:

    
package guestList;

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import net.miginfocom.swing.*;
import java.sql.*;

public class btnSaveListener implements ActionListener 
{
  private gList glist;
  private connect c;
  private SQLShizzle sql;
  String inputName;
  String inputAddress;
  int number;
  int guestNo;
  String kidsYN;
  int kidsNo;
  String replyYN;
  String dietRes;
  String dietInfo;
  String attend;
  String invite;
  //connect c = new connect();
  //SQLShizzle sql = new SQLShizzle();
  
  public btnSaveListener(gList glist, connect connect, SQLShizzle SQLShizzle)
  {
    this.glist = glist;
    this.c = connect;
    this.sql = SQLShizzle;
  }
  public void actionPerformed(ActionEvent event)
  {
    //Get the name and address into a string
    inputName = glist.tfName.getText();
    inputAddress = glist.taAddress.getText();
    //Get the number and convert it to an int
    String num = glist.tfNumber.getText();
    number = Integer.parseInt( num );
    //Get the no. of guests from a combobox and conver to an int
    String guest = (String)glist.guestNoCombo.getSelectedItem();
    guestNo = Integer.parseInt( guest );
    //Convert kids checkbox result into a string
    //Get no. of kids into an int if applicable
    if(glist.kidBox.isSelected())
    {
      kidsYN = "Y";
      String kids = (String)glist.kidNoCombo.getSelectedItem();
      kidsNo = Integer.parseInt( kids );
    }
    else
    {
      kidsYN = "N";
      kidsNo = '0';
    }
    //Get reply result into a string
    if(glist.replyBox.isSelected())
    {
      replyYN = "Y";
    }
    else
    {
      replyYN = "N";
    }
    //Get the attendance
    attend = (String)glist.attendCombo.getSelectedItem();
    invite = (String)glist.inviteCombo.getSelectedItem();
    if(invite == "All Day")
    {
      if(glist.dietBox.isSelected())
      {
        dietRes = "Y";
        dietInfo = glist.tfDiet.getText();
      }
      else
      {
        dietRes = "N";
      }
    }
    else
    {
      dietRes = "N";
    }
  c.dbConnect();
  sql.insert(inputName,inputAddress,number,guestNo,kidsYN,kidsNo,
              replyYN,attend,invite,dietRes,dietInfo);
  }
}

[Henry Wong]

Have another problem know though; one that just randomly appeared. I don't know why, but it has been working.

Stuff like this don't randomly appear. If it has been working, then this is caused by something that you changed.

By chance, did you happen to change the constructor of the btnSaveListener class? Maybe added a few parameters?

Henry

[John Pisci]

Henry Wong wrote:

Have another problem know though; one that just randomly appeared. I don't know why, but it has been working.

Stuff like this don't randomly appear. If it has been working, then this is caused by something that you changed.

By chance, did you happen to change the constructor of the btnSaveListener class? Maybe added a few parameters?

Henry

Yep It was a bad java day yesterday!

This is the 'before' code which worked:

    
public btnSaveListener(gList glist)  
{  
  this.glist = glist;  
}  

The 'after' code that doesn't work:

    
public btnSaveListener(gList glist, connect connect, SQLShizzle SQLShizzle)  
{  
  this.glist = glist;  
  this.c = connect;  
  this.sql = SQLShizzle;  
}  

How would I get this to work?

I can't imagine it would be as simple as doing this?!:

    
btnSave.addActionListener(new btnSaveListener(this, this, this)); 

[Vijitha Kumara]

 public btnSaveListener(gList glist, connect connect, SQLShizzle SQLShizzle)    

This constructor accept three different types of arguments (gList,connect,SQLShizzle) . So you need to call that constructor passing instances which are compatible with those types. (Passing "this" for all three types would work only if the current instance which you are calling the constructor is assignable for all those three types, which is not in this case).