4. Problem mit Kapselung! Hilfe?

Problem mit Kapselung! Hilfe?

  • ?

    Hi,

    ich habe folgendes Problem und hoffe mir kann wer helfen. Der nachfolgende Quelltext ist meine Version eines TextEditors, dessen einzelne Bestandteile und dessen Nachrichtenbeahandlung mehr oder weniger in eine einzige Klasse gestopft ist. Also nicht wirklich elegant nach den Regeln der OOP. Er funktioniert aber einwandfrei.

    import java.io.*;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

public class JTextEditorBETA
extends JFrame
implements ActionListener
{
    static final private String OPEN = "Öffnen";
    static final private String SAVE = "Speichern";
    static final private String END ="Beenden";

    private JMenuBar mbar;
    private JMenu menu;
    private JMenuItem item;
    private JTextArea textarea;
    private JToolBar toolBar;
    private JButton button;

    public JTextEditorBETA(){
        super("Patrick Mittelbach`s  JTextEditor (BETA Version 1)");
        this.setLocation(200,200);
        this.setSize(600,500);
        mbar = new JMenuBar();
        menu = new JMenu("Datei");
        item = new JMenuItem("Öffnen");
        item.addActionListener(this);
        menu.add(item);
        item = new JMenuItem("Speichern");
        item.addActionListener(this);
        menu.add(item);
        item = new JMenuItem("Beenden");
        item.addActionListener(this);
        menu.add(item);
        mbar.add(menu);
        this.setJMenuBar(mbar);
        textarea = new JTextArea(); //vordefinierte Größe hier nicht sinnvoll

        //Erzeugung einer eingenen ContentPane
        JPanel myContentPane = new JPanel();
        myContentPane.setOpaque(true);
        this.setContentPane(myContentPane);

        //Erzeugen der Werzeugleiste und Steuerelemente
        toolBar = new JToolBar();
        button = new JButton(new ImageIcon("neu.gif"));
        button.setActionCommand(OPEN);
        button.setToolTipText("Neues Dokument");
        button.addActionListener(this);
        toolBar.add(button);
        button = new JButton(new ImageIcon("oeffnen.gif"));
        button.setActionCommand(SAVE);
        button.setToolTipText("Dokument öffnen");
        button.addActionListener(this);
        toolBar.add(button);
        button = new JButton(new ImageIcon("suchen.gif"));
        button.setActionCommand(END);
        button.setToolTipText("Speichern unter");
        toolBar.add(button);
        toolBar.setRollover(true);
        //evt falsch
        myContentPane.setPreferredSize(new Dimension(400,30));
        myContentPane.setLayout(new BorderLayout()); //BorderLayout dringend notwendig!!!!
        myContentPane.add(toolBar, BorderLayout.PAGE_START);
        myContentPane.add(new JScrollPane(textarea), BorderLayout.CENTER);
        this.setVisible(true);
    }

    public static void main(String args[]){
        JTextEditorBETA editor = new JTextEditorBETA();
        editor.setDefaultLookAndFeelDecorated(true);
        editor.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }

    public void actionPerformed(ActionEvent event) {
        String cmd = event.getActionCommand();
        if(cmd.equals("Öffnen")){
            String openPath = (String)JOptionPane.showInputDialog(
                this, 
                "Geben sie den Pfad der günschten Datei ein",
                "Öffnen",
                JOptionPane.QUESTION_MESSAGE
            );
            textarea.setText(readChoosenFile(openPath));
        } else
            if(cmd.equals("Speichern")) {
                String savePath = (String)JOptionPane.showInputDialog(
                    this,
                    "Geben Sie den gewünschten Dateipfad ein",
                    "Speichern",
                    JOptionPane.QUESTION_MESSAGE
                );
                File file = new File(savePath);
                if(file.exists()) {
                    JOptionPane.showMessageDialog(
                        this, 
                        "Die Datei existiert bereits",
                        "Warnung",
                        JOptionPane.WARNING_MESSAGE
                    );
                    System.exit(0);
                }else
                    if(file.isDirectory()) {
                        JOptionPane.showMessageDialog(
                            this, 
                            "Der Angegebende Datei ist ein Verzeichnis",
                            "Warnung", 
                            JOptionPane.WARNING_MESSAGE
                        );
                        System.exit(0);
                }
                else {
                    //try {
                        String string = textarea.getText();
                        this.saveFile(savePath, string);
                    /*}catch (IOException exception) {
                        JOptionPane.showMessageDialog(
                            this, 
                            "Es ist ein Fehler beim Erstellen der Datei aufgetreten",
                            "Warnung", 
                            JOptionPane.INFORMATION_MESSAGE
                        );
                    }*/
                }
            }
            else
                if(cmd.equals("Beenden")) {
                    this.setVisible(false);
                    dispose();
                    System.exit(0);
                }     
        }

    protected void saveFile(String file, String text) {
            if(file != "") {
                if(text.length() == 0) {
                    System.out.println("Kein Text zum schreiben da!");
                } else {
                    StringBuffer savebuffer = new StringBuffer();
                    try {
                        BufferedWriter buffwr = new BufferedWriter(
                            new FileWriter(file));
                        String str = new String();
                        str = textarea.getText();
                        BufferedReader strrd = new BufferedReader(
                            new StringReader(str));
                        while( (str = strrd.readLine()) != null){
                            savebuffer.append(str);
                            savebuffer.append(System.getProperties().getProperty("line.separator"));
                        }
                        buffwr.write(savebuffer.toString());
                        buffwr.close();
                        System.out.println("Der Text wurde in Datei geschrieben");
                    }catch (Exception e) {
                        System.out.println("Da lief was ganz gewaltig schief!");
                        System.out.println(e);

                    }
                }
            }
        }

    protected String readChoosenFile(String file) {
            StringBuffer buffer = new StringBuffer();
            try {
                BufferedReader buffrd = new BufferedReader(
                    new InputStreamReader(
                        new FileInputStream(file)));
                String line = new String();
                while( (line = buffrd.readLine()) != null)  {
                    buffer.append(line);
                    //buffer.append("\r");  --> Zeilentrenner unter Linux
                    //buffer.append("\r\n"); --> Zeilentrenner unter Windows
                    buffer.append(System.getProperties().getProperty("line.separator"));
                }
                buffrd.close();
                return buffer.toString();

            }catch(IOException e) {
                e.printStackTrace();
                return null;
            }
        }
 }

    Um dem objektorientierten Prinzipien gerecht zu werden, habe ich sämtliche Bestandteil des Editors in eigenen Klassen gekapselt und herausgekommen ist dabei dies; der leider nur zur "Hälfte" funktioniert.

    import java.io.*;
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;

