[JAVA] menubar in een Panel

[Zynth]

Ik heb pas een vraag gesteld over een grafisch venster.
Mij werdt toen aangeraden om het met Panels te doen.
Maar nu zit ik met een probleem met de menubar:

code:

import java.awt.*;
import java.awt.event.*;
// Hoofdletter voor klassenaam
public class ComposeMail extends Panel
{
    // Gebruik 1 declaratie per regel

    private TextField toField;
    private TextField fromField;
    private TextArea messageField;
    private MenuItem exitMenuItem;
    private MenuItem sendMenuItem;
    private MenuItem resetMenuItem;
    private MenuItem aboutMenuItem;

    private CloseAction closeAction;

    public ComposeMail(Window window)
    {
        super(new BorderLayout());

        MenuBar menuBar = new MenuBar();
        Menu fileMenu   = new Menu("File");
        fileMenu.add(sendMenuItem = new MenuItem("Send"));
        fileMenu.add(resetMenuItem = new MenuItem("Reset"));
        fileMenu.add(exitMenuItem = new MenuItem("Exit"));
        menuBar.add(fileMenu);
        Menu aboutMenu  = new Menu("Help");
        aboutMenu.add(aboutMenuItem = new MenuItem("About"));
        menuBar.add(aboutMenu);

         //DEZE KAN NIET!!
         setMenuBar(menuBar);


        // assignments als parameter (expressie ) gebruiken
        // is niet mooi -> aparte assignments

        Button sendButton   = new Button("Send");
        Button resetButton  = new Button("reset");
        Button backButton   = new Button("Go Back");

        closeAction = new CloseAction(window);

        // vergeet niet je ActionListeners ook toe te voegen!
        sendButton.addActionListener(new SendAction());
        resetButton.addActionListener(new ResetAction());
        backButton.addActionListener(new GoBackAction());
        sendMenuItem.addActionListener(new SendAction());
        resetMenuItem.addActionListener(new ResetAction());
        exitMenuItem.addActionListener(new GoBackAction());
        aboutMenuItem.addActionListener(new AboutAction());

        Panel p = new Panel();
        p.add(sendButton);
        p.add(resetButton);
        p.add(backButton);

        // Gebruik geen "South" maar BorderLayout.SOUTH

        add(BorderLayout.SOUTH, p);

        fromField = new TextField(30);
        toField   = new TextField(30);
        // hum, dit wil je later vast een TextArea maken?
        messageField = new TextArea(20,40);
            
        Panel n = new Panel();
        n.setLayout(new FlowLayout());
        n.add(new Label("From:"));
        n.add(fromField);
        n.add(new Label("To:"));
        n.add(toField);
    
        // pack is hier niet zinvol omdat nog niet al je
        // componten zijn geplaatst
        // pack();
        
        // Ook hier weer BorderLayout.NORTH ipv "North"

        add(BorderLayout.NORTH, n);
        add(BorderLayout.CENTER, messageField);

        setBackground(Color.lightGray);

        // beter: laat de layout-manager zijn werk doen:
        // gebruik daarom pack() bij een Window.
        //    setBounds(100, 100, 400, 400);
        
        // Visible maken in een constructor is niet handig
        // en in dit geval ook niet meer mogelijk (panel).
        //setVisible(true);
    }

    // dit is ff een makkelijke methode om een Frame te maken
    public static Frame createFrame()
    {
        Frame frame = new Frame("E-mail composer");
        ComposeMail panel = new ComposeMail(frame);

        frame.addWindowListener(panel.getCloseAction());
        frame.add(panel);
        frame.pack();
        frame.setLocation(100,100);

        return frame;
    }

    //entry point, gewoon om ff te testen

    public CloseAction getCloseAction()
    {
        return closeAction;
    }

    // liever aparte ActionListeners dan gevalsonderscheid in 1 ActionListener

    class CloseAction extends WindowAdapter implements ActionListener
    {
        private Window window;

