forの中のtextfieldに入力した内容を表示できない

Question

現在javaを用いて割り勘をしてくれるシステムを作ろうとしています。それにあたって、textfieldで名前の入力欄を設けて入力した値をJLabelで表示するコードを作成いたしました。しかし、フィールドが初期化されてしまっているせいか、何も表示されない結果となっています。どなたかお力添えをいただけると助かります。

細かい説明(下記コードを参照)
ソースコード１の12行目のforの部分でtextfieldを任意の数生成できるシステムを組み、入力した値を15行目のgetTextで取得できるコードを記入した。forの中の値を参照することができないため、あらかじめフィールドとしてソースコード１の4行目にあるString strを作成した。そしてソースコード２の6行目でそれを表示しようとしても白紙になってしまう。

該当のソースコード

Java

ソースコード１

public class ChangeNumberPractice extends JPanel {
    String str;
    JLabel label = new JLabel("入力した人数をテキストボックスの数に変える");
    
    public ChangeNumberPractice(PanelPracticeNumber ppn){
        this.setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
        this.add(label);
        String textNumber = ppn.getText();
        int tn = Integer.parseInt(textNumber);
        for(int i = 0; i < tn; i++){
               JTextField text = new JTextField();
               this.add(text);
               str = text.getText();
        }
        this.add(new ButtonInCnp(ppn));
        System.out.println(str);
    }
}

ソースコード２

public class PanelInCnp extends JPanel{
    JLabel label = new JLabel("記入した名前の表示");
    public PanelInCnp(ChangeNumberPractice cnp){
        JLabel nameLabel = new JLabel(cnp.str);→１の方のフィールドの値を表示
        this.setLayout(new BoxLayout(this, BoxLayout.Y_AXIS));
        this.add(label);
        this.add(nameLabel);
    }
}

試したこと

switch-caseで動的でないtextfieldも設けたが、そちらも結局は値が表示されなかった。
例case1: jtextfield text1 = new jtextfield();
String str = text1.getText();
break;

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

versionはjava11
その他質問があればぜひぜひお尋ねください。

Comments

[y_waiwai]

このままではコードが読みづらいので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[Beginner.java]

ご指摘ありがとうございます。変更いたしました。

[dodox86]

現状ご提示されているコードは断片的なものだと思いますが、問題を再現できる最低限の全体の動くコードを示した方が良いと思います。どのタイミング、例えばボタンを押したときに入力した値（文字列）を取得したいのかなどが分かりません。

[jimbe]

> 12行目のforの部分でtextfieldを任意の数生成できるシステムを組み、入力した値を15行目のgetTextで取得できるコード

> JTextField text = new JTextField();
> this.add(text);
> str = text.getText();

ユーザが何の入力も出来ない(パネルの表示さえされていない)画面生成の段階で getText しても意味が無いでしょう。
プログラムの動作イメージが GUI になっていないようです。

そもそもといいますか、普通に割り勘なら合計金額を人数で割るだけで、個々人の名前など不要に思いますが、どういうプログラムにされるのでしょう。

[Beginner.java]

数々のご指摘ありがとうございます。説明不足な点は申し訳ありません。後日より分かりやすい形でGUIの画像を含めて再提示させていただきます。
イメージとしては、金額と人数を入力したら、人名ごとに料金を表示するという感じです。
例:2000円と人数を入力し決定ボタンを押す→次に立て替えてもらった人を選択する(今回は2000円を4人が立て替えてもらったという想定)

aさん500円
bさん500円
cさん500円
dさん500円
を表示するといった感じです。最終的には、これまで入力した額をまとめて表示してくれるというイメージです。
合計額
aさん1500円
bさん3000円
cさん2000円
dさん900円

作る意味は全くもってありません。あくまで目的は勉強です。

[jimbe]

> 最終的には、これまで入力した額をまとめて表示してくれる
最終的…というのは、例えば一か月間に色々なグループで飲み会をした割り勘を月末に「あの日はaさんbさんで3000円、別の日はbさんcさんdさんで5000円、…でそれぞれの割り勘の合計は…」で清算という感じでしょうか。

[Beginner.java]

ご質問ありがとうございます。
どちらかと言えばもう少し短期間の想定です。例えば、友人と2泊、3泊の旅行とかで払った代金の精算とかを目的としていました。

[jimbe]

