Swing 3 JComboBoxen ausgelesen Elemente einzel ausgeben

[sandaime]

Hallo an alle,
dies ist mein erster Beitrag ich hoffe ihr könnt mir weiterhelfen. Ich habe eine GUI mit 3 ComboBoxen. In diese habe ich Items über ein Array manuell hinzugefügt. Nun möchte ich per Button Klick den Inhalt des ComboBoxes und einer TextArea in der Konsole ausgeben. Es findet auch eine Ausgabe statt jedoch von allen ComboBoxen und diese 3 mal mit dem Inhalt des TextAreas. Ich will jedoch dass die Ausgabe nur von der ausgewählten ComboBox und dem TextArea ausgegeben wird.

Hier ist der Code dazu vom ActionEvent Abschnitt:

    public void actionPerformed(ActionEvent ae) {
       
        /*Ein Versuch die Elemente von der JComboBox auszulesen und spaeter so in die txt datei zu uebertragen
         * aber auch auf der GUI die aktuellen Eintraege zu zeigen
         * */
       
       
        if(ae.getSource()==this.eingabeKnopf){
            if ("parameterKuerzel".equals(ae.getActionCommand()));{
                System.out.println(parameterKuerzel.getSelectedItem()+ " "+eingabeFeld.getText());
                parameterKuerzel.setSelectedItem("--");
            }
            if ("gaseKuerzel".equals(ae.getActionCommand()));{
                System.out.println(gaseKuerzel.getSelectedItem()+ " "+eingabeFeld.getText());
                gaseKuerzel.setSelectedItem("--");
            }
            if ("staubKuerzel".equals(ae.getActionCommand()));{
                System.out.println(staubKuerzel.getSelectedItem()+ " "+eingabeFeld.getText());
                staubKuerzel.setSelectedItem("--");
            }
        }

 

[incluedu]

Hallo sandaime

Dein Problem ist der ; nach der if abfrage, wenn du das Semikolon entfernst dann sollte es funktionieren so wie du gerne möchtest.

 if ("parameterKuerzel".equals(ae.getActionCommand())){
                System.out.println(parameterKuerzel.getSelectedItem()+ " "+eingabeFeld.getText());
                parameterKuerzel.setSelectedItem("--");
            }

 

[sandaime]

Hallo incluedu,
danke für deine antwort, aber diesmal tut sich einfach gar nicht. Also keine Ausgabe nichts.

danke

 

[incluedu]

Dann muss es daran liegen das du das 'actionCommand' nicht richtig gesetzt hast. Das sollte irgendwie so ausschauen dort wo du deine 'Combobox' initialisierst.

        parameterKuerzel = new JComboBox<>(new String[]{"--", "param1", "param2", "param3"});
        parameterKuerzel.setActionCommand("parameterKuerzel");
        parameterKuerzel.addActionListener(this);

Vielleicht kannst du auch mal ein wenig mehr von deinem Code posten

 

[sandaime]

Danke für deine Antwort.
Hier der gesamte Code

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

public class EingabeGui extends JFrame implements ActionListener {

    /**
     Komische Anferoderun von Eclipe einfach hinzugefuegt ohne Wissen darueber
     */
    private static final long serialVersionUID = 1L;
   
    //Es werden die Variablen definiert
    JPanel eingabePanel;
    JButton eingabeKnopf;
    JButton loeschKnopf;
    JComboBox parameterKuerzel;
    JComboBox gaseKuerzel;
    JComboBox staubKuerzel;
    JList aktuelleEingabeListe;
    JLabel listTitel;
    JTextArea eingabeFeld;
   
    EingabeGui(){
       
        //Eine ComboBox mit allen moeglichen Parameter eintraegen
        String comboBoxListeParameter[] = {"--","ab","aq","as","az","bb","bq","cb",
        "cq","d0","dd","dq","gh","gx","gy","ha","hh","hp","hq","lc","lq",
        "nx","ny","nz","os","os NESTING","os -NESTING","os SCINOTAT",
        "BS=CBS","os PRFMOD","os SORRELAX","os SPECTRUM","os SPREAD"
        ,"qb","qq","qs","rb","rq","sd","sq","ti","tq","ux","uy",
        "vq","wb","wq","xO","xa","xb","xp","xq","y0","ya","yb","yp","yq","z0"};
       
        //Eine ComboBox mit allen moeglichen Gas eintraegen
        String comboBoxListeGase[] = {"--","so2", "no","no2","nox","bzl","tce","f","nh3",
                "hg","xx","odor","odor_nnn"};
       
        //Eine ComboBox mit allen moeglichen Staub eintraegen
        String comboBoxListeStaub[] = {"--","pm","pm-1","pm-2" ,"as","pb","cd","ni","hg","tl",
                "xx",};
       
        //Die ComboBoxen werden erst jetzt erstellt, damit die Eintraege von Arrays in den ComboBoxen aufgenommen werden
        parameterKuerzel = new JComboBox(comboBoxListeParameter);
        gaseKuerzel = new JComboBox(comboBoxListeGase);
        staubKuerzel = new JComboBox(comboBoxListeStaub);
        loeschKnopf = new JButton("entfernen");
        eingabeKnopf = new JButton("Parameter eingabe");
        aktuelleEingabeListe = new JList();
        eingabePanel = new JPanel();
        listTitel = new JLabel("Die aktuellen Eingaben sind");
        eingabeFeld = new JTextArea(1,8);
       
        //Dem Panel werden die Elemente hinzugefuegt Button, ComboBox, Liste, Label
        eingabePanel.add(parameterKuerzel);
        eingabePanel.add(gaseKuerzel);
        eingabePanel.add(staubKuerzel);
        eingabePanel.add(eingabeKnopf);
        eingabePanel.add(loeschKnopf);
        eingabePanel.add(listTitel);
        eingabePanel.add(aktuelleEingabeListe);
        eingabePanel.add(eingabeFeld);
       
        //Die Elemente bekommen einen ActionListener zugewisen
        parameterKuerzel.addActionListener(this);
        gaseKuerzel.addActionListener(this);
        staubKuerzel.addActionListener(this);
        eingabeKnopf.addActionListener(this);
        loeschKnopf.addActionListener(this);
       
        //Das Panel wird dem Fenster hinzugefuegt
        this.add(eingabePanel);
       
        //Der Titel, die Groesse und die Postition des Fensters werden festgelegt. Zudem kommt die Schliessfunktion hinzu
        this.setTitle("Parameter Eingaben");
        this.setDefaultCloseOperation(DISPOSE_ON_CLOSE);
        this.setSize(550,300);
        this.setLocation(375, 175);
    }
   
    public static void main(String[] args) {
        EingabeGui eingabeFenster=new EingabeGui();
        eingabeFenster.setVisible(true);
    }

   
   
    public void actionPerformed(ActionEvent ae) {
       
        /*Ein Versuch die Elemente von der JComboBox auszulesen und spaeter so in die txt datei zu uebertragen
         * aber auch auf der GUI die aktuellen Eintraege zu zeigen
         * */
       
       
        if(ae.getSource()==this.eingabeKnopf);{
            if ("parameterKuerzel".equals(ae.getActionCommand()));{
                System.out.println(parameterKuerzel.getSelectedItem()+ " "+eingabeFeld.getText());
                parameterKuerzel.setSelectedItem("--");
            }
            if ("gaseKuerzel".equals(ae.getActionCommand()));{
                System.out.println(gaseKuerzel.getSelectedItem()+ " "+eingabeFeld.getText());
                gaseKuerzel.setSelectedItem("--");
            }
            if ("staubKuerzel".equals(ae.getActionCommand()));{
                System.out.println(staubKuerzel.getSelectedItem()+ " "+eingabeFeld.getText());
                staubKuerzel.setSelectedItem("--");
            }
        }
       
        if(ae.getSource()==this.loeschKnopf){
            try
            {
                File file = new File("C:/Users/user/Desktop/Beispiel.txt");
                FileWriter fw = new FileWriter(file);
                fw.write(parameterKuerzel.getSelectedItem() + " "+ eingabeFeld.getText()+"\r\n");
                fw.flush();
                fw.close();
            }
            catch( IOException ev )
            {
                ev.printStackTrace();
            }
        }
    }
}

 

[incluedu]

Hm ...

Bin mir nicht sicher ob ich genau verstehe was du machen möchtest.

Aber soweit ich es verstanden habe willst du das die Ausgabe nur dann erfolgt wenn du bei mindestens einer deiner drei 'ComboBoxen' etwas anderes als '--' ausgewählt hast.

Um das zu erreichen würde ich als erstes deinen 'ComboBoxen' einen Type zuweisen in deinem Fall den Type String. Das hat den Vorteil das du dich nicht mehr darum kümmern must was in der 'ComboBox' gespeichert ist. Außerdem sollte man variablen 'private' definieren.

    //Es werden die Variablen definiert
    private JPanel eingabePanel;
    private JButton eingabeKnopf;
    private JButton loeschKnopf;
    private JComboBox<String> parameterKuerzel;
    private JComboBox<String> gaseKuerzel;
    private JComboBox<String> staubKuerzel;
    private JList aktuelleEingabeListe;
    private JLabel listTitel;
    private JTextArea eingabeFeld;

Auf die 'ActionListener' bei den 'ComboBoxen' könntest du in diesem falle eigentlich verzichten da du die Action ja mit dem 'eingabeKnopf' machen möchtest.

        //Die Elemente bekommen einen ActionListener zugewisen
//        parameterKuerzel.addActionListener(this);
//        gaseKuerzel.addActionListener(this);
//        staubKuerzel.addActionListener(this);
        eingabeKnopf.addActionListener(this);
        loeschKnopf.addActionListener(this);

Bei deinem Event Handling must die wie ich schon sagte die Semikolon entfernen und statt der Abfrage auf das 'ActionCommand' das aktuell selektierte item von jeder 'ComboBox' mit '--' vergleichen. Wenn die Bedingung nicht erfüllt ist dann kannst du dein Ausgabe auf der Console machen.

        if (ae.getSource() == this.eingabeKnopf) {
            if (!parameterKuerzel.getSelectedItem().equals("--")) {
                System.out.println(parameterKuerzel.getSelectedItem() + " " + eingabeFeld.getText());
                parameterKuerzel.setSelectedItem("--");
            }
            if (!gaseKuerzel.getSelectedItem().equals("--")) {
                System.out.println(gaseKuerzel.getSelectedItem() + " " + eingabeFeld.getText());
                gaseKuerzel.setSelectedItem("--");
            }
            if (!staubKuerzel.getSelectedItem().equals("--")) {
               
System.out.println(staubKuerzel.getSelectedItem() + " " + eingabeFeld.getText());
                staubKuerzel.setSelectedItem("--");
            }
        }

Was mir auch noch aufgefallen ist an deinem Code, ist das du deinen JFrame falsch aufrufst, man sollte das in Swing immer mit 'InvokeLater' machen, wenn es dich interessiert warum dann google danach.
Ich würde mir das aber dennoch angewöhnen das ich es von Anfang an richtig machen auch wenn es so wie du es machst funktioniert.

    public static void main(String[] args) {
        SwingUtilities.invokeLater(new Runnable() {
            @Override public void run() {
                EingabeGui eingabeFenster = new EingabeGui();
                eingabeFenster.setVisible(true);
            }
        });
    }

Hoffe das hilft ein wenig, wenn nicht dann versuch doch nochmal mir zu erklären wie denn genau das gewünschte Ergebnis ausschauen soll (hab grad Zeit also kein Problem helfe gerne wenn ich denn kann).

 

[sandaime]

Danke dir vielmals. Es hat mir sehr geholfen und mich zu der Lösung gebracht was ich brauchte.