class ApplicationMenuBar
extends JMenuBar
{
    private JMenu m_menu;
    private JMenuItem m_item;

    public ApplicationMenuBar(JFrame frame, JTextArea textarea){
        this.add(createJMenu(frame,textarea));
    }

    public JMenu createJMenu(JFrame frame,JTextArea textarea){
        m_menu = new JMenu("Datei");
        m_item = new JMenuItem("Neu");
        m_item.addActionListener(new MenuListener(frame,textarea));
        m_menu.add(m_item);
        m_item = new JMenuItem("Öffnen");
        m_item.addActionListener(new MenuListener(frame, textarea));
        m_menu.add(m_item);
        m_item = new JMenuItem("Speichern");
        m_item.addActionListener(new MenuListener(frame, textarea));
        m_menu.add(m_item);
        m_item = new JMenuItem("Beenden");
        m_item.addActionListener(new MenuListener(frame, textarea));
        m_menu.add(m_item);
        return m_menu;
    }

    class MenuListener
    implements ActionListener
    {  
        private JFrame m_frame;
        final JTextArea m_textarea;

        public MenuListener(JFrame frame, JTextArea textarea){
            m_frame = frame;
            m_textarea = textarea;
        }

        public void actionPerformed(ActionEvent actionEvent) {
            String cmd = actionEvent.getActionCommand();
            if(cmd.equals("Neu")){
                m_textarea.setText("");
            }
            if(cmd.equals("Öffnen")){ 
                new OpenFile(m_frame, m_textarea);
            }
            if(cmd.equals("Speichern")){
                new SaveFile(m_frame, m_textarea);
            }
            if(cmd.equals("Beenden")){
                m_frame.setVisible(false);
                m_frame.dispose();
                System.exit(0);
            }
        }
    }
}

