Frage zu ComboBox

[Muffelhs]

Hallo

wenn meine ComboBox aktiviert wird, dann wird automatisch das 1. objekt ausgewählt.

Wenn ich eine anderes Object in der Combobox auswähle passiert nichts

Warum klappt meine abfrage nicht?

  public void setFarben(){
	  	  
	  farben.setEnabled(true);
     	  
      if(farben.getSelectedItem().equals("schwarz")){ farbe = Color.black; console.setEnabled(true);}
      else if(farben.getSelectedItem().equals("blau")){ farbe = Color.blue;console.setEnabled(true);}
      else if(farben.getSelectedItem().equals("grün")){ farbe = Color.green;  console.setEnabled(true);}
      else if(farben.getSelectedItem().equals("orange")){ farbe = Color.orange; console.setEnabled(true);}
      else if(farben.getSelectedItem().equals("rot")){ farbe = Color.red;console.setEnabled(true);}	 
      
  }

wie kann ich die methode schrieben das diese auswahl funktioniert ?
Es wird immer das 1. Element selected somit klappen meine abfragen nicht egal ob ich etwas anderes auswähle
kann mir einer helfen ?

MfG

Muffel

 

[hdi]

Für sowas nutzt man "Listener" - das Programm muss ja wissen was passieren
soll wenn du in der ComboBox auf etwas klickst.
Und wenn du klickst wird ein Event gefeuert, dass du im Listener abfangen und behandeln
kannst.

[HIGHLIGHT="Java"]myComboBox.addActionListener(new ActionListener(){
@Override
public void actionPerformed(ActionEvent e){
JComboBox box = (JComboBox)e.getSource();
if(box.getSelectedItem().equals usw...)
}
});[/HIGHLIGHT]

 

[Muffelhs]

das problem ist ich habe mit getActionCommand schon von der auwahl davor die abfrage gemacht. Das ist sozusagen in dem anderen listener drin ..
hier nochmal der code dann wird es verständlich

