Swing MenuBar als eigene Klasse

[chnooorzi]

Hallo

Bin ein Java-Anfänger und spiele zur Zeit ein wenig mit Swing.
Ich möchte meine MenuBar als eigenständige Klasse machen.
Leider funktioniert dies bei mir nicht.
Ich habe 3 Klassen: MenuTestViewer, MenuTestFrame & MenuBar.

import javax.swing.JFrame;

public class MenuTestViewer {

	public static void main(String[] args) {
		// TODO Auto-generated method stub

		JFrame frame = new MenuTestFrame();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setTitle("Bill");
		frame.setVisible(true);

	}

}

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

public class MenuTestFrame extends JFrame{
	
	private static final int FRAME_WIDTH = 300;
	private static final int FRAME_HEIGHT = 400;

	private JLabel label1;
	
	private JMenuBar menuBar;
	private JMenu fileMenu;
	private JMenuItem menuItem;
	
	public MenuTestFrame() {
		// TODO Auto-generated constructor stub

		label1 = new JLabel("Hallo");
		add(label1);
		
		JMenuBar menuBar = new MenuBar();
		
		//menuBar = new JMenuBar();
		//fileMenu = new JMenu("File");
		//menuItem = new JMenuItem("Exit");
		//fileMenu.add(menuItem);
		//menuBar.add(fileMenu);
		
		setJMenuBar(menuBar);
		
		setSize(FRAME_WIDTH, FRAME_HEIGHT);	
	}

}

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

public class MenuBar  extends JMenuBar{

	private JMenuBar menu;
	private JMenu fileMenu;
	private JMenuItem menuItem;
	
	public MenuBar() {
		// TODO Auto-generated constructor stub
		menu = new JMenuBar();
		
		fileMenu = new JMenu("File");
		menuItem = new JMenuItem("Exit");
		fileMenu.add(menuItem);
		menu.add(fileMenu);
	}
	
}

Wenn ich in der MenuTestFrame - Klasse genau das selbe definiere wie in der MenuBar-Klasse
funktioniert es (das ausgeklammerte unter: JMenuBar menuBar = new MenuBar()
Hat jemand einen Tipp?
Macht es überhaupt Sinn das Menü in einer eigenen Klasse zu definieren?

Danke

Freundliche Grüsse
chnooorzi

 

[eRaaaa]

Bevor du dich mit Swing beschäftigst solltest du dir die Grundlagen von Java und speziell OOP aneignen.
Zeilen 13 und 18 machen keinen Sinn in deiner MenuBar Klasse, wieso erstellst du dort noch einmal eine MenuBar? Die Instanz selbst ist doch schon eine!

->

import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
 
public class MenuBar  extends JMenuBar{
 
    private JMenu fileMenu;
    private JMenuItem menuItem;
    
    public MenuBar() {
        // TODO Auto-generated constructor stub
        fileMenu = new JMenu("File");
        menuItem = new JMenuItem("Exit");
        fileMenu.add(menuItem);
        this.add(fileMenu);
    }
    
}

Probier`s mal eher so (this.add(fileMenu) und Zeile 13 und 7 löschen!)

 

[chnooorzi]

Hallo

Danke für die rasche Antwort.
Hat so geklappt.

Werde mich dann noch einmal hinter die Grundlagen setzen.
Macht es den überhaupt Sinn eine eigene Klasse für das Menü zu machen?
Klar bei ein,zwei Menüitems nicht, aber bei grösseren ist es sinnvoll oder irre
ich mich?

Gruss

 

[eRaaaa]

Man erbt dann aber normalerweise nicht, denn du erweiterst ja so gesehen keine Funktionen von MenuBar. Man würde dann eher so vorgehen wie du es schon gemacht hattest, sprich dann ein neues Objekt von JMenuBar in deiner eigener Klassen erzeugen

 

[chnooorzi]

Supi danke für die Antwort.