JFrameの挙動がよくわかりません。

Question

ボタンを押すごとに画面内のTextFieldの内容と個数が切り替わるようにしたいです。
ボタンを1回押したときは正常に動作するのですが、2回目を押した際に何も反応しないです。
原因を教えていただきたいです。

import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.*;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextField;

public class Test {
    public static void main(String[] args) {
        new a().showa();
    }
}

class a extends JFrame implements ActionListener {
    JPanel p;
    ArrayList<JTextField> t;
    JButton b;
    int flg = 0;

    public void showa() {
        this.setTitle("a");
        this.setSize(1000, 100);
        this.setResizable(false);

        p = new JPanel();
        t = new ArrayList<>();
        b = new JButton();

        if (flg == 0) {
            for (int i = 0; i < 10; i++) {
                t.add(new JTextField());
                t.get(i).setText("a");
                p.add(t.get(i));
            }
        } else {
            for (int i = 0; i < 5; i++) {
                t.add(new JTextField());
                t.get(i).setText("b");
                p.add(t.get(i));
            }
        }

        b.setText("button");

        b.addActionListener(this);
        this.add(b, BorderLayout.SOUTH);

        this.add(p);
        this.show();
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        if (e.getSource() == this.b) {
            if (flg == 0) {
                this.flg = 1;
            } else {
                flg = 0;
            }
            this.p.removeAll();
            this.showa();
        }
    }
}

Comments

[jimbe]

コンポーネントを随時増減するような画面は UI としてあまり良いとは思えません。
最初に全て設定しておけば良いのではないでしょうか。

Answer 1

BorderLayout は一つの方向には一つしか表示出来ません。また、レイアウトは保存されているコンポーネントを並べるだけで、保存の管理には関係しません。
一方 JFrame の add メソッドはコンポーネントを追加するメソッドで、第2引数が同じだからとコンポーネントを置き換えたりはせずそのまま追加します。
つまりボタンを一回押した時、 JFrame には最初のボタンと二つ目のボタンがともに SOUTH で保存されていることになります。
同一方向のコンポーネントが複数あった場合に BorderLayout がどれを配置するかは、ドキュメント上では明言されていなかったと思います。

試した所では、二回目のボタン押下でも actionPerformed に渡される イベントのソース (getSource() で返されるオブジェクト) は最初のボタンのままで、従って e.getSource() == this.b が成立せず、二回目のボタンの処理は行われませんでした。

ボタンを押す度に JFrame から全部設定し直すような乱暴なことをしなくても、 「ボタンを押すごとに画面内のTextFieldの内容と個数が切り替わる」なら表示状態の設定(setVisible)や再配置要求(revalidate)等で出来ると思います。

import java.awt.BorderLayout;
import java.util.ArrayList;
import java.util.List;

import javax.swing.*;

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

  TestFrame() {
    super("TestFrame");
    setDefaultCloseOperation(EXIT_ON_CLOSE);
    setSize(1000, 100);
    setResizable(false);

    TestPanel panel = new TestPanel();
    add(panel, BorderLayout.CENTER);

    JButton button = new JButton("button");
    button.addActionListener(e -> panel.reverseFlag());
    add(button, BorderLayout.SOUTH);
  }

  private class TestPanel extends JPanel {
    private List<JTextField> list = new ArrayList<JTextField>();
    private int flag = 0;
    TestPanel() {
      super();
      for(int i=0; i<10; i++) {
        JTextField textField = new JTextField();
        add(textField);
        list.add(textField);
      }
      setFields();
    }
    private void setFields() {
      for(int i=0; i<10; i++) {
        JTextField textField = list.get(i);
        textField.setText(flag == 0 ? "a" : "b");
        textField.setVisible(i < 5 || flag == 0);
      }
      revalidate();
    }
    void reverseFlag() {
      flag ^= 1;
      setFields();
    }
  }
}

Comments

[Keittio]

詳しい解説ありがとうございます！

Answer 2

ボタンを1回押したときは正常に動作するのですが、2回目を押した際に何も反応しないです。
原因を教えていただきたいです。

↓とすれば想定通りになりますが、個人的にはイヤですね（パネル内だけ入れ替えればいいのに全部作り直し）

//this.p.removeAll();
this.getContentPane().removeAll();

---

自分だったらこうですかねぇ？

import javax.swing.*;
import java.awt.BorderLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;


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

    private final JPanel panel = new JPanel();
    private final ArrayList<JTextField> textFields = new ArrayList<>();
    private int flg;

    public Test() {
        setTitle("a");
        setSize(1000, 100);
        setLocationRelativeTo(null);
        setResizable(false);

        var button = new JButton("button");
        button.addActionListener(this);
        add(button, BorderLayout.SOUTH);

        add(panel);
        update();
    }

    private void update() {
        panel.removeAll();
        textFields.clear();

        if (flg == 0) {
            for (var i = 0; i < 10; i++) {
                var textField = new JTextField("a");
                textFields.add(textField);
                panel.add(textField);
            }
        } else {
            for (var i = 0; i < 5; i++) {
                var textField = new JTextField("b");
                textFields.add(textField);
                panel.add(textField);
            }
        }

        // 再レイアウト＆再描画
        panel.revalidate();
        panel.repaint();
    }

    @Override public void actionPerformed(ActionEvent e) {
        flg = flg == 0 ? 1 : 0;
        update();
    }
}

Comments

[TN8001]

あと提示コードだとボタンがリークしてそうです（removeActionListenerが必要）

[Keittio]

ありがとうございます。
JFrame触り始めたばかりなのでもっと勉強頑張ります・v・