Java. JComboBox - No funciona el programa

Orici

Hola no consigo que funcione como debe, este es el código tal como lo tengo ahora, podéis ejecutarlo para ver lo que no funciona.
Saludos.

/*JComboBox
	Al seleccionar una opcion aparece el texto asociado en un campo de texto.
*/
 
import javax.swing.*;
import java.awt.FlowLayout;
import java.awt.event.*;
import java.io.PrintWriter;

public class ComboBox01 extends JFrame implements ActionListener, ItemListener {
	
	JLabel lab1 = new JLabel("El mar es:");
	JLabel lab2 = new JLabel("El desierto es:");
	JComboBox combo1 = new JComboBox();
	JComboBox combo2 = new JComboBox();
	JTextField campo = new JTextField(38);
	JButton limpiar = new JButton(" - Limpiar - ");
	JButton cerrar = new JButton(" - Cerrar - ");
	
	FlowLayout flow = new FlowLayout();
	
	String[] cb1 = {"azul","verde","oscuro","profundo"};
		int largoCb1 = cb1.length;
	String[] cb2 = {"silencioso","mortal","misterioso","eterno"};
		int largoCb2 = cb2.length;

//Constructor:
	public ComboBox01() {
		super("Elige opciones");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setSize(450, 120);
		setResizable(false);
		setLayout(flow);
		flow.setHgap(15);
		
		//Añadir componentes a la ventana:
		add(lab1);
		add(combo1);
		add(lab2);
		add(combo2);
		add(campo);
		add(limpiar);
		add(cerrar);
		
		//Rellenar los JComboBoxes:
		for (int i=0; i<largoCb1; i++)		combo1.addItem(cb1[i]);
		for (int i=0; i<largoCb2; i++)		combo2.addItem(cb2[i]);
		
		//Añadir listeners a los componentes:
		combo1.addItemListener(this);
		combo2.addItemListener(this);
		limpiar.addActionListener(this);
		cerrar.addActionListener(this);
	}
	
//Implementa métodos de ActionListener:
	@Override
	public void actionPerformed(ActionEvent ev) {
		if (ev.getSource() == cerrar)		System.exit(0);
		if (ev.getSource() == limpiar)	campo.setText("");
	}

//Implementa métodos de ItemListener:
	@Override
	public void itemStateChanged(ItemEvent ev) {
		String salida = "";
		
		Object origen1 = combo1.getSelectedItem();
		Object origen2 = combo2.getSelectedItem();
			
			if (origen1 == "azul") {
				salida = "El mar es azul.";
				campo.setText(salida);
			}
			if (origen1 == "verde")	{
				salida = "El mar es verde.";
				campo.setText(salida);
			}
			if (origen1 == "oscuro") {
				salida = "El mar es oscuro.";
				campo.setText(salida);
			}
			if (origen1 == "profundo") {
				salida = "El mar es profundo.";
				campo.setText(salida);
			}
		
		
			if (origen2 == "silencioso") {
				salida = "El desierto es silencioso.";
				campo.setText(salida);
			}
			if (origen2 == "mortal") {
				salida = "El desierto es mortal.";
				campo.setText(salida);
			}
			if (origen2 == "misterioso") {
				salida = "El desierto es misterioso.";
				campo.setText(salida);
			}
			if (origen2 == "eterno") {
				salida = "El desierto es eterno.";
				campo.setText(salida);
			}
	}
	
//Main:
	public static void main(String args[]) {
		new ComboBox01().setVisible(true);
	}
}

JooseLuis

edito: no es lo que pensaba

voy a verlo

JooseLuis

Si "el fallo" es el que creo está en la implementación de los ItemListener,
al ocurrir un evento de este tipo no le decimos qué JComboBox ha sido modificado,
y siempre llena el campo de texto con el último JComboBox en orden descendente.

Modifico el archivo para que funcione (según creo que quieres que lo haga):

Orici escribió

Hola no consigo que funcione como debe, este es el código tal como lo tengo ahora, podéis ejecutarlo para ver lo que no funciona.
Saludos.

/*JComboBox
	Al seleccionar una opcion aparece el texto asociado en un campo de texto.
*/
 
