Problem mit JComboBox

[Guest]

Hallo,

ich habe:
-1GUI(eigene Klasse)
-1Panel (eigene Klasse)

Auf dem Panel befindet sich eine JComboBox!
Nun packe ich das Panel in die GUI,das klappt auch alles und die JComboBox wird angezeigt.
Drücke ich nun den Pfeil neben der JComboBox öffnet sich die "Liste" und ich kann auswählen,schließe ich jedoch die Box wieder oder wähle etwas aus,so "verschwindet" die ComboBox und der Hintergrund der GUI wird sichtbar.
Allerdings kann cih sie immer noch anklicken,an der Stelle wo sie sich befindet!

          cbTest=new JComboBox();
          cbTest.setEditable(false);
          cbTest.setBounds(300,20,100,30);
          cbTest.setFont(new Font( "Verdana",Font.PLAIN,16 ));
          cbTest.addItem(Team1.getSpieler(0).getName());
          cbTest.addItem(Team1.getSpieler(1).getName());
          cbTest.addActionListener(this);
          this.add(cbTest);

Hängt das "verschwinden" mit der GUI zusamm oder mit den Einstellungen der JComboBox?

 

[SlaterB]

hast du ein AWT-Panel oder ein AWT-Frame statt Swing-JPanel und Swing-JFrame?

vollständiges kurzes Programm hilft ansonsten,
ohne Team usw., einfache Strings "a", "b" reichen

 

[Ebenius]

          /* ... */
          cbTest.setBounds(300,20,100,30);
          /* ... */

Hängt das "verschwinden" mit der GUI zusamm oder mit den Einstellungen der JComboBox?

Wahrscheinlich mit nem layout manager. Das erkennt man an dem Beispiel nicht. Lies Dich am besten hier ein.

Grüße, Ebenius

 

[Gast]

Also,mein Frame ist durch BorderLayout in 5 Bereiche eingeteilt.
Ich möchte nun das Panel mit der ComboBox in den Central Bereich einfügen,das klappt wie gesagt auch!
Das Panel mit der ComboBox hat das "null-Layout".
Gibt's dann da irgendwelche Probleme mit dem LayoutManager?!

 

[Ebenius]

Gibt's dann da irgendwelche Probleme mit dem LayoutManager?!

Möglich.

Wenn Deine ComboBox in einem Panel liegt das einen LayoutManager benutzt, wieso setzt Du der ComboBox dann bounds?

Ebenius

 

[Gast]

nein also das panel sieht wie folgt aus :

public class MeinPanel extends JPanel implements ActionListener
{
       //...

        public MeinPanel()
        {
          this.setLayout(null);
          this.setBackground(Background);
          //...   
         }
//...
}

Also muss ich ja bounds setzen!
....

 

[Ebenius]

Wo Du Recht hast ... Scheiße wenn man (ich) nicht lesen kann.

Poste einfach mal den (abgespeckten) Source Code, ich find den Fehler schon.

Cheers, Ebenius

 

[Landei]

Ich würde mal testweise einen einfachen LayoutManager setzen. Wäre interessant zu wissen, ob das Problem dann weggeht.

 

[Guest]

@Landei: Die Idee hat ich auch schon,aber zB beim FlowLayout-Manager besteht das selber Problem =(

@Ebenius: Kein Problem,das sollte kein Vorwurf sein Aber danke schonmal das du dir die Mühe machst hier die vielleicht nich ganz perfekten Source Codes:

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

public class GUI extends JFrame implements ActionListener
{
    JPanel Knöpfe=new JPanel(new GridLayout(5,1,0,30));
    JPanel Haupt=new JPanel(null);
    JPanel Infos=new JPanel();
    JPanel Über=new JPanel();
    JPanel Unter=new JPanel(new FlowLayout(FlowLayout.RIGHT,50,20));

   public GUI()

   {
        super("Java-Facharbeit");
        this.getContentPane().setBackground(Color.black);
        this.getContentPane().setLayout(new BorderLayout(5, 5));
        //...  
        Haupt.setBackground(Background);
        lbHintergrund=new JLabel(new ImageIcon("images/background1.png"));
        lbHintergrund.setBounds(0,0,800,600);
        Haupt.add(lbHintergrund);

        lbKonto = new JLabel();
        lbKonto.setBounds(560,20, 70, 20);
        lbKonto.setText("Kontostand:");
        Haupt.add(lbKonto);
        //...  
        tfKonto = new JTextField (20);
        tfKonto.setBounds (640, 20, 100, 25);
        tfKonto.setBackground(Menu);
        tfKonto.setText("120.000.000");
        Haupt.add(tfKonto);                
        //...  
        this.add(Haupt);
        this.add(Knöpfe,BorderLayout.WEST);
        this.add(Über,BorderLayout.NORTH);
        this.add(Unter,BorderLayout.SOUTH);
        
        //...  
            }
         });
    }

    public void actionPerformed(ActionEvent event)
    {
        String cmd = event.getActionCommand();
         if (cmd.equals("Aufstellung"))
        {
          JPanel AufstellungPanel=new Aufstellung();
          this.getContentPane().add(AufstellungPanel);
          this.validate();
        }

        else if (cmd.equals("Team-Info"))
        {
          JPanel TeamInfoPanel=new Team_Info();
          this.getContentPane().add(TeamInfoPanel);
          this.validate();
        }
        
        else if (cmd.equals("Spieler-Info"))
        {
          JPanel SpielerInfoPanel=new Spieler_Info();
          this.getContentPane().add(SpielerInfoPanel);
          this.validate();
        }

        else if (cmd.equals("Ende"))
        {
            setVisible(false);
            dispose();
            System.exit(0);
        }
    }
}

und das"AufstellungPanel" um das es geht^^ :

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

