Swing Problem beim 2. Öffnen eines JFrames

[coree]

Hallo,
ich habe ein Problem mit meinem Einstellungs-Frame.
Ich starte den JFrame über ein JMenuItem in meiner JMenuBar.
Wenn ich mein Programm gestartet habe und die Options öffne funktioniert noch alles einwandfrei. Wenn ich dann allerdings schließe und neu aufrufe, bekomme ich einen Darstellungsfehler.
Zur Veranschaulichung mal 2 Bilder:

1. Aufruf

2. Aufruf

Auf Bild 1 wird die JList noch richtig dargestellt (hier noch ohne einträge), beim 2. Bild ist die JList plötzlich verschwunden und nur noch das ScrollPane wird dargestellt (erkennbar an der Border).

Aufruf Options durch menuItem

        menuItem = new JMenuItem("Einstellungen");
        menuItem.setAccelerator(KeyStroke.getKeyStroke(
                KeyEvent.VK_1, ActionEvent.ALT_MASK));
        menuItem.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(final ActionEvent evt) {
                new Options();
            }
        });

Options

public class Options extends JFrame {

    private static JFrame options = new JFrame("Einstellungen");
    
    public static JFrame getOptions() {
        return options;
    }

    public Options() {

        options.setPreferredSize(new Dimension(510, 400));
        options.setResizable(false);
        options.setDefaultCloseOperation(JFrame.HIDE_ON_CLOSE);

        final OptionsFolder folderPanel = new OptionsFolder();
        final OptionsMail mailPanel = new OptionsMail();

        final JTabbedPane tabbedPane = new JTabbedPane();
        tabbedPane.setPreferredSize(new Dimension(400, 300));
        tabbedPane.addTab("Ordner", folderPanel);
        tabbedPane.addTab("Mail", mailPanel);

        options.getContentPane().add(tabbedPane, BorderLayout.CENTER);
        options.pack();
        options.setVisible(true);

    }
}

closeButton in OptionsFolder

        final JButton closeButton = new JButton(new ImageIcon(getClass()
                .getResource("images/close.jpg")));
        closeButton.setPreferredSize(new Dimension(100, 31));
        closeButton.setMinimumSize(new Dimension(100, 31));
        closeButton.setMaximumSize(new Dimension(100, 31));
        closeButton.setRolloverEnabled(true);
        closeButton.setRolloverIcon(new ImageIcon(getClass()
                .getResource("images/close_hover.jpg")));
        closeButton.setCursor(new Cursor(Cursor.HAND_CURSOR));
        closeButton.setFocusPainted(false);
        closeButton.setContentAreaFilled(false);
        closeButton.setBorder(null);
        closeButton.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent evt) {
                Options.getOptions().dispatchEvent(new WindowEvent(
                        Options.getOptions(), WindowEvent.WINDOW_CLOSING));
            }
        });

Evtl kann mir jemand dabei weiter helfen. ich hab keine ahnung woran es liegen könnte oder gar was ich zu tun hab.
danke schonmal

 

[KrokoDiehl]

Ich weiß nciht woran es genau liegt, aber was mir spanisch vorkommt, ist der ActionListener:

        menuItem.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(final ActionEvent evt) {
                new Options();
            }
        });

Hier wird jedes Mal der Konstruktor von Options durchlaufen. Auch wenn das dortige Objekt statisch ist, wird der Frame immer neu belegt.
Erzeuge dein Einstellungsfenster einmalig und zeig es dann nur an, anstatt es immer wieder aufzubauen. Ich benutze in solchen Fällen meist etwas á la:

public class MainClass
{
    private JFrame options = null;
    ...
    
    public JFrame getOptionsDialog()
    {
        // lazy creation:
        if (options == null)
        {
            options = new Options();
            //sonstige (einmalige) Intialisierugnen
        }

        return options;
    }    

    ...

        menuItem.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent event)
            {
                final JFrame frame = getOptionsDialog();
                frame.setVisible(true);
            }
        });
}

Damit hast du nur ein Einstellungsfenster und es wird immer da gleiche angezeigt. Vllt hilft das.

 

[Marco13]

Statt da einen ominösen Event zum Schließen hinzuschicken könnte man vielleicht getOptions().setVisible(false) machen...

 

[coree]

Erzeuge dein Einstellungsfenster einmalig und zeig es dann nur an, anstatt es immer wieder aufzubauen. Ich benutze in solchen Fällen meist etwas á la:

public class MainClass
{
    private JFrame options = null;
    ...
    
    public JFrame getOptionsDialog()
    {
        // lazy creation:
        if (options == null)
        {
            options = new Options();
            //sonstige (einmalige) Intialisierugnen
        }

        return options;
    }    

    ...

        menuItem.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent event)
            {
                final JFrame frame = getOptionsDialog();
                frame.setVisible(true);
            }
        });
}

Damit hast du nur ein Einstellungsfenster und es wird immer da gleiche angezeigt. Vllt hilft das.

Hm... danke schonmal, aber so ganz funktioniert dies noch nicht. wenn ich die Options nun öffne, kommt der Optionsframe wie gewünscht und noch ein 2. leerer Frame. wenn ich nun beide schließe und die options erneut aufrufen möchte, kommt nur noch der 2. leere frame.

 

[Marco13]

Steht da noch irgendwoanders ein "new Options()" ?

 

[coree]

ne, ich post mal die komplette menuBar, evtl fällt dann irgendwann auf:

