Action Listener Funktioniert nicht

[schili85]

ich habe ein tree mit popup Menü. Das popup Menü öffnet sich auch aber es reagiert nicht wenn ich einen menüpunkt auswähle. hab es auch schon im DebugModus durchlaufen lassen aber der ActionListener springt nicht an.

Mein Code

public class Sca1View extends FrameView implements ActionListener{

.......


public void createPopupMenu() {
        JMenuItem menuItem;
 
       //Create the popup menu.
        JPopupMenu popup = new JPopupMenu();
        menuItem = new JMenuItem(New_Project);
        menuItem.setActionCommand(New_Project);
        menuItem.addActionListener(this);
        popup.add(menuItem);
        menuItem = new JMenuItem("New_Build");
        menuItem.setActionCommand(New_Build);
        menuItem.addActionListener(this);
        popup.add(menuItem);
        menuItem = new JMenuItem("Delete");
        menuItem.setActionCommand(Delete);
        menuItem.addActionListener(this);
        popup.add(menuItem);
        
        //Add listener to the text area so the popup menu can come up.
        MouseListener popupListener = new PopupListener(popup);
        jTree1.addMouseListener(popupListener);

}

  public void actionPerformed(ActionEvent e,String Wert) {
    	String url = "jdbc:postgresql://localhost:5432/QualityDB"
                + "?user=postgres&password=xxx";
        String NodeName = null;
       Integer depth = null;
       
       
    	DefaultMutableTreeNode selectedNode = (DefaultMutableTreeNode)
    		jTree1.getLastSelectedPathComponent();
 
    	if (selectedNode == null){
    		return;
    	}
    	if (e.getActionCommand().equals("Delete")){
    		if (selectedNode.getParent() != null)
                    depth = selectedNode.getChildCount();
                    if (depth == 1) 
                    NodeName = "'" + selectedNode.toString() + "'";
                    try
                        {
                        Class.forName("org.postgresql.Driver");
                        Connection conn = DriverManager.getConnection(url);
                        stmt = conn.createStatement();
                        String query = "DELETE FROM project WHERE(project_name = "+NodeName+")";
                        stmt.executeQuery(query);
                                    
                        }
                        catch (Exception a){
                          jLabel1.setText("Fehler: " + a.getMessage());
                        }        
                      if (depth == 0)
                       NodeName = "'" + selectedNode.toString() + "'";
                        try
                            {
                            Class.forName("org.postgresql.Driver");
                            Connection conn = DriverManager.getConnection(url);
                            stmt = conn.createStatement();
                            String query = "DELETE FROM build WHERE(project_name = "+NodeName+")";
                            stmt.executeQuery(query);
                                    
                             }
                            catch (Exception a){
                            jLabel1.setText("Fehler: " + a.getMessage());
                             }          
                   return;
            }
    		DefaultMutableTreeNode newNode = new DefaultMutableTreeNode("New");
    	           
                if (e.getActionCommand().equals("New Project")){
    		DefaultMutableTreeNode parent = (DefaultMutableTreeNode)
    			selectedNode.getParent();
    		if (parent != null){
    			int selectedIndex = parent.getIndex(selectedNode);
    //			model.insertNodeInto(newNode, parent, selectedIndex+1);
    //
                        
                        
    //		}
    	}
    	else if (e.getActionCommand().equals("New Build")){
    //		model.insertNodeInto(newNode, selectedNode, selectedNode.getChildCount());
	  	}
    	    	
    //	

    //    Code um neuen Build anzulegen!!!       
    //            
                
    }
    }

 

[KimKaze]

ich weiss nicht genau wie das aussieht, aber musst du evtl jedem menuitem den actionlistener zuweisen... du sprichst ja da oben kein menuitem an sondern sagst einfach nur menuitem.add
gibts da ne möglichkeit wie
menuitem.delete.add
oder
New_Project.add ???

 

[Marco13]

Dass bei http://www.java-forum.org/de/topic71170_popup-men-ffnen.html niemand geantwortet hat, liegt nicht an Faulheit (unsererseits) oder mangelnder Auforpferungsbereitschaft (unsererseits), sondern daran, dass die Frage unbeantwortbar gestellt war. Zumindest müßte man wissen, was "PopupListener" ist. Welche DriverConnections du verwendest, und wie deine SQL-Statements aussehen, interessiert aber nicht nur am Rande, sondern überhaupt nicht.

 

[schili85]

Also Menuitem.delete und auch New_Project.add klappt nicht!

 

[Marco13]

Welche DriverConnections du verwendest, und wie deine SQL-Statements aussehen, interessiert aber nicht nur am Rande, sondern überhaupt nicht.

... das Passwort könnte aber ggf. schon den einen oder anderen interessieren :shock: :wink:

 

[KimKaze]

hmm du willst irgendwann mal wieder deine menuitems aufrufen, wie denn wenn du die ersten beiden überschreibst?

public void createPopupMenu() {
        JMenuItem menuItem1, menuItem2, menuItem3 ;

       //Create the popup menu.
        JPopupMenu popup = new JPopupMenu();
        menuItem1 = new JMenuItem(New_Project);
        menuItem1.setActionCommand(New_Project);
        menuItem1.addActionListener(this);
        popup.add(menuItem1);
        menuItem2 = new JMenuItem("New_Build");
        menuItem2.setActionCommand(New_Build);
        menuItem2.addActionListener(this);
        popup.add(menuItem2);
        menuItem3 = new JMenuItem("Delete");
        menuItem3.setActionCommand(Delete);
        menuItem3.addActionListener(this);
        popup.add(menuItem3);

und aus

 if (e.getActionCommand().equals("Delete")){

müsste dann werden

 if (e.getActionCommand().equals("menuitem1")){

denn du hast kein actioncommand das delete heißt... da gibts evtl. auch ne möglichkeit aber die sieht nicht so aus.. ich hab leider keine unterlagen griffbereit aber versuchs mal so

 

[Marco13]

@KimKaze: Eine MenuItem Referenz reicht.
@Topic:
public void actionPerformed(ActionEvent e,String Wert) {
sieht schonmal falsch aus, weil es eigentlich nur
public void actionPerformed(ActionEvent e) {
heißen dürfte. Und sonst... naja, ihr kriegt das schon irgendwie hin.