JMenuBar und EventHandling

[BMAS]

Könnt ihr mir weiterhelfen?

Ich habe eine JMenuBar mit ein paar JMenus und JMenuItems erstellt und hab´ den JMenuItems dann einen ActionListener zugewiesen aber die feuern kein Event ab! :confused:

Sprich wenn ich die anklicke passiert nichts...

Hier erstelle ich mein MenuBar und füge es dann meinem JFrame hinzu...

                hauptMenu.add(ansichtMenu);


	ansichtMenu.add(designMenu);

	designMenu.add("Metal");

	designMenu.add("Motif");	

	designMenu.add("Windows");


	hauptMenu.add(verbMenu);

	verbMenu.add("Connectingstring");

	verbMenu.add("Servername");


	hauptMenuBar.add(hauptMenu);

	setJMenuBar(hauptMenuBar);


	metal.addActionListener(this);

	motif.addActionListener(this);

	windows.addActionListener(this); 

[/PHP]

Und das soll dann passieren, wenn man die JMenuItems metal, motif oder windows anklickt... (die InternalFrames einfach mal ignorieren, die sind für was anderes...)

[PHP]
public void actionPerformed(ActionEvent e) {

if (e.getSource() == bEnde)
System.exit(0);

if (e.getSource() == bSuche) {

JInternalFrame jif = new JInternalFrame("Suche", false, true, false, true);
jif.getContentPane().add(new SearchPanel());
jif.setVisible(true);
jif.setPreferredSize(new Dimension(420, 220));
jif.setFrameIcon(new ImageIcon("C:/images/Find16.gif"));
desktop.add(jif);
desktop.revalidate();
}

/* if (e.getSource() == bEinstellungen) {

JInternalFrame jif =
new JInternalFrame("Einstellungen", false, true, false, true);
jif.getContentPane().add(new EinstellungPanel());
jif.setVisible(true);
jif.setPreferredSize(new Dimension(420, 220));
jif.setFrameIcon(new ImageIcon("C:/images/Preferences16.gif"));
desktop.add(jif);
desktop.revalidate();
}*/

if (e.getSource() == bDatenpflege) {

}

if (e.getSource() == bBewert) {

}

if (e.getSource() == bAdmin) {

}

//HIER GEHT´s LOS!

if (e.getSource() == metal) {

try {

UIManager.setLookAndFeel("javax.swing.plaf.metal.MetalLookAndFeel");
SwingUtilities.updateComponentTreeUI(this);

} catch (Exception ex) {
}
}
if (e.getSource() == motif) {

try {

UIManager.setLookAndFeel("com.sun.java.swing.plaf.motif.MotifLookAndFeel");
SwingUtilities.updateComponentTreeUI(this);

} catch (Exception ex) {
}
}
if (e.getSource() == windows) {

try {

UIManager.setLookAndFeel("com.sun.java.swing.plaf.windows.WindowsLookAndFeel");
SwingUtilities.updateComponentTreeUI(this);

} catch (Exception ex) {
}
}
}

Die Anzeige funktioniert einwandfrei, nur passiert wie gesagt einfach ncihts, wenn ich im JMenu auf metal, windows oder motif klicke...

Es wird erst gar kein Event gefeuert, habe ich getestet (mit nem simplen System.out.println("Hallo"); am Anfang der Methode, die wird gar nicht erst aufgerufen...)

[Jaraz]

Hallo,

Objekte vergleicht man mit if(object1.equals(object2))

Außerdem kennt Java auch else if.

Und drittens ist der Aufbau deines Menus glaube ich nicht richtig. Kann man aber schlecht sehen, da du nicht alles gepostet hasts.

Gruß Jaraz

[Snowghost]

Versuche doch einmal, die JMenuItems selbst zu erzeugen, den ActionListener hinzuzufügen und dann einem anderen JMenuItem zu übergeben.

Also so:

JMenuItem look = new JMenuItem("Look");

JMenuItem metal = new JMenuItem("Metal");

metal.addActionListener(...);

look.add(metal);

Gruß

Snowghost

[BMAS]

@Jaraz hier nochmal der ganze Code der Klasse:

import javax.swing.BorderFactory;

import java.awt.Color;

import java.awt.Dimension;

import java.awt.FlowLayout;

import javax.swing.JInternalFrame;

import java.util.StringTokenizer;

