スライダーを表示させたい

Question

前提・実現したいこと

JSliderを使って0から255の間を動かせるスライダーを作りたい。
JColorChooserは使えないということでお願いします。

発生している問題・エラーメッセージ

Exception in thread "main" java.lang.NullPointerException: Cannot invoke "javax.swing.JPanel.add(java.awt.Component)" because "this.panel" is null
        at MainFrame.<init>(ColorFrame.java:50)
        at MainFrame.main(ColorFrame.java:64)

このエラーがどういうことをすると吐かれるのかも教えていただきたいです。

該当のソースコード

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
 
class SliderPanel extends JPanel { // JFrame -> JPanelに変更しました
    JSlider rSlider;

    public SliderPanel() {
        rSlider = new JSlider(JSlider.HORIZONTAL, 0, 255, 0);
    }
}

class MainFrame extends JFrame {
    JPanel panel;

    public MainFrame(){
      SliderPanel sp = new SliderPanel();
      this.setTitle("Test");
      
      panel = new JPanel();
      panel.add(sp);
 
      this.add(panel, BorderLayout.CENTER);

      this.pack();
      this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      this.setVisible(true);
    }
    public static void main(String argv[]) {
      new MainFrame();
    }
}

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

SliderPanel クラスは Panel と言いながら JFrame を継承するのはなぜでしょう。
同クラスコンストラクタで JSlider オブジェクトを生成していますが、それをどこにも配置していないのはなぜでしょう。

[yomoanyan]

JFrameを継承していたのはミスです。すみません。

配置の件については、自分としてはMainFrameの方でpanelにaddしてそれを表示させているつもりです（できていませんが）ので、配置のさせ方？を教えていただきたいです。

[jimbe]

Swing のコンポーネントの配置は、レイアウトマネージャによって行われます。レイアウトマネージャは複数種類あり、必要なものを JPanel 等のコンテナに設定しておいてそのコンテナにコンポーネントを追加すると、レイアウトマネージャが配置してくれます。

Answer 1

「adding a window to a container」
「コンテナにウィンドウを追加する」
by Google翻訳

---

スライダー3つでRGBを選ぶパネルなら、こんな感じでしょうか。

package teratail_java.q367113;

import java.awt.*;

import javax.swing.*;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;

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

  MainFrame() {
    super();
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    JTextField rgbTextField = new JTextField();
    rgbTextField.setEditable(false);

    RGBSliderPanel rgbSliderPanel = new RGBSliderPanel();
    rgbSliderPanel.setChangeListener(new RGBSliderPanel.Listener() {
      @Override
      public void onChange(int red, int green, int blue) {
        rgbTextField.setText("red="+red+", green="+green+", blue="+blue);
      }
    });

    add(rgbTextField, BorderLayout.NORTH);
    add(rgbSliderPanel, BorderLayout.CENTER);

    pack();
  }
}

class RGBSliderPanel extends JPanel {
  interface Listener {
    void onChange(int red, int green, int blue);
  }
  private Listener listener;
  private int red, green, blue;

  RGBSliderPanel() {
    super(new GridBagLayout());
    setBorder(BorderFactory.createEmptyBorder(10,10,10,10));

    JLabel redLabel = new JLabel("R");
    JSlider redSlider = new JSlider(JSlider.HORIZONTAL, 0, 255, 0);
    redSlider.addChangeListener(new ChangeListener() {
      @Override
      public void stateChanged(ChangeEvent e) {
        red = ((JSlider)e.getSource()).getValue();
        if(listener != null) listener.onChange(red, green, blue);
      }
    });

    JLabel greenLabel = new JLabel("G");
    JSlider greenSlider = new JSlider(JSlider.HORIZONTAL, 0, 255, 0);
    greenSlider.addChangeListener(new ChangeListener() {
      @Override
      public void stateChanged(ChangeEvent e) {
        green = ((JSlider)e.getSource()).getValue();
        if(listener != null) listener.onChange(red, green, blue);
      }
    });

    JLabel blueLabel = new JLabel("B");
    JSlider blueSlider = new JSlider(JSlider.HORIZONTAL, 0, 255, 0);
    blueSlider.addChangeListener(new ChangeListener() {
      @Override
      public void stateChanged(ChangeEvent e) {
        blue = ((JSlider)e.getSource()).getValue();
        if(listener != null) listener.onChange(red, green, blue);
      }
    });

    GridBagConstraints gbc = new GridBagConstraints();
    gbc.gridx = 0;
    gbc.gridy = 0;
    add(redLabel, gbc);
    gbc.gridx ++;
    add(Box.createHorizontalStrut(5), gbc);
    gbc.gridx ++;
    add(redSlider, gbc);
    gbc.gridx = 0;
    gbc.gridy ++;
    add(Box.createVerticalStrut(10), gbc);
    gbc.gridx = 0;
    gbc.gridy ++;
    add(greenLabel, gbc);
    gbc.gridx = 2;
    add(greenSlider, gbc);
    gbc.gridx = 0;
    gbc.gridy ++;
    add(Box.createVerticalStrut(10), gbc);
    gbc.gridx = 0;
    gbc.gridy ++;
    add(blueLabel, gbc);
    gbc.gridx = 2;
    add(blueSlider, gbc);
  }

  void setChangeListener(Listener listener) {
    this.listener = listener;
    if(listener!= null) listener.onChange(red, green, blue); //初期表示用
  }
}

Comments

[yomoanyan]

ありがとうございます。参考にさせていただきます。

[jimbe]

ちょっとバグあったので直しました。
ついでにスライダーにラベルを付けてみました。

[yomoanyan]

ありがとうございます。

Answer 2

書いていなかったのですが、最終的にRGBの値をスライダーで指定して、その結果作られる色を表示させる、ということをやろうとしている最中なので、

であれば、JColorChooserを使えばいいのでは? そういうはなしではないのかな。

Comments

[yomoanyan]

書いてませんでしたが、使ってはいけないです。追記しておきます。

Answer 3

こんな感じでしょうか？
スライダーを追加するだけであれば、クラスを分ける必要は無いように思います。

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

class MainFrame extends JFrame {
    JPanel panel;

    public MainFrame(){
      SliderPanel sp = new SliderPanel();
      this.setTitle("Test");

      panel = new JPanel();

      JSlider slider = new JSlider(JSlider.HORIZONTAL, 0, 255, 0);
      panel.add(slider);

      this.add(panel, BorderLayout.CENTER);

      this.pack();
      this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
      this.setVisible(true);
    }
    public static void main(String argv[]) {
      new MainFrame();
    }
}

■ 実行結果

＜参考＞
■ スライダーの値を設定する - JSliderクラス - Swing
https://www.javadrive.jp/tutorial/jslider/index4.html

Comments

[yomoanyan]

回答ありがとうございます。書いていなかったのですが、最終的にRGBの値をスライダーで指定して、その結果作られる色を表示させる、ということをやろうとしている最中なので、この様な書き方、聞き方になってしまいました。そのためクラスを分けた書き方も教えていただきたいです。

余談ですが、自分としては、色を表示させるためのパネルを作るクラス、スライダーを作るクラス、RGB値を表示させるクラス、に分けようと考えています。
よろしくお願いします。