Исключение в теме "AWT-EventQueue-1"

Question

Я сделаю это быстро и просто.

Я получаю ошибку NPE ...

Row: 0
You pressed on Item 1
Removed cancel for Item 1
Exception in thread "AWT-EventQueue-1" java.lang.NullPointerException
        at inv$MousePopupListener.checkPopup(inv.java:118)
        at inv$MousePopupListener.mouseReleased(inv.java:104)
        at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:2
73)
        at java.awt.Component.processMouseEvent(Component.java:6267)
        at javax.swing.JComponent.processMouseEvent(JComponent.java:3267)
        at java.awt.Component.processEvent(Component.java:6032)
        at java.awt.Container.processEvent(Container.java:2041)
        at java.awt.Component.dispatchEventImpl(Component.java:4630)
        at java.awt.Container.dispatchEventImpl(Container.java:2099)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4577
)
        at java.awt.LightweightDispatcher.processMouseEvent(Container.java:4238)

        at java.awt.LightweightDispatcher.dispatchEvent(Container.java:4168)
        at java.awt.Container.dispatchEventImpl(Container.java:2085)
        at java.awt.Component.dispatchEvent(Component.java:4460)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:599)
        at java.awt.EventDispatchThread.pumpOneEventForFilters(EventDispatchThre
ad.java:269)
        at java.awt.EventDispatchThread.pumpEventsForFilter(EventDispatchThread.
java:184)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThre
ad.java:174)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:169)

        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:161)

        at java.awt.EventDispatchThread.run(EventDispatchThread.java:122)

Я искал все выше и ниже и не знаю, как это сделать.исправить это:

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 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);


        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());

        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.
}
    }



    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

Научитесь читать свою трассировку стека:

Exception in thread "AWT-EventQueue-1" java.lang.NullPointerException
        at inv$MousePopupListener.checkPopup(inv.java:118)
        at inv$MousePopupListener.mouseReleased(inv.java:104)
        at java.awt.AWTEventMulticaster.mouseReleased(AWTEventMulticaster.java:2
73)

Там написано, что в строке 118

было исключение NullPointerException

...checkPopup(inv.java:118)....

Который, в свою очередь, был вызван строкой 104:

... useReleased(inv.java:104).. 

Если вы видите эти строки, у вас будет:

Как видите, в этой строке вы вызываете:

popup.remove(itemSelectx);

Единственный способ, которым эта строка могла вызвать исключение NullPointerException, это то, что popup является нулевым, поскольку null.remove.. выдает исключение NullPointerException.

Итак, инициализируйте ваше всплывающее окно с тем, что имеет смысл.

Надеюсь, это поможет вам в этом и в будущих исключениях.

Answer 2

Я думаю, что всплывающее окно JPopUpMenu null, вы должны инициализировать его, используя:

popup = new JPopUpMenu();

или назначить его одному из других определенных вами всплывающих окон ... исключение достаточно явное, онодаже говоря вам, какая строка содержит код, вызывающий оскорбления!

Answer 3

Я не вижу назначения для popup где-либо в этом коде, поэтому его значение по умолчанию всегда будет null.

Вы забыли сказать

popup = useDropPopup;

или

popup = useCancelPopup;

или, может быть,

popup = new JPopupMenu();

где-нибудь?В противном случае эта подпрограмма просто не знает, о чем popup вы говорите.

Кроме того, поскольку вы мутируете popup, удаляя элемент, позаботьтесь о том, чтобы он был только что создан / скопирован JPopupMenu, или что вы положили удаленный предмет обратно в тех случаях, когда вы хотите, чтобы он был там.