Rich Charlton

right i think i have sorted my addClient method now:
public void addClient (String name, String address, String telephonenum) { if (index == collection.length) { System.out.println("Error"); } else { collection[index] = new clienttest(name,address,telephonenum); System.out.println("Client has been stored"); index++; } } public clienttest findClient(String name) { for (int i = 0; i < collection.length; i++) { if (collection[i].getName().equals(name)) { System.out.println("Client Found"); return (collection[i]); } else { System.out.println("ERROR CLIENT NOT FOUND"); } } return (new clienttest("NOT FOUND", "", "")); }

Output after entering 5 clients:
Client has been stored Client has been stored Client has been stored Client has been stored Client has been stored Error Error Error

The above is what i want which is cool..

Now to my findClient method....This works to a point...It will find a specific name...however when i enter a name not stored i get null exception error then my "Error Client not Found" message. How can i avoid this..or have you allready explained this to me...ive probably got my loop slightly wrong again...for the 400th time...

thanks

Hi,

i have removed making a new branch every time i click the find button on my GUI, and thats seems to of helped.

Heres what i have now..public class CustomerCollection { //define class attributes private clienttest[] collection; private int arraySize = 5; private int index; private String name, address, telephonenum, temp, temp1; // Constructor: Creates an initially empty collection. public CustomerCollection () { collection = new clienttest[arraySize]; } // Adds a Client to the collection public void addClient (String name, String address, String telephonenum) { if (index <= collection.length) { collection[index] = new clienttest(name,address,telephonenum); System.out.println("Client has been stored"); index++; } else { System.out.println("Error"); } } public clienttest findClient(String name) { //System.out.println(collection.length); for (int i = 0; i <= collection.length; i += index) { if (collection[i].getName().equals(name)) //temp = collection[i].getAddress(); //temp1 = collection[i].getPhone(); //System.out.println(name); //System.out.println(temp); //System.out.println(temp1); System.out.println(i); return (collection[i]); } return (new clienttest("NOT FOUND", "", "")); } }

Only problem now is that when i do click find it just returns whatever i type into the textfield. It is not searching the array for stored items.

Any ideas....

Hi, sorry about this cant seem to get the logic right..
Firstly my GUI gets the text name input
private void addclientActionPerformed(java.awt.event.ActionEvent evt) { name = nametextfield.getText(); address = addresstextfield.getText(); telephonenum = numtextfield.getText(); branch newbranch = new branch(); newbranch.add(name, address, telephonenum); }
which passes the values name, address, telephonenum over to branch class by calling the method add()
public branch() {} public CustomerCollection clients = new CustomerCollection(); public String temp; public void add(String name, String address, String telephonenum) { clients.addClient(name,address,telephonenum); }
The add() calls the method addClient() from customercollection class below
public class CustomerCollection { //define class attributes public Client[] collection; private int arraySize = 5; public int index = 0; public String name, address, telephonenum; // Constructor: Creates an initially empty collection. public CustomerCollection () { collection = new Client[arraySize]; } // Adds a Client to the collection public void addClient (String name, String address, String telephonenum) { if(index == collection.length) { collection[index] = new Client(name,address,telephonenum); System.out.println("Stored"); } else { System.out.println("Error"); } index++; } }

Output is my system.out.println("Error"); obviously my my values are wrong somewhere and the statement if (index == collection.length) has got to be incorrect.

Im not sure why this is not working.
To clarify :[code]
private int arraySize = 5; //this is how many elements i would like to have in the array
public int index = 0; //this is an index for each element

If you have any more help, that would be great

Thanks again..

Sorry to bother you again...

I have a question...

I have slightly changed the addClient method now so that the user cannot store more than 5 users, although when testing my GUI out i can add as many clients as i like without any error message what so ever.

Here's my addClient method:

// Adds a Client to the collection public void addClient (String name, String address, String telephonenum) { if(index <= collection.length) { collection[index] = new Client(name,address,telephonenum); } else { System.out.println("Error"); } index++; }

Thanks again..

[ February 23, 2005: Message edited by: Rich Charlton ]
[ February 23, 2005: Message edited by: Rich Charlton ]

Hi,

Great help thankyou...

I have sorted my add client method out which is much more logical now..

Many thanks

Hi all,
I am having trouble figuring out how to search a collection. The collection holds three strings (name, address, telephonenum) i would like to be able to search using a name as the search criteria.

public class estateGUI extends javax.swing.JFrame { public estateGUI() { initComponents(); } public String name, address, telephonenum, find; private void initComponents() { addclient = new javax.swing.JButton(); findclient = new javax.swing.JButton(); logo = new javax.swing.JLabel(); cleartext = new javax.swing.JButton(); nametextfield = new javax.swing.JTextField(); addresstextfield = new javax.swing.JTextField(); numtextfield = new javax.swing.JTextField(); jLabel1 = new javax.swing.JLabel(); jLabel2 = new javax.swing.JLabel(); jLabel3 = new javax.swing.JLabel(); findnametextfield = new javax.swing.JTextField(); jLabel4 = new javax.swing.JLabel(); displaydetails = new javax.swing.JTextPane(); getContentPane().setLayout(new org.netbeans.lib.awtextra.AbsoluteLayout()); setDefaultCloseOperation(javax.swing.WindowConstants.EXIT_ON_CLOSE); setTitle("UWE Estates"); setCursor(new java.awt.Cursor(java.awt.Cursor.DEFAULT_CURSOR)); setMaximizedBounds(new java.awt.Rectangle(500, 500, 400, 420)); setName("MainFrame"); setResizable(false); addWindowListener(new java.awt.event.WindowAdapter() { public void windowClosing(java.awt.event.WindowEvent evt) { exitForm(evt); } }); addclient.setText("Add Client"); addclient.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { addclientActionPerformed(evt); } }); getContentPane().add(addclient, new org.netbeans.lib.awtextra.AbsoluteConstraints(10, 210, 130, -1)); findclient.setText("FIND"); findclient.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { findclientActionPerformed(evt); } }); getContentPane().add(findclient, new org.netbeans.lib.awtextra.AbsoluteConstraints(170, 300, 90, -1)); logo.setBackground(new java.awt.Color(255, 255, 255)); logo.setFont(new java.awt.Font("Arial", 1, 24)); logo.setForeground(new java.awt.Color(51, 51, 255)); logo.setHorizontalAlignment(javax.swing.SwingConstants.CENTER); logo.setText("UWE Estates"); getContentPane().add(logo, new org.netbeans.lib.awtextra.AbsoluteConstraints(120, 20, 180, 30)); cleartext.setText("Clear Text"); cleartext.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { cleartextActionPerformed(evt); } }); getContentPane().add(cleartext, new org.netbeans.lib.awtextra.AbsoluteConstraints(270, 210, 130, -1)); nametextfield.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { nametextfieldActionPerformed(evt); } }); getContentPane().add(nametextfield, new org.netbeans.lib.awtextra.AbsoluteConstraints(140, 80, 130, -1)); addresstextfield.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { addresstextfieldActionPerformed(evt); } }); getContentPane().add(addresstextfield, new org.netbeans.lib.awtextra.AbsoluteConstraints(140, 130, 130, -1)); numtextfield.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { numtextfieldActionPerformed(evt); } }); getContentPane().add(numtextfield, new org.netbeans.lib.awtextra.AbsoluteConstraints(140, 180, 130, -1)); jLabel1.setHorizontalAlignment(javax.swing.SwingConstants.CENTER); jLabel1.setText("Add Name"); getContentPane().add(jLabel1, new org.netbeans.lib.awtextra.AbsoluteConstraints(140, 60, 130, -1)); jLabel2.setHorizontalAlignment(javax.swing.SwingConstants.CENTER); jLabel2.setText("Add Address"); getContentPane().add(jLabel2, new org.netbeans.lib.awtextra.AbsoluteConstraints(140, 110, 130, -1)); jLabel3.setHorizontalAlignment(javax.swing.SwingConstants.CENTER); jLabel3.setText("Add Telephone Number"); getContentPane().add(jLabel3, new org.netbeans.lib.awtextra.AbsoluteConstraints(130, 160, 160, -1)); findnametextfield.addActionListener(new java.awt.event.ActionListener() { public void actionPerformed(java.awt.event.ActionEvent evt) { findnametextfieldActionPerformed(evt); } }); getContentPane().add(findnametextfield, new org.netbeans.lib.awtextra.AbsoluteConstraints(90, 270, 260, -1)); jLabel4.setHorizontalAlignment(javax.swing.SwingConstants.CENTER); jLabel4.setText("Input Name and Click Find"); getContentPane().add(jLabel4, new org.netbeans.lib.awtextra.AbsoluteConstraints(140, 250, 160, -1)); getContentPane().add(displaydetails, new org.netbeans.lib.awtextra.AbsoluteConstraints(90, 330, 260, 90)); pack(); } private void findclientActionPerformed(java.awt.event.ActionEvent evt) { find = findnametextfield.getText(); branch newbranch = new branch(); newbranch.find(find); } private void findnametextfieldActionPerformed(java.awt.event.ActionEvent evt) { } private void numtextfieldActionPerformed(java.awt.event.ActionEvent evt) { } private void addresstextfieldActionPerformed(java.awt.event.ActionEvent evt) { } private void cleartextActionPerformed(java.awt.event.ActionEvent evt) { clearFields(); } private void addclientActionPerformed(java.awt.event.ActionEvent evt) { name = nametextfield.getText(); address = addresstextfield.getText(); telephonenum = numtextfield.getText(); branch newbranch = new branch(); newbranch.add(name, address, telephonenum); } private void nametextfieldActionPerformed(java.awt.event.ActionEvent evt) { } private void exitForm(java.awt.event.WindowEvent evt) { System.exit(0); } private void clearFields() { nametextfield.setText(""); addresstextfield.setText(""); numtextfield.setText(""); } public static void main(String args[]) { new estateGUI().show(); } private javax.swing.JButton addclient; private javax.swing.JTextField addresstextfield; private javax.swing.JButton cleartext; private javax.swing.JTextPane displaydetails; private javax.swing.JButton findclient; private javax.swing.JTextField findnametextfield; private javax.swing.JLabel jLabel1; private javax.swing.JLabel jLabel2; private javax.swing.JLabel jLabel3; private javax.swing.JLabel jLabel4; private javax.swing.JLabel logo; private javax.swing.JTextField nametextfield; private javax.swing.JTextField numtextfield; }