class ApplicationToolBar
extends JToolBar
{   
    private JButton m_button;

    public ApplicationToolBar(JFrame frame, JTextArea textarea){
        super("Patrick`s Tools");
        m_button = new JButton(new ImageIcon("neu.gif"));
        m_button.setActionCommand("new");
        m_button.addActionListener(new ToolBarListener(frame, textarea));
        this.add(m_button);
        m_button = new JButton(new ImageIcon("oeffnen.gif"));
        m_button.setActionCommand("open");
        m_button.addActionListener(new ToolBarListener(frame, textarea));
        this.add(m_button);
        m_button = new JButton(new ImageIcon("suchen.gif"));
        m_button.setActionCommand("save");
        m_button.addActionListener(new ToolBarListener(frame, textarea));
        this.add(m_button);

    }

    class ToolBarListener
    implements ActionListener
    {
        private JFrame d_frame;
        private JTextArea d_textarea;

        public ToolBarListener(JFrame frame, JTextArea textarea){
            d_frame = frame;
            d_textarea = textarea;
        }

        public void actionPerformed(ActionEvent actionEvent) {
            String cmd = actionEvent.getActionCommand();
            if(cmd.equals("new")){
                d_textarea.setText("");
            }
            if(cmd.equals("open")){
                new OpenFile(d_frame, d_textarea);
            }
            if(cmd.equals("save")){
                new SaveFile(d_frame, d_textarea);
            }
        }
     }   
}

class OpenFile
{   
    public OpenFile(JFrame m_frame, JTextArea m_textarea){
        String openPath = (String)JOptionPane.showInputDialog(
                m_frame, 
                "Geben sie den Pfad der günschten Datei ein",
                "Öffnen",
                JOptionPane.QUESTION_MESSAGE
        );
        m_textarea.setText(readChoosenFile(openPath));
    }

    protected String readChoosenFile(String file) {
            StringBuffer buffer = new StringBuffer();
            try {
                BufferedReader buffrd = new BufferedReader(
                    new InputStreamReader(
                        new FileInputStream(file)));
                String line = new String();
                while( (line = buffrd.readLine()) != null)  {
                    buffer.append(line);
                    //buffer.append("\r");  --> Zeilentrenner unter Linux
                    //buffer.append("\r\n"); --> Zeilentrenner unter Windows
                    buffer.append(System.getProperties().getProperty("line.separator"));
                }
                buffrd.close();
                return buffer.toString();  
            }catch(IOException e) {
                e.printStackTrace();
                return null;
            }
        } 
}

class SaveFile
{   
    public SaveFile(JFrame m_frame, JTextArea m_textarea){
        String savePath = (String)JOptionPane.showInputDialog(
                    m_frame,
                    "Geben Sie den gewünschten Dateipfad ein",
                    "Speichern",
                    JOptionPane.QUESTION_MESSAGE
                );
                File file = new File(savePath);
                if(file.exists()) {
                    JOptionPane.showMessageDialog(
                        m_frame, 
                        "Die Datei existiert bereits",
                        "Warnung",
                        JOptionPane.WARNING_MESSAGE
                    );
                    System.exit(0);
                }else
                    if(file.isDirectory()) {
                        JOptionPane.showMessageDialog(
                            m_frame, 
                            "Der Angegebende Datei ist ein Verzeichnis",
                            "Warnung", 
                            JOptionPane.WARNING_MESSAGE
                        );
                        System.exit(0);
                }
                else {
                    //try {
                        String string = m_textarea.getText();
                        this.saveFile(savePath, string, m_textarea);
                }}

    protected void saveFile(String file, String text, JTextArea m_textarea) {
            if(file != "") {
                if(text.length() == 0) {
                    System.out.println("Kein Text zum schreiben da!");
                } else {
                    StringBuffer savebuffer = new StringBuffer();
                    try {
                        BufferedWriter buffwr = new BufferedWriter(
                            new FileWriter(file));
                        String str = new String();
                        str = m_textarea.getText();
                        BufferedReader strrd = new BufferedReader(
                            new StringReader(str));
                        while( (str = strrd.readLine()) != null){
                            savebuffer.append(str);
                            savebuffer.append(System.getProperties().getProperty("line.separator"));
                        }
                        buffwr.write(savebuffer.toString());
                        buffwr.close();
                        System.out.println("Der Text wurde in Datei geschrieben");
                    }catch (Exception e) {
                        System.out.println("Da lief was ganz gewaltig schief!");
                        System.out.println(e);

                    }
                }
            }
        } 
}

