JavaのSwingにて、JPanelの内容を継承クラスに書きたいがぬるぽが発生してしまう

Question

実現したいこと

JPanelをアコーディオン風に展開　こちらのサイトを参考にして設計しています。

ラベルをクリックするとラベル下部のパネルの表示・非表示が切り替わる仕組みになっていて、書く内容を何パターンかに分けて3つほどの継承クラスを作成し、配列などで一気に作成できるようにしたいです。

class Main {
	public static void main(String[] args){
		JFrame f = new JFrame();
		JPanel p = new JPanel();
		CheckBox_Accordion instance = new CheckBox_Accordion("mockTitle", "chk1", "chk2", "chk3");
		f.add(p);
		p.add(instance);
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setBounds(100, 100, 640, 400);
		f.setVisible(true);
	}
}
class CheckBox_Accordion extends Abstract_Accordion {
	List<String> list = new LinkedList<>();

	CheckBox_Accordion(String i_title, String... chkbox_title){
		super(i_title);
		Stream.of(chkbox_title).forEach(list::add);
	}
	
	JPanel makePanel(){
		JPanel panel = new JPanel();
		panel.setLayout(new BoxLayout(panel, BoxLayout.Y_AXIS));
		list.stream().map(JCheckBox::new).forEach(panel::add);

		return panel;
	}
}
abstract class Abstract_Accordion extends JPanel {
	private JPanel panel = makePanel();

	Abstract_Accordion(String i_title){
		JLabel label = new JLabel(i_title);
		label.addMouseListener(/* 省略 */);

		add(label, BorderLayout.NORTH);
		add(panel, BorderLayout.CENTER);
	}

	abstract JPanel makePanel();
}

発生している問題・分からないこと

しかし継承先のmakePanel()メソッドでpanelの内容を書こうとすると実行順の影響でぬるぽが発生してしまいます。
今はぬるぽを回避するために継承クラスのコンストラクタ全てにmakePanel()を書かなくてはならず、面倒ですしコーディングミスも起きやすいですし何よりきれいじゃないと思います。

該当のソースコード

Exception in thread "main" java.lang.NullPointerException
	at CheckBox_Accordion.makePanel(Main.java:33)
	at Abstract_Accordion.<init>(Main.java:40)
	at CheckBox_Accordion.<init>(Main.java:26)
	at Main.main(Main.java:13)

試したこと・調べたこと

• teratailやGoogle等で検索した
• ソースコードを自分なりに変更した
• 知人に聞いた
• その他

上記の詳細・結果

テンプレートパターンが最適かと思っているのですが上手く組み立てられていません。

補足

特になし

Comments

[jimbe]

>書く内容を何パターンかに分けて3つほどの継承クラスを作成し、配列などで一気に作成
>しかし継承先のmakePanel()メソッドでpanelの内容を書こうとすると実行順の影響でぬるぽが発生

ということですが、ご提示のコードでご提示のエラーが発生しているのでしょうか。そうは見えないのですが・・・。
パターン別の3つの継承クラスは見当たりませんし、配列で何を作ろうと言われているのかもよく分かりません。makePanel はそれぞれ中身が違くなるはずですので書かなければならないのは当然のように思いますが…恐らく私がイメージしたこととが違うのでしょうね。
ですので、その辺り言葉での説明よりも実際に書きたい(動作させると例外の出る)コードをご提示頂けないでしょうか。

[TN8001]

「グッド」をいただいておりますが、質問は解決したのでしょうか？