public class MenuBar {
    private JFrame options = null;

    public JFrame getOptionsDialog()
    {
        // lazy creation:
        if (options == null)
        {
            options = new Options();
            //sonstige (einmalige) Intialisierugnen
        }

        return options;
    }

   /**
    * Erstellt eine Menüleiste.
    * @return menuBar
    */
   public final JMenuBar createMenuBar() {
        JMenuBar menuBar;
        JMenuItem menuItem;
        JMenu menu;

        menuBar = new JMenuBar();
        menuBar.setBackground(new Color(239, 239, 239));
        menuBar.setBorder(BorderFactory.createMatteBorder(
                                    0, 0, 1, 0, new Color(34, 34, 34)));

        menu = new JMenu();
        menu.setText("Datei");
        menu.setPreferredSize(new Dimension(45, 20));
        menu.setHorizontalAlignment(SwingConstants.CENTER);
        menu.setHorizontalTextPosition(SwingConstants.CENTER);
        menu.setIconTextGap(0);
        menu.setMnemonic(KeyEvent.VK_D);
        menuBar.add(menu);

        menuItem = new JMenuItem("Einstellungen");
        menuItem.setAccelerator(KeyStroke.getKeyStroke(
                KeyEvent.VK_1, ActionEvent.ALT_MASK));
        menuItem.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(final ActionEvent evt) {
                final JFrame frame = getOptionsDialog();
                frame.setVisible(true);
            }
        });
        menu.add(menuItem);

        /*
        menu.addSeparator();
        menuItem = new JMenuItem("Starte Verbindung");
        menuItem.addActionListener(new ActionListener() {
        @Override
        public void actionPerformed(final ActionEvent evt) {
            newEdgeActionPerformed(evt);
        }
        });
        menu.add(menuItem);*/

        menu = new JMenu("Hilfe");
        menu.setMnemonic(KeyEvent.VK_H);
        menuBar.add(menu);

        return menuBar;
    }
    /*private void newEdgeActionPerformed(final ActionEvent evt) {
       EdgePeer newEdge = new EdgePeer("");
       newEdge.start();
    }*/
    /**
     * Returns an ImageIcon, or null if the path was invalid.
     * @param path Pfad zum Icon
     * @return ImageIcon
     */
    protected static ImageIcon createImageIcon(final String path) {
        final java.net.URL imgURL = MenuBar.class.getResource(path);
        if (imgURL != null) {
            return new ImageIcon(imgURL);
        } else {
            ErrorHandling.error("MenuBar", "createImageIcon", 0,
                        "Konnte Datei " + path + "nicht finden.");
            return null;
        }
    }
}

Der Code des Frames selbst sieht so aus:

public class Options extends JFrame {

    /**
     * Der Frame des Einstellungsfensters.
     * @see Options()
     * @see ButtonListener()
     */
    private static JFrame options = new JFrame("Einstellungen");
    
    /**
     * Get-Methode zur Übergabe des Frames der Optionen.
     * @return options
     */
    public static JFrame getOptions() {
        return options;
    }

    /**
     * Einstellungsfenster. 
     */
    public Options() {

        options.setPreferredSize(new Dimension(510, 400));
        options.setResizable(false);
        options.setDefaultCloseOperation(JFrame.HIDE_ON_CLOSE);

        final OptionsFolder folderPanel = new OptionsFolder();
        final OptionsMail mailPanel = new OptionsMail();

        final JTabbedPane tabbedPane = new JTabbedPane();
        tabbedPane.setPreferredSize(new Dimension(400, 300));
        tabbedPane.addTab("Ordner", folderPanel);
        tabbedPane.addTab("Mail", mailPanel);

        options.getContentPane().add(tabbedPane, BorderLayout.CENTER);
        options.pack();
        options.setVisible(true);

    }
}

oder könnte der fehler mit der nicht angezeigten liste bei Options liegen, da ich dort auch beim aufruf der options jedes mal new OptionsFolder() quasi neu mache.

 

[Marco13]

Niemand weiß, was ein OptionsFolder ist. Poste am besten mal ein KSKB

 

[coree]

in OptionsFolder oder OptionsMail werden die zwei Tabs des TabbedPanes erstellt:

public class OptionsFolder extends JPanel {

    /**
     * Einstellungsfenster.
     */
    public OptionsFolder() {

        super(new GridBagLayout());
        ....
   }

}

 

[Michael...]

Könnte an der Art und Weise liegen wie das programmiert ist:
statische Variablen: private static JFrame options = new JFrame("Einstellungen");
--> unschön
Options erbt von JFrame wird aber nie als solcher genutzt. warum?
Warum erzeugst Du nicht einmalig ein Frameobjekt und blendest es bei Bedarf ein und wieder aus?

 

[coree]

Könnte an der Art und Weise liegen wie das programmiert ist:
statische Variablen: private static JFrame options = new JFrame("Einstellungen");
--> unschön
Options erbt von JFrame wird aber nie als solcher genutzt. warum?
Warum erzeugst Du nicht einmalig ein Frameobjekt und blendest es bei Bedarf ein und wieder aus?

Hab nun deinen Code zum Laufen gebracht und auch, dass sich das Fenster beim erneuten Öffnen richtig verhält.
hab an OptionsFolder und OptionsMail jeweils ne Referenz auf den OptionsFrame gegeben und dann per setVisible(false) einfach ausgeblendet.
danke 🙂