import javax.swing.*;
import java.awt.FlowLayout;
import java.awt.event.*;
import java.io.PrintWriter;

public class ComboBox01 extends JFrame implements ActionListener, ItemListener {
	
	JLabel lab1 = new JLabel("El mar es:");
	JLabel lab2 = new JLabel("El desierto es:");
	JComboBox combo1 = new JComboBox();
	JComboBox combo2 = new JComboBox();
	JTextField campo = new JTextField(38);
	JButton limpiar = new JButton(" - Limpiar - ");
	JButton cerrar = new JButton(" - Cerrar - ");
	
	FlowLayout flow = new FlowLayout();
	
	String[] cb1 = {"azul","verde","oscuro","profundo"};
		int largoCb1 = cb1.length;
	String[] cb2 = {"silencioso","mortal","misterioso","eterno"};
		int largoCb2 = cb2.length;

//Constructor:
	public ComboBox01() {
		super("Elige opciones");
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setSize(450, 120);
		setResizable(false);
		setLayout(flow);
		flow.setHgap(15);
		
		//Añadir componentes a la ventana:
		add(lab1);
		add(combo1);
		add(lab2);
		add(combo2);
		add(campo);
		add(limpiar);
		add(cerrar);
		
		//Rellenar los JComboBoxes:
		for (int i=0; i<largoCb1; i++)		combo1.addItem(cb1[i]);
		for (int i=0; i<largoCb2; i++)		combo2.addItem(cb2[i]);
		
		//Añadir listeners a los componentes:
		combo1.addItemListener(this);
		combo2.addItemListener(this);
		limpiar.addActionListener(this);
		cerrar.addActionListener(this);
	}
	
//Implementa métodos de ActionListener:
	@Override
	public void actionPerformed(ActionEvent ev) {
		if (ev.getSource() == cerrar)		System.exit(0);
		if (ev.getSource() == limpiar)	campo.setText("");
	}

//Implementa métodos de ItemListener:
	@Override
	public void itemStateChanged(ItemEvent ev) {
		String salida = "";
		JComboBox combo = (JComboBox) ev.getSource(); 
		Object origen = combo.getSelectedItem();
		
			
			if (origen == "azul") {
				salida = "El mar es azul.";
				campo.setText(salida);
			}
			if (origen == "verde")	{
				salida = "El mar es verde.";
				campo.setText(salida);
			}
			if (origen == "oscuro") {
				salida = "El mar es oscuro.";
				campo.setText(salida);
			}
			if (origen == "profundo") {
				salida = "El mar es profundo.";
				campo.setText(salida);
			}
		
		
			if (origen == "silencioso") {
				salida = "El desierto es silencioso.";
				campo.setText(salida);
			}
			if (origen == "mortal") {
				salida = "El desierto es mortal.";
				campo.setText(salida);
			}
			if (origen == "misterioso") {
				salida = "El desierto es misterioso.";
				campo.setText(salida);
			}
			if (origen == "eterno") {
				salida = "El desierto es eterno.";
				campo.setText(salida);
			}
	}
	
//Main:
	public static void main(String args[]) {
		new ComboBox01().setVisible(true);
	}
}

jjy

Como te han dicho el problema es que no diferencias que campo es el que se ha modificado, por lo tanto se ejecuta el código de forma secuencial dando como resultado el ultimo setText posible, te paso una propuesta del itemStateChanged mejorada.

import javax.swing.*;
import java.awt.FlowLayout;
import java.awt.event.*;
import java.io.PrintWriter;
 
public class ComboBox01 extends JFrame implements ActionListener, ItemListener {
  
    JLabel lab1 = new JLabel("El mar es:");
 JLabel lab2 = new JLabel("El desierto es:");
    JComboBox combo1 = new JComboBox();
 JComboBox combo2 = new JComboBox();
 JTextField campo = new JTextField(38);
  JButton limpiar = new JButton(" - Limpiar - ");
 JButton cerrar = new JButton(" - Cerrar - ");
    
    FlowLayout flow = new FlowLayout();
  
