Nested Menues und ActionListener

[chris1507]

Hallo zusammen,

ich habe folgendes Probleme mit Nested Menus und dessen Action Listener in Swing.

Folgende Bsp Klasse:

public class popup1 extends JPopupMenu implements ActionListener
{
//..

public getMenu()
{
JMenuItem jmi;

jmi=new JMenuItem("test1");
jmi.addActionListener(this);
add(jmi);

JMenu submenu;
submenu=new JMenu("untermenue1");
submenu.addActionListener(this);
submenu.add("u1");
submenu.addActionListener(this);

add(submenu);


}
public void actionPerformed(ActionEvent e) 
{
    String command = e.getActionCommand();
 
if(command.equals("test1"))
....

 if(command.equals("u1"))
....
}

}

Das ist doch richtige Vorgehensweise um einen ActionListener zu einem Nested Menu zu adden oder?

Das Problem dabei ist, dass nur auf den Eintrag test1 reagiert wird. Auf die Untermenues erfolgt keine Reaktion. Im Debug Modus springt er auch nicht mal in die Methode actionPerformed rein.

Warum nicht? Wer kann mir helfen? :meld:

 

[chris1507]

kann mir niemand helfen?

 

[Beni]

Hallo.

Den ActionListener interessiert es überhaupt nicht, wie dein Menü aufgebaut ist (nichtmal, ob du ein Menü, oder z.B. Buttons verwendest...).
Du musst den Listener direkt bei dem JMenuItem hinzufügen, welches du anklicken kannst.

Also z.B. für das "ul":

JMenuItem ulItem = submenu.add( "ul" );
uiItem.addActionListener( this );

P.S. vielleicht sind auch mehrere ActionListener praktischer, für jedes MenuItem einen. Dann kannst du dir die "if"'s sparen.

 

[L-ectron-X]

@chris1507: Nee, dein Code ist komplett Quark. Schau dir mal das Erstellen von Menüs noch mal an:
http://www.galileocomputing.de/openbook/javainsel5/javainsel15_018.htm#Xxx999379
http://www.galileocomputing.de/openbook/javainsel5/javainsel15_018.htm#Xxx999496

 

[chris1507]

sorry aber des geht leider nicht. Bei deinem Bsp Beni ist kein Untermenue mehr da.

Kann mir jemand ein funktonierendes Bsp posten indem auch die Untermenues mit dem Listener verbungen sind?

Danke

 

[chris1507]

Laut http://javaalmanac.com/egs/javax.swing/NestedPopups.html
wird so ein nested menu erstellt, was auch geht. Allerdings klappt der Verbund mit dem ActionListener nicht. Warum??

final JPopupMenu popupMenu = new JPopupMenu();
    
    // Create a submenu with items
    JMenu submenu = new JMenu("SubMenu1");
    submenu.add(action1);
    submenu.add(action2);
    
    // Add submenu to popup menu
    popupMenu.add(submenu);

 

[Beni]

Weil "action" kein ActionListener ist! Sowas könnte man in der API nachschlagen...

***

So kann man das mit einem ActionListener lösen. Hier wird nur einer verwendet, aber du kannst selbst mehrere verschiedene implementieren.

package forum;

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseAdapter;
import java.awt.event.MouseEvent;

import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JPanel;
import javax.swing.JPopupMenu;

public class Test {
	public static void main(String[] args) {
		JPanel panel = new JPanel();
		JFrame frame = new JFrame();
		frame.add( panel );
		frame.setSize( 500, 500 );
		frame.setLocationRelativeTo( null );
		
		final JPopupMenu popup = makePopup();
		
		panel.addMouseListener( new MouseAdapter(){
			public void mouseClicked( MouseEvent e ){
				popup.show( e.getComponent(), e.getX(), e.getY() );
			}
		});
		
		frame.setDefaultCloseOperation( JFrame.EXIT_ON_CLOSE );
		frame.setVisible( true );
	}
	
	public static JPopupMenu makePopup(){
		ActionListener listener = new ActionListener(){
			public void actionPerformed( ActionEvent e ){
				System.out.println( e.getActionCommand() );
			}
		};
		
		JPopupMenu menu = new JPopupMenu();
		menu.add( "toplevel" ).addActionListener( listener );
		
		JMenu subA = new JMenu( "sub A");
		JMenu subB = new JMenu( "sub B");
		
		menu.add( subA );
		menu.add( subB );

		// "add" gibt gerade ein JMenuItem zurück, da kann man den Listener hinzufügen.
		subA.add( "sub A 1" ).addActionListener( listener );
		subA.add( "sub A 2" ).addActionListener( listener );
		subB.add( "sub B 1" ).addActionListener( listener );
		subB.add( "sub B 2" ).addActionListener( listener );
		
		return menu;
	}
}

 

[chris1507]

Danke vielmals. jetzt funktonierts. Bin a bissle auf dem Schlauch gestanden.