Эффективный способ добавления JMenuItems

Question

ОК, так что до того, как моя программа INV удаляла пункты меню, но мне было все равно ... это слишком. Так что, если бы я ДОБАВЛЯЛ пункты меню для определенных элементов, щелкаемых правой кнопкой мыши, ВМЕСТО удаляемых каждый раз?

Итак, если вы щелкнете правой кнопкой мыши по пункту 1, вы получите «Use» и «Drop», добавленные в меню. Затем, как только вы выберете свой вариант, JMenu удалит все, так что было бы там, где мы начали. Затем, если вы щелкнете правой кнопкой мыши по ITem 2, он добавит «Использовать» и «Отмена». Видишь, куда я иду?

Я пытался сделать это сам, но я просто не могу понять, как это сделать - например, чтобы добавить новый JMenuItem, вам нужно сделать это:

popup.add(item = new JMenuItem("Cancel"));
item.addActionListener(menuListener);

и, как видите, добавьте список действий. Я не могу сделать это под if (actItemx == "Item 1") {, так что ... что мне делать?

В любом случае, вот что у меня есть:

import java.awt.*;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.event.MouseAdapter;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.*;
import javax.swing.border.*;
import javax.swing.event.*;

public class inv extends JApplet implements MouseListener
{
    public JList listbox;
    public JPopupMenu popup;
    public JMenuItem item;

    public void init()
    {
        ActionListener menuListener = new ActionListener()
        {
            public void actionPerformed(ActionEvent event)
            {
                String invAction = event.getActionCommand();


                int itemSelect = listbox.getSelectedIndex();
                Object actItem = listbox.getModel().getElementAt(itemSelect);

                System.out.println("Popup menu item [" + invAction + "] [ " + actItem + " ] was pressed.");
            }
        };

        popup = new JPopupMenu();

        popup.add(item = new JMenuItem("Use"));
        item.addActionListener(menuListener);

        popup.add(item = new JMenuItem("Drop"));
        item.addActionListener(menuListener);

        popup.add(item = new JMenuItem("Cancel"));
        item.addActionListener(menuListener);



        String listData[] =
        {
            "Item 1","Item 2","Item 3","Item 4"
        };

        listbox = new JList( listData );
        listbox.addMouseListener( new MouseAdapter()
        {
            public void mousePressed(MouseEvent e)
            {
                if ( SwingUtilities.isRightMouseButton(e) )
                {
                    System.out.println("Row: " + getRow(e.getPoint()));
                    listbox.setSelectedIndex(getRow(e.getPoint()));
                }
            }
        }
        );

        listbox.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
        add(listbox);
        listbox.setVisible(true);
        listbox.setFocusable(false);


        listbox.addMouseListener(new MousePopupListener());
    }

    class MousePopupListener extends MouseAdapter
    {
        public void mousePressed(MouseEvent e)
        {
            checkPopup(e);
        }

        public void mouseClicked(MouseEvent e)
        {
            checkPopup(e);
        }

        public void mouseReleased(MouseEvent e)
        {
            checkPopup(e);
        }

        private void checkPopup(MouseEvent e)
        {
            if (e.isPopupTrigger())
            {

                int itemSelectx = listbox.getSelectedIndex();
                Object actItemx = listbox.getModel().getElementAt(itemSelectx);
                System.out.println("You pressed on " + actItemx);

            if (actItemx == "Item 1") {
                System.out.println("Removed cancel for " + actItemx);
                popup.remove(itemSelectx); // So upon right-click on Item 1, you won't see "Cancel" menu.
            }

                popup.show(inv.this, e.getX(), e.getY());
                popup.revalidate();
            }
        }
    }

    private int getRow(Point point)
    {
        return listbox.locationToIndex(point);
    }

    public void mouseEntered(MouseEvent e)
    {
    }

    public void mouseReleased(MouseEvent e)
    {
    }

    public void mousePressed(MouseEvent e)
    {
    }

    public void mouseClicked(MouseEvent e)
    {
    }

    public void mouseExited(MouseEvent e)
    {
    }
}

Answer 1

Как насчет настройки различных меню JPopup для каждого типа элемента.Под этим я подразумеваю, что у вас есть что-то похожее на это:

public JPopupMenu useDropPopup;
public JPopupMenu useCancelPopup;

public void init() {
   ActionListener menuListener = new ActionListener()
        {
            public void actionPerformed(ActionEvent event)
            {
                String invAction = event.getActionCommand();


                int itemSelect = listbox.getSelectedIndex();
                Object actItem = listbox.getModel().getElementAt(itemSelect);

                System.out.println("Popup menu item [" + invAction + "] [ " + actItem + " ] was pressed.");
            }
        };

   useDropPopup = new JPopupMenu();
   useCancelPopup = new JPopupMenu();

   JMenuItem useMenuItem = new JMenuItem("Use");
   useMenuItem.addActionListener(menuListener);
   JMenuItem dropMenuItem = new JMenuItem("Drop");
   dropMenuItem.addActionListener(menuListener);
   JMenuItem cancelMenuItem = new JMenuItem("Cancel");
   cancelMenuItem.addActionListener(menuListener);

   useDropPopup.add(useMenuItem);
   useDropPopup.add(dropMenuItem);

   useCancelPopup.add(useMenuItem);
   useCancelPopup.add(cancelMenuItem);

   // ... etc bring up the appropriate popup depending on the item.
}

Кроме того, вам не следует назначать JMenuItem для элемента внутри вызова метода.Это плохая практика.Также рассмотрите возможность использования различных списков действий для каждого пункта меню, чтобы вы могли разделить функциональный код каждого пункта меню, например:

useMenuItem.addActionListener(new ActionListener() {
   public void actionPerformed(ActionEvent event) {
      useMenuAction(event);
   }
};

// ... after the init method

public void useMenuAction(ActionEvent evt) {
   // Add specific use menu code here.
}

Answer 2

Это в основном дополнение к ответу от jluzwick:

Вместо создания JMenuItem и добавления к нему ActionListener, вы можете использовать Action - это в основном комбинация ActionListener с именемопционально Icon и некоторые другие свойства.(Большинство просто расширяется от AbstractAction, переопределяя actionPerformed-Method.)

Затем добавьте Action в ваше JMenu, и он сам создаст JMenuItem.(Вы можете также использовать тот же объект Action в других местах, таких как кнопки, «обычная» строка меню и т. Д.)