Fragen zur JMenuBar & Co.

[qexxler]

Schönen guten Abend bzw. guten Morgen,

nachdem ich vergangene Tage meine erste kleine Software geschrieben habe, dessen Code ich bald einmal für alle zur Verfügung stellen werde, habe ich einige Fragen zum Design von einer JMenuBar und Co. Mit Co meine ich JMenus und JMenuItems.

Ich habe in meiner Start Klasse, wie auch immer sie heißen mag eine Instanz-Variable

 private JFrame frame;

Im Konstruktor initialisiere ich alle wichtige Daten und erstelle mir zum erstellen meiner JMenuBar eine entsprechende Methode

 private void initMenuBar()

.
In dieser Methode implentiere ich für jedes JMenuItem meiner MenuBar und Co als anonyme Klasse jeweilse das ActionListener-Interface und rufe ich der entsprechenenden actionPerformed - Methode eine entsprechende Funktion auf, die nach dem Klick, z.B. Datei -> Neu, ausgeführt werden soll.

Nun bin ich mir nicht sicher, ob das der richtige Weg ist, ein solches Menu aufzubauen, oder ob es einen objektorientierten Weg gibt, sich eine JMenuBar aufzubauen.

Wenn ihr Code benötigt, was das ganze noch besser zur Geltung bringt, lasst es mir wissen.

Mir scheint es nur der Fall zu sein, dass das kein gutes Design ist. Und da ich es gerne richtig machen möchte und hierzu keine speziellen Informationen im Internet gefunden habe, wollte ich hier einmal nachfragen.

Denn wie eine JMenuBar aufgebaut wird ist klar.

Ich hoffe, ihr versteht meine Frage. Ist schon spät, ich weiß.

 

[qexxler]

Einen schönen guten Morgen zusammen,

hier noch einmal ein kleines, ausführbares Programm was obiges Thema zeigen soll.

Spoiler

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

public class JMenuBarDemo {

	private JFrame frame;
	private JMenuBar menuBar;
	private JMenu menuDatei;
	private JMenuItem menuDateiSpeichern, menuDateiLaden;

	public JMenuBarDemo() {
		frame = new JFrame();
		frame.setTitle("JMenuBarDemo - Java-forum.org");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		/* Initialisieren der Komponenten */
		initMenuBar();
		addAction();

		frame.pack();
		frame.setLocationRelativeTo(null);
	}

	/* Initialisieren der MenuBar */
	private void initMenuBar() {

		menuBar = new JMenuBar();
		menuDatei = new JMenu("Datei");
		menuDateiSpeichern = new JMenuItem("Speichern");
		menuDateiLaden = new JMenuItem("Laden");

		menuDatei.add(menuDateiSpeichern);
		menuDatei.add(menuDateiLaden);
		menuBar.add(menuDatei);

		frame.setJMenuBar(menuBar);

	}

	private void addAction() {

		/* ActionListener : menuDateiSpeichern */
		menuDateiSpeichern.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				actionSpeichern();
			}
		});

		/* ActionListener : menuDateiLaden */
		menuDateiLaden.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				actionLaden();
			}
		});
	}

	private void actionLaden() {
		/* Hier wird eine beliebige Aktion ausgeführt */
		System.out.println("... laden ...");
	}

	private void actionSpeichern() {
		/* Hier wird eine beliebige Aktion ausgeführt */
		System.out.println("... speichern ...");
	}

	/* Setzt die Sichtbarkeit auf 'visible' */
	public void setVisible(boolean visible) {
		frame.setVisible(visible);
	}

	/* Applikation wird gestartet */
	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			@Override
			public void run() {
				JMenuBarDemo frame = new JMenuBarDemo();
				frame.setVisible(true);
			}
		});
	}
}

Was möchte ich nun genau wissen?
Es geht mir darum, wie man es in einer größeren Software mit der JMenuBar und deren Items richtig strukturiert. Ich bin mir nicht ganz sicher, ob mein obiges Programm, als Grundgerüst dienen kann. Mein zweiter Gedanken heute Nacht war, ob es nicht auch funktionieren würde, wenn ich für jedes JMenu eine eigene Klasse mache und dann einfach nur eine neue Instanz an das JMenuBar des Frame adde. Da stellte sich mir aber das Problem, wie ich die Informationen des Frame gut in die Klasse des entsprechenden JMenus bekomme, ohne im entsprechenden des JMenus Konstruktur eine lange Liste von Parameter mitzugeben, die es braucht um an die Informationen zu kommen, zumal es mir scheint, dass sich dadurch Redundanzen ergeben, da ich das bei allen anderen JMenus auch machen muss.


Um es zu zeigen, habe ich obigen Code etwas erweitert.

Spoiler