public void actionPerformed(ActionEvent ae){
    String command = ae.getActionCommand();
    
    if(command.equals("figuren")){

      if(figuren.getSelectedItem().equals("Vieleck")){
        erweitert.setEnabled(true);
        
        if(erweitert.getSelectedItem().equals("Vieleck")){
        
        	setFarben();
       
        }
        else if(erweitert.getSelectedItem().equals("Rechteck")){
        	//EDIT!!!
        }
      }
      else if(figuren.getSelectedItem().equals("Kreis")){
    	  
        setFarben(); 
        
      }
    }

ich habe in fioguren erstmal den action listener drin mit
addActionlistener(this)
dann den command
setAction Commadn Figuren..

wenn ich auf figurne klicke und kreis auswähle dann soll ich zu der methode setFarben kommen

da klappt das noch also da kann ich abfragen mit figuren.getSelectedItem.equals("Kreis");
wenn ich das gleich dann in setFarben mache geht das nicht..

das mit dem action listener geht nicht weil ich ja schon figuren als action Command habe und das ist ja in figuren drin ..

ist das problem verständlich `?

 

[hdi]

Ja okay, ich glaube ich weiss wo das Problem sein könnte.
Du reaktivierst die Box "farben" jedesmal neu in der setFarben() Methode.
Daher ist der evlt vorher gespeicherte selektierte Index verloren. Wenn
du nun prüfst was selected ist, ist es default-mässig der erste Wert.
Sprich: Entweder du lässt das deaktivieren und reaktiveren (setEnabled()) dieser
Box zur Laufzeit sein, oder du musst dir per Hand den selektierten Index irgendwo
abspeichern, bevor du sie deaktivierst, und dann in der setFarben() darauf zurückgreifen.

 

[Muffelhs]

else if(figuren.getSelectedItem().equals("Kreis")){
    	  //farben.setEnabled(true);
     	  
          if(farben.getSelectedItem().equals("schwarz")){ farbe = Color.black;console.setEnabled(true);}
          else if(farben.getSelectedItem().equals("blau")){ farbe = Color.blue;}
          else if(farben.getSelectedItem().equals("grün")){ farbe = Color.green;}
          else if(farben.getSelectedItem().equals("orange")){ farbe = Color.orange;}
          else if(farben.getSelectedItem().equals("rot")){ farbe = Color.red;}	 
          
        
      }

ich habe die combobox jetzt von anfang an aktiviert und es geht immernoch nicht ..
dh daran kann es nicht liegen

es ist so konfus das ich bei der oberen combobox die abfrage machen kann getSelectedObject.equals("Kreis");
und hier direkt dadrunter geht es nichtmehr...

woran kann es noch liegen ?

=/

hiernochmal der ganze code..

es gibt zwar vieleck rechteck usw.

Mache aber erstmal die Funktionen für Kreis fertig ...

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import javax.swing.event.*;

/**
  *
  * Beschreibung
  *
  * @version 1.0 vom 27.03.2009
  * @author
  */

public class FigurenProgramm extends JFrame implements ActionListener {
  // Anfang Attribute
  private Color farbe = Color.white;
  private Point[] punkte = new Point[1];
  private int x = 0;
  private int y = 0;
  private String name = "";
  private double radius;
  
  private Canvas zf = new Canvas();
  private String[] jList1Data = {};
  private String[] jComboBox1Data = {};
  private JComboBox figuren = new JComboBox(jComboBox1Data);
  private String[] jComboBox2Data = {};
  private JComboBox erweitert = new JComboBox(jComboBox2Data);
  private String[] jComboBox3Data = {};
  private JComboBox erweitert_ = new JComboBox(jComboBox3Data);
  private String[] jComboBox4Data = {};
  private JComboBox farben = new JComboBox(jComboBox4Data);
  private JLabel namelb = new JLabel();
  private JTextField nameed = new JTextField();
  private JButton console = new JButton();
  private JButton erstellen = new JButton();
  // Ende Attribute

  public FigurenProgramm(String title) {
    // Frame-Initialisierung
    super(title);
    setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);
    int frameWidth = 982;
    int frameHeight = 533;
    setSize(frameWidth, frameHeight);
    Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
    int x = (d.width - getSize().width) / 2;
    int y = (d.height - getSize().height) / 2;
    setLocation(x, y);
    Container cp = getContentPane();
    cp.setLayout(null);
    // Anfang Komponenten
    zf.setSize(600,600);
    zf.setBackground(Color.white);
    cp.add(zf);
    
    namelb.setBounds(610, 20, 50, 16);
    namelb.setText("Name:");
    cp.add(namelb);

    nameed.setBounds(665,20,60,20);
    cp.add(nameed);
    
    figuren.setBounds(610, 50, 145, 24);
    figuren.addItem("Vieleck");
    figuren.addItem("Kreis");
    figuren.addActionListener(this);
    figuren.setActionCommand("figuren");
    cp.add(figuren);
    
    erweitert.setBounds(610, 72, 145, 24);
    erweitert.addItem("Auswahl");
    erweitert.addItem("Vieleck");
    erweitert.addItem("Rechteck");
    erweitert.addActionListener(this);
    erweitert.setActionCommand("erweitert");
    cp.add(erweitert);
    erweitert.setEnabled(false);
    
    erweitert_.setBounds(610, 95, 145, 24);
    erweitert_.addItem("Auswahl");
    erweitert_.addItem("Rechteck");
    erweitert_.addItem("Quadrat");
    erweitert_.addActionListener(this);
    erweitert.setActionCommand("erweitert_");
    cp.add(erweitert_);
    erweitert_.setEnabled(false);
    
    farben.setBounds(610, 136, 145, 24);
    farben.addItem("Auswahl");
    farben.addItem("schwarz");
    farben.addItem("blau");
    farben.addItem("grün");
    farben.addItem("orange");
    farben.addItem("rot");
    farben.addActionListener(this);
    farben.setActionCommand("farben");
    cp.add(farben);
    farben.setEnabled(false);

    console.setBounds(610, 176, 90, 25);
    console.setText("Konsole");
    console.addActionListener(this);
    console.setActionCommand("console");
    cp.add(console);
    console.setEnabled(false);
    
    erstellen.setBounds(610, 210, 90, 25);
    erstellen.setText("erstellen");
    erstellen.addActionListener(this);
    erstellen.setActionCommand("erstellen");
    cp.add(erstellen);
    erstellen.setEnabled(false);
    // Ende Komponenten

    setResizable(false);
    setVisible(true);
  }

  // Anfang Methoden
  public void setFarben(){
	  	  
	  farben.setEnabled(true);
     	  
      if(farben.getSelectedItem().equals("schwarz")){ farbe = Color.black;}
      else if(farben.getSelectedItem().equals("blau")){ farbe = Color.blue;}
      else if(farben.getSelectedItem().equals("grün")){ farbe = Color.green;}
      else if(farben.getSelectedItem().equals("orange")){ farbe = Color.orange;}
      else if(farben.getSelectedItem().equals("rot")){ farbe = Color.red;}	 
      console.setEnabled(true);
  }
  
  public void actionPerformed(ActionEvent ae){
    String command = ae.getActionCommand();
    
    if(command.equals("figuren")){

      if(figuren.getSelectedItem().equals("Vieleck")){
        erweitert.setEnabled(true);
        
        if(erweitert.getSelectedItem().equals("Vieleck")){
        
        	//EDIT!!!
       
        }
        else if(erweitert.getSelectedItem().equals("Rechteck")){
        	//EDIT!!!
        }
      }
      else if(figuren.getSelectedItem().equals("Kreis")){
    	  farben.setEnabled(true);
     	  
         setFarben();
      }
    }
    else if(command.equals("console")){

      TextIO.putln("");
      TextIO.put("Bitte geben Sie den X Wert an: ");
      x = TextIO.getlnInt();
      TextIO.put("Bitte geben Sie den Y Wert an: ");
      y = TextIO.getlnInt();

      Point punkt = new Point(x,y);
      punkte[0]= punkt;

      TextIO.putln("");
      TextIO.put("Bitte geben Sie den Radius an: ");
      radius = TextIO.getlnDouble();
      erstellen.setEnabled(true);
    }
    else if(command.equals("erstellen")){

      Kreis k = new Kreis(radius,name,farbe,punkte);
      zf.setForeground(k.getFarbe());
      k.zeichne(zf.getGraphics());
    }
    
    
  }
    // Ende Methoden

    public static void main(String[] args) {
      new FigurenProgramm("FigurenProgramm");
    }

}

 

[SlaterB]

was genau ist das Problem anhand des gepostenen Beispielprogramms?
anfangs ist die Farben-Box deaktiviert, man kann sie nur aktivieren, wenn man Kreis auswählt,
bei dieser Aktivierung ist natürlich immer noch das erste Element 'Auswahl' selektiert, kein anderes,
willst du einen anderen Startwert?

jedenfalls trifft dann zunächst keiner der if-Fälle in setFarben() zu, kein Wunder,

da die Box nun aktiviert ist, kann man aber eine Farbe auswählen, daraufhin findet keine besondere Ereignisbehandlung statt, da nicht programmiert,
wenn man aber den Kreis wegwählt und wieder auswählt,
dann gelangt man erneut in die setFarben()-Methode und diesmal ist ja eine der Farben ausgewählt, das klappt auch

was genau ist bei all diesen normal verständlichen logischen Vorgängen das Problem?

 

[Marco13]

Vielleicht im Konstruktor nach dem initialisieren un dHinzufügen der "farben"-ComboBox noch

    farben.addActionListener(new ActionListener()
    {
        public void actionPerformed(ActionEvent e)
        {
            setFarben();
        }
    });

einfügen!?

 

[Muffelhs]

dankeschön

mir war nicht klar das der actionlistener obwohl ich in dem command figuren war auch noch gleichzeitik ein anderes event steuern kann..

vielen dank

mit marcos vorschlag hat es geklappt!