public class JTextEditorRC 
extends JFrame
{
    private JTextArea textarea;

    public JTextEditorRC(){
        super("Patrick Mittelbach´s JTextEditor (RELEASE CANDIDATE)");
        this.setLocation(200,200);
        this.setSize(500,400);
        this.setJMenuBar(new ApplicationMenuBar(this, textarea));
        textarea = new JTextArea(); //vordefinierte Größe hier nicht sinnvoll
        //Erzeugung einer eingenen ContentPane
        JPanel myContentPane = new JPanel();
        myContentPane.setOpaque(true);
        this.setContentPane(myContentPane);
        myContentPane.setPreferredSize(new Dimension(400,30));
        myContentPane.setLayout(new BorderLayout()); //BorderLayout dringend notwendig!!!!
        myContentPane.add(new ApplicationToolBar(this, textarea), BorderLayout.PAGE_START);
        myContentPane.add(new JScrollPane(textarea), BorderLayout.CENTER);
        this.setVisible(true);
    }

    public static void main(String args[]){
        JTextEditorRC editor = new JTextEditorRC();
        editor.setDefaultLookAndFeelDecorated(true);
        editor.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }
}

    Beim Speichern und Öffnen über die Werkzeugleiste funktionieren die Vorgänge, wird jedoch Speichern bzw. Öffnen über das Menü aufgerufen kommt folgende Fehlermeldung:

    java.lang.NullPointerException
        at TextEditor.SaveFile.<init>(JTextEditorRC.java:196)
        at TextEditor.ApplicationMenuBar$MenuListener.actionPerformed(JTextEditorRC.java:72)
        at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1834)
        at javax.swing.AbstractButton$Handler.actionPerformed(AbstractButton.java:2152)
        at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:420)
        at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:258)
        at javax.swing.AbstractButton.doClick(AbstractButton.java:302)
        at javax.swing.plaf.basic.BasicMenuItemUI.doClick(BasicMenuItemUI.java:1000)
        at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(BasicMenuItemUI.java:1041)
        at java.awt.Component.processMouseEvent(Component.java:5463)
        at javax.swing.JComponent.processMouseEvent(JComponent.java:3052)
        at java.awt.Component.processEvent(Component.java:5228)
        at java.awt.Container.processEvent(Container.java:1961)
        at java.awt.Component.dispatchEventImpl(Component.java:3931)
        at java.awt.Container.dispatchEventImpl(Container.java:2019)
        at java.awt.Component.dispatchEvent(Component.java:3779)
        at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4203)
        at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3883)
        at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3813)
        at java.awt.Container.dispatchEventImpl(Container.java:2005)
        at java.awt.Window.dispatchEventImpl(Window.java:1757)
        at java.awt.Component.dispatchEvent(Component.java:3779)
        at java.awt.EventQueue.dispatchEvent(EventQueue.java:463)
        at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:234)
        at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
        at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
        at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

    Sitze jetzt schon zwei geschlagene Tage über dem Problem und komme nicht auf die Fehlerquelle. Kann mir wer helfen.

    0
  • C

    Dein Problem liegt hier (im Code ausgezeichnet)

    public JTextEditorRC(){ 
        super("Patrick Mittelbach´s JTextEditor (RELEASE CANDIDATE)"); 
        this.setLocation(200,200); 
        this.setSize(500,400);
        // ************** HIER IST DAS PROBLEM ****************** 
        this.setJMenuBar(new ApplicationMenuBar(this, textarea)); 
        textarea = new JTextArea(); //vordefinierte Größe hier nicht sinnvoll 
        // ************** HIER IST DAS PROBLEM ******************
        //Erzeugung einer eingenen ContentPane 
        JPanel myContentPane = new JPanel(); 
        myContentPane.setOpaque(true); 
        this.setContentPane(myContentPane); 
        myContentPane.setPreferredSize(new Dimension(400,30)); 
        myContentPane.setLayout(new BorderLayout()); //BorderLayout dringend notwendig!!!! 
        myContentPane.add(new ApplicationToolBar(this, textarea), BorderLayout.PAGE_START); 
        myContentPane.add(new JScrollPane(textarea), BorderLayout.CENTER); 
        this.setVisible(true); 
    }

    Du erzeugst die Menubar zu einem Zeitpunkt, an dem dein Member textarea null ist. Deswegen haut er dir auch eine NullPointerException an den Kopf.

    0
Anmelden zum Antworten