        public CloseAction(Window window)
        {
            super();
            this.window = window;
        }

        public void windowClosing(WindowEvent e)
        {
            window.dispose();
      }

        public void actionPerformed(ActionEvent event)
        {
            window.dispose();
        }
    }

    class SendAction implements ActionListener
    {
        public void actionPerformed(ActionEvent event)
        {
            Frame newmaill = ComposeMail.createFrame();
            new SentMail();

        }
    }

    class ResetAction implements ActionListener
    {
        public void actionPerformed(ActionEvent event)
        {
            messageField.setText("");
        }
    }

    class GoBackAction implements ActionListener
    {
        public void actionPerformed(ActionEvent event)
        {
            Venster.newMail.dispose();
        }
    }

    class AboutAction implements ActionListener
    {
        public void actionPerformed(ActionEvent event)
        {
          new About();
        }
    }

}

Het specificeren van een menubar gaat goed,
maar setMenuBar(); kan hij niet vinden...

[mbravenboer]

In principe is een MenuBar gewoon een Component dus je kan hem aan het Panel toevoegen op de gebruikelijke methode. Je wilt hem dan waarschijnlijk in het noorden hebben, maar helaas is die plek al bezet . Je moet dan even een Panel nesten (of een complexere LayoutManager gebruiken).