    String[] cb1 = {"azul","verde","oscuro","profundo"};
        int largoCb1 = cb1.length;
  String[] cb2 = {"silencioso","mortal","misterioso","eterno"};
       int largoCb2 = cb2.length;
 
//Constructor:
    public ComboBox01() {
       super("Elige opciones");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     setSize(450, 120);
      setResizable(false);
        setLayout(flow);
        flow.setHgap(15);
        
        //Añadir componentes a la ventana:
      add(lab1);
      add(combo1);
        add(lab2);
      add(combo2);
        add(campo);
     add(limpiar);
       add(cerrar);
         
        //Rellenar los JComboBoxes:
     for (int i=0; i<largoCb1; i++)       combo1.addItem(cb1[i]);
     for (int i=0; i<largoCb2; i++)       combo2.addItem(cb2[i]);
      
        //Añadir listeners a los componentes:
       combo1.addItemListener(this);
       combo2.addItemListener(this);
       limpiar.addActionListener(this);
        cerrar.addActionListener(this);
 }
    
//Implementa métodos de ActionListener:
 @Override
   public void actionPerformed(ActionEvent ev) {
       if (ev.getSource() == cerrar)       System.exit(0);
     if (ev.getSource() == limpiar)  campo.setText("");
  }
 
//Implementa métodos de ItemListener:
  @Override
   public void itemStateChanged(ItemEvent ev) {
        JComboBox c = (JComboBox) ev.getSource(); 
        StringBuffer salida = new StringBuffer(); 
        if(c == combo1) { 
            salida.append("El mar es ");
        } else { 
            salida.append("El desierto es ");
        }
        salida.append(c.getSelectedItem()+".");
        campo.setText(salida.toString());
   }
    
//Main:
 public static void main(String args[]) {
        new ComboBox01().setVisible(true);
  }
}

JooseLuis

jjy escribió

import javax.swing.*;
import java.awt.FlowLayout;
import java.awt.event.*;
import java.io.PrintWriter;
 
public class ComboBox01 extends JFrame implements ActionListener, ItemListener {
  
    JLabel lab1 = new JLabel("El mar es:");
 JLabel lab2 = new JLabel("El desierto es:");
    JComboBox combo1 = new JComboBox();
 JComboBox combo2 = new JComboBox();
 JTextField campo = new JTextField(38);
  JButton limpiar = new JButton(" - Limpiar - ");
 JButton cerrar = new JButton(" - Cerrar - ");
    
    FlowLayout flow = new FlowLayout();
  
    String[] cb1 = {"azul","verde","oscuro","profundo"};
        int largoCb1 = cb1.length;
  String[] cb2 = {"silencioso","mortal","misterioso","eterno"};
       int largoCb2 = cb2.length;
 
//Constructor:
    public ComboBox01() {
       super("Elige opciones");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     setSize(450, 120);
      setResizable(false);
        setLayout(flow);
        flow.setHgap(15);
        
        //Añadir componentes a la ventana:
      add(lab1);
      add(combo1);
        add(lab2);
      add(combo2);
        add(campo);
     add(limpiar);
       add(cerrar);
         
        //Rellenar los JComboBoxes:
     for (int i=0; i<largoCb1; i++)       combo1.addItem(cb1[i]);
     for (int i=0; i<largoCb2; i++)       combo2.addItem(cb2[i]);
      
        //Añadir listeners a los componentes:
       combo1.addItemListener(this);
       combo2.addItemListener(this);
       limpiar.addActionListener(this);
        cerrar.addActionListener(this);
 }
    
//Implementa métodos de ActionListener:
 @Override
   public void actionPerformed(ActionEvent ev) {
       if (ev.getSource() == cerrar)       System.exit(0);
     if (ev.getSource() == limpiar)  campo.setText("");
  }
 
//Implementa métodos de ItemListener:
  @Override
   public void itemStateChanged(ItemEvent ev) {
        JComboBox c = (JComboBox) ev.getSource(); 
        StringBuffer salida = new StringBuffer(); 
        if(c == combo1) { 
            salida.append("El mar es ");
        } else { 
            salida.append("El desierto es ");
        }
        salida.append(c.getSelectedItem()+".");
        campo.setText(salida.toString());
   }
    
//Main:
 public static void main(String args[]) {
        new ComboBox01().setVisible(true);
  }
}

Yo de más mayor quiero saber programar como tú... xD

PD: Lo digo en serio