JPanelの表示

Question

ボタンを押すことでパネルが表示されるプログラムを作成しています。
想定結果としては以下の通りです。

1. drawボタン押下でオレンジ色のパネルが描画される
2. 描画されたオレンジ色のパネルの中には数値がランダムで表示される
3. 再び押下することで、パネルが上書きされる

しかし、現状としては

1. drawボタンを押下してもパネルが表示されない。ウィンドウをリサイズすることで表示される。
2. パネルが上書きされず、追加されてしまう。

なぜこのような現象が起きてしまったのでしょうか？また、どのようにすれば解決できるでしょうか？
わかる方いらっしゃいましたら、よろしくお願いします。

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

public class Sample extends JFrame implements Actionlistener{

JButton draw;
JPamel p;
MyPanel mp;

public static void main (String[] args) {
new sample(). setVisible(true);
}

Sample () {
setsize(500, 500);
setDefaultCloseOperation (JFrame.EXIT_ON_CLOSE);
draw = new JButton("draw");
draw.addActionListener (this);

p = new JPanel ();
p.setPreferredSize(new Dimension (300, 300));

setLayout(new FlowLayout ()); getContentPane().add(draw);
getContentPane().add(p);
}

@Override
public void actionPerformed(ActionEvent e) {
if (e.getSource () == draw) {
mp = new MyPanel();
p. add(mp);
}
}

public class MyPanel extends JPanel {
MyPanel() {
setBackground(Color.orange);
add(new JLabel(String.valueOf((int) (Math.random()*100))));
}
}
}

Answer 1

まず「drawボタンを押下してもパネルが表示されない。ウィンドウをリサイズすることで表示される。」という問題ですが、これはパネルを追加したことをSwingが理解しておらず再描画が行なわれないからです。
revalidateというメソッドを呼べば解決できます。

次に「パネルが上書きされず、追加されてしまう。」ですが、これは以前に追加したパネルを削除せずに追加し続けているからです。新しいパネルを追加する前に、以前に追加したパネルを削除するようにしましょう。

これらの改善を施したコードを貼っておきます。

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

public class Sample extends JFrame implements ActionListener {

  JButton draw;
  JPanel p;
  MyPanel mp;

  public static void main (String[] args) {
    new Sample(). setVisible(true);
  }

  Sample () {
    setSize(500, 500);
    setDefaultCloseOperation (JFrame.EXIT_ON_CLOSE);
    draw = new JButton("draw");
    draw.addActionListener (this);

    p = new JPanel ();
    p.setPreferredSize(new Dimension (300, 300));

    setLayout(new FlowLayout ());
    getContentPane().add(draw);
    getContentPane().add(p);
  }

  @Override
  public void actionPerformed(ActionEvent e) {
    if (e.getSource () == draw) {
      mp = new MyPanel();
      p.removeAll(); // <= ここで以前に追加したパネルを削除（一度も追加していない場合は何もしない）
      p.add(mp);
      revalidate(); // <= ここで再描画を要請
    }
  }

  public class MyPanel extends JPanel {
    MyPanel() {
      setBackground(Color.orange);
      add(new JLabel(String.valueOf((int) (Math.random()*100))));
    }
  }
}

Answer 2

下記のように変更すればよいと思います。

	public void actionPerformed(ActionEvent e) {
		if (e.getSource() == draw) {
			p.removeAll();
			mp = new MyPanel();
			p.add(mp);
			p.revalidate();
		}
	}