comboBox problem need help bad

[Kendra Payne]

Hi,
newbie here. I have a program I am working on and-it opens with a splash screen then the following code runs. The problem is when I do a sort by...it comes up please enter comboBoxChanged with a field below it. I am so confused and don't know what the problem is. I realllllly appreciate your help.

 
/*
Programmer:
 

 

*/
 
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.text.*;
 
public class CHARM extends JFrame implements ActionListener
{
//construct components
JLabel sortPrompt = new JLabel("Sort by:");
JComboBox fieldCombo = new JComboBox();
JTextPane textPane = new JTextPane();
 
//initialize data in arrays
String title[] = {"Puffy Heart", "Octopus", "Goat","Cat"};
String supplier[] = {"Plum Island A", "Plum Island A", "Sunwest B", "Monster Slayer C"};
String charm[] = {"A-155", "A-558", "B-4567", "C-6247"};
 
//construct instance of Charm
public CHARM()
{
super("Charm Farm Manager");
}
 
//create the menu system
public JMenuBar createMenuBar()
{
//create an instance of the menu
JMenuBar mnuBar = new JMenuBar();
setJMenuBar(mnuBar);
 
//construct and populate the File menu
JMenu mnuFile = new JMenu("File", true);
mnuFile.setMnemonic(KeyEvent.VK_F);
mnuFile.setDisplayedMnemonicIndex(0);
mnuBar.add(mnuFile);
 
JMenuItem mnuFileExit = new JMenuItem("Exit");
mnuFileExit.setMnemonic(KeyEvent.VK_X);
mnuFileExit.setDisplayedMnemonicIndex(1);
mnuFile.add(mnuFileExit);
mnuFileExit.setActionCommand("Exit");
mnuFileExit.addActionListener(this);
 
//construct and pouplate the Edit menu
JMenu mnuEdit = new JMenu("Edit", true);
mnuEdit.setMnemonic(KeyEvent.VK_E);
mnuEdit.setDisplayedMnemonicIndex(0);
mnuBar.add(mnuEdit);
 
JMenuItem mnuEditInsert = new JMenuItem("Insert new Charm");
mnuEditInsert.setMnemonic(KeyEvent.VK_I);
mnuEditInsert.setDisplayedMnemonicIndex(0);
mnuEdit.add(mnuEditInsert);
mnuEditInsert.setActionCommand("Insert");
mnuEditInsert.addActionListener(this);
 
JMenu mnuEditSearch = new JMenu("Search");
mnuEditSearch.setMnemonic(KeyEvent.VK_R);
mnuEditSearch.setDisplayedMnemonicIndex(3);
mnuEdit.add(mnuEditSearch);
 
JMenuItem mnuEditSearchByTitle = new JMenuItem("by Title");
mnuEditSearchByTitle.setMnemonic(KeyEvent.VK_T);
mnuEditSearchByTitle.setDisplayedMnemonicIndex(3);
mnuEditSearch.add(mnuEditSearchByTitle);
mnuEditSearchByTitle.setActionCommand("title");
mnuEditSearchByTitle.addActionListener(this);
 
JMenuItem mnuEditSearchBySupplier = new JMenuItem("by Supplier");
mnuEditSearchBySupplier.setMnemonic(KeyEvent.VK_S);
mnuEditSearchBySupplier.setDisplayedMnemonicIndex(3);
mnuEditSearch.add(mnuEditSearchBySupplier);
mnuEditSearchBySupplier.setActionCommand("supplier");
mnuEditSearchBySupplier.addActionListener(this);
 
JMenuItem mnuEditSearchByCharm = new JMenuItem("by Charm");
mnuEditSearchByCharm.setMnemonic(KeyEvent.VK_Y);
mnuEditSearchByCharm.setDisplayedMnemonicIndex(3);
mnuEditSearch.add(mnuEditSearchByCharm);
mnuEditSearchByCharm.setActionCommand("Charm");
mnuEditSearchByCharm.addActionListener(this);
 
return mnuBar;
}
 
//create content pane
public Container createContentPane()
{
//populate the JCombobox
fieldCombo.addItem("Title");
fieldCombo.addItem("Supplier");
fieldCombo.addItem("Charm");
fieldCombo.addActionListener(this);
fieldCombo.setToolTipText("click the drop down arrow to diplay sort fields.");
 
//construct and populate the north panel
JPanel northPanel = new JPanel();
northPanel.setLayout(new FlowLayout());
northPanel.add(sortPrompt);
northPanel.add(fieldCombo);
 
//create the JTextPane and cneter panel
JPanel centerPanel = new JPanel();
setTabsAndStyles(textPane);
textPane = addTextToTextPane();
JScrollPane scrollPane = new JScrollPane(textPane);
scrollPane.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_ALWAYS);
scrollPane.setPreferredSize(new Dimension(500, 200));
centerPanel.add(scrollPane);
 
//create Container and set attributes
Container c = getContentPane();
c.setLayout(new BorderLayout(10,10));
c.add(northPanel,BorderLayout.NORTH);
c.add(centerPanel,BorderLayout.CENTER);
 
return c;
}
 
