Я получаю NullPointerException, используя CardLayout - PullRequest
Новая
       26

Я получаю NullPointerException, используя CardLayout

1 голос
/ 31 марта 2012

Эй, ребята, я пытаюсь создать графический интерфейс, который может перемещаться по JTextAreas при нажатии кнопки «Далее» или «Предыдущий». Я использую CardLayout для переключения между JTextArea. Я могу заставить программу компилироваться, но каждый раз, когда я нажимаю следующую или предыдущую кнопку, я всегда получаю исключение NullPointerException. Я не знаю, почему объект CardLayout имеет значение null в следующем и предыдущем методах, и это делает меня действительно запутанным.

Ошибка исходит из этого куска кода 

cl.show(cardPanel, "" + (currentCard));

Любая помощь приветствуется, ребята! 

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
import java.io.*;


public class AsciiCanvas extends JTextArea
{

private JPanel jp1;
private JPanel jp2;
private JPanel jp3;
private JPanel jp4;
private JPanel jp5;
private JPanel jp6;
private JPanel jp7;
private JPanel jp8;
private JPanel jp9;
private JPanel jp10;
private JTextArea text1;
private JTextArea text2;
private JTextArea text3;
private JTextArea text4;
private JTextArea text5;
private JTextArea text6;
private JTextArea text7;
private JTextArea text8;
private JTextArea text9;
private JTextArea text10;

private JPanel cardPanel;
private CardLayout cl;
private int currentCard = 1;

private JFileChooser fc;


public AsciiCanvas()
{
   Font mono = new Font("Monospaced", Font.PLAIN, 12);

    cardPanel = new JPanel();
    CardLayout cl = new CardLayout();
    cardPanel.setLayout(cl);


    JTextArea text1 = new JTextArea();
    JTextArea text2 = new JTextArea();
    JTextArea text3 = new JTextArea();
    JTextArea text4 = new JTextArea();
    JTextArea text5 = new JTextArea();
    JTextArea text6 = new JTextArea();
    JTextArea text7 = new JTextArea();
    JTextArea text8 = new JTextArea();
    JTextArea text9 = new JTextArea();
    JTextArea text10 = new JTextArea();



   JPanel jp1 = new JPanel();
   JPanel jp2 = new JPanel();
   JPanel jp3 = new JPanel();
   JPanel jp4 = new JPanel();
   JPanel jp5 = new JPanel();
   JPanel jp6 = new JPanel();
   JPanel jp7 = new JPanel();
   JPanel jp8 = new JPanel();
   JPanel jp9 = new JPanel();
   JPanel jp10 = new JPanel();

   jp1.add(text1);
   jp2.add(text2);
   jp3.add(text3);
   jp4.add(text4);
   jp5.add(text5);
   jp6.add(text6);
   jp7.add(text7);
   jp8.add(text8);
   jp9.add(text9);
   jp10.add(text10);

   cardPanel.add(jp1, "1");
   cardPanel.add(jp2, "2");
   cardPanel.add(jp3, "3");
   cardPanel.add(jp4, "4");
   cardPanel.add(jp5, "5");
   cardPanel.add(jp6, "6");
   cardPanel.add(jp7, "7");
   cardPanel.add(jp8, "8");
   cardPanel.add(jp9, "9");
   cardPanel.add(jp10, "10");

   setBorder(BorderFactory.createTitledBorder("Animat ion here"));
   setFont(mono);

   fc = new JFileChooser();
}


public void Next()
{
  if(currentCard < 10)
  {

     currentCard +=1;
     cl.show(cardPanel, "" + (currentCard));
  }


}

public void Previous()
{
  if(currentCard > 1)
  {
     currentCard -= 1;
     cl.show(cardPanel, "" + (currentCard));

  }
}

Сообщение об ошибке ***

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
at AsciiCanvas.Next(AsciiCanvas.java:108)

Ответы [ 2 ]

2 голосов
/ 31 марта 2012

Проблема в том, что cl является нулем, и вот почему: 

private CardLayout cl;

...

public AsciiCanvas()
{
    ...
    CardLayout cl = new CardLayout();

В этом конструкторе вы объявили новую локальную переменную, а не присваиваете значение переменной экземпляра. Локальная переменная shadows или скрывает переменную экземпляра. Ваш код поставщика должен быть: 

public AsciiCanvas()
{
    ...
    cl = new CardLayout();

(Кроме того, я настоятельно рекомендую использовать String.valueOf(...) вместо "" + ... для преобразования значения в строку безопасным для нуля способом. Вы не действительно заинтересованы в строке конкатенация, ты?)

1 голос
/ 31 марта 2012

Это локальная переменная в конструкторе, которая скрывает член cl: 

CardLayout cl = new CardLayout();

, означающий, что член cl равен null в методах Next() и Previous().

Изменить на: 

cl = new CardLayout();
...