import java.awt.Insets;

import java.awt.event.ActionEvent;

import java.awt.event.ActionListener;

import javax.swing.ImageIcon;

import javax.swing.JButton;

import java.awt.GridLayout;

import javax.swing.JPanel;

import javax.swing.JSeparator;

import javax.swing.JToolBar;

import java.awt.BorderLayout;

import java.awt.GridBagConstraints;

import java.awt.GridBagLayout;

import javax.swing.UIManager;

import javax.swing.SwingUtilities;

import javax.swing.JDesktopPane;

import javax.swing.JMenu;

import javax.swing.JMenuBar;

import javax.swing.JMenuItem;


public final class HauptFrame extends javax.swing.JFrame implements ActionListener {


    private JPanel norden = new JPanel();


    private JButton bSuche;

    private JButton bDatenpflege;

    private JButton bBewert;

    private JButton bEinstellungen;

    private JButton bAdmin;

    private JButton bEnde;


    private JMenuBar hauptMenuBar = new JMenuBar();

    private JMenu hauptMenu = new JMenu("Einstellungen");

    private JMenu verbMenu = new JMenu("Verbindung");

    private JMenu ansichtMenu = new JMenu("Ansicht");

    private JMenu designMenu = new JMenu("GUI-Design");


    private JMenuItem metal = new JMenuItem("Metal");

    private JMenuItem motif = new JMenuItem("Motif");

    private JMenuItem windows = new JMenuItem("Windows");


    private TableFrame tableframe;


    private JDesktopPane desktop = new JDesktopPane();


    private JPanel buttonPanel = new JPanel();


    private JButton bOK = new JButton(new ImageIcon("C:/images/tick_blue.gif"));

    private JButton bZur = new JButton(new ImageIcon("C:/images/cross_red.gif"));