なるほど。そうすると人数的にもせいぜい数人想定でしょうか。名前は登録制にして選択する感じも良さそうですね。
計算は(入力間違いを考えなければ)何かボタンを押したら即加算できるでしょうから、名前とその人の合計のリストは常に表示状態にして、割り勘対象はそこから選択(チェックボックス)とし、名前追加のパネルを常に出しておき必要なら名前をリストに追加して選択、割り勘対象金額とボタンを付けて金額を入力してボタンを押したらリストの選択されている人たちに割り勘が合計される・・・とすればダイアログ等無くても一通りの処理が出来そうです。

[Beginner.java]

@jimbeさん　ご回答ありがとうございます。さっそく動作確認をしてみます。
ご指摘ありがとうございます。ある程度作り方が分かるようになりましたら、そちらのイメージで試してみます。

Answer 1

非常にざっくりとですが、 人数をスピナーで選択してセットボタンを押すと指定した数の TextField が表示され、そこに入力すると最下部の TextField に各入力が "," 区切りで表示されるようにしてみました。

ただ、このような場合はリストにしたほうが良いように思います。

package teratail_java.q366141;

import java.awt.BorderLayout;
import java.awt.Rectangle;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.*;

import javax.swing.*;
import javax.swing.event.DocumentEvent;
import javax.swing.event.DocumentListener;

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

  Q366141() {
    super("割り勘？");
    setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
    setBounds(new Rectangle(300,300));

    JPanel northPanel = new JPanel(null);
    northPanel.setLayout(new BoxLayout(northPanel, BoxLayout.X_AXIS));
    JLabel numberOfPeopleLabel = new JLabel("人数:");
    northPanel.add(numberOfPeopleLabel);
    JSpinner numberOfPeopleSpinner = new JSpinner(new SpinnerNumberModel(2, 2, 10, 1));
    northPanel.add(numberOfPeopleSpinner);
    JButton numberOfPeopleButton = new JButton("セット");
    northPanel.add(numberOfPeopleButton);
    add(northPanel, BorderLayout.NORTH);

    JPanel centerPanel = new JPanel(null);
    centerPanel.setLayout(new BoxLayout(centerPanel, BoxLayout.Y_AXIS));
    add(centerPanel, BorderLayout.CENTER);

    JTextField namesField = new JTextField("");
    namesField.setEditable(false);
    add(namesField, BorderLayout.SOUTH);

    Coordinator coordinator = new Coordinator(new Coordinator.Listener() {
      @Override
      public void onChangeNames(String[] names) {
        StringJoiner sj = new StringJoiner(",");
        for(String name : names) sj.add(name);
        namesField.setText(sj.toString());
      }
    });

    numberOfPeopleButton.addActionListener(new ActionListener() {
      @Override
      public void actionPerformed(ActionEvent e) {
        centerPanel.removeAll();
        coordinator.clear();

        int n = (Integer)(numberOfPeopleSpinner.getValue());
        for(int i=0; i<n; i++) {
          JTextField nameField = new JTextField();
          coordinator.add(nameField);
          nameField.getDocument().addDocumentListener(coordinator);
          centerPanel.add(nameField);
        }
        centerPanel.revalidate();
      }
    });
  }

  static class Coordinator implements DocumentListener {
    interface Listener {
      void onChangeNames(String[] names);
    }
    private List<JTextField> nameFieldList = new ArrayList<>();
    private Listener listener;
    Coordinator(Listener l) {
      this.listener = l;
    }
    void add(JTextField nameField) {
      nameFieldList.add(nameField);
    }
    void clear() {
      nameFieldList.clear();
      listener.onChangeNames(new String[0]);
    }
    private String[] getNames() {
      String[] names = new String[nameFieldList.size()];
      int i = 0;
      for(JTextField field : nameFieldList) {
        names[i++] = field.getText();
      }
      return names;
    }
    @Override
    public void insertUpdate(DocumentEvent e) {
      listener.onChangeNames(getNames());
    }
    @Override
    public void removeUpdate(DocumentEvent e) {
      listener.onChangeNames(getNames());
    }
    @Override
    public void changedUpdate(DocumentEvent e) {
      listener.onChangeNames(getNames());
    }
  }
}

Comments

[jimbe]

ご質問へのコメントで書いたような仕様から作ってみたコードは以下に入れてあります。
(JList でチェックボックス操作は面倒だったので JTable に変えました。)

https://github.com/Jimbe-github/teratail_java/tree/master/workspace2/teratail_java/src/teratail_java/q366141

[Beginner.java]

@jimbeさん　ご回答ありがとうございます。何から何まで本当に助かりました。また、機会があればぜひご教授いただけますと幸いです。本日はお忙しいところありがとうございました。