public class Aufstellung extends JPanel implements ActionListener
{
       private JComboBox Torwart;
       //...
        public Aufstellung()
        {
          this.setLayout(new FlowLayout());
          this.setBackground(Background);
          //
          Torwart=new JComboBox();
          Torwart.setEditable(false);
          Torwart.setFont(new Font( "Verdana",Font.PLAIN,16 ));
          Torwart.setBounds(20,20,100,30);
          Torwart.addItem(Schalke.getSpieler(0).getName());
          Torwart.addItem(Schalke.getSpieler(1).getName());
          Torwart.addActionListener(this);
          this.add(Torwart);
         }

        public void actionPerformed(ActionEvent event)
    {
        String cmd = event.getActionCommand();

         if (cmd.equals("Neuer"))
        {

        }


        else if (cmd.equals("Beenden"))
        {

        }
    }
}

Also wie gesagt das sind nur Ausschnitte,aber ich glaube iwo in diesen Zeilen steckt der Fehler.
Eigentlich ist auch der Teil in "GUI" der das Label & das Textfeld hinzufügt auskommentiert,aber der Fehler hängt NICHT damit zusamm.
Danke für eure Hilfe =)

 

[Ebenius]

@Ebenius: Kein Problem,das sollte kein Vorwurf sein

Hab ich auch nicht so aufgefasst.

Zum Thema:

• Klassennamen fangen mit einem Großbuchstaben an, Variablen (auch die Felder einer Klasse) mit einem kleinen Buchstaben. Umlaute sollte man vermeiden.
• In der actionPerformed(...)-Methode fügst Du dem ContentPane dynamisch Komponenten hinzu, die Du aber nie wieder entfernst. Das geht schief. Ich nehme an, Du überlagerst Komponenten übereinander und dann überzeichnet eine die andere. Die ComboBox ist nicht schuld, sie verursacht nur ein Repaint, woraufhin der Fehler zum tragen kommt.
• Man montiert niemals Komponenten auf ein JFrame, sondern immer auf das content pane (JFrame.getContentPane()). Das da ist also falsch:
  

  this.add(Haupt);
  this.add(Knöpfe,BorderLayout.WEST);
  this.add(Über,BorderLayout.NORTH);
  this.add(Unter,BorderLayout.SOUTH);

Wenn ich das alles richtig verstehe, dann willst Du auf Knopfdruck das mittlere Panel umschalten. Mach Dir doch ein JPanel, welches ein CardLayout benutzt, pack da alle Deine möglichen Panels drauf und schalte diese je nach Knopfdruck nach vorn. How-To use CardLayout

Hilft das erstmal?

Grüße, Ebenius

 

[Gast]

Ok danke
Das die Namen noch nicht ganz perfekt sind hab ich mir schon gedacht,hab mich bis jetzt auch noch nich außerhalb der Schule mit JAVA beschäftigt und da muss ich nochma genauer hinschauen!

Ich glaub ich hab das richtig verstanden und versuch's jetzt erstmal umzusetzen !

Falls das Problem weiterhin besteht( was ich aber nich glaube) melde ich mich nochmal...

Danke für die Hilfe =)

 

[Ebenius]

Gern.

 

[Gast]

Also ich hab jetzt noch ne leichtere Lösung gefunden,da ich deine Lösung doch nich so ganz verstanden hab =(
Ich hab dem Button noch den Befehl hinzugefügt,erst das "HauptPanel" wieder zu löschen.
Nun funktioniert alles wie ich es möchte,meine Frage ist nur ,wie effektiv diese Lösung ist & ob es vielleicht doch besser wäre das CardLayout einzuführen?
Zur Info,ich fertige eine Facharbeit für die Stufe 12 an

 

[Ebenius]

Nun funktioniert alles wie ich es möchte,meine Frage ist nur ,wie effektiv diese Lösung ist & ob es vielleicht doch besser wäre das CardLayout einzuführen?

Kommt darauf an, was Du noch möchtest. Hübsch ist Deine Lösung nicht unbedingt. Aber ich hab schon so viele Dinge getan die nicht hübsch sind. ;-)

Wenn Du noch mehr lernen magst, dann versuch Dich einfach nochmal am CardLayout. Der Aufbau ist sicher schöner.

Nur für die Facharbeit ist's wahrscheinlich besser, den Aufwand in gut dokumentierten & strukturierten Quelltext zu stecken. Die Punkte a und c (siehe oben) solltest Du aber in jedem Fall beheben.

Grüße, Ebenius

 

[Gast]

Ok danke,ich werd das noch einmal probieren mit dem CardLayout,schaden kann's ja eigentlich nicht!^^
An der Zeit mangelts auch nicht,weil wir gerade erst angefangen haben....
Die Punkte a und c hab ich schon versucht teilweise umzusetzen und ich werde mich dran begeben das komplett zu ändern.
Eine Frage noch zu c):
Meinst du das so das ich dann schreibe :

this.getContentPane().add(Haupt);
this.getContentPane().add(Knöpfe,BorderLayout.WEST);
this.getContentPane().add(Über,BorderLayout.NORTH);
this.getContentPane().add(Unter,BorderLayout.SOUTH);

oder komplett anders?

 

[Ebenius]

Meinst du das so das ich dann schreibe[...]

Im Prinzip ja. So ist's noch etwas schöner und lesbarer (In Zeile 2, Constraint mit angeben; das liest sich IMHO besser):

final Container mainPane = this.getContentPane();
mainPane.add(Haupt, BorderLayout.CENTER);
mainPane.add(Knöpfe, BorderLayout.WEST);
mainPane.add(Über, BorderLayout.NORTH);
mainPane.add(Unter, BorderLayout.SOUTH);

Viel Spaß & Erfolg, Ebenius