Problem mit JPopupMenu

[claudy79]

Hi Leutz,

ich programmiere noch nicht so lange mit Java. Und nun hat sich ein Problem aufgetan welches ich nicht verstehe. Ich versuche in einem Baum, einzelne Knoten anzuklicken(also in einem Diagramm) und möchte dann, dass dieser Knoten zB gelöscht wird. Nun habe ich mir ne Klasse geschrieben:

import java.awt.*;
import java.awt.event.*;

import javax.swing.JMenuItem;
import javax.swing.JPopupMenu;


public class MyPopupMenu extends JPopupMenu{
		JPopupMenu jM = new JPopupMenu();
		
		public MyPopupMenu(){
			
			JMenuItem mi1= new JMenuItem("löschen");
			mi1.addActionListener(new java.awt.event.ActionListener()
				 {
				  public void actionPerformed(ActionEvent e)
				      {
				        mi1_actionPerformed(e);
				      }
				    });
			jM.add(mi1);
			
			JMenuItem mi2 = new JMenuItem("gehe zu");
			mi2.addActionListener(new java.awt.event.ActionListener()
				{
				 public void actionPerformed(ActionEvent e)
				      {
				        mi2_actionPerformed(e);
				      }
				    });
		    jM.add(mi2);
		 
		    JMenuItem mi3 = new JMenuItem("kopieren");
		    mi3.addActionListener(new java.awt.event.ActionListener()
					{
					 public void actionPerformed(ActionEvent e)
					      {
					        mi3_actionPerformed(e);
					      }
					    });
		    jM.add(mi3);
		 
		    JMenuItem mi4 = new JMenuItem("einfuegen");
		    mi4.addActionListener(new java.awt.event.ActionListener()
					{
					 public void actionPerformed(ActionEvent e)
					      {
					        mi4_actionPerformed(e);
					      }
					    });
		    jM.add(mi4);
		}
		
		public void mi1_actionPerformed(ActionEvent e){
			
		}
		public void mi2_actionPerformed(ActionEvent e){
			
		}
		public void mi3_actionPerformed(ActionEvent e){
			
		}
		public void mi4_actionPerformed(ActionEvent e){
			
		}



}

und rufe diese in ner anderen Klasse mit MouseListener drin auf, also e ist der MouseListener:

if(e.getClickCount()==1){
        	if(e.getButton()==3){
        		MyPopupMenu test= new MyPopupMenu();
        		test.setToolTipText("test");
        		test.setPopupSize(200,200);
        		test.show(e.getComponent(),e.getX(),e.getY());
        		test.setVisible(true);
        		
        	}
}

Nun malt er mir da ein graues Fenster hin und ich weiss nich warum??? Knoten im Kopf!

Mfg Claudia


L-ectron-X hat diesen Beitrag am 25.01.2006 um 14:56 Uhr editiert.
Code-Tags eingefügt

 

[L-ectron-X]

Du musst die mouseRelease()-Methode überschreiben. Z.B. so:

//im Konstruktor
this.addMouseListener(new MouseAdapter() {
  public void mouseRelease(MouseEvent e) {
    if(e.isPopupTrigger()) {
      myPopupMenu.show(e.getComponent(), e.getX(), e.getY());
    }
  }
});

Links dazu:
http://java.sun.com/j2se/1.5.0/docs/api/java/awt/event/MouseEvent.html#isPopupTrigger()
http://java.sun.com/j2se/1.5.0/docs/api/javax/swing/JPopupMenu.html#show(java.awt.Component, int, int)

 

[claudy79]

Du musst die mouseRelease()-Methode überschreiben. Z.B. so:

//im Konstruktor
this.addMouseListener(new MouseAdapter() {
  public void mouseRelease(MouseEvent e) {
    if(e.isPopupTrigger()) {
      myPopupMenu.show(e.getComponent(), e.getX(), e.getY());
    }
  }
});

Links dazu:
http://java.sun.com/j2se/1.5.0/docs/api/java/awt/event/MouseEvent.html#isPopupTrigger()
http://java.sun.com/j2se/1.5.0/docs/api/javax/swing/JPopupMenu.html#show(java.awt.Component, int, int)

Naja es geht mir erstmal darum, dass das PopupMenu aufgeht wenn ich die rechte Maustaste drücke. und zwar ordentlich, also nich blos nen graues Fenster. ???? Weil wenn nich mal das passiert, brauch ich mit dem release ja noch nix machen, oder?

Mfg Claudia

 

[flanker]

Hallo.
etwa so sollte es gehen:

Step-1:
JTree (jTree) und JPopupMenu (jpp) und JMenuItem-s, z.B.: JMenuItem (jppRem) bauen
am besten als Instance-Variablen (Zumindest jTree und jpp)

// Step-2:
jppRem.addActionListener(new ActionListener() {
  public void actionPerformed(ActionEvent ae) {
		TreeNode node = getSelectedNode(); //Holen, siehe API (Achtung, kann null sein)
		node.removeFromParent();
		// gegebenenfalls updaten und 
		// vielleicht noch nächste oder vorherige Node selektieren
  }
});
jpp.add(jppRem);


// Step-3:
jTree.addMouseListener(new MouseAdapter() {
  public void mouseReleased(final MouseEvent me) {
    if (me.isPopupTrigger()) {
      int row = jTree.getRowForLocation(me.getX(), me.getY());
      if (row >= 0) {
        jTree.setSelectionRow(row); //Mit Rechtsklick selektieren
        jpp.show(jTree, me.getX(), me.getY());
      }
    }
  }
});

 

[L-ectron-X]

@claudy79: Hast Recht, habe ich noch übersehen.

Nicht

jM.add(menuItem);

sondern

this.add(menuItem);

Und die Instanziierung von JPopupMenu zu Beginn deiner Klasse löschen.
Du erbst doch bereits von JPopupMenu.

 

[claudy79]

@claudy79: Hast Recht, habe ich noch übersehen.

Nicht

jM.add(menuItem);

sondern

this.add(menuItem);

Und die Instanziierung von JPopupMenu zu Beginn deiner Klasse löschen.
Du erbst doch bereits von JPopupMenu.

Ja danke das werde ich mal testen.

Mfg Claudia :wink:

 

[claudy79]

super das funzt, iss ja auch klar man sagt zwar doppelt hält besser, aber hier passt wohl eher doppelt gemoppelt