Het is trouwens wel zo handig om de API documentatie erbij te houden ( http://java.sun.com/j2se/1.4/docs/api/ ) dan had je zelf kunnen vinden dat die methode setMenuBar in een Frame zit...

Overigens is dit inderdaad wel een nadeel van 'de Panel aanpak' en eigenlijk zou je dat ook iets beter moeten doen (met WindowHandles). Maar dat wordt een beetje te complex...

[Zynth]

Ik ben er nog niet uitgekomen.
Ik heb het commando om de menubar te maken aangepast:

code:

//DEZE KAN NIET!!
         add(BorderLayout.NORTH, menuBar);

maar dit werkt ook niet.
owja; ik heb ff het panel dat eerst op north stond, op left gezet.

[Verwijderd]

daar heb je ook geen layout voor nodig, gewoon add(menuBar) doen...

[mbravenboer]

Hum dat zou toch moeten werken... Ik zal zo wel ff kijken of het hier werkt.

add(...) voegt een component trouwens op de default plek toe. Ik denk dat een layout positie hier dus toch wel handig is.

[mbravenboer]

Hum, ik heb te lang niet meer in AWT gewerkt .
MenuBar is helemaal geen component en je kan hem dus ook niet aan een Panel toevoegen. Je kan hem alleen maar instellen op een Frame (zelfs niet op een Dialog). In Swing is een menubar wel gewoon een component.

Ik heb het opgelost door een methode configure te maken die een Window configureert. Merk op dat nu ook de CloseAction niet meer publiek beschikbaar is, dat werkt wel aardig .

In Swing zou dit allemaal veel makkelijker en mooier kunnen, maar goed...

Hier is de code (kijk uit: ik heb niet al jouw code overgenomen, je moet hem dus ff mergen met jouw versie):

code:

import java.awt.*;
import java.awt.event.*;

// Hoofdletter voor klassenaam
public class SendMailPanel extends Panel
{
    // Gebruik 1 declaratie per regel

    private TextField toField;
    private TextField fromField;
    private TextField messageField;

    private CloseAction closeAction;
    private MenuBar menuBar;

    public SendMailPanel(Window window)
    {
        super(new BorderLayout());

        closeAction = new CloseAction(window);

        MenuItem sendMenuItem   = new MenuItem("Send");
        MenuItem resetMenuItem  = new MenuItem("Reset");
        MenuItem exitMenuItem   = new MenuItem("Exit");
        MenuItem aboutMenuItem  = new MenuItem("About");

        sendMenuItem.addActionListener(new SendAction());
        resetMenuItem.addActionListener(new ResetAction());
        exitMenuItem.addActionListener(closeAction);
        aboutMenuItem.addActionListener(new AboutAction());

        menuBar = new MenuBar();
      Menu fileMenu   = new Menu("File");
          fileMenu.add(sendMenuItem);
        fileMenu.add(resetMenuItem);
        fileMenu.add(exitMenuItem);
      menuBar.add(fileMenu);
      Menu aboutMenu  = new Menu("Help");
        aboutMenu.add(aboutMenuItem);
      menuBar.add(aboutMenu);

        // assignments als parameter (expressie ) gebruiken 
        // is niet mooi -> aparte assignments

        Button sendButton   = new Button("Send");
        Button resetButton  = new Button("reset");
        Button backButton   = new Button("Go Back");

        // vergeet niet je ActionListeners ook toe te voegen!
        sendButton.addActionListener(new SendAction());
        resetButton.addActionListener(new ResetAction());
        backButton.addActionListener(new GoBackAction());

        Panel p = new Panel();
        p.add(sendButton);
        p.add(resetButton);
        p.add(backButton);

        // Gebruik geen "South" maar BorderLayout.SOUTH

        add(BorderLayout.SOUTH, p);

        fromField = new TextField(30);
        toField   = new TextField(30);
        // hum, dit wil je later vast een TextArea maken?
        messageField = new TextField(40);
            
        Panel n = new Panel();
        n.setLayout(new FlowLayout());
        n.add(new Label("From:"));
        n.add(fromField);
        n.add(new Label("To:"));
        n.add(toField);
    
        // pack is hier niet zinvol omdat nog niet al je
        // componten zijn geplaatst
        // pack();
        
        // Ook hier weer BorderLayout.NORTH ipv "North"

        add(BorderLayout.NORTH, n);
        add(BorderLayout.CENTER, messageField);

        setBackground(Color.lightGray);

        // beter: laat de layout-manager zijn werk doen:
        // gebruik daarom pack() bij een Window.
        //    setBounds(100, 100, 400, 400);
        
        // Visible maken in een constructor is niet handig
        // en in dit geval ook niet meer mogelijk (panel).
        //setVisible(true);
    }

    // dit is ff een makkelijke methode om een Frame te maken
    public static Frame createFrame()
    {
        Frame frame = new Frame("E-mail composer");

        SendMailPanel panel = new SendMailPanel(frame);
        panel.configure(frame);

        return frame;
    }

    //entry point, gewoon om ff te testen
    public static void main(String[] ps)
    {
        Frame frame = createFrame();
        frame.show();
    }
    
    public void configure(Window window)
    {
        window.addWindowListener(closeAction);
        window.add(this);

        if(window instanceof Frame)
        {
            Frame frame = (Frame) window;
            frame.setMenuBar(menuBar);
        }

        window.pack();
    }

    // liever aparte ActionListeners dan gevalsonderscheid in 1 ActionListener
    class CloseAction extends WindowAdapter implements ActionListener
    {
        private Window window;

        public CloseAction(Window window)
        {
            super();
            this.window = window;
        }

        public void windowClosing(WindowEvent e)
        {
            window.dispose();
      }

        public void actionPerformed(ActionEvent event)
        {
            window.dispose();
        }
    }

    class SendAction implements ActionListener
    {
        public void actionPerformed(ActionEvent event)
        {
            // doe iets
        }
    }

    class ResetAction implements ActionListener
    {
        public void actionPerformed(ActionEvent event)
        {
            // doe iets
        }
    }

    class GoBackAction implements ActionListener
    {
        public void actionPerformed(ActionEvent event)
        {
            // doe iets
        }
    }

    class AboutAction implements ActionListener
    {
        public void actionPerformed(ActionEvent event)
        {
                // doe iets
        }
    }
}

Overigens zou je dit ook allemaal best zelf kunnen uitzoeken. Het is slechts een simpele combinatie van API documentatie leren lezen en kennis opdoen van Java.