我在菜單中切換項目時遇到麻煩

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

您在第一堂課的哪里為您的guess變量分配一個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的唯一原因。你描述它正在發生。