Не могу повторно добавить ту же панель - PullRequest
4 голосов
/ 10 февраля 2012

Это сводит меня с ума. Я пытаюсь изменить панели, когда вы нажимаете на 3 разные кнопки, и это работает, но для одной панели - только один раз.

Если вы нажмете addPerson - на панели «Персона» появится Если вы затем нажмете addCD - отобразится панель CD; (то же самое для просмотра магазина) Если вы затем нажмете на addPerson - это не сработает. Выдает исключение нулевого указателя. Даже если вы нажмете на addCD / viewstore и ТОГДА добавите Person, он отобразится, но просто не будет отображаться во второй раз.

В тестовом файле я создал графический интерфейс с надстройкой и удалением: если я нажал кнопку «Добавить», возникла исключительная ситуация нулевого указателя, но если я только добавил его и скомпилировал, все было хорошо ...

/* PERSON PANEL */
public JPanel create_PersonPnl()
{
    personPnl = new JPanel(new FlowLayout(FlowLayout.CENTER));
    personPnl.setBackground(Color.WHITE);
    personPnl.setPreferredSize(minPnl);

    /* VERTICAL BOX for Person boxes */
    Box personBox = Box.createVerticalBox();
    personBox.setBorder(new TitledBorder(new LineBorder(Color.DARK_GRAY), "Person"));

        /* Horizontal Box for Name Lbl & TF */
        Box nameBox = Box.createHorizontalBox();
            nameBox.add(Box.createHorizontalStrut(10));
            nameLbl = new JLabel("Name: ");
            nameBox.add(nameLbl);
            nameBox.add(Box.createHorizontalStrut(5));
            nameTF = new JTextField();
            nameBox.add(nameTF);
            nameBox.add(Box.createHorizontalStrut(10));

        personBox.add(nameBox);
        personBox.add(Box.createVerticalStrut(10));
        Box limitBox = Box.createHorizontalBox();
            limitBox.add(Box.createHorizontalStrut(10));
            limitLbl = new JLabel("CD Limit: ");
            limitBox.add(limitLbl);
            limitTF = new JFormattedTextField();
            limitBox.add(limitTF);
            limitBox.add(Box.createHorizontalStrut(10));
        personBox.add(limitBox);
        personBox.add(Box.createVerticalStrut(10));
        personBox.add(addPersonBtn = new JButton("Add Person"));
        personBox.add(Box.createVerticalStrut(10));
            personList = new JList(temp);
            personList.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
            scrollp = new JScrollPane(personList);
                scrollp.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
                scrollp.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED);
            scrollp.setSize(100, 45);
            personBox.add(scrollp);
            personBox.add(Box.createVerticalStrut(10));
        Box optionsBox = Box.createHorizontalBox();
            editPrsnBtn = new JButton("Edit");
            editPrsnBtn.addActionListener(this);
            optionsBox.add(editPrsnBtn);
            removePrsnBtn = new JButton("Remove");
            removePrsnBtn.addActionListener(this);
            optionsBox.add(removePrsnBtn);
        personBox.add(optionsBox);

    personPnl.add(personBox);
    return personPnl;
}

Это то, что есть в моем методе ActionPerformed.

if(e.getSource() == addPersonBtn)
    {
        changePnl.removeAll();
        changePnl.add(create_PersonPnl());
        changePnl.revalidate();
        System.out.println("PersonPnl added");
    }

    if(e.getSource() == addCDBtn)
    {
        changePnl.removeAll();
        changePnl.add(create_CDPnl());
        changePnl.revalidate();
    }

    if(e.getSource() == viewStoreBtn)
    {
        changePnl.removeAll();
        changePnl.add(create_StorePnl());
        changePnl.revalidate();
    }

Только код для ChangePnl.

  changePnl = new JPanel();
  changePnl.setBackground(Color.WHITE);
defaultPnl = new JPanel();
defaultPnl.setBackground(Color.WHITE);
defaultPnl.add(new JLabel("Welcome to the CD Store"));
defaultPnl.add(new JLabel("Click an option from the left"));
changePnl.add(defaultPnl);

S.O.P должен был отладить, чтобы увидеть, что запускается ... и он печатается только один раз, и все, если я не вычеркну .add (create_PersonPnl ()); поэтому я его сузил, но понятия не имею, так как он работает с первого раза.

Спасибо заранее!

Отдельный тестовый файл, чтобы доказать, что он create_PersonPnl ()

import java.awt.Color;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.*;
import javax.swing.border.LineBorder;
import javax.swing.border.TitledBorder;

