コンボボックスで選ばれたものを正常に出力したい

Question

複数のコンボボックスから来た信号を選別し、表示させるコードを書いたのですが、ラベルを使ってパネルに書き込む方は正常なのですがコンソール出力の方だけなぜがそれぞれ2文づつ出力されてしまいます。対処法を教えていただきたいです。

import java.awt.BorderLayout;
import java.awt.event.ItemEvent;
import java.awt.event.ItemListener;

import javax.swing.JComboBox;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;

public class Ctest extends JFrame implements ItemListener {
	String[] combodata = { "Swing", "Java2D", "Java3D", "JavaMail" };
	String[] combodata2 = { "A", "B", "C", "D" };
	JComboBox combo1 = new JComboBox(combodata);
	JComboBox combo2 = new JComboBox(combodata2);
	JLabel label;

	public static void main(String[] args) {
		Ctest frame = new Ctest();

		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setBounds(10, 10, 300, 200);
		frame.setTitle("タイトル");
		frame.setVisible(true);
	}

	Ctest() {
		combo1.addItemListener(this);
		combo2.addItemListener(this);
		JPanel p = new JPanel();
		p.add(combo1);
		p.add(combo2);

		label = new JLabel();

		JPanel labelPanel = new JPanel();
		labelPanel.add(label);

		getContentPane().add(p, BorderLayout.CENTER);
		getContentPane().add(labelPanel, BorderLayout.PAGE_END);
	}

	public void itemStateChanged(ItemEvent e) {
		if (e.getItemSelectable() == combo1) {
			String data = "combo1から" + (String) combo1.getSelectedItem() + "が選ばれた";
			label.setText(data);
			System.out.println("combo1から" + (String) combo1.getSelectedItem() + "が選ばれた");
		} else if (e.getItemSelectable() == combo2) {
			String data = "combo2から" + (String) combo2.getSelectedItem() + "が選ばれた";
			label.setText(data);
			System.out.println("combo2から" + (String) combo2.getSelectedItem() + "が選ばれた");
		}
	}
}

Comments

[jimbe]

改行が変になっているようですが…。

> コンソール出力の方だけなぜがそれぞれ2文づつ出力されて
label のほうは例え 2 回書き込んでも上書きされるだけですから、判断付かないだけかと思います。

itemStateChanged メソッドはどのような時に呼ばれるでしょう。

(itemStateChanged でパラメータ e の表示をしてみると分かり易いかと思います。)

[raiboz1115]

itemStateChanged メソッドって戻り値intですよね？
どちらのコンボボックスから信号を受け取ったかをitemStateChanged メソッドで判別できますか？

[raiboz1115]

追記
改行のズレは直しました。

[jimbe]

ありがとうございます。

itemStateChanged の戻りは void ですが・・・表示のお勧めをしているのは
> itemStateChanged(ItemEvent e)
の ItemEvent 型の仮引数 e です。この中に itemStateChanged が呼ばれた意味が含まれていますので。詳細は java のドキュメントに書かれていると思います。
public void itemStateChanged(ItemEvent e) {
System.out.println("itemStateChanged: ItemEvent="+e);
　　　　:
といった具合ですね。

どのコンボボックスからかは、お使いの通り getItemSelectable で良いのではないでしょうか。

[raiboz1115]

申し訳ありません、getStateChangeと勘違いしていました。
eを出力させて読んでみたところ、原因は初期設定されているコンボボックスの要素でも呼び出されているということであっていますでしょうか？
コンボボックスの判別はgetItemSelectableでいいのですね、了解しました。

[jimbe]

e の中身やドキュメント等を参照されましたら、e の中に「選択(SELECTED)か、選択解除(DESELECTED)か」を示すデータがあることにお気付きになられましたでしょうか。
itemStateChanged というメソッド名を訳すと、「アイテムの状態が変わった」という風になりますね。
この場合のアイテムとは、コンボボックスに設定されている個々の選択肢のことです。(getSelectedItem の Item もその意味ですね。)
コンボボックスは単一選択、つまり、複数の中で同時に一つしか選択出来ません。一つしか選択できないということは、別のものを選択したら、それまで選択していたものは選択解除される、ということです。
これが2回 itemStateChanged が呼ばれる理由です。

[raiboz1115]

ありがとうございます、完璧に理解できました！

Answer 1

JComboBox の選択が変わったら一回だけ通知を受ければ良い、ということでしたら、　JButton と同じく ActionListener が使えます。