//method to create tab stops and set font styles
protected void setTabsAndStyles(JTextPane textPane)
{
//create Tab Stops
TabStop[] tabs = new TabStop[2];
tabs[0] = new TabStop(200, TabStop.ALIGN_LEFT, TabStop.LEAD_NONE);
tabs[1] = new TabStop(350, TabStop.ALIGN_LEFT, TabStop.LEAD_NONE);
TabSet tabset = new TabSet(tabs);
 
//set Tab Style
StyleContext tabStyle = StyleContext.getDefaultStyleContext();
AttributeSet aset =
tabStyle.addAttribute(SimpleAttributeSet.EMPTY, StyleConstants.TabSet, tabset);
textPane.setParagraphAttributes(aset, false);
 
//set Font Style
Style fontStyle =
StyleContext.getDefaultStyleContext().getStyle(StyleContext.DEFAULT_STYLE);
 
Style regular = textPane.addStyle("regular", fontStyle);
StyleConstants.setFontFamily(fontStyle, "SanSerif");
 
Style s = textPane.addStyle("italic", regular);
StyleConstants.setItalic(s, true);
 
s = textPane.addStyle("bold", regular);
StyleConstants.setBold(s, true);
 
s = textPane.addStyle("large", regular);
StyleConstants.setFontSize(s, 16);
}
 
//method to add new text to the JTextPane
public JTextPane addTextToTextPane()
{
Document doc = textPane.getDocument();
try
{
//clear previous text
doc.remove(0, doc.getLength());
 
//insert title
doc.insertString(0,"TITLE\tSUPPLIER\tCHARM\n", textPane.getStyle("large"));
 
//insert detail
for (int j = 0; j<title.length; j++)
{
doc.insertString(doc.getLength(), title[j] + "\t", textPane.getStyle("bold"));
doc.insertString(doc.getLength(), supplier[j] + "\t", textPane.getStyle("italic"));
doc.insertString(doc.getLength(), charm[j] + "\n", textPane.getStyle("regular"));
}
}
catch(BadLocationException ble)
{
System.err.println("Couldn't insert text.");
}
 
return textPane;
}
 
