java itemlisetenerの質問

Question

itemlistenerの部分で質問です
表示がうまくいきません

import java.awt.Color;
import java.awt.GridLayout;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;

import javax.swing.ButtonGroup;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JRadioButton;

@SuppressWarnings("serial")
public class No1 extends JFrame implements ItemListener {
	JRadioButton[] rb;
	String[] str = { "3本", "4本", "5本", "6本" };
	JPanel p, p1, p2;
	JLabel[][] s;
	JLabel[] l, a;

	No1() {
		p = new JPanel();
		p1 = new JPanel();
		p2 = new JPanel();
		rb = new JRadioButton[4];
		l = new JLabel[5];
		a = new JLabel[5];
		s = new JLabel[5][5];
		ButtonGroup bg = new ButtonGroup();
		for (int i = 0; i < rb.length; i++) {
			rb[i] = new JRadioButton(str[i]);
			bg.add(rb[i]);
			rb[i].addItemListener(this);

		}

		p.setLayout(new GridLayout(2, 1));
		p1.add(rb[0]);
		p1.add(rb[1]);
		p1.add(rb[2]);
		p1.add(rb[3]);

		p2.setLayout(new GridLayout(5, 1));
		for (int i = 0; i < l.length; i++) {
			l[i] = new JLabel();
			p2.add(l[i]);
		}

		p.add(p1);
		p.add(p2);

		add(p);

		a[0] = new JLabel("赤いワイヤー");   a[0].setForeground(Color.RED);
		a[1] = new JLabel("青いワイヤー");   a[1].setForeground(Color.BLUE);
		a[2] = new JLabel("黄色いワイヤー"); a[2].setForeground(Color.YELLOW);
		a[3] = new JLabel("黒いワイヤー");   a[3].setForeground(Color.BLACK);;



		s[0][0] = new JLabel( "ア）"+a[0]+"ない場合、ワイヤー２を切断します。");
		s[0][1] = new JLabel("イ）一番下のワイヤーが白い場合、そのワイヤーを切断します。");
		s[0][2] = new JLabel("ウ）青いワイヤーが複数ある場合、青いワイヤーの中で一番下のものを切断します。");
		s[0][3] = new JLabel("エ）ワイヤー３を切断します。");

		s[1][0] = new JLabel( "ア）赤いワイヤーが複数あり、かつシリアルナンバーの最後の桁が奇数なら、赤いワイヤーの中で一番下のものを切断します。");
		s[1][1] = new JLabel("イ）一番下のワイヤーが黄色く、かつ赤いワイヤーがない場合、ワイヤー１を切断します。");
		s[1][2] = new JLabel("ウ）青いワイヤーが一本だけの場合、ワイヤー１を切断します。");
		s[1][3] = new JLabel("エ）黄色いワイヤーが複数ある場合、ワイヤー４を切断します。");
		s[1][4] = new JLabel("オ）ワイヤー２を切断します。" );

		s[2][0] = new JLabel( "ア）一番下のワイヤーが黒く、かつシリアルナンバーの最後の桁が奇数なら、ワイヤー４を切ります");
		s[2][1] = new JLabel("イ）赤いワイヤーが一本だけ、かつ黄色いワイヤーが複数ある場合、ワイヤー１を切断します。");
		s[2][2] = new JLabel("ウ）黒いワイヤーがない場合、ワイヤー２を切断します。");
		s[2][3] = new JLabel("エ）ワイヤー１を切断します。" );

		s[3][0] = new JLabel("ア）黄色いワイヤーがなく、かつシリアルナンバーの最後の桁が奇数なら、ワイヤー３を切断します");
		s[3][1] = new JLabel("イ）黄色いワイヤーが一本だけ、かつ白いワイヤーが複数ある場合、ワイヤー４を切断します。");
		s[3][2] = new JLabel("ウ）赤いワイヤーがない場合、ワイヤー６を切断します。");
		s[3][3] = new JLabel("エ）ワイヤー４を切断します。");

	}

