addとremoveActionListener

質問

以下のプログラムは作っているプログラムの簡易版で、他のクラスからaddActionListenerを追加しようとしているですが、やり方が分かりません。
どうプログラミングすればいいか教えてください。
あと、removeActionListenerを使いたいのですが、これについて書かれているサイトが少なく、使い方がよくわからないので、こちらも教えて欲しいです。

import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
import javax.swing.JPanel;

public class Test1 extends JFrame {
public Test1(){
super("test1");
Set set=new Set(this);
setBounds(100,100,806,659);
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
setResizable(false);
add(set);
JMenuBar menubar = new JMenuBar();
JMenu menu1=new JMenu("メニュー");
menubar.add(menu1);
JMenuItem menuitem1=new JMenuItem("終了");
menu1.add(menuitem1);
setJMenuBar(menubar);
}
public static void main(String[] args){
new Test1().setVisible(true);
}
}
class Set extends JPanel{
Set(final JFrame frame){
/*ここにaddを入れたかった
frame.menuitem1.addActionListener(new ActionListener(){
public void actionPerformed(ActionEvent e){
System.exit(0);
}
});*/
//removeActionListener　使用方法不明
}
}

回答No.2

LancerVII · ベストアンサー

こんにちは。

ほかのクラスからaddしたい理由はわかりませんが、やるとしたら現状のソースの変数のスコープとか考えないといけません。

まず、現状のソースで出来ない理由。
menuitem1がローカル変数であるため外部から参照できない。
仮にmenuitem1をフィールド変数にしても現状の呼び出しだとnullの為、NullPointerExceptionが発生する。
SetにJFrameを渡す必要があるとしたらJFrameではなくTest1ではないか？
(これだけのソースを見る限りでは渡す必要性がわからない)

以上を踏まえて直してみると
public class Test1 extends JFrame {

// フィールド変数にすることにより参照が可能
private JMenuItem menuitem1;

public Test1() {
super("test1");
setBounds(100,100,806,659);
setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
setResizable(false);
JMenuBar menubar = new JMenuBar();
JMenu menu1=new JMenu("メニュー");
menubar.add(menu1);
menuitem1=new JMenuItem("終了");
menu1.add(menuitem1);
setJMenuBar(menubar);
// menuitem1=new JMenuItem("終了")以降に呼ぶ
Set set=new Set(this);
add(set);

}

public static void main ( String[] args ) {
new Test1().setVisible ( true );
}

class Set extends JPanel{
Set(final Test1frame){
// menuitem1=new JMenuItem("終了")以降に呼ばないとmenuitem1がnullの為エラーになる。
menuitem1.addActionListener(new ActionListener(){
public void actionPerformed(ActionEvent e){
System.out.println ( "押された");
// removeActionListenerは設定されたリスナーを削除するので以下のような感じで使う
// menuitem1に設定されているリスナーをすべて取得(配列)して最初のリスナーを削除する
// すなわち、一度メニューが選択されると次からはリスナーが削除されているため「押された」が表示されなくなる
menuitem1.removeActionListener(menuitem1.getActionListeners()[0]);
}
});
}
}
}

お礼

わかりやすかったです
返信ありがとうございます

回答No.1

noname#177743

いろいろ突っ込むところはあるのですが、もっとも根本的な部分で。

プログラムの流れをよく見てください。

まず、Setインスタンスを作っていますね。そしてそれをJFrameにaddしています。

続いて、JMenuBarを作り、組み立てています。

/*ここにaddを入れたかった　という部分は、Setのコンストラクタ、つまりSetインスタンスを作るときに呼び出される部分です。

ということは、/*ここにaddを入れたかった　の部分が呼び出されるとき、まだメニューは存在しないわけですね。存在しないメニューにどうやってイベントを組み込むのでしょう？

なぜ、そういう妙なことになってるのかといえば、「そのオブジェクトに関することはそのオブジェクトの中で完結させる」というオブジェクト指向の基本ができていないからです。

その上のTest1のコンストラクタ内でJMenuItemインスタンスは組み込まれていますね。なぜ、JFrameで組み込んだJMenuItemのイベントリスナーを、全く関係のないJPanelのコンストラクタで設定するのでしょう？　どういう理由でやっているのか、何をやろうとしているのかがわからないと、なんとも回答のしようがないのですが……。

お礼

返信ありがとうございます。
正直、作成している時はJMenuBarがSetの前に作成されていないことに気づかなかったです・・・
JMenuItemをSetでやっているのは、このプログラムは簡略化したもので、実際は色々処理がある中でこの処理を入れようとしているから、といった感じでしょうか