解決したのであれば質問を「解決済」にしてください（「意見交換」と違って「Q&A」は、質問者自ら閉じる必要があります）
[ヘルプ｜質問を解決済みにしたい](https://teratail.com/help#resolve-question)

解決していないのであれば、「回答の〇〇が理解できない」・「望んだ回答でない」等々解決していない理由をコメントしてください。

回答を確認する時間がないということなら、目安の期間をコメントしてください。

[perfectibility]

すみません。
すこし忙しくてグッドを付けるしか出来ておりませんでした。
直ぐに解決済みまたはコメントします。

[TN8001]

せかす意図はありませんのでゆっくり確認されて結構です。
回答に不明点や疑問点があればお気軽にコメントください^^

[perfectibility]

お二方どちらのコードもとても勉強になりました！
そもそもAccordionと自分が実装したいものの概念の名称は違うのですね、そこも勉強になりました…

TN8001さんの仰る通りabstractでないExpanderクラスを作り、そのコンストラクタで内容を渡すのが簡単かつ分かりやすいですね。
データが多いので配列で渡して一気に作成するのですが、それこそ内容のみのクラスに分けた方がコーディング時も分かり易いのでとても良いと思いました！

ベストアンサーを選ぶのはとても迷いましたが、
jimbeさんのコードは知らない事が多く理解するのに時間が掛かりましたが、その分多くのことを学べたので今回はjimbeさんとさせていただきました！
本当にありがとうございました！

Answer 1

アコーディオンというと「開くのはひとつだけ」というイメージがあります（だから「風」なんでしょうが^^;
展開・折り畳みが個別にできるようなので、Expanderとかのほうがあっていそうに思います（以後Expanderと呼称します）

参考サイトの作りは（個人的には）あまりいい作りに思えません（サンプルを作りやすい構造にしたかった等の都合があったのかもしれません）

Expanderはヘッダーのテキストとなんらかのコンテンツを持ち、ヘッダーのクリックでコンテンツの展開・折り畳みをするコンポーネントですね？

Expanderはコンテンツがなんであるかには興味はなく、単に渡されたコンテンツをそのまま使うだけでいいはずです。
なのでabstractである必要はないですし、継承する必要もありません。

パターンごとにクラスを作りたければ、「コンテンツ」として好きに作ればいいでしょう^^

import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.util.stream.Stream;


public class Main extends JFrame {
    public static void main(String[] args) {
        new Main().setVisible(true);
    }

    private final JLabel label = new JLabel(" ");

    public Main() {
        setDefaultCloseOperation(EXIT_ON_CLOSE);
        setSize(300, 200);
        setLocationRelativeTo(null);

        add(label, BorderLayout.NORTH);

        var box = Box.createVerticalBox();

        var cp = new CheckBoxPanel("111", "222");
        box.add(new Expander("CheckBox", cp));
        box.add(Box.createVerticalStrut(5));

        var rp = new RadioButtonPanel("aaa", "bbb", "ccc");
        box.add(new Expander("RadioButton", rp));
        box.add(Box.createVerticalStrut(5));

        box.add(Box.createVerticalGlue());
        add(new JScrollPane(box));
    }

    class CheckBoxPanel extends JPanel {
        CheckBoxPanel(String... texts) {
            setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
            Stream.of(texts).map(JCheckBox::new).forEach(b -> {
                add(b);
                b.addActionListener(e -> label.setText((b.isSelected() ? "selected " : "unselected ") + b.getText()));
            });
        }
    }

    class RadioButtonPanel extends JPanel {
        RadioButtonPanel(String... texts) {
            setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
            var bg = new ButtonGroup();
            Stream.of(texts).map(JRadioButton::new).forEach(b -> {
                b.setSelected(getComponentCount() == 0);
                add(b);
                bg.add(b);

                b.addItemListener(e -> {
                    if (e.getStateChange() == ItemEvent.SELECTED) {
                        label.setText("selected " + b.getText());
                    }
                });
            });
        }
    }
}

class Expander extends JPanel {
    Expander(String header, JComponent content) {
        super(new BorderLayout());

        var label = new JLabel("▼ " + header);
        label.addMouseListener(new MouseAdapter() {
            @Override public void mousePressed(MouseEvent e) {
                content.setVisible(!content.isVisible());
                label.setText(String.format("%s %s", content.isVisible() ? "△" : "▼", header));
                revalidate();
                EventQueue.invokeLater(() -> scrollRectToVisible(content.getBounds()));
            }
        });
        add(label, BorderLayout.NORTH);

        content.setVisible(false);
        add(content, BorderLayout.CENTER);
    }

    @Override public Dimension getMaximumSize() {
        var d = getPreferredSize();
        d.width = Short.MAX_VALUE;
        return d;
    }
}

Answer 2

質問のコードは makePanel でフィールドを初期化していますが、そこまで行っていればあと少しで「コンストラクタパラメータで渡してしまえば」と気付けたように思います。
回答としては TN8001 さんと同じく「コンテンツの生成をリンク先のようにメソッドで持つ必要は無い」ということになります。

class Main {
  public static void main(String[] args){
      JFrame f = new JFrame();
      JPanel p = new JPanel();
      Accordion instance = new Accordion("mockTitle", new CheckBoxContents("chk1", "chk2", "chk3"));
      f.add(p);
      p.add(instance);
      f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      f.setBounds(100, 100, 640, 400);
      f.setVisible(true);
  }
}
//CheckBox_Accordion の makePanel で作っていたヤツ
class CheckBoxContents extends JPanel {
  CheckBoxContents(String... chkboxTitle) {
    super(null);
    setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
    Stream.of(chkboxTitle).map(JCheckBox::new).forEach(this::add);
  }
}
//makePanel で作るのでは無くコンストラクタパラメータで貰う
class Accordion extends JPanel {
  Accordion(String iTitle, JPanel panel){
      JLabel label = new JLabel(iTitle);
      label.addMouseListener(/* 省略 */);

      add(label, BorderLayout.NORTH);
      add(panel, BorderLayout.CENTER);
  }
}

---

リンク先の GitHub のコードを弄ってみました。
コンテンツクラスを流用ということで CheckPanel を 2 回使っていますが、配列にしてというのは必要か分からなかったのでやっていません。
また、ラベルとコンテンツパネルの組のクラスを土台のパネルに並べるよりも(ラベルとコンテンツの組は複数有るのが通常でしょうから)土台に直接ラベルとパネルを並べたほうが階層が減って良いと思います。

ついでに、タイトルにはラベルで無くチェックボックスを使えば操作はマウス以外でも出来るようになります(チェック部分の画像は差し替えられます)し、 BoxLayout で無く GridBagLayout なら大きさの調整の為にメソッドをオーバーライドする必要もありません。

import java.awt.*;

import javax.swing.*;
import javax.swing.event.AncestorEvent;
import javax.swing.event.AncestorListener;

public class MainFrame extends JFrame {
  public static void main(String[] args) {
    SwingUtilities.invokeLater(() -> new MainFrame().setVisible(true));
  }

  MainFrame() {
    super("ExpandableContentsPanel");
    setDefaultCloseOperation(WindowConstants.EXIT_ON_CLOSE);

    add(new JScrollPane(new ExpandableContentsPanel()));

    setSize(320, 400);
    //pack();
    setLocationRelativeTo(null);
  }
}

class ExpandableContentsPanel extends JPanel {
  private static final AncestorListener visibleToScroll = new AncestorListener() {
    @Override
    public void ancestorRemoved(AncestorEvent event) { /*no process*/ }
    @Override
    public void ancestorMoved(AncestorEvent event) { /*no process*/ }
    @Override
    public void ancestorAdded(AncestorEvent event) { scrollRectToVisible(event.getComponent()); }
  };

  private static void scrollRectToVisible(JComponent c) {
    ((JComponent)c.getParent()).scrollRectToVisible(c.getBounds());
  }

  ExpandableContentsPanel() {
    super(new GridBagLayout());
    setBackground(new Color(180, 180, 255));
    setBorder(BorderFactory.createEmptyBorder(10, 5, 10, 5));

    GridBagConstraints gbc = new GridBagConstraints();
    gbc.gridx = 0;
    gbc.fill = GridBagConstraints.HORIZONTAL;
    gbc.weightx = 1;
    add("System Tasks", new CheckPanel("1111","222222"), gbc);
    add("User Tasks", new CheckPanel("33333","444","55"), gbc);
    add("Other Places", new LabelPanel("Desktop","My Network Places","My Documents","Shared Documents"), gbc);
    add("Details", new RadioPanel("aaa","bbb","ccc","ddd"), gbc);

    gbc.weighty = 1;
    add(Box.createVerticalStrut(0), gbc);

    KeyboardFocusManager.getCurrentKeyboardFocusManager().addPropertyChangeListener("focusOwner", e -> {
      if(!(e.getNewValue() instanceof JComponent)) return;
      JComponent focused = (JComponent)e.getNewValue();
      if(this.isAncestorOf(focused)) scrollRectToVisible(focused);
    });
  }

  private void add(String titleText, JComponent contents, GridBagConstraints gbc) {
    Title title = new Title(titleText);
    title.addItemListener(e -> contents.setVisible(title.isSelected()));
    contents.setVisible(title.isSelected());
    contents.addAncestorListener(visibleToScroll);

    if(getComponentCount() > 0) add(Box.createVerticalStrut(5), gbc);
    add(title, gbc);
    add(contents, gbc);
  }
}

class Title extends JCheckBox {
  private static Icon icon = new ExpandedStateIcon();

  Title(String text) {
    super(text, icon);
    setOpaque(false);
    setForeground(Color.BLUE);
    setBorder(BorderFactory.createEmptyBorder(2, 5, 2, 2));
  }

  @Override
  protected void paintComponent(Graphics g) {
    Graphics2D g2 = (Graphics2D)g;
    g2.setPaint(new GradientPaint(50, 0, Color.WHITE, getWidth(), getHeight(), new Color(200, 200, 255)));
    g2.fillRect(0, 0, getWidth(), getHeight());
    super.paintComponent(g);
  }
}

class ExpandedStateIcon implements Icon {
  private final int iconWidth, iconHeight;
  private final int dx, dy; //位置補正
  private final Polygon selectedPolygon, polygon;

  ExpandedStateIcon() {
    Icon orgIcon = UIManager.getIcon("CheckBox.icon");
    iconWidth = orgIcon.getIconWidth();
    iconHeight = orgIcon.getIconHeight();
    int pw = (int)(iconWidth * 0.8); //大きさはテキトウ
    int ph = (int)(iconHeight * 0.6); // 〃
    dx = (iconWidth-pw) / 2;
    dy = (iconHeight-ph) / 2;
    selectedPolygon = new Polygon(new int[]{pw/2,0,pw}, new int[]{0,ph,ph}, 3); //上向き三角
    polygon = new Polygon(new int[]{0,pw,pw/2}, new int[]{0,0,ph}, 3); //下向き三角
  }

  @Override
  public void paintIcon(Component c, Graphics g, int x, int y) {
    Graphics2D g2 = (Graphics2D)g.create();
    g2.translate(x+dx, y+dy);
    g2.setColor(c.isEnabled() ? c.getForeground() : Color.GRAY);
    g2.setRenderingHint(RenderingHints.KEY_ANTIALIASING, RenderingHints.VALUE_ANTIALIAS_ON);
    if(c instanceof AbstractButton && ((AbstractButton)c).isSelected()) {
      g2.draw(selectedPolygon);
    } else {
      g2.fill(polygon);
    }
    g2.dispose();
  }
  @Override
  public int getIconWidth() { return iconWidth; }
  @Override
  public int getIconHeight() { return iconHeight; }
}

class Contents extends JPanel {
  Contents(LayoutManager layout) {
    super(layout);
    setBackground(new Color(240, 240, 255));
    setBorder(BorderFactory.createCompoundBorder(
        BorderFactory.createMatteBorder(0, 2, 2, 2, Color.WHITE),
        BorderFactory.createEmptyBorder(10, 10, 10, 10)));
  }
}

class CheckPanel extends Contents {
  CheckPanel(String... labels) {
    super(new GridLayout(0, 1));

    for(String s : labels) {
      JCheckBox b = new JCheckBox(s);
      b.setOpaque(false);
      add(b);
    }
  }
}

class LabelPanel extends Contents {
  LabelPanel(String... labels) {
    super(new GridLayout(0, 1));

    for(String s : labels) {
      add(new JLabel(s));
    }
  }
}

class RadioPanel extends Contents {
  RadioPanel(String... labels) {
    super(new GridLayout(0, 1));

    ButtonGroup bg = new ButtonGroup();
    for(String s : labels) {
      JRadioButton b = new JRadioButton(s);
      b.setOpaque(false);
      b.setSelected(getComponentCount() == 0);
      add(b);
      bg.add(b);
    }
  }
}