Double effect from itemStateChanged

Question

Hi!
I�d like this to happen: when clicking on 3rd alternative a new menu to come along.
But there comes two menus!
This happens only when I call a function (createMine)
that returns a JComboBox OR when i call a public variable
(that is a JComboBox) in another class.
Why is the effect double?
Kind Regards//jF

Answer 1

Thanks!
It seems the itemStateChanged and the if (if (minMeny.getSelectedIndex() == 2))is entered twice. But that�s mysterious. (The new JComboBox has not yet
an itemListener. ) What is the DESELECTED variable for? Do I have to tell
the itemStateChanged somehow that "Now I don�t select this item anymore?"
hmm that can�t be it. :)
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.util.*;
* Write a description of class newGUI here.
* @author (your name)
* @version (a version number or a date)
//denna klass ska ge alt:
//boka
//avboka
//visa biljetter
public class IP implements ActionListener, ItemListener
    // instance variables - replace the example below with your own
    JFrame tfnFrame;
    JPanel tfnPanel;
    JTextField tfnNummer;
    JLabel bekr�ftelseLabel, testLabel;
    JButton skickaKnapp;
    JComboBox minMeny ;
    JComboBox testCombo ;
    String kundnummer ;
    Bio Frasses  ;
     * Constructor for objects of class newGUI
     //Create and set up the window.
    public IP()
        //bara vid test f�r Frasse initieras s� h�r
        //obs �ndra till long
        Frasses = new Bio(new Kund("24")) ;
        tfnFrame = new JFrame("Frasses Bio");
        tfnFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        tfnFrame.setSize(new Dimension(120, 40)); //Create and set up the panel.
        tfnPanel = new JPanel(new GridLayout(2, 2)); //Add the widgets.
        addWidgets(); //Set the default button.
        tfnFrame.getRootPane().setDefaultButton(skickaKnapp); //Add the panel to the window.
        tfnFrame.getContentPane().add(tfnPanel, BorderLayout.CENTER); //Display the window.
        tfnFrame.pack();
        tfnFrame.setVisible(true);
    private void addWidgets()
    { //Create widgets.
        tfnNummer = new JTextField(2);
        testLabel = new JLabel("Test*******", SwingConstants.LEFT);
        skickaKnapp = new JButton("Hall�");
        bekr�ftelseLabel = new JLabel("Resultat", SwingConstants.LEFT);
        //Listen to events from the Convert button.
        skickaKnapp.addActionListener(this);
      minMeny = new JComboBox() ;
       minMeny.addItem("L�gg till Film") ;
       minMeny.addItem("Ta bort Film") ;
       minMeny.addItem("L�gg till F�rest�llning ") ;
       minMeny.addItem("Ta bort F�rest�llning ") ;
       minMeny.addItem("Visa aktuella Filmer") ;
       minMeny.addItem("Visa aktuella F�rest�llningar") ;
       minMeny.addItem("Visa biljetter") ;
       minMeny.addItem("Visa antal lediga platser") ;
      minMeny.addItemListener(this) ; //hmm this intressant
      minMeny.addActionListener(this) ;
        //Add the widgets to the container.
        testCombo = new JComboBox() ;
        testCombo.addItem("Hej!") ;
        tfnPanel.add(tfnNummer);
        tfnPanel.add(testLabel);
        tfnPanel.add(skickaKnapp);
        tfnPanel.add(bekr�ftelseLabel);
        tfnPanel.add(minMeny) ;
        testLabel.setBorder(BorderFactory.createEmptyBorder(5,5,5,5));
        testLabel.setBorder(BorderFactory.createEmptyBorder(5,5,5,5));
    public void actionPerformed(ActionEvent event)
        String dittNummer = tfnNummer.getText() ;
        dittNummer = fixaNummer(dittNummer) ;
        if (�rSiffra(dittNummer))
            bekr�ftelseLabel.setText("true");
         else
            bekr�ftelseLabel.setText("false");
            if( event.getSource() == minMeny)
                bekr�ftelseLabel.setText("hallonpaj") ;
              System.out.println("itemlistener11") ;
                //Bokning nu = new Bokning() ;
                 tfnPanel.remove(minMeny) ;
                 tfnPanel.add(createNow()) ;
            //   testLabel.setText("hejhej") ;
               //    tfnFrame.getContentPane().add(tfnPanel, BorderLayout.CENTER); //Display the window.
        tfnFrame.pack();
        tfnFrame.setVisible(true);
                //alt �ndra minMeny till skapaMeny...
               //ide� k�ra create ..
               //m�ste jag addera den en g�ng till
    private static String fixaNummer(String min)
        StringTokenizer st1 = new StringTokenizer(min, "/") ;
        StringTokenizer st2 = new StringTokenizer(f�Ihop(st1) , "-") ;
        StringTokenizer st3 = new StringTokenizer(f�Ihop(st2), " ") ;
        return f�Ihop(st3) ;   
      private static String f�Ihop(StringTokenizer s)
          String temp = "" ;
        while (s.hasMoreElements())
                temp += s.nextToken() ;
        return temp ;
   ///** * Create the GUI and show it. For thread safety, * this method should be invoked from the * event-dispatching thread. */
//changed from private static to -->
    private static void createAndShowGUI()
        { //Make sure we have nice window decorations.
                System.out.println("creatAndShow") ;
            JFrame.setDefaultLookAndFeelDecorated(true);
            IP minTest = new IP();
    private JLabel createMine()
        JLabel nu = new JLabel("Mine", SwingConstants.LEFT);
        return nu ;
    private JComboBox createNow()
        JComboBox temp = new JComboBox() ;
        temp.addItem("Nummer 24") ;
        temp.addItem("Nummer 25") ;
        return temp ;
     public void itemStateChanged(ItemEvent y)
             System.out.println("item") ;
             MenyBoka nu = new MenyBoka(Frasses) ;
        if (y.getSource() == minMeny)
            if (minMeny.getSelectedIndex() == 0)
              System.out.println("itemlistener11") ;
                //Bokning nu = new Bokning() ;
                 tfnPanel.remove(minMeny) ;
                 tfnPanel.add(createNow()) ;
            //   testLabel.setText("hejhej") ;
               //    tfnFrame.getContentPane().add(tfnPanel, BorderLayout.CENTER); //Display the window.
             tfnFrame.pack();
             tfnFrame.setVisible(true);
                //alt �ndra minMeny till skapaMeny...
               //ide� k�ra create ..
               //m�ste jag addera den en g�ng till
             if (minMeny.getSelectedIndex() == 1)
                      System.out.println("select1") ;
               testLabel.setText("snusmumrik") ;
              if (minMeny.getSelectedIndex() == 2)
                    System.out.println("inuti selcted 2") ;
                 tfnPanel.remove(minMeny) ;
                 tfnPanel.add(nu.bokaMeny) ;
                 tfnFrame.pack();
                 tfnFrame.setVisible(true);
     } //itemstatechanged
   public static void main(String[] args)
   { //Schedule a job for the event-dispatching thread: //creating and showing this application's GUI.
    System.out.println("main") ;
    javax.swing.SwingUtilities.invokeLater(new Runnable() { public void run() { createAndShowGUI(); } }); } 
    public static boolean �rSiffra(String nr)
         try
            int minTry = Integer.parseInt(nr) ;
            return true ;
         catch (NumberFormatException ne)
          //   ne.printStackTrace()  ;
             return false ;
    }//�rSiffra