public class Test implements ActionListener
{
JButton add, remove;
JButton addPersonBtn, editPrsnBtn, removePrsnBtn;
JFrame frame;
JFormattedTextField limitTF;
JLabel nameLbl, limitLbl;
JList personList;
JPanel  TotalGUI, welcomePnl, mainPnl, imagePnl, changePnl, defaultPnl, cdPnl,       storePnl;
JPanel personPnl;
JScrollPane scrollp;
JTextField nameTF;

Dimension dim = Toolkit.getDefaultToolkit().getScreenSize();
Dimension minPnl = new Dimension(300, 400);

/* Test for JList */
String[] temp = {"1", "2", "3", "4", "1", "2", "3", "4","1", "2", "3", "4","1", "2", "3", "4" };

public Test()
{
    frame = new JFrame("CD Store");
    frame.setExtendedState(JFrame.NORMAL);

    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    //frame.pack(); //sets size based on components size in TotalGUI
    //frame.setJMenuBar(create_MenuBar());
    //frame.setMinimumSize(minDim);

    frame.setSize(725, 550);

    //set frame location (central to screen)
    int fw = frame.getSize().width;
    int fh = frame.getSize().height;
    int fx = (dim.width-fw)/2;
    int fy = (dim.height-fh)/2;


    frame.getContentPane().add(create_Content_Pane());
    frame.setVisible(true);

    //moves the frame to the centre
    frame.setLocation(fx, fy);
}

public JPanel create_Content_Pane()
{
    JPanel TotalGUI = new JPanel();

    TotalGUI.add(remove = new JButton("Remove"));
    remove.addActionListener(this);

    TotalGUI.add(add = new JButton("Add")); <- this crashes when selected
    add.addActionListener(this);

    //TotalGUI.add(create_PersonPnl()); <- works just fine

    return TotalGUI;
}

public JPanel create_PersonPnl()
{
    personPnl = new JPanel(new FlowLayout(FlowLayout.CENTER));
    personPnl.setBackground(Color.WHITE);
    personPnl.setPreferredSize(minPnl);

    /* VERTICAL BOX for Person boxes */
    Box personBox = Box.createVerticalBox();
    personBox.setBorder(new TitledBorder(new LineBorder(Color.DARK_GRAY), "Person"));

        /* Horizontal Box for Name Lbl & TF */
        Box nameBox = Box.createHorizontalBox();
            nameBox.add(Box.createHorizontalStrut(10));
            nameLbl = new JLabel("Name: ");
            nameBox.add(nameLbl);
            nameBox.add(Box.createHorizontalStrut(5));
            nameTF = new JTextField();
            nameBox.add(nameTF);
            nameBox.add(Box.createHorizontalStrut(10));

        personBox.add(nameBox);
        personBox.add(Box.createVerticalStrut(10));
        Box limitBox = Box.createHorizontalBox();
            limitBox.add(Box.createHorizontalStrut(10));
            limitLbl = new JLabel("CD Limit: ");
            limitBox.add(limitLbl);
            limitTF = new JFormattedTextField();
            limitBox.add(limitTF);
            limitBox.add(Box.createHorizontalStrut(10));
        personBox.add(limitBox);
        personBox.add(Box.createVerticalStrut(10));
        personBox.add(addPersonBtn = new JButton("Add Person"));
        personBox.add(Box.createVerticalStrut(10));
            personList = new JList(temp);
            personList.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
            scrollp = new JScrollPane(personList);
                scrollp.setHorizontalScrollBarPolicy(JScrollPane.HORIZONTAL_SCROLLBAR_AS_NEEDED);
                scrollp.setVerticalScrollBarPolicy(JScrollPane.VERTICAL_SCROLLBAR_AS_NEEDED);
            scrollp.setSize(100, 45);
            personBox.add(scrollp);
            personBox.add(Box.createVerticalStrut(10));
        Box optionsBox = Box.createHorizontalBox();
            editPrsnBtn = new JButton("Edit");
            editPrsnBtn.addActionListener(this);
            optionsBox.add(editPrsnBtn);
            removePrsnBtn = new JButton("Remove");
            removePrsnBtn.addActionListener(this);
            optionsBox.add(removePrsnBtn);
        personBox.add(optionsBox);

    personPnl.add(personBox);
    return personPnl;
}

public static void main(String[] args)
{
    new Test();
}

public void actionPerformed(ActionEvent e) 
{
    if(e.getSource() == remove)
    {
        TotalGUI.removeAll();
        TotalGUI.revalidate();
    }

    if(e.getSource() == add)
    {
        TotalGUI.add(create_PersonPnl());
        TotalGUI.revalidate();
    }
}
}

1 Ответ

1 голос
/ 10 февраля 2012

В вашем методе

public JPanel create_Content_Pane()
{
    JPanel TotalGUI = new JPanel();

    TotalGUI.add(remove = new JButton("Remove"));
    remove.addActionListener(this);

    TotalGUI.add(add = new JButton("Add")); <- this crashes when selected
    add.addActionListener(this);

    //TotalGUI.add(create_PersonPnl()); <- works just fine

    return TotalGUI;
}

Исключение NullPointerException возникает из-за того, что ваше личное поле TotalGUI имеет значение null ... Удалите объявление JPanel перед TotalGUI = new JPanel(); Это решит проблему нулевого указателя. Это, вероятно, решает только вашу проблему в тестовом сценарии ... Чтобы решить проблему в исходном сценарии, было бы неплохо иметь полный исходный код класса ..

...