Блог

Итак, у меня есть 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 и т.д. Если вы считаете, что чего-то не хватает, чего я оставил много для экономии места, спросите. Заранее спасибо. Как я уже сказал, когда я нажимаю элемент в списке, отличный от значения по умолчанию, система считывает исключение нулевого указателя, и я не знаю почему.

Где вы назначаете объект GuessDoc для своей переменной guess в своем первом классе? Я не вижу, чтобы вы когда-либо делали это, а тем более делали это перед созданием ваших ItemListeners. Таким образом, вы передаете nulls в качестве второго параметра ваших ItemListeners.

Рекомендации:

• не делайте этого. Не создавайте свои ItemListeners до того, как все объекты параметров будут полностью реализованы.
• Кроме того, я настоятельно рекомендую вам избегать использования AWT и предпочитать использование Swing GUI.

Редактировать
Вы заявляете:

Я назначаю его в самом начале со всеми другими переменными как GuessDoc guess; Я думаю, что это в строке 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 это значение равно нулю. Я пытаюсь показать вам, почему это значение равно null — потому что, когда вы вызываете этот конструктор:

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

здесь:

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

вы передаете нулевое значение для guess, поскольку guess никогда не инициализируется в вашей основной программе перед выполнением вызова выше.

Я не говорю, что это единственная проблема с вашим кодом, поскольку есть и другие проблемы, которые в конечном итоге необходимо будет решить, но опять же, проблемы, которые я описал выше, являются единственной причиной того, что ваша программа выдает исключение NullPointerException, где вы описываете, что это происходит.