java - 我在切换菜单中的项目时遇到问题

Question

所以我有一个菜单，它会在每个项目上调用一个项目监听器。

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

    public class GuessWindow extends Frame implements Constants
    {
     MenuBar menuBar = new MenuBar();
     GuesstimateGraph anApp;
     GuessDoc guess;
     ScrollPane drawArea = null;
     int addType;

     CheckboxMenuItem Yaxis = new CheckboxMenuItem("Y-axis", DEFAULT_TYPE==YAXIS);
     CheckboxMenuItem Xaxis = new CheckboxMenuItem("X-axis", DEFAULT_TYPE==XAXIS);
     CheckboxMenuItem Zaxis = new CheckboxMenuItem("Z-axis", DEFAULT_TYPE==ZAXIS);
     CheckboxMenuItem Yaxislabel = new CheckboxMenuItem("Y label", DEFAULT_TYPE==YLABEL);
     CheckboxMenuItem Xaxislabel = new CheckboxMenuItem("X label", DEFAULT_TYPE==XLABEL);
     CheckboxMenuItem Zaxislabel = new CheckboxMenuItem("Z label", DEFAULT_TYPE==ZLABEL);
     CheckboxMenuItem axisbreak = new CheckboxMenuItem("Break", DEFAULT_TYPE==BREAK);

     CheckboxMenuItem select = new CheckboxMenuItem("Select", DEFAULT_TYPE==SELECT);
     CheckboxMenuItem delete = new CheckboxMenuItem("Delete", DEFAULT_TYPE==DELETE);
     CheckboxMenuItem move = new CheckboxMenuItem("Move", DEFAULT_TYPE==MOVE);
     //make panel with file edit and import picture to add x,y,z axis and 2 labels on each and line breaks
     public GuessWindow(String title, GuesstimateGraph anApp)
     {
      this.anApp = anApp;
      setMenuBar(menuBar);

      Menu fileMenu = new Menu("File");

      MenuItem item;
      fileMenu.add(item = new MenuItem("New", new MenuShortcut('N')));
      fileMenu.add(item = new MenuItem("Open", new MenuShortcut('O')));
      fileMenu.add(item = new MenuItem("Close", new MenuShortcut('C')));
      fileMenu.addSeparator();
      fileMenu.addSeparator();
      fileMenu.add(item = new MenuItem("Save", new MenuShortcut('S')));
      fileMenu.add(item = new MenuItem("Save as..."));
      fileMenu.addSeparator();
      fileMenu.add(item = new MenuItem("Print", new MenuShortcut('P')));
      MenuItem exit = new MenuItem("Exit", new MenuShortcut('a'));
      fileMenu.add(exit);
      exit.addActionListener(new ActionListener() {
             public void actionPerformed(ActionEvent e) {
                        System.exit(0);
                    }
                });

      Menu addMenu = new Menu("Add");

      Yaxis.addItemListener(new AxisCommand(YAXIS, guess));
      Xaxis.addItemListener(new AxisCommand(XAXIS, guess));
      Zaxis.addItemListener(new AxisCommand(ZAXIS, guess));
      Yaxislabel.addItemListener(new AxisCommand(YLABEL, guess));
      Xaxislabel.addItemListener(new AxisCommand(XLABEL, guess));
      Zaxislabel.addItemListener(new AxisCommand(ZLABEL, guess));
      axisbreak.addItemListener(new AxisCommand(BREAK, guess));

      addMenu.add(Yaxis);
      addMenu.add(Xaxis);
      addMenu.add(Zaxis);
      addMenu.add(Yaxislabel);
      addMenu.add(Xaxislabel);
      addMenu.add(Zaxislabel);
      addMenu.add(axisbreak);

      Menu editMenu = new Menu("Edit");
      editMenu.add(select);
      editMenu.add(delete);
      editMenu.add(move);

      addMenu.add(editMenu);

      Menu ImportMenu = new Menu("Import");

      ImportMenu.add(item = new MenuItem("Select Pic"));
      ImportMenu.add(item = new MenuItem("Resize"));
      ImportMenu.add(item = new MenuItem("Crop"));

      //if further developed add a draw menu to point out specific features

      menuBar.add(fileMenu);
      menuBar.add(addMenu);
      menuBar.add(ImportMenu);





    }

    public void setaddcheck(int addType)
    {
     Yaxis.setState(YAXIS==addType);
     Xaxis.setState(XAXIS==addType);
     Zaxis.setState(ZAXIS==addType);
     Yaxislabel.setState(YLABEL==addType);
     Xaxislabel.setState(XLABEL==addType);
     Zaxislabel.setState(ZLABEL==addType);
     axisbreak.setState(BREAK==addType);
    }

    public void seteditcheck(int editType)
    {
     select.setState(SELECT==editType);
     delete.setState(DELETE==editType);
     move.setState(MOVE==editType);
    }

    public void addView(GuessView view)
    {
     drawArea = new ScrollPane();
     drawArea.add(view);
     add(drawArea, BorderLayout.CENTER);
    }

    }

ItemListener 代码如下所示

    import java.awt.event.*;

    class AxisCommand implements ItemListener, Constants
    {
     int addTypeid;
     GuessDoc guess;

     public AxisCommand(int addTypeid, GuessDoc guess)
     {
      this.addTypeid = addTypeid;
      this.guess = guess;
     }

     public void itemStateChanged(ItemEvent e)
     {
      guess.setType(addTypeid);
     }

    }

目前的问题实际上是系统说“guess.setType(addTypeid);”出现空指针异常。 GuessDoc.setType(int id) 读起来像这样

    int addType;

    public void setType(int addType)
    {
     this.addType = addType;
     window.setaddcheck(addType);
    }

常量是列出的所有整数，即 YAXIS、XAXIS、YLABEL 等。如果您认为缺少某些内容(由于篇幅原因我省略了很多内容)，请询问。提前致谢。就像我说的，当我单击列表中除默认值之外的项目时，系统读取空指针异常，我不知道为什么。

Answer 1

在你的第一个类中，你在哪里为你的猜测变量分配一个 GuessDoc 对象？我不认为您曾经这样做过，更不用说在创建 ItemListener 之前这样做了。因此，您将 null 作为 ItemListeners 的第二个参数传递。

建议:

• 不要这样做。在所有参数对象完全实现之前，请勿创建 ItemListener。
• 顺便说一句，我强烈建议您避免使用 AWT，而更喜欢使用 Swing GUI。

<小时/>

编辑
您声明:

I assign it in the very beginning with all of the other variables as GuessDoc guess; I think it is on line 8. I don't know what your first suggestion means.

请告诉我您指的是哪一行。我看到这个:

GuessDoc guess;

请理解，这不是赋值，而只是变量声明，与此完全相同:

GuessDoc guess = null;

我看不到您在任何地方创建新的 GuessDoc 对象。如果您要为变量分配一个值，那么必须像这样完成:

GuessDoc guess = new GuessDoc();

或

    GuessDoc guess;
    guess = new GuessDoc();

考虑阅读一本不错的 Java 介绍教科书(例如 Head First Java)的前几章，因为它将帮助您快速熟悉 Java。我知道，因为它确实对我有帮助。

<小时/>

编辑2
您在评论中指出:

That's not the problem, but ok.

是的。你的问题是你在这条线上得到了一个 NPE:

guess.setType(addTypeid);

这意味着guess为空。我试图向您展示为什么它是 null ——因为当您调用这个构造函数时:

 public AxisCommand(int addTypeid, GuessDoc guess)
 {
  this.addTypeid = addTypeid;
  this.guess = guess;
 }

这里:

Yaxis.addItemListener(new AxisCommand(YAXIS, guess));

您正在为猜测传递一个空值，因为在进行上述调用之前，猜测从未在主程序中初始化。

我并不是说这是您的代码的唯一问题，因为最终还需要解决其他问题，但同样，我上面描述的问题是唯一 你的程序在你描述的地方抛出 NullPointerException 的原因。