	@Override
	public void itemStateChanged(ItemEvent e) {

		if (rb[0].isSelected()) {
			for (int i = 0; i < 5; i++) {
				if (i == 4) {
					l[4].setText("");
				} else {
					l[i].add(s[0][i]);
				}
			}
		} else if (rb[1].isSelected()) {
			for (int i = 0; i < 5; i++) {
				l[i].add(s[1][i]);
			}
		} else if (rb[2].isSelected()) {
			for (int i = 0; i < 5; i++) {
				if (i == 4) {
					l[4].setText("");
				} else {
					l[i].add(s[2][i]);
				}
			}
		} else if (rb[3].isSelected()) {
			for (int i = 0; i < 5; i++) {
				if (i == 4) {
					l[4].setText("");
				} else {
					l[i].add(s[3][i]);
				}
			}
		}
		for(int i=0; i<l.length; i++) {
			p2.add(l[i]);
		}
		p.add(p2);
		add(p);
		p.repaint();
	};

	public static void main(String[] args) {
		No1 f = new No1();

		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setTitle("配線");
		f.setSize(760, 300);
		f.setVisible(true);
	}

}

Comments

[swordone]

コードマークダウンは'(SHIFT+7)ではなく`(SHIFT+@)です。

[TakuyaAsaka]

編集しました。失礼しました

[LouiS0616]

どういうときに上手く行って、どういうときに上手く行かないんですか？どういう風な動作を期待しているんですか？

[TakuyaAsaka]

ラジオボタンからの入力でラベルへの出力をしたいのですが、ラジオボタンのみ表示され、他反応なしです。

[LouiS0616]

addItemListenerは使っていますか？

[TakuyaAsaka]

使っています

[LouiS0616]

メソッド自体は確かに呼び出されているんですか？

[TakuyaAsaka]

更新しました

Answer 1

レンダリング属性(色)付きの文字列(テキスト)を簡単に配置してリッチテキストのような表現としたいということですね。

###何も表示されない原因
JLabelへ別のJLabelをaddしているからです。itemStateChangedで、l[i].add(s[0][i])のようなコードとしていますが、l[i]はJLabelでs[0][i]もJLabelですね。addメソッドが定義されているのであたかもJLabelの子供コンポーネントとして別のJLabelが配置できるかのように思えてしまいますが、実際はできません(※1)。

###対処方法
色付きの文字列をJLabelで表現するという前提なら、それを配置する親コンポーネントl[i]はJPanelとします。JPanelのレイアウトはデフォルトのFlowLayoutでよいでしょう。
ボタンをクリックする度にテキストを置き換える必要がありますが、例えば次のようにする方法が考えられます。

class ColorLabel extends JLabel {
  ColorLabel(String text) {
    super(text);
  }

  ColorLabel(String text, Color color) {
    this(text);
    setForeground(color);
  }
}

public class No1 ... {
  JPanel[] l = new JPanel[5];
  JLabel[][][] s = new JLabel[5][][]; //いわゆるジャグ配列にする
  ...
  No1() {
    ...
    for (int i = 0; i < l.length; i++) {
      l[i] = new JPanel();
      p2.add(l[i]);
    }
    ...
    s[0] = new JLabel[][] = {
      {
        new ColorLabel("ア）"),
        new ColorLabel("赤いワイヤー", Color.RED),
        new ColorLabel("ない場合、ワイヤー２を切断します。"),
      },
      { new ColorLabel("イ）一番下のワイヤーが白い場合、そのワイヤーを切断します。") },
      {
        new ColorLabel("ウ）"),
        new ColorLabel("青いワイヤー", Color.BLUE),
        new ColorLabel("が複数ある場合、青いワイヤーの中で一番下のものを切断します。"),
      },
      ...
    };
    s[1] = new JLabel[][] = {
      ...
    };
    ...
  }