Branch class that recieves input from estateGUI.
import javax.swing.*; import java.awt.*; import javax.swing.JOptionPane; import java.awt.event.*; public class branch { public branch() {} public int customerID = 1; public int maxclients = 5; public int count; private int index = 0; public CustomerCollection clients = new CustomerCollection(); public void add(String name, String address, String telephonenum) { if (count < 2) { count++; clients.addClient(name,address,telephonenum); JOptionPane.showMessageDialog(null, "Client Stored \n" +name+ "\n" + address + "\n" + telephonenum); } else { JOptionPane.showMessageDialog(null, "Error. Maximum number of" + "clients exceeded"); } } public void amend() {} public void delete() {} public void find(String details) { clients.findclient(details); } public void print() {} }

CustomerCollection class which builds the collection and has the method for add/find client etc:
import javax.swing.*; import java.awt.*; import javax.swing.JOptionPane; import java.awt.event.*; public class CustomerCollection { public Client[] collection; private int arraySize = 5; private int index = 0; public CustomerCollection () { collection = new Client[arraySize]; } public void addClient (String name, String address, String telephonenum) { if(index == collection.length) { collection[index] = new Client (name,address,telephonenum); } index++; } public void findclient(String details) { }

Does anyone have any ideas on how i can find a specific client in a collection.

Thanks

Thanks for your help guys....

I have re written my program logic now...although i have another problem...
I have started a new thread in the intermediate java forum if you can spare a look.

Thanks

Hi,

Im working on a basic email filter, which will read in a text file of several hundred email messages, increment a counter for an individual message, tokenise it, count number of occurences of token in that email then store in a dictionary. I have built a couple of classes that do certain things and they work ok. Although i cant figure out how i can split up each individual message in the text file. At the moment i just read the whole text file in, with no method of splitting the text file up into individual messages. The messages in the text file are split by 10 dashes (----------). Here are my classes so far:
Tokeniser.javaimport java.io.*; import java.util.*; import javax.swing.JOptionPane; import java.util.ArrayList; public class Tokeniser { public HashSet unCommonHS = new HashSet(); public int numberOfTokens = 0; public Tokeniser() { String fileName = "SPAM.txt"; String aLine; ComWords cm = new ComWords(); HashSet cwSet = cm.getCommonWordsHS(); if (cm==null) return; try { BufferedReader bR = new BufferedReader(new FileReader(fileName)); while((aLine=bR.readLine())!= null) { checkForCommonWord (aLine, cwSet, unCommonHS); } bR.close(); Object[] ucw = unCommonHS.toArray(); for (int i=0; i<ucw.length; i++) { numberOfTokens++; System.out.print(ucw[i]+" "); if (i!=0 && (i%1)==0) System.out.println(); } Set sm = new TreeSet(unCommonHS); for (Iterator it = sm.iterator(); it.hasNext(); ) { numberOfTokens++; it.next(); } System.out.println("Number of Tokens In File :" + numberOfTokens); } catch(IOException e) { System.out.println("Train Failed"); e.printStackTrace(); return; } } private void checkForCommonWord(String line, HashSet commonSet, HashSet uncommonHS) { StringTokenizer st = new StringTokenizer(line.toLowerCase(), "!-/.,@?><#~'��,'��� - �'�� �� :;][}{|�$%`'�/-^�&�*()1234567890-=_+*`�"); while (st.hasMoreElements()) { Object ob = st.nextElement(); if (!commonSet.contains(ob)) uncommonHS.add(ob); } } public String returnString() { return unCommonHS.toString(); } public static void main(String[] args) { Tokeniser t = new Tokeniser(); } }
Comwords.java (removes common words like 'the', 'a' etc)import java.io.*; import java.util.*; import javax.swing.JOptionPane; import java.util.ArrayList; import java.util.Hashtable; class ComWords { public ComWords() {} public HashSet getCommonWordsHS () { HashSet cwSet = new HashSet(); String fileName = "cwords.txt"; try { BufferedReader bR = new BufferedReader(new FileReader(fileName)); String aLine=null; while((aLine=bR.readLine())!= null) storeWord(cwSet, aLine); bR.close(); return cwSet; } catch(IOException e) { System.out.println("CommonWords Failed"); e.printStackTrace(); return null; } } private void storeWord(HashSet set, String line) { StringTokenizer st=new StringTokenizer(line," "); while (st.hasMoreElements()) { set.add(st.nextElement()); } } }
Does anyone have any ideas how i might solve this problem?
Thanks

thats great am able to get the items added in the commonwords class. although im still having a problem removing the items from the initial hashmap. i have changed it slightly now to use an arraylist for the commonwords instead of a hash table as i had no need for a value next to the string. ();//instance of CommonWords Class CommonWords cwords = new CommonWords(); System.out.println(cwords); //this displays CommonWords@18d107f only [B]counts.remove(cwords);[/B]

im not sure whether the remove statement is going to work now.
as counts is a hashmap and cwords is an arraylist of string objects.

so problems at the moment are that i cant instantiate the arraylist and cannot remove commonwords from hashmap.

any ideas thanks.

Hi all,

im trying to build a text classification program that reads in a text file, tokenises it and stores it in a hashmap. Along the way i would like to instantiate another class which reads a text file full of commonwords. I would then like to remove these common words from the initial hashmap. I have set up two classes one called CommonWords.java and the other called Train.java. I am getting an error which i have posted after my code.

Train.javaimport java.io.*; import java.util.*; import javax.swing.JOptionPane; public class Train { public FileReader file; public StreamTokenizer st; public HashMap counts = new HashMap(); public Hashtable cword; /** Creates a new instance of train */ public Train() { String fileName = "SPAM.txt"; String data; HashMap counts = new HashMap(); int tokenType = 0; int numberOfTokens = 0; try { FileReader file = new FileReader(fileName); StreamTokenizer st = new StreamTokenizer(new BufferedReader(file)); st.ordinaryChar('!'); st.ordinaryChar('$'); st.ordinaryChar('"'); st.whitespaceChars('/','/'); st.whitespaceChars('\\','\\'); st.whitespaceChars('.','.'); st.whitespaceChars(',',','); st.whitespaceChars(';',';'); st.whitespaceChars(':',':'); st.whitespaceChars('=','='); st.whitespaceChars('\'','\''); st.whitespaceChars('`','`'); st.whitespaceChars('[',']'); while(st.nextToken() != StreamTokenizer.TT_EOF) { String s; switch(st.ttype) { case StreamTokenizer.TT_EOL: s = new String("EOL"); break; /** case StreamTokenizer.TT_NUMBER: s = Double.toString(st.nval); break;**/ case StreamTokenizer.TT_WORD: s = st.sval; // Already a String // System.out.println("Token Extracted = " + st.sval); st.lowerCaseMode(true); numberOfTokens++; break; default: s = String.valueOf((char)st.ttype); } if(counts.containsKey(s)) ((Counter)counts.get(s)).increment(); else counts.put(s, new Counter()); } } catch(IOException e) { System.out.println("st.nextToken() unsuccessful"); System.out.println("Problem reading " + fileName ); System.out.println("Exception: " + e); e.printStackTrace(); } //create instance of commonwords CommonWords c = new CommonWords(); cword = c.cwords; //remove the commonwords from HashMap counts.remove(cword); System.out.println("File "+fileName+" was succesfully loaded and Stored."); System.out.println("Tokens in File :" + numberOfTokens); System.out.println(cword); } public static void main(String[] args) { Train t = new Train(); } }

CommonWords.javaimport java.io.*; import java.util.*; import javax.swing.JOptionPane; public class CommonWords { public Hashtable cwords; public FileReader file; public StreamTokenizer st; public CommonWords() { String fileName = "cwords.txt"; int tokenType = 0; int numberOfTokens = 0; try { Reader cWords = new BufferedReader(new FileReader(fileName)); StreamTokenizer stc = new StreamTokenizer(cWords); int i=0; while(stc.nextToken() != StreamTokenizer.TT_EOF) { cwords.put(stc.sval.toLowerCase(), new Integer(i)); System.out.println("Token Extracted = " + stc.sval); i=i++; } } catch(IOException e) { System.out.println("st.nextToken() unsuccessful"); System.out.println("Problem reading " + fileName ); System.out.println("Exception: " + e); e.printStackTrace(); } } }

Error Message:java.lang.NullPointerException at CommonWords.<init>(CommonWords.java:26) at Train.<init>(Train.java:80) at Train.main(Train.java:92) Exception in thread "main"

I think i have something wrong with the adding of the commonwords to the hashtable..

Does anyone have any ideas.

Thanks

Hi all,

I have sorted my parameter passing problem now. everything is working

Big thanks to Michael for explaining my problems.

Hi all,

Im trying to do a simple value passing operation i cant seem to get it working correctly. I have two classes one which produces a value and the other that needs that value to do something with it. I have written the code, although i do not get the right result, when i click the confirm button and enter how many tickets i would like the showID integer to be passed over to Cost class using method at the bottom of panel class. I think i have a problem with my call method as it is duplicating the fram twice after i enter a number into the input dialog. Heres my code:

Panel class:import javax.swing.*; import java.awt.*; import javax.swing.JOptionPane; import java.awt.event.*; public class Panel extends JFrame { public JComboBox days, months, years; private JLabel dateresult; public String date, day, month, year; private JTextArea area; public int num; public static void main(String[] args) { Panel frame = new Panel(); } public Panel() { String[] Month = {"Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"}; String[] Year = {"2004"}; String[] Day = new String[32]; for (int i = 1; i < 32; i++) { Day[i] = Integer.toString(i); } JButton okbutton; JButton confirmbutton; JFrame frame = new JFrame(); frame.setLocation(100, 100); frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE); JPanel primary = new JPanel(new FlowLayout(FlowLayout.CENTER, 30000, 5)); primary.setPreferredSize(new Dimension(500, 500)); JPanel datecombos = new JPanel(new FlowLayout(FlowLayout.CENTER, 10, 10)); days = new JComboBox(Day); days.addItemListener(new SelectButton(datecombos)); days.setSelectedIndex(1); months = new JComboBox(Month); months.addItemListener(new SelectButton(datecombos)); years = new JComboBox(Year); years.addItemListener(new SelectButton(datecombos)); datecombos.add(days); datecombos.add(months); datecombos.add(years); Cost cost; cost = new Cost(); cost.BookingType(); JPanel showinfo = new JPanel(); primary.add(datecombos); primary.add(showinfo); dateresult = new JLabel(); okbutton = new JButton("SELECT"); okbutton.addActionListener(new SelectButton(datecombos)); datecombos.add(okbutton); area = new JTextArea(); primary.add(cost); showinfo.add(area); frame.getContentPane().add(primary, BorderLayout.NORTH); frame.pack(); frame.setVisible(true); } public class SelectButton implements ItemListener, ActionListener { JPanel parent; public SelectButton(JPanel p) { parent = p; } public void actionPerformed(ActionEvent e) { String showID; day = (String)days.getSelectedItem(); month = (String)months.getSelectedItem(); year = (String)years.getSelectedItem(); if (day != "1" || month != "Apr" || year != "2004") { JOptionPane.showMessageDialog(parent, "There are no showings Available for this Date. " + "Please Select Again", "Error", JOptionPane.ERROR_MESSAGE); } else { dateresult.setText("Date Selected : " + day + " " + month + " " + year); [B] try //select { showID = JOptionPane.showInputDialog(parent, "Enter A Show ID Please."); num = Integer.parseInt(showID); if (num == 1) { JOptionPane.showMessageDialog(parent, " bla bla bla"); area.setText(" "); } else if (num == 2) { JOptionPane.showMessageDialog(parent, "bla bla bla"); area.setText(""); } else if (num == 3) { JOptionPane.showMessageDialog(parent, "bla bla bla"); area.setText(""); } else { JOptionPane.showMessageDialog(parent, "You have not picked correctly" + " Please Select Again", "Error", JOptionPane.ERROR_MESSAGE); } } catch (NumberFormatException ex) { JOptionPane.showMessageDialog(parent, "Non Numerical Data Exception."); System.exit(0); // Exits Program } } } public void itemStateChanged(ItemEvent e) { } } public int getnum() { return num; } }[/B]

Cost class: import javax.swing.*; import java.awt.*; import javax.swing.JOptionPane; import java.awt.event.*; import java.awt.*; import javax.swing.text.*; import javax.swing.event.*; public class Cost extends JPanel implements ItemListener, ActionListener { public String[] type = {"Select", "Adult", "Child", "Student"}; public String typechosen, ticketno; public JComboBox typecombo, numbercombo; public int ticketcost, number, totalcost; private JLabel typeresult; public int num1; public int screen1 = 50; public int screen2 = 75; public int screen3 = 100; public JTextField tickettext; public JButton ticketbutton, confirmbutton; /** * Creates a new instance of BookingInfo */ public void BookingType() { JPanel typeinfo = new JPanel(new FlowLayout(FlowLayout.CENTER, 30000, 5)); typeinfo.setBorder(BorderFactory.createLoweredBevelBorder()); typeinfo.setBackground(Color.yellow); typeinfo.setPreferredSize(new Dimension(200, 100)); JLabel costlabel1 = new JLabel("Select Ticket Type"); costlabel1.setFont(new Font("Serif", Font.BOLD, 14)); costlabel1.setForeground(Color.black); typecombo = new JComboBox(type); typecombo.addItemListener(this); typeinfo.add(costlabel1); typeinfo.add(typecombo); typeresult = new JLabel(); typeinfo.setVisible(true); add(typeinfo); JPanel okPanel = new JPanel(); okPanel.setBorder(BorderFactory.createLoweredBevelBorder()); okPanel.setBackground(Color.red); okPanel.setPreferredSize(new Dimension(500, 60)); confirmbutton = new JButton("CONFIRM BOOKING"); confirmbutton.addActionListener(this); typeinfo.add(confirmbutton); } public void itemStateChanged(ItemEvent event) { typechosen = (String)typecombo.getSelectedItem(); if (typechosen.equals("Adult")) { ticketcost = 5; } else if (typechosen.equals("Child")) { ticketcost = 3; } else if (typechosen.equals("Student")) { ticketcost = 4; } } public void actionPerformed(ActionEvent e) { try { ticketno = JOptionPane.showInputDialog(Cost.this, "Please enter the number of tickets required. Thanks"); number = Integer.parseInt(ticketno); totalcost = ticketcost * number; } catch (NumberFormatException ex) { JOptionPane.showMessageDialog(Cost.this, "Non Numerical Data Exception."); System.exit(0); // Exits Program } [B]Panel cust; cust = new Panel(); num1 = cust.getnum(); System.out.println(cust.getnum());[/B] if (num1 > screen1) { JOptionPane.showMessageDialog(Cost.this, "Booking Accepted"); } else if (num1 > screen2) { JOptionPane.showMessageDialog(Cost.this, "Booking Accepted"); } else if (num1 > screen3) { JOptionPane.showMessageDialog(Cost.this, "Booking Accepted"); } else { JOptionPane.showMessageDialog(Cost.this, "Booking Not Accepted"); } } }

I have highlighted areas of code that are the problem areas.

Does anyone have any ideas why im not getting the desired result ?

Thanks



[ edited to break long lines of code -ds ]
[ November 27, 2004: Message edited by: Dirk Schreckmann ]

Hi,

I looked it up and got it working. Never new about that specific method for JComboBox.

Many Thanks.

Hi,

Thanks for taking a look, ive changed my name now hope thats cool.

As for the answer to my question..where do i put the setselectedindex(0);

Do you have any more clarification...

Thanks

Hi all,

I have written a program with three combo boxes asking user to pick a date. I have one combo for day, one for month and one for year. My problem is that i have designed a for loop to make the numbers 1 to 31 appear in the combo box, although when you run the program you get a blank space where the number 1 should be. Is their way of setting the combo box so that one is shown first, instead of a blank box.

Program: import javax.swing.*; import java.awt.*; import java.awt.event.*; public class TestCombo extends JPanel { public JComboBox days, months, years; public static void main(String[] args) { String[] Month = {"Jan", "Feb", "Mar", "Apr", "May", "Jun", "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"}; String[] Year = { "2004"}; [B] String [] Day = new String[32]; for(int i = 1; i < 32; i++){ Day[i] = Integer.toString(i); }[/B] JFrame frame = new JFrame(); frame.setLocation(100,100); frame.setDefaultCloseOperation (JFrame.EXIT_ON_CLOSE); JPanel datecombos = new JPanel(); datecombos.setPreferredSize(new Dimension(500,500)); JComboBox days = new JComboBox(Day); JComboBox months = new JComboBox(Month); JComboBox years = new JComboBox(Year); datecombos.add (days); datecombos.add (months); datecombos.add (years); frame.getContentPane().add(datecombos, BorderLayout.NORTH); frame.pack(); frame.setVisible(true); } }

Any ideas ?

Thanks