import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

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

public class JMenuBarDemo {

	private JFrame frame;
	private JMenuBar menuBar;
	private JMenu menuDatei;
	private JMenuItem menuDateiSpeichern, menuDateiLaden;

	public JMenuBarDemo() {
		frame = new JFrame();
		frame.setTitle("JMenuBarDemo - Java-forum.org");
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		/* Initialisieren der Komponenten */
		initMenuBar();
		addAction();

		frame.pack();
		frame.setLocationRelativeTo(null);
	}

	/* Initialisieren der MenuBar */
	private void initMenuBar() {

		menuBar = new JMenuBar();
		menuDatei = new JMenu("Datei");
		menuDateiSpeichern = new JMenuItem("Speichern");
		menuDateiLaden = new JMenuItem("Laden");

		menuDatei.add(menuDateiSpeichern);
		menuDatei.add(menuDateiLaden);
		menuBar.add(menuDatei);
		menuBar.add(new MenuOptionen().getJMenu());

		frame.setJMenuBar(menuBar);

	}

	private void addAction() {

		/* ActionListener : menuDateiSpeichern */
		menuDateiSpeichern.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				actionSpeichern();
			}
		});

		/* ActionListener : menuDateiLaden */
		menuDateiLaden.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				actionLaden();
			}
		});
	}

	private void actionLaden() {
		/* Hier wird eine beliebige Aktion ausgeführt */
		System.out.println("... laden ...");
	}

	private void actionSpeichern() {
		/* Hier wird eine beliebige Aktion ausgeführt */
		System.out.println("... speichern ...");
	}

	/* Setzt die Sichtbarkeit auf 'visible' */
	public void setVisible(boolean visible) {
		frame.setVisible(visible);
	}

	/* Applikation wird gestartet */
	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Runnable() {
			@Override
			public void run() {
				JMenuBarDemo frame = new JMenuBarDemo();
				frame.setVisible(true);
			}
		});
	}
	
	/* ***  Private innere Klasse für die Optionen *** */
	
	public class MenuOptionen {

		private JMenu menuOptionen;
		private JMenuItem menuOptionenSprache, menuOptionenEinstellungen;
		
		public MenuOptionen() {
			menuOptionen = new JMenu("Optionen");
			menuOptionenSprache = new JMenuItem("Sprache wählen");
			menuOptionenEinstellungen = new JMenuItem("Einstellungen");
			
			
			menuOptionen.add(menuOptionenSprache);
			menuOptionen.add(menuOptionenEinstellungen);
			
			addAction();
		}
		
		private void addAction() {
			menuOptionenSprache.addActionListener(new ActionListener() {
				@Override
				public void actionPerformed(ActionEvent e) {
					actionChoice();
				}
			});
			
			menuOptionenEinstellungen.addActionListener(new ActionListener() {
				@Override
				public void actionPerformed(ActionEvent e) {
					actionEinstellungen();
				}
			});
		}

		private void actionChoice() {
			System.out.println("... Sprache wählen ...");
		}
		
		
		private void actionEinstellungen() {
			System.out.println("... Einstellungen ...");
		}
		
		public JMenu getJMenu() {
			return menuOptionen;
		}
	}

}

Ihr müsste euch vorstellen, dass diese innere Klasse MenuOptionen eben keine innere Klasse ist, sondern eine eigenständige. Habe es nur zur Verfeinfachung so gewählt.

 

[Bartertown]

Was würde dagegen sprechen nur Action-Objekte zu benutzen.
z.B.

private Action actSave=new AbstractAction(){
@Override
public void actionPerformed(ActionEvent e){
//...handle event
}
};

Man hat die Möglichkeit elegant Werte festzulegen:

private void putActionValues(){
actSave.putValue(Action.NAME,"Datei speichern");
actSave.putValue(Action.ACCELERATOR_KEY,Keystroke.getKeystroke(KeyEvent.VK_S,InputEvent.CTRL_MASK);
actSave.putValue(Action.SMALL_ICON,new IconImage(getClass().getRessource("icon.png"))
//...
}

Jedes Menu kann man mit einer Methode erzeugen:

private JMenu createFileMenu(){
JMenu jmenu=new JMenu("File");
jmenu.add(new JMenuItem(actSave));
//jmenu.add...
return jmenu;
}


Viele Grüße (-_-J)

 

[qexxler]

Guten Morgen zusammen und hallo Bartertown,

zunächst einmal vielen Dank für deinen Post und dafür, dass du dir die Zeit genommen hast, dir meinen Code eventuell durchzulesen.

Was deinen Vorschlag angeht, werde ich heute mal darüber nachdenken und schauen, ob es mir in meinem Denken weiter hilf.

Einen schönen Sonntag.