  public void itemStateChanged(ItemEvent e) {
    for (int buttonIndex = 0; buttonIndex < rb.length; buttonIndex++) {
      if (rb[buttonIndex].isSelected()) {
        for (int rowIndex = 0; rowIndex < l.length; rowIndex++) {
          l[rowIndex].removeAll(); // 以前表示していたものを全て取り去る
          if (rowIndex < s[buttonIndex].length) {
            for (JLabel label : s[buttonIndex][rowIndex]) {
              l[row].add(label);
            }
          }
        }
      }
    }
    // コンポーネント階層を変更したのでrevalidateを呼び出す
    // revalidateはJPanelの下に配置した子供コンポーネントの位置や大きさや
    // JPanel自身の大きさを全て再計算し適切に配置しなおしてくれる。
    // またレイアウトしなおすことは表示内容が変わることを意味するので
    // repaintを呼び出さなくてもswingが自動的に呼び出してくれる。
    p2.revalidate();
  }
}

---

補足1: 元のコードでは配列aに色を付けたJLabelがあらかじめ定義されているので、自分のコード例にあるnew ColorLabel("赤いワイヤー", Color.RED)の代わりにa[0]と書きたくなるかも知れません。しかしそれはお勧めしません。swingのコンポーネントはあるインスタンスを画面上の複数個所に配置できないからです。面倒でも個別にnew ColorLabel...のようにしてインスタンスを生成する必要があります。

補足2: 色付きのテキストをJLabelの並びで表現するのは最も素直な方法とはいえないかも知れません。JLabelの設定テキストにはHTML形式の文字列を指定できるので以下のようにするのも手段の一つになるかも知れません。
new JLabel("<html><head></head><body>ア）<font color='red'>赤いワイヤー</font>...")
またJTextPaneには色などの属性付きのテキストを表現できるのでそれを使うのも一つの方法ですが、いずれにしても若干面倒なコードになります。

---

※1: JLabelへaddメソッドがあるのに使ってはいけない理由
JLabelはswingにおける「子供を持たないコンポーネントクラス(=JComponent)」として定義されています。しかしJComponentは古いGUIライブラリーであるawtでの「子供を持てるコンテナークラス(=Container)」の派生として定義されています。
これはJComponentがawtの複数のComponentで構成されているという実装の都合によるもので、実際には「子供を配置してはいけないもの」です。addメソッドがコンパイルエラーにならずに呼び出せてしまう理由は単にJComponentがContainerの派生でありContainerにaddメソッドが定義されていたからなんですね。このあたりawt/swingの関係に注意していないとわかりにくいですね・・・(余談ですがJava8からの標準GUIライブラリーであるJavaFXは、swingとは継承関係がない新規のGUIライブラリーとして設計されており、こうしたいびつなインターフェース上の問題はなくなっています)

Answer 2

簡単な修正で直りました。
文字列自体を扱う際は、JLabelではなくStringを用います。
つまり、質問者様が使っているlとsをString型に変更してください。

この場合、lに関する処理を次のように変更することになります。
l[i].add(s[num][i]);→l[i] += s[num][i];

---

また、同様のミスを減らすためにも、変数名を変えることを強くお勧めします。
今のコードでは、クラス名とリテラル以外の情報がほぼありません。
適切に命名しなおすか、適切なコメントを付けてください。

Comments

[TakuyaAsaka]

Stringにした場合、色つけはどうなりますか？

[LouiS0616]

例えば文字列に色を付けたいときは、その文字をJLabelに貼り付けてください。
前もってラベル配列coloredLabel[]を用意しておいて、

```Java
for(int i=0; i<coloredLabel.length; i++ ) {
coloredLabel[i].setText(l[i]);
coloredLabel[i].setForeground(Color.BLUE); // 例えばね
p2.add(coloredLabel[i]);
}
```

このとき予想されるエラーはいわゆるヌルポインタアクセスです。
それらしいエラーが出たときは、おそらくcoloredLabelのスコープの問題ですから、宣言する位置を変えて色々試してみてください。