JButtonの画面を消す・ダイアログを指定のボタンで開く

Question

質問が2点あります。
どちらもJボタンに関してのことです。

1
現状は、Aframeで (if(e.getsource()==Button)Bframe.setVisible(true))
の処理をおこなって、Bframeを表示している状態です。
そこで、Bframeの閉じるボタン（B_toziru)を押して
Bframeを非表示・破棄したいです。
イメージ的には(JFrame.DISPOSE_ON_CLOSE)の感じです。

下記に自分が書いたコードを記載します。
Bframe.setVisible(false)では非表示になりませんでした。
ネット等で調べましたが、エラー等もでないので原因が分かりません。

使用するメソッドが違うのか、文の書き方が違うのか、ご教授お願いします。

public void actionPerformed(ActionEvent e) {
　BFraem f =new BFrame();
　if(e.getSource()== B_toziru);{
	f.dispose();
		}

2
Bframe内で閉じるボタン（B_toziru）とファイルの参照ボタン（B_sannsyou）が2つあります。
参照ボタンを押すとJFileChooserメソッドのダイアログを表示したいのですが、
現状では閉じるボタンを押しても、参照ボタンを押しても、
どちらでもダイアログが開いてしまいます。

エラーは出力されず原因がわかりません。
上記と同様で、ご教授お願いします。

下記のコードは現状、私が書いたコードになります。

public void actionPerformed(ActionEvent e) {
　if(e.getSource() == B_sannsyou);
　JFileChooser();}

public void JFileChooser() {
　JFileChooser fc = new JFileChooser();
　fc.setDialogTitle("フォルダーの参照");
　fc.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);

　int selected = fc.showSaveDialog(this);
　if (selected == JFileChooser.APPROVE_OPTION){
　File file = fc.getSelectedFile();
　txtCusersmasamdesktop.setText(file.getName());｝

Answer 1

1の解決方法
swingのJFrameクラスの作成時、自動生成されるmainクラスに
BFrame frame= new Bframe();
がrunメソッド内にありましたが、これをmainクラスから外に出し、下記のように記述しました。
private static　BFrame frame= new Bframe();

どこからでも参照できるようにし、下記のコードで非表示にはできました。

public void actionPerformed(ActionEvent e) {
//戻るボタンの設定
		if (e.getSource() == B_toziru) {
			Bframe.frame.setVisible(false);}
　　　　　}

Aframeクラスの(if(e.getsource()==Button)Bframe.setVisible(true))を
(if(e.getsource()==Button)Bframe.main(null);
に修正しました。

2の解決方法
共通のactionPerformedで振り分けが出来なかったため、回答者さんのURLを参考に
ボタンごとに書きなおしました。

Answer 2

１
BFraem f =new BFrame(); が不要です。
この f は２つめのまだ表示されていない Bframe 画面です。
オジェクト指向では、new するごとに、いくつでも分身画面ができてしまうのです。
閉じるべき画面は
「Aframeで (if(e.getsource()==Button)Bframe.setVisible(true))」
した f です。newしたとき f をインスタンス変数に代入しておいて、それを閉じればＯＫ。

２
両ボタンとも、同じactionPerformedにきてしまっているからです。
ActionListener、actionPerformed をボンタごとに記述するか、
あるいは、共通でいくならactionPerformedで振り分ける、です。

こちら様[Let'sプログラミング]のサイトがわかりやすいかと思います。
両方のやり方がのってます。
https://www.javadrive.jp/tutorial/event/index1.html
https://www.javadrive.jp/tutorial/event/index2.html

Comments

[mmatt]

回答ありがとうございます。
＜＜new するごとに、いくつでも分身画面ができてしまうのです。
知らなかったです、勉強になりました。

＜＜newしたとき f をインスタンス変数に代入しておいて
インスタンス変数や参照の仕方を勉強中でして、上記の意味が理解できていません。。
ｆは既にインタンス変数になっているのでは、ないでしょうか？
詳細な説明して頂ければ、ありがたいです。

＜＜共通でいくならactionPerformedで振り分ける
これの通り下記コードで試してみたのですが、質問時と同様で、ボタンの区別が出来ていないようです。
if (e.getSource() == B_toziru) {
System.out.println("a");　　こちらはテスト用で出力aにしています。

}else if(e.getSource() == B_sannsyou);{
JFileChooser();
コンソール結果はaと出力され、ダイアログも開いてしまいます。
なにかお気づきの点ありましたら、再度ご教授お願いします。
現在、下記の方法を試しています。
＜＜ActionListener、actionPerformed をボンタごとに記述するか