//event to process user clicks
public void actionPerformed(ActionEvent e )
{
String arg = e.getActionCommand();
 
//user clicks tthe sort by combo box
if (e.getSource() == fieldCombo)
{
switch(fieldCombo.getSelectedIndex())
{
case 0:
sort(title);
break;
case 1:
sort(supplier);
break;
case 2:
sort(charm);
break;
}
}
 
//user clicks Exit on the File menu
if (arg == "Exit")
System.exit(0);
 
//user clicks Insert New CHARM on the Edit menu
if (arg == "Insert")
{
//accept new data
String newTitle = JOptionPane.showInputDialog(null,"Please enter the new charm's title");
String newSupplier = JOptionPane.showInputDialog(null,"Please enter the supplier for" + newTitle);
String newCharm = JOptionPane.showInputDialog(null,"Please enter the charm# for" + newTitle);
 
//enlarge arrays
title = enlargeArray(title);
supplier = enlargeArray(supplier);
charm = enlargeArray(charm);
 
//add new data to arrays
title[title.length-1] = newTitle;
supplier[supplier.length-1] = newSupplier;
charm[charm.length-1] = newCharm;
 
//call sort method
sort(title);
fieldCombo.setSelectedIndex(0);
}
 
//user clicks title on the search submenu
if (arg == "title");
search(arg, title);
 
//user clicks studio on the search submenu
if (arg == "supplier");
search(arg, supplier);
 
//user clicks Year on the search submenu
if (arg == "charm#");
search(arg, charm);
}
 
//method to enlarge an array by 1
public String[] enlargeArray(String[] currentArray)
{
String[] newArray = new String[currentArray.length + 1];
for (int i = 0; i<currentArray.length; i++)
newArray[i] = currentArray[i];
return newArray;
}
 
//method to sort arrays
public void sort(String tempArray[])
{
//loop to control number of passes
for (int pass = 1; pass < tempArray.length; pass++)
{
for (int element = 0; element < tempArray.length - 1; element++)
     if (tempArray[element].compareTo(tempArray[element + 1])>0)
     {
 swap(title, element, element + 1);
 swap(supplier, element, element + 1);
 swap(charm, element, element + 1);
 }
 }
 addTextToTextPane();
 }
 
 //method to swap two elements of an array
 public void swap(String swapArray[], int first, int second)
 {
 String hold;  //temporary holding area for swap
 hold = swapArray[first];
 swapArray[first] = swapArray[second];
 swapArray[second] = hold;
 }
 
 public void search(String searchField, String searchArray[])
 {
 try
 {
 Document doc = textPane.getDocument(); //assign text to document object
 doc.remove(0,doc.getLength()); //clear previous text
 
 //display column titles
 doc.insertString(0,"TITLE\tSUPPLIER\tCHARM\n",textPane.getStyle("large"));
 
 //prompt user for search data
 String search = JOptionPane.showInputDialog(null, "Please enter the "+ searchField);
 boolean found = false;
 
 //search arrays
 for  (int i = 0; i<title.length; i++)
 {
 if (search.compareTo(searchArray[i])==0)
 {
doc.insertString(doc.getLength(), title[i] + "\t", textPane.getStyle("bold"));
doc.insertString(doc.getLength(), supplier[i] +"\t", textPane.getStyle("italic"));
doc.insertString(doc.getLength(), charm[i] + "\n", textPane.getStyle("regular"));
found = true;
}
}
if (found == false)
{
JOptionPane.showMessageDialog(null, "Your search produced no results.", "No results found",JOptionPane.INFORMATION_MESSAGE);
sort(title);
}
}
catch(BadLocationException ble)
{
System.err.println("Couldn't insert text.");
}
}
 
//main method executes at run time
public void createAndShowGUI()
{
 
JFrame.setDefaultLookAndFeelDecorated(true);
CHARM f = new CHARM();
f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
f.setJMenuBar(f.createMenuBar());
f.setContentPane(f.createContentPane());
f.setSize(600,375);
f.setVisible(true);
}
}

[Michael Dunn]

looks like this is the offending bit of code

//user clicks title on the search submenu
if (arg == "title");
  search(arg, title);
 
//user clicks studio on the search submenu
if (arg == "supplier");
  search(arg, supplier);
 
//user clicks Year on the search submenu
if (arg == "charm#");
  search(arg, charm);

if you step through from the first line:
if (arg == "title");
1) should be arg.equals("title")
2) whether true or false, the if() won't do anything because of the trailing ; (semi-colon)
3) again, whether true of false, the code will continue to the next line
search(arg, title);
which appears to produce a JOptionPane.

this will be repeated for the other if()'s

[Kendra Payne]

THANK YOU, THANK YOU, THANK YOU!