Java・JLabelの3つの画像変更方法を思いつきましたが全部エラー。。。原因と改善をお願いします。

Question

JLabelで1つディスプレイを用意し、その下に3つのJButtonを用意します。
3つのJButtonにはそれぞれ画像が張り付けてあり、各ボタンを押すと、そのボタンの画像がディスプレイに表示されるというプログラムです。

actionPerformed(ActionEvent e)に画像変更の方法を3つ思い付き、if文で分けて実行してみましたが、どれもうまくいきませんでした。

その3つの案は
1, 新しくJLabelを作り、ディスプレイ用に代入して入れ替える
2, ディスプレイは最初にaddしているので、remove(0)で削除してから新しいLabelをaddする
3, JlabelのsetIconで画像を指定して変更する

それぞれの実行結果は
1, 何も変わらない
2, 画面がバグる（フリーズしてしまう）

3, 大量のエラー(エラーをみても理由がわかりませんでした)

この3つの方法がうまくいかない理由と改善案をご教授していただきたいです。よろしくお願いします。

import java.awt.*;
import javax.swing.*;
class ImageDisplayEvt extends JFrame implements ActionListener{
	JLabel lb1;
	JButton bt1,bt2,bt3;
	ImageIcon img1,img3_base,img4_base;

	public ImageDisplayEvt(String title) {
		setTitle(title);
		setLocation(100,100);
		setSize(640,600);
		setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		setLayout(new FlowLayout(FlowLayout.LEFT,20,20));

		//1枚目(Label)
		img1=new ImageIcon("img/sakura1.jpg");
		int width=img1.getIconWidth();
		int height=img1.getIconHeight();
		JLabel lb1=new JLabel(new ImageIcon("img/sakura1.jpg"));
		add(lb1);

		//2枚目(Button1)
		Image img2=img1.getImage().getScaledInstance((int) (width/(height/100.0)),
				100, Image.SCALE_SMOOTH);
		bt1=new JButton(new ImageIcon(img2));
		bt1.setPreferredSize(new Dimension((int) (width/(height/100.0)),100));
		add(bt1);
		bt1.addActionListener(this);

		//3枚目(Button2)
		img3_base=new ImageIcon("img/sakura2.jpg");
		Image img3=img3_base.getImage().getScaledInstance(
				(int) ((img3_base.getIconWidth())/(( img3_base.getIconHeight()/100))),
				100, Image.SCALE_SMOOTH);
		bt2=new JButton(new ImageIcon(img3));
		bt2.setPreferredSize(new Dimension(
				(int) ((img3_base.getIconWidth())/(( img3_base.getIconHeight()/100)))
				,100));
		add(bt2);
		bt2.addActionListener(this);

		//4枚目(Button3)
		img4_base=new ImageIcon("img/sakura3.png");
		Image img4=img4_base.getImage().getScaledInstance(
				(int) ((img4_base.getIconWidth()/(img4_base.getIconHeight()/100.0))),
				100, Image.SCALE_SMOOTH);
		bt3=new JButton(new ImageIcon(img4));
		bt3.setPreferredSize(new Dimension(
				(int) ((img4_base.getIconWidth()/(img4_base.getIconHeight()/100.0))),
				100));
		add(bt3);
		bt3.addActionListener(this);

	}

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

	public void actionPerformed(ActionEvent e) {
		if(e.getSource().equals(bt1)) {
			System.out.println("bt1 was pushed!");

			JLabel lb2=new JLabel(img3_base);
			this.lb1=lb2;//エラーはないが画像が変わらない
		}
		if(e.getSource().equals(bt2)) {
			System.out.println("bt2 was pushed!");

			JLabel lb2=new JLabel(img3_base);
			remove(0);add(lb2,0);//画面が止まる
		}
		if(e.getSource().equals(bt3)) {
			System.out.println("bt3 was pushed!");

			this.lb1.setIcon(img4_base);//大量のエラー
		}
	}
}

（追記）
それぞれの画像のサイズがバラバラですので、getIconWidth()とgetIconHeight()を用いて、高さが100になるように縮小しています。

Answer 1

コンストラクタ内で完結するようにすれば、フィールドが無く(or 減ら)せて、代入忘れによるバグが減らせると思います。

import java.awt.BorderLayout;
import java.awt.Image;

import javax.swing.*;

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

  private static final String[] filenames = { "img/sakura1.png", "img/sakura2.png", "img/sakura3.png" };

  public ImageDisplayEvt() {
    setTitle("ImageDisplayEvent");
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    JLabel display = new JLabel();
    add(display, BorderLayout.CENTER);

    JPanel selectPanel = new SelectPanel(filenames, display);
    add(selectPanel, BorderLayout.SOUTH);

    pack();
    setLocationRelativeTo(null);
  }

  private static class SelectPanel extends JPanel {
    SelectPanel(String[] filenames, JLabel display) {
      super();

      JButton firstButton = null;
      for(String filename : filenames) {
        JButton button = createButton(filename, display);
        if(firstButton == null) firstButton = button;
        add(button);
      }
      firstButton.doClick();
    }

    private static JButton createButton(String filename, JLabel display) {
      ImageIcon imageIcon = new ImageIcon(filename);
      JButton button = new JButton(createResizeIcon(imageIcon, 100));
      button.addActionListener(v -> display.setIcon(imageIcon));
      return button;
    }

    private static ImageIcon createResizeIcon(ImageIcon src, int height) {
      return new ImageIcon(src.getImage().getScaledInstance(
          src.getIconWidth() * height / src.getIconHeight(),
          height,
          Image.SCALE_SMOOTH));
    }
  }
}

Answer 2

3つの方法がうまくいかない理由と改善案

・ 案１（左側のボタン）
論外です。残念ながら、this.lb1 の値を上書きしただけでは、画面上は何も変化しません。

・ 案２（中央のボタン）
フリーズする理由は、JFrame から直接 remove(0) しているためです。

通常の Container から部品を抜く際は直接 remove(0) すれば良いのですが、JFrame から部品を抜くときだけは、getContentPane().remove(0) とすることになっています。なお、コンポーネント階層の構造を変えた後は、validate() する必要もあります。

	JLabel lb2 = new JLabel(img3_base);
	getContentPane().remove(0);
	add(lb2, 0);
	validate();
	this.lb1 = lb2;  // <- 忘れずに

・ 案３（右のボタン）
this.lb1 の初期値が null であるため、NullPointerException が発生しているだけです。コンストラクタの中で、

	JLabel lb1 = new JLabel(new ImageIcon("img/sakura1.jpg"));
	add(lb1);
	this.lb1 = lb1;  // <- これが足りない

とでもしておけば良いでしょう。

Comments

[abe_sinzo]

lb1の初期値がNullであることに気づけました。大変分かりやすいご説明ありがとうございます。