[`B]

Hi,

ich habe am Anfang folgenden CodeSchnipssel zu präsentieren:

 
/** Die Menuleiste. */
private MenuBar mb;
/** Die Menupunkte. */
private Menu datei, bearbeiten, filter, hilfe, spiegeln;
/** Die Menuunterpunte. */
private MenuItem beenden, fragez, laden, farbwertveraenderung,
skalieren, graustufen, weichzeichnen, horizontalSpiegeln,
vertikalSpiegeln, speichernUnter, speichern, kontrast;
/** Undo / Redo Menuunterpunte. */
private MenuItem undo, redo;
 
/**
* Menübar wird erzeugt mit jeweiligen Menü´s und Einträgen.
*/
private void erzeugeMenue() {
// Menüanzeige initialiseren
mb = new MenuBar();
// Menüs initialisieren
datei = new Menu(MENUTITEL_DATEI);
bearbeiten = new Menu(MENUTITEL_BEARBEITEN);
filter = new Menu(MENUTITEL_FILTER);
filter.setEnabled(FILTERMENU);
hilfe = new Menu(MENUTITEL_HILFE);
spiegeln = new Menu(MENU_FILTER_SPIEGELN);
spiegeln.setEnabled(SPIEGELNMENU);
// Menüeinträge initialisieren
laden = new MenuItem(MENU_OEFFNEN);
speichern = new MenuItem(MENU_SPEICHERN);
speichern.setEnabled(BOOL_SPEICHERMENU);
speichernUnter = new MenuItem(MENU_SPEICHERNUNTER);
speichernUnter.setEnabled(BOOL_SPEICHERUNTERMENU);
beenden = new MenuItem(MENU_BEENDEN);
undo = new MenuItem(MENU_UNDO);
undo.setEnabled(BOOL_UNDOMENU);
redo = new MenuItem(MENU_REDO);
redo.setEnabled(BOOL_REDOMENU);
farbwertveraenderung = new MenuItem(MENU_FILTER_FARBWERT);
skalieren = new MenuItem(MENU_FILTER_GROESSE);
graustufen = new MenuItem(MENU_FILTER_GRAUSTUFE);
kontrast = new MenuItem(MENU_FILTER_KONTRAST);
weichzeichnen = new MenuItem(MENU_FILTER_WEICHZEICHNEN);
horizontalSpiegeln = new MenuItem(MENU_FILTER_SPIEGEL_H);
vertikalSpiegeln = new MenuItem(MENU_FILTER_SPIEGEL_V);
fragez = new MenuItem(MENU_HILFE_FRAGEZ);
// Menüeinträge einfügen
datei.add(laden); // Datei->Datei öffnen...
datei.addSeparator(); // Datei->--------
datei.add(speichern); // Datei->Speichern
datei.add(speichernUnter); // Datei->Speichern unter...
datei.addSeparator(); // Datei->--------
datei.add(beenden); // Datei->Beenden
bearbeiten.add(undo);
bearbeiten.add(redo);
filter.add(farbwertveraenderung); // Filter->Farbwerte ändern...
filter.add(skalieren); // Filter->Größe ändern...
filter.add(graustufen); // Filter->Graustufenfilter
filter.add(weichzeichnen); // Filter->Weichzeichnen
filter.add(kontrast); // Filter->Kontrastriere
filter.add(spiegeln); // Filter->Spiegeln
spiegeln.add(vertikalSpiegeln); // Filter->Spiegeln->Horizontal
spiegeln.add(horizontalSpiegeln); // Filter->Spiegeln->Vertikal
hilfe.add(fragez); // Hilfe->?
// Menü(s) in Menübar einfügen
mb.add(datei);
mb.add(bearbeiten);
mb.add(filter);
mb.setHelpMenu(hilfe);
// Menübar in Frame einfügen
setMenuBar(mb);
laden.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Datei laden"); } });
speichern.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Datei speichern"); } });
speichernUnter.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Datei speichern unter"); } });
beenden.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Anwendungbeenden"); } });
undo.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Rueckgaengig"); } });
redo.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Wiederherstellen"); } });
farbwertveraenderung.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Farbwertveränderung"); } });
skalieren.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Skalieren"); } });
graustufen.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Graustufe"); } });
weichzeichnen.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Weichzeichnen"); } });
kontrast.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("Kontrast"); } });
vertikalSpiegeln.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("horizontales Spiegeln"); } });
horizontalSpiegeln.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener("vertikales Spiegeln"); } });
repaint();
}

Da dies viel zu unübersichtlich war habe ich das alles ein wenig verkürzt. Dadurch stellen sich aber nun neue Probleme.

final String[] KONST_MENUE_AUFSCHRIFTEN = new String[] {"Datei", "Bearbeiten", "Filter", "Hilfe"}; 
final String[] KONST_MENUE_EINTRAEGE = new String[] {
   "Laden...#-#Speicher#Speichern unter...#-#Beenden", 
   "R\u00fcckg\u00e4ngig#Wiederherstellen#-#Kopieren#Einfuegen#Ausschneiden",  "Farbwertver\u00e4nderung...#Skalieren...#Graustufen#Weichzeichnen#Horizontal spiegeln#Vertikal spiegeln#Kontrast",
"?"};
 
private void erzeugeMenue() {
// Menüanzeige initialiseren
mb = new MenuBar();
// Menüs initialisieren
liste = KONST_MENUE_AUFSCHRIFTEN;
for(int i=0; i<liste.length - 1; i++) {
	Menu eintrag		= new Menu(liste[i]);
	hilfsliste = KONST_MENUE_EINTRAEGE[i];
	liste2 = hilfsliste.split("#");
	for(int j=0; j<liste2.length -1; j++) {
		if(liste[j].equals("-")) {
		 eintrag.addSeparator();
		} else {
		 MenuItem menueEintrag = new MenuItem(liste[j]);
		 eintrag.add(menueEintrag);
		}
	}
	hatMenue[i] = eintrag;
}
this.fuegeActionListenerHinzu();
}

Nun stellt sich mir nur die Frage:

1. wie Spreche ich die MenuItems nun direkt an um sie enabled bzw. disabled setzen zu können.
2. wie kann ich sobald ich sie ansprechen kann diesen langen addActionListener Teil von oben verkürzen? Aufgerufen werden soll IMMER main.bearbeiteEreignis(KONST_EREIGNIS_***MenüEintr agName (zB.***LADEN).

Anmerkung noch zum Schluss alle Worte die nur in Großbuchstaben geschrieben sind, sind deklarierte Variablen aus einem Interface auf die zugegriffen werden können.

[peschmae]

Zitat von `B

1. wie Spreche ich die MenuItems nun direkt an um sie enabled bzw. disabled setzen zu können.

Da musst du dich wohl durch den Menübaum hangeln und bei jedem gucken ob das jetzt da ist - ausgehend von der MenuBar. Das tönt allerdings recht ineffizient. Eventuell ne Map für das Zeugs? Kommt halt drauf an ausgehend von welchen Daten (Name?) du den Menüpunkt jeweils finden willst.

Damit kannst du den Listener Teil doch auch recht effizient machen. Also einfach ne for Schleife die alle Menüs durchgeht und dort mit getItem() bis zu getItemCount() und immer mit dem aktuellen Eintrag ein

item.addActionListener(new ActionListener() {
public final void actionPerformed(final ActionEvent e) {
main.menuListener(item.getLabel()); } });

macht.

MfG peschmä

[`B]

habe nun ne Lösung für mein Problem, weiß net ob die so 100% fein ist aber ich poste sie gleich mal hier.

Also erstmal drei ewig lange Zeilen, die großen Wörter darin sind wieder finals die zuvor deklariert wurden.

PHP-Code:

 // Menueerzeugung
    final String[] KONST_MENUE_AUFSCHRIFTEN       = new String[]   {"Datei", "Bearbeiten", "Filter", "Hilfe"};
    final String[][] KONST_MENUE_EINTRAEGE        = new String[][] { {"Laden...", "-", "Speicher", "Speichern unter...", "-", "Beenden"},
                                                                     {"R\u00fcckg\u00e4ngig", "Wiederherstellen", "-", "Kopieren", "Einfuegen", "Ausschneiden"},
                                                                     {"Farbwertver\u00e4nderung...", "Skalieren...", "Graustufen", "Weichzeichnen", "Horizontal spiegeln", "Vertikal spiegeln", "Kontrast"},
                                                                     {"?"} };
    final int[][] KONST_MENUE_EINTRAG_EREIGNIS      = new int[][] { {KONST_EREIGNIS_LADE, 0, KONST_EREIGNIS_SPEICHERN, KONST_EREIGNIS_SPEICHERN_UNTER, 0, KONST_EREIGNIS_BEENDEN},
                                                                    {KONST_EREIGNIS_RUECKGAENGIG, KONST_EREIGNIS_WIEDERHERSTELLEN, 0, KONST_EREIGNIS_KOPIEREN, KONST_EREIGNIS_EINFUEGEN, KONST_EREIGNIS_AUSSCHNEIDEN},
                                                                    {KONST_FILTEREREIGNIS_FARBWERT, KONST_FILTEREREIGNIS_SKALIEREN, KONST_FILTEREREIGNIS_GRAUSTUFE, KONST_FILTEREREIGNIS_WEICHZ, KONST_FILTEREREIGNIS_H_SPIEGEL, KONST_FILTEREREIGNIS_V_SPIEGEL, KONST_FILTEREREIGNIS_KONTRAST},
                                                                    {KONST_EREIGNIS_HILFE} }; 


So das war die Vorraussetzung für folgende Funktion:

PHP-Code:

private void erzeugeMenue() {
    // Menüleiste initialiseren
    mb = new MenuBar();
    // Menüs initialisieren
    final String[]    kenntMenuePunkte     = KONST_MENUE_AUFSCHRIFTEN;
    final String[][] kenntMenueEintraege = KONST_MENUE_EINTRAEGE;
    final int[][]     kenntMenueEreignisse = KONST_MENUE_EINTRAG_EREIGNIS;
    for(int i=0; i<kenntMenuePunkte.length - 1; i++) {
        Menu hatMenuePunkt = new Menu(kenntMenuePunkte[i]);
        // Menüunterpunkte hinzufügen
        for(int j=0; j<kenntMenueEintraege[i].length -1; j++) {
            // Abfrage ob Eintrag nur ein Separator sein soll
            if(kenntMenueEintraege[i][j].equals("-")) {
                hatMenuePunkt.addSeparator();
            } else { 
                // MenuItem erzeugen
                MenuItem hatMenueEintrag = new MenuItem(kenntMenueEintraege[i][j]);
                // MenuItem dem Menu hinzufügen
                hatMenuePunkt.add(hatMenueEintrag);
                // Action Listener hinzufügen
                final int a = i; final int b = j;
                hatMenueEintrag.addActionListener(new ActionListener() {
                        public final void actionPerformed(final ActionEvent e) {
                        kenntEB.bearbeiteEreignis(kenntMenueEreignisse[a][b]); } });
            }
        }
        // Fertiges Menü der Menüleiste hinzufügen
        mb.add(hatMenuePunkt);
    }
} 


Hab nun php-Tag genommen zum eintrücken. Wüßte gerne noch wie die zeile
final int a = i; final int b = j;
unüberflüssig machbar ist.

[pago]

Heiliger Joseph...

Les mal im Java-Tutorial das Kapitel über Actions durch. Das ist ziehmlich genau das, was du da beschreibst.

[`B]

hm nu hab ich das ja so gemacht aber die MB wird mir nicht angezeigt ... woran liegt das?

Ich poste hier mal den gesammten Quelltxt der entsprechenden Klasse:

PHP-Code:

import java.awt.Frame;
import java.awt.MenuBar;
import java.awt.Menu;
import java.awt.MenuItem;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.awt.event.WindowEvent;
import java.awt.event.WindowAdapter;
/**
  * Diese Klasse dient dazu um die eigentliche GUI zu erzeugen und deren
  * Objekte zu erzeugen.
  *
  * @author Florian Pfeifer & Dirk Weber
  * @version 1.0
  */
 public class MenueAnzeige extends Frame implements konfiguration {
    // Objekte deklarieren
    /** Die Kenntbeziehung zum EB. */
    private EreignisBearbeiter kenntEB;
    /** Die Menuleiste. */
    private MenuBar mb;
    private final String[]    kenntMenuePunkte      = KONST_MENUE_AUFSCHRIFTEN;
    private final String[][]  kenntMenueEintraege  = KONST_MENUE_EINTRAEGE;
    private final int[][]       kenntMenueEreignisse = KONST_MENUE_EINTRAG_EREIGNIS;   
    /**
      * Der Konstruktor erzeugt den Frame und das Menü.
      *
      * @param pEB die Ereignisanwendung
      */
    public MenueAnzeige(final EreignisBearbeiter pEB) {
        super(KONST_AUFSCHRIFT_TITEL);
        kenntEB = pEB;
        // Menüleiste initialiseren
        mb = new MenuBar();
        this.erzeugeFrame();
    }
    private void erzeugeMenue() {
        // Menüs initialisieren
        for(int i=0; i<kenntMenuePunkte.length - 1; i++) {
            Menu hatMenuePunkt = new Menu(kenntMenuePunkte[i]);
            // Menüunterpunkte hinzufügen
            for(int j=0; j<kenntMenueEintraege[i].length -1; j++) {
                // Abfrage ob Eintrag nur ein Separator sein soll
                if(kenntMenueEintraege[i][j].equals("-")) {
                    hatMenuePunkt.addSeparator();
                } else { 
                    // MenuItem erzeugen
                    MenuItem hatMenueEintrag = new MenuItem(kenntMenueEintraege[i][j]);
                    // MenuItem dem Menu hinzufügen
                    hatMenuePunkt.add(hatMenueEintrag);
                    // Action Listener hinzufügen
                    final int a = i; final int b = j;
                    hatMenueEintrag.addActionListener(new ActionListener() {
                            public final void actionPerformed(final ActionEvent e) {
                            kenntEB.bearbeiteEreignis(kenntMenueEreignisse[a][b]); } });
                }
            }
            // Fertiges Menü der Menüleiste hinzufügen
            mb.add(hatMenuePunkt);
        }
    }
    private void erzeugeFrame() {
        this.setResizable(false);
        this.setBounds( KONST_BILDSCHIRM_BREITE / 2,
                        KONST_BILDSCHIRM_HOEHE / 2,
                        KONFIG_MENUE_BREITE,
                        KONFIG_MENUE_HOEHE);
        this.setBackground(KONFIG_MENUE_HINTERGUNDFARBE);
        this.setVisible(true);
        this.toFront();
        repaint();
        this.erzeugeMenue();
        this.addWindowListener(new WindowAdapter() {
                public final void windowClosing(final WindowEvent e) {
                    kenntEB.bearbeiteEreignis(KONST_EREIGNIS_BEENDEN);
                }
            });
    }
    
} 


Das Menü wird mir halt nicht angezeigt und ich habe keine Ahnung warum.

[Sym]

Jau,

in Deiner Initialisierungsmethode fehlt ein

this.setMenuBar(mb)