帮我看看这段代码哪里出问题了(简单问题，求解决啊)

Question

package notepad;import java.util.logging.Level;
import java.util.logging.Logger;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
import java.io.*;public class NotePad extends JFrame implements ActionListener{    JTextArea jta = null;
    JMenuBar jmb = null;
    JMenu jm = null;
    JMenuItem jm1 = null;
    JMenuItem jm2 = null;    public static void main(String[] args) {
        NotePad np = new NotePad();
    }    public NotePad() {
        setSize(400, 300);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        jta = new JTextArea();
        jmb = new JMenuBar();
        jm = new JMenu("文件");
        //设置助记符（单引号）
        jm.setMnemonic('F');
        jm1 = new JMenuItem("打开");    
        //注册监听器
        jm1.addActionListener(this);
        jm1.setActionCommand("open");
        jm2 = new JMenuItem("保存");
        //注册监听器
        jm2.addActionListener(this);
        jm2.setActionCommand("save");
        //将组建添加到JFrame中
        jm.add(jm1);
        jm.add(jm2);
        jmb.add(jm);
        setJMenuBar(jmb);
        add(jta);
        setVisible(true);
    }
    @Override
    public void actionPerformed(ActionEvent e) {
           if (e.getActionCommand().equals("save"))
        {
                //文件选择组建
                JFileChooser jfc=new JFileChooser();
                jfc.setDialogTitle("保存为：");
                //按默认的方式显示
                jfc.showSaveDialog(null);
                jfc.setVisible(true);
                String file=jfc.getSelectedFile().getAbsolutePath();
                FileWriter fw=null;
                BufferedWriter bw=null;
            try {
                fw =new FileWriter(file);
                bw=new BufferedWriter(fw);
                bw.write(this.jta.getText());
            } catch (IOException ex) {
                Logger.getLogger(NotePad.class.getName()).log(Level.SEVERE, null, ex);
            }
            finally{
                try {
                    fw.close();
                    bw.close();
                } catch (IOException ex) {
                    Logger.getLogger(NotePad.class.getName()).log(Level.SEVERE, null, ex);
                }
               
            }
                
        }else
        if(e.getActionCommand().equals("open"));
       {
           
                //文件选择组建
                JFileChooser jfc=new JFileChooser();
                jfc.setDialogTitle("请选择组建：");
                //按默认的方式显示
                jfc.showOpenDialog(null);
                jfc.setVisible(true);
                String filename=jfc.getSelectedFile().getAbsolutePath();
                FileReader fr=null;
                BufferedReader br=null;
            try {
                fr=new FileReader(filename);
                br=new BufferedReader(fr);
                String s="";
                String con="";
                try {
                    while((s=br.readLine())!=null)
                    {
                        con+=s+"\r\n";
                    }
                } catch (IOException ex) {
                    Logger.getLogger(NotePad.class.getName()).log(Level.SEVERE, null, ex);
                }
                jta.setText(con);
            } catch (FileNotFoundException ex) {
                Logger.getLogger(NotePad.class.getName()).log(Level.SEVERE, null, ex);
            }finally{
                try {
                    fr.close();              
                    br.close();
                }catch (IOException ex) {
                    Logger.getLogger(NotePad.class.getName()).log(Level.SEVERE, null, ex);
                }
            }
       }
        
  
            }
}

Answer 1

1.String file=jfc.getSelectedFile().getAbsolutePath();要对file进行判断，不选择存文件的时候会返回null值导致你的程序异常；
2.fr.close();   
  br.close();
去掉fr.close()。已经用BufferedWriter将FileWriter包装了，只要关br，就会自动去关fr的。你这样写，fr先关闭了，会导致IO异常，因为fr流已经被关闭了。
3.程序逻辑还有问题，会导致保存的对话框重复弹出一次，自己检查。

Answer 2

  if(e.getActionCommand().equals("open"));
  {
楼主好好反省吧
还有bw.flush();要养成好习惯

Answer 3

问题找到了，原来是我不小心在for语句的括号后面多了个分号。你说的第一个可能在我这个简单的程序里面体现不出来。