    private JSeparator sepp = new JSeparator();


public HauptFrame() {


    init();


    JToolBar toolBar = new JToolBar();

    toolBar = createToolBar(toolBar);


    JSeparator sep = new JSeparator(javax.swing.SwingConstants.HORIZONTAL);

    JSeparator sepp = new JSeparator(javax.swing.SwingConstants.HORIZONTAL);


    this.getContentPane().setLayout(new BorderLayout());


    this.getContentPane().add(norden, BorderLayout.NORTH);

    norden.setLayout(new GridBagLayout());


    GridBagConstraints csep = new GridBagConstraints();

    csep.gridx = 0;

    csep.gridy = 1;

    csep.fill = GridBagConstraints.HORIZONTAL;

    csep.weightx = 1.0;

    csep.gridwidth = 1;

    csep.anchor = GridBagConstraints.WEST;

    csep.gridwidth = GridBagConstraints.REMAINDER;


    GridBagConstraints ctoolBar = new GridBagConstraints();

    ctoolBar.gridx = 0;

    ctoolBar.gridy = 0;

    ctoolBar.weightx = 1.0;

    ctoolBar.gridwidth = 1;

    ctoolBar.anchor = GridBagConstraints.WEST;

    ctoolBar.gridwidth = GridBagConstraints.REMAINDER;


    norden.add(toolBar, ctoolBar);

    norden.add(sep, csep);


    hauptMenu.add(ansichtMenu);


	ansichtMenu.add(designMenu);

	designMenu.add("Metal");

	designMenu.add("Motif");	

	designMenu.add("Windows");


	hauptMenu.add(verbMenu);

	verbMenu.add("Connectingstring");

	verbMenu.add("Servername");


	hauptMenuBar.add(hauptMenu);

	setJMenuBar(hauptMenuBar);


	metal.addActionListener(this);

	motif.addActionListener(this);

	windows.addActionListener(this); 


    desktop.setBorder(BorderFactory.createLoweredBevelBorder());


    this.getContentPane().add(desktop, BorderLayout.CENTER);


    setVisible(true);

    this.pack();

    setResizable(true);


    setTitle("Lieferanten und Dienstleiter Datenbank, Vers. 0.9");

    this.validate();

}

public void actionPerformed(ActionEvent e) {


    if (e.getSource() == bEnde)

        System.exit(0);


    if (e.getSource() == bSuche) {


        JInternalFrame jif = new JInternalFrame("Suche", false, true, false, true);

        jif.getContentPane().add(new SearchPanel());

        jif.setVisible(true);

        jif.setPreferredSize(new Dimension(420, 220));

        jif.setFrameIcon(new ImageIcon("C:/images/Find16.gif"));

        desktop.add(jif);

        desktop.revalidate();

    }


    if (e.getSource() == bDatenpflege) {


    }


    if (e.getSource() == bBewert) {


    }


    if (e.getSource() == bAdmin) {


    }


    if (e.getSource().equals(metal)) {


        try {


            UIManager.setLookAndFeel("javax.swing.plaf.metal.MetalLookAndFeel");

            SwingUtilities.updateComponentTreeUI(this);


        } catch (Exception ex) {

        }

    }

    if (e.getSource().equals(motif)) {


        try {


            UIManager.setLookAndFeel("com.sun.java.swing.plaf.motif.MotifLookAndFeel");

            SwingUtilities.updateComponentTreeUI(this);


        } catch (Exception ex) {

        }

    }

    if (e.getSource().equals(windows)) {


        try {


            UIManager.setLookAndFeel("com.sun.java.swing.plaf.windows.WindowsLookAndFeel");

            SwingUtilities.updateComponentTreeUI(this);


        } catch (Exception ex) {

        }

    }

}


public JToolBar createToolBar(JToolBar toolBar) {


    toolBar.setOrientation(javax.swing.SwingConstants.HORIZONTAL);


    toolBar.add(bSuche);

    toolBar.add(bDatenpflege);

    toolBar.add(bBewert);

    toolBar.addSeparator();

    toolBar.add(bEinstellungen);

    toolBar.add(bAdmin);

    toolBar.addSeparator();

    toolBar.add(bEnde);

    toolBar.addSeparator();

    toolBar.setFloatable(false);


    return toolBar;

}

public void init() {


    bSuche = new JButton(new ImageIcon("C:/images/Find24.gif"));

    bEinstellungen = new JButton(new ImageIcon("C:/images/Preferences24.gif"));

    bBewert = new JButton(new ImageIcon("C:/images/BARCHART.gif"));

    bDatenpflege = new JButton(new ImageIcon("C:/images/Edit24.gif"));

    bAdmin = new JButton(new ImageIcon("C:/images/Server24.gif"));

    bEnde = new JButton(new ImageIcon("C:/images/door2.gif"));


    bSuche.setToolTipText("Suche");

    bEinstellungen.setToolTipText("Einstellungen");

    bBewert.setToolTipText("Lieferantenbewertung");

    bDatenpflege.setToolTipText(

        "Datenpflege: Erstellen, Löschen und Ändern von Datensätzen");

    bAdmin.setToolTipText("Administration: Vergabe von Benutzerrechten");

    bEnde.setToolTipText("Ende");


	desktop.setBackground(Color.gray);


    bSuche.addActionListener(this);

    bDatenpflege.addActionListener(this);

    bBewert.addActionListener(this);

    bEinstellungen.addActionListener(this);

    bAdmin.addActionListener(this);

    bEnde.addActionListener(this);

    desktop.setLayout(new FlowLayout());



}


public static void main(java.lang.String[] args) {

    new HauptFrame();

}

}

[/PHP]

Ich habe jetzt das == durch equals() ersetzt, macht aber keinen Unterschied, das Event wird noch immer nicht gefeuert. :confused:

@Snowghost

Ich werde es mal versuchen, melde mich dann...

[BMAS]

Habe den Fehler gerade gefunden, (was bin ich auch blöd!)

seht euch mal das an:

designMenu.add("Metal");

designMenu.add("Motif");

designMenu.add("Windows");

was fällt euch daran auf? Genau, das sind Strings und keine MenuItems die ich da dem Menu hinzufüge...

Trotzdem vielen Dank!

[Jaraz]

Funktioniert doch.

Gruß Jaraz

[Jaraz]

Original geschrieben von BMAS

Habe den Fehler gerade gefunden, (was bin ich auch blöd!)

seht euch mal das an:

designMenu.add("Metal");

designMenu.add("Motif");

designMenu.add("Windows");

was fällt euch daran auf? Genau, das sind Strings und keine MenuItems die ich da dem Menu hinzufüge...

Trotzdem vielen Dank!

Achso, ich dachte der ActionListener würde generell nicht funktionieren. Deswegen habe ich da gar nicht drauf geachtet.

Gruß Jaraz