Question

所以我有Menu在每个项目上调用itemlistener。

    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对象？我没有看到你做过这件事，更不用说在创建你的ItemListeners之前做到这一点。因此，您将传递空值作为ItemListeners的第二个参数。

建议：

不要这样做。在完全实现所有参数对象之前，不要创建ItemListeners。
另外，我强烈建议您避免使用AWT并且更喜欢使用Swing GUI。

修改
你说：

我在一开始就将所有其他变量分配为GuessDoc猜测;我认为这是第8行。我不知道你的第一个建议是什么意思。

请告诉我你的意思。我明白这一点：

GuessDoc guess;

请理解不作业但只是变量声明，并且与此完全相同：

GuessDoc guess = null;

我不知道你在哪里创建一个新的GuessDoc对象在任何地方。如果你要为变量赋值，则必须这样做：

GuessDoc guess = new GuessDoc();

或

GuessDoc guess; guess = new GuessDoc();

请仔细阅读Java教科书的前几章，例如Head First Java，因为它可以帮助您快速掌握Java的速度。我知道因为它确实帮助了我。

编辑2
您在评论中说明：

这不是问题，但确定。

是的。你的问题是你在这条线上获得了NPE：

guess.setType(addTypeid);

表示guess为空。我试图告诉你为什么它为空 - 因为当你调用这个构造函数时：

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

这里：

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

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

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

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

1 个答案: