内部クラスとaddActionListenerについて

今、swingを使って、文字列の入力をした後に、ボタンを押すと、ある関数によって文字列が変換され、それがアウトプットのテキストフィールドに表示されるアプリケーションを作成しています。内部クラスを使ってaddActionListenerを使用しようと思っているのですが、ボタンの関連付けのところで困っています。外部クラス内で宣言、内部クラス内で宣言の二パターンを試したのですが、両方ともコンパイルエラーになってしまいます。よろしくお願いします。

java
1import java.awt.*;
2import java.awt.event.*;
3import javax.swing.JButton;
4import javax.swing.JComboBox;
5import javax.swing.JFrame;
6import javax.swing.JLabel;
7import javax.swing.JPanel;
8import javax.swing.JTextField;
9
10public class CaesarFrame{
11	/**
12	 * 
13	 */
14	private static final long serialVersionUID = 1L;
15	JFrame frame;
16	JTextField textField1, textField2;
17	JButton button1;
18	JPanel panel1, panel2;
19	JComboBox<Object> comboBox1;
20	Object[] ob = {'A','B','C','D','E','F','G','H','I','J','K','L'
21			,'M','N','O','P','Q','R','S','T','U','V','W','X','Y','Z'};
22	Container contentPane;
23	JLabel label;
24	
25	CaesarFrame(){
26		
27		
28		frame = new JFrame("SwingLab");
29		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
30		frame.setSize(400, 110);
31		frame.setLayout(new BorderLayout());
32		frame.setLocationRelativeTo(null);
33		
34		
35		contentPane = frame.getContentPane();
36		textField1 = new JTextField(20);
37		textField2 = new JTextField(28);
38		textField2.setEditable(false);
39		button1 = new JButton("Code!");
40		
41		comboBox1 = new JComboBox<Object>(ob);
42		
43		
44		label = new JLabel("Output:");
45		
46		panel1 = new JPanel();
47		panel2 = new JPanel();
48		
49		panel1.add(comboBox1);
50		panel1.add(textField1);
51		panel1.add(button1);
52		
53		panel2.add(label);
54		panel2.add(textField2);
55		
56		contentPane.add(panel1, BorderLayout.NORTH);
57		contentPane.add(panel2,BorderLayout.WEST);
58		
59		frame.setVisible(true);
60		
61		class OkButtonActionListener implements ActionListener{
62			String message = textField1.getText();
63			char offset = (char) comboBox1.getSelectedItem();
64　　　　　　　　　　　　　//これがエラーになってしまう
65			button1.addActionListener(new OkButtonActionListener());
66                 
67			
68			@Override
69			public void actionPerformed(ActionEvent e) {
70				textField2.setText(Caesar.caesarCode(message,offset));
71			}		
72		}
73	}		
74}

退会済みユーザー

2017/10/17 23:32

無限ループなコンパイルエラーだな。(笑)

tkturbo

2017/10/18 01:07

コンパイルエラーならその時のエラーメッセージ載せようよ。。。

行動規範の内容に同意します

回答1件

ベストアンサー

###文法エラーの原因
まず処理の正しさについては脇に置いて、Javaの文法についてのみ説明しますと・・・

OkButtonActionListenerのクラス定義本体は内部クラスでないクラスと同様、定義本体のトップレベルにはフィールド定義、メソッド定義、初期化ブロック等々記述できるものが決まっています。

button1.addActionListener(...);

は言語仕様的に表現すると文(より正確にはexpression statement)です。文はクラスの定義本体のトップレベルには書けませんのでコンパイルエラーとなります。

###やりたいことに対する対処方法
文法エラーを単に直すのではなく、本来やりたいであろうことをどうするかを考えます。

イベントハンドラーの登録とはどういうふうにやるべきなのか質問者さんは勘違いしておられるような気もしますが・・・
普通、イベントハンドラーの登録は画面を初期化する際に全部やるものであって、イベントハンドラーの中でイベントハンドラーを設定(addXxxListener)をしようとする考え方はちょっと異質です。質問者さんのコードはそういうことをしようとしています。

画面の初期化の際にやればよいので、OkButtonActionListenerという名前付きの内部クラスを定義する必要は必ずしもないです。またボタンが押されたときにやるべきことだけをハンドラーメソッドの中に書くようにします。

java
1frame.setVisible(true);
2button1.addActionListener(new ActionListener() {
3  @Override
4  public void actionPerformed(ActionEvent e) {
5    String originalText = textField1.getText();
6    String convertedText = ....; // 文字列の変換処理
7    textField2.setText(convertedText);
8  }
9});

元々のコードにあるメソッド定義が書いてないので本来はどういう動きにしたかったのか質問文だけからはわかりませんでした。上記は単にtextFiled1に入力されている文字列をボタンを押した時になんらかの方法で変換し、その結果をtextField2へ設定する例です。

もしこのハンドラーが呼び出される度に（ボタンが押される度に）異なることをさせるといった場合、ActionListenerのフィールドとして状態を覚えておくこともできます。例えば1回目にボタンを押したらtextField1の最初の文字をtextField2へ表示、2回目は2番目の文字、・・・とするような場合次のように書けます。

java
1frame.setVisible(true);
2button1.addActionListener(new ActionListener() {
3  int pos = 0; // これはローカル変数ではなくフィールド定義であることに注意
4
5  @Override
6  public void actionPerformed(ActionEvent e) {
7    String s = textField1.getText();
8    if (s.isEmpty()) return;
9    pos = pos % s.length();
10    textField2.setText(String.format("char at %d is %s", pos, s1.charAt(pos)));
11    pos++;
12  }
13});