nicht-statische Inhalte auf statische Inhalte verweisen

[OutOfBounds]

Ich grüße euch,

ich steht vor einem kleinen Problem was ich euch anhand code snippets von mir darstellen möchte:

public static void createGUI(){
	
        final JFrame f = new JFrame("Analyzer");
        f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        f.getContentPane().add(display);
        f.getContentPane().setLayout(new FlowLayout());
        f.setSize(display.getHeight(), display.getWidth());
        f.pack();
        f.setVisible(true);       
	}


public void addMenu(JFrame f){
		
		/* Sets the menubar */
        f.setMenuBar(this);
        Menu fileMenu = new Menu("Datei");
        Menu fileMenu2 = new Menu("Bearbeiten");
        Menu fileMenu3 = new Menu("Info");
        add(fileMenu);
        add(fileMenu2);
        add(fileMenu3);

//noch mehr code - ist jedoch hier nicht relevant

}


public static void main(String[] args) {
		
                Interface i = new Interface();

		i.addMenu(f);         // <== Hier ist "f" nicht einsetzbar 

		display = new PerformanceProfile();

		javax.swing.SwingUtilities.invokeLater(new Runnable() {
                public void run() {
                createGUI();              
                   }
                });
}

Nur zur Problematik :

Ich habe ein JFrame in einer statischen Methode erstellt (ja es muss leider static sein - gründe liegen in unserer Projektorganisation)

Analog zu diesem JFrame würde ich gerne noch eine MenuBar erstellen - dies liegt in einer "normalen" void methode. Wie bekomme ich es nun hin die Menubar zu diesem JFrame zu erstellen - ich habe hierbei primär Probleme damit statischen Inhalt auf nicht-statischen Inhalt (der Menubar) zu verweisen ?

danke schonmal im vorraus für Hints oder Denkansätze

OutOfBounds

 

[bygones]

(ja es muss leider static sein - gründe liegen in unserer Projektorganisation)

na dann gratulation zu dieser Projektorganisation... *kopfschüttel*

einfachste Lösung:

alles static machen und sich über das Ergebnis freuen....

 

[Guest]

"Herzlichen Dank" von meiner Seite aus - diese Gratulation darfst du nun neun mal (aber bitte nicht öfter) noch machen ^^

Ich würde zudem gerne alles auf static setzten, doch bei Menubar geht es nicht - sprich diesen Block:

public void addMenu(JFrame f){
      
      /* Sets the menubar */
        f.setMenuBar(this);
        Menu fileMenu = new Menu("Datei");
        Menu fileMenu2 = new Menu("Bearbeiten");
        Menu fileMenu3 = new Menu("Info");
        add(fileMenu);
        add(fileMenu2);
        add(fileMenu3);

//noch mehr code - ist jedoch hier nicht relevant

}

kann man nicht einfach in static umschreiben - da add(fileMenu); & co aus einem non-static Bereich hervorgehen.

 

[bygones]

9X Gratulation

kannst du mal die Klasse posten - was für eine Klasse ist das - eine die von JMenuBar erbt ?

dann krausts mich noch mehr - warum einen static frame darin definieren - bitte erklär mir - es muss ein Sinn haben ....

 

[Mag1c]

Hi,

erzeuge eine Instanz und rufe addMenu daran auf und übergib das Frame. Wenn du nicht mehrmals eine Instanz erzeugen willst, mach ein Singleton draus.

Gruß
Mag1c

 

[OutOfBounds]

@deathbyaclown

kann die ganze klasse leider nicht posten zumindest jetzt nicht - ich versuche morgen mit den Kollegen mich abzusprechen.

@Mag1c

Dies hört sich sehr gut an - doch in wiefern eine Instanz erzeugen ? Ich komme da noch nicht ganz mit

 

[bygones]

Dies hört sich sehr gut an - doch in wiefern eine Instanz erzeugen ? Ich komme da noch nicht ganz mit

hihi - nun versteh ich warum static wenn du nicht weißt wie eine Instanz erzeugen

Instanzen werden per new erzeugt
also

List eineListe = new ArrayList();

somit hast du eine Instanz einer ArrayList erzeugt. Mit dieser Instanz kannst du dann auf methoden der Klasse zugrifen

edit:
.

Interface i = new Interface();

hier habt ihr doch eine Instanz erzeugt ?!

ich tipp mal wieder so ins Blaue was ihr machen wollt bzw. geb mal n tipp ab.

Das ganze soll eine Gui werden - Interface ist irgendeine Klasse, die ihr zum anzeigen verwendet wollt (schließlich willst du darin dein menu haben). Bitte überlegt euch euer Design noch 1, 2x.

Z.b. macht eine Klasse (z.b. Interface) die von JFrame erbt. Somit könnt ihr diese dann als Hauptklasse für die GUI ansehen. in dieser Klasse könnt ihr dann z.b. eine Methode (nicht static *g) haben, die euch ein Menu erstellt.....

 

[OutOfBounds]

*lol* sehr schön - die Problematik ist nicht "wie" eine Instanz erzeugen - sondern "wovon" ?

 

[bygones]

*lol* sehr schön - die Problematik ist nicht "wie" eine Instanz erzeugen - sondern "wovon" ?

eine Instanz der Klasse die du z.T. gepostet hast, da ja addMenu eine Methode dieser Klasse ist

 

[Guest]

erzeuge eine Instanz und rufe addMenu daran auf und übergib das Frame. Wenn du nicht mehrmals eine Instanz erzeugen willst, mach ein Singleton draus.

Ich hab eine Instanz von der Klasse - und ich habe daran addMenu aufgerufen und dem als Parameter das Frame übergeben - oder versteh ich das jetzt völlig falsch ? *noch mehr verwirrt sei*

public static void main(String[] args) {
		Interface i = new Interface();  //<== Instanz erzeugt
		i.addMenu(f);                         //<== addMenu aufgerufen und als Parameter JFrame f übergeben
}

 

[Mag1c]

Hi,

also die static-Geschichte ist eigentlich was Besonderes was man nur aus ganz bestimmten (Projekt- :lol: ) Gründen einsetzt. Da ich nicht weiß, wie deine Klasse heißt, denk ich mir mal eine aus:

public class ProjektMenu extends JMenuBar {
  public void addMenu (JFrame f) {
     /* Sets the menubar */
     f.setMenuBar(this);
     Menu fileMenu = new Menu("Datei");
     Menu fileMenu2 = new Menu("Bearbeiten");
     Menu fileMenu3 = new Menu("Info");
     add(fileMenu);
     add(fileMenu2);
     add(fileMenu3); 
    //noch mehr code - ist jedoch hier nicht relevant 
  }
}

in der static-Methode kann das dann so verwendet werden:

public static void createGUI () {
    final JFrame f = new JFrame("Analyzer");
    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    f.getContentPane().add(display);
    f.getContentPane().setLayout(new FlowLayout());
    f.setSize(display.getHeight(), display.getWidth());

    // und jetzt das Menu
    ProjektMenu menu = new ProjektMenu();
    menu.addMenu(f);

    f.pack();
    f.setVisible(true);
}

und wenn ich mir das so anschaue, würde ich dir empfehlen, das leicht abzuwandeln, und zwar so:

public class ProjektMenu extends JMenuBar {
  public ProjektMenu () {
    createMenu();
  }

  public void createMenu () {
     Menu fileMenu = new Menu("Datei");
     Menu fileMenu2 = new Menu("Bearbeiten");
     Menu fileMenu3 = new Menu("Info");
     add(fileMenu);
     add(fileMenu2);
     add(fileMenu3); 
    //noch mehr code - ist jedoch hier nicht relevant 
  }
}

das ist nun das reine Menu ohne Bezug auf irgendein Frame. Der kommt dann hier:

public static void createGUI () {
    final JFrame f = new JFrame("Analyzer");
    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    f.getContentPane().add(display);
    f.getContentPane().setLayout(new FlowLayout());
    f.setSize(display.getHeight(), display.getWidth());

    // und jetzt das Menu
    ProjektMenu menu = new ProjektMenu();
    f.setMenuBar(menu);

    f.pack();
    f.setVisible(true);
}

Gruß
Mag1c

 

[OutOfBounds]

Herzlichen Dank, das war genau das i-Tüpfelchen -

1. Danke für die schnellen Antworten von deathbyaclown und Mag1c

und

2. Danke für den hint wie man es besser strukturieren kann

3. Danke für die aufgebrachte Zeit - hier geht alles nur husch husch ... zeit für Erklärungen untereinander nimmt man sich kaum.

 

[OutOfBounds]

Grüße,

dank eurer Hilfe konnte ich diverse weitere Probleme lösen, doch nun steh ich vor einem Problem welches ich nicht so leicht lösen kann.

basierend auf die methode:

public static void createGUI () {
    final JFrame f = new JFrame("Analyzer");
    f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

    ProjektMenu menu = new ProjektMenu();
    f.setMenuBar(menu);

    f.pack();
    f.setVisible(true);
}

hab ich noch eine methode

public void saveFile(JFrame f){
        FileDialog fd = new FileDialog(f, "Save File",
                FileDialog.SAVE);
        fd.show(); // Display dialog and wait for response  

// --einige try-catch komponenten die für das Problem nicht relevant sind
	}

Nun hab ich noch die methode "ActionPerformed" vom actionListener

public void actionPerformed(ActionEvent e){		

        if (e.getActionCommand()=="WLA"){
           saveFile(f); //<== Problem 
        }   

    }

Der ActionListener bzw. die methode ActionPerformed dient dazu um beim klicken auf ein MenuItem den Speicher Dialog zu öffnen.

Nun besteht genau hierbei das Problem, das ich in der methode actionPerformed nicht "saveFile(f)" schreiben kann, da JFrame f in der methode nicht bekannt ist - wie ist es möglich dennoch einen erfolgreichen Verweis in actionPerformed auf JFrame zu machen ?

 

[Illuvatar]

Wo ist denn das actionPerformed ist das ne eigene Klasse dann solltest du das im Konstruktor übergeben ist das die gleiche Klasse kannst du this verwenden.

Hab ich sowas nicht heute schon mal geschrieben ???:L

 

[OutOfBounds]

actionPerformed ist eine methode von ActionListener innerhalb der Klasse "Interface" (entschuldigung das ich das net miterwähnt habe !)

das würde bei der Klasse dann so aussehen:

public class Interface implements ActionListener{

public void actionPerformed(ActionEvent e){

 5
 6
 7
 8
 9

	

public void actionPerformed(ActionEvent e){      

        if (e.getActionCommand()=="WLA"){
           saveFile(f); //<== Problem
        }  

}

}

Das Problem ist das (f) sprich das JFrame leider als unbekannt in der methode actionPerformed galt

und wenn ich "this.saveFile(f);" steht dann "f cannot be resolved"

 

[Sky]

Nur mal so zur Info:

if (e.getActionCommand()=="WLA"){

Guck doch mal in der FAQ zum Thema "Stringvergleiche".

 

[OutOfBounds]

Auch wenn es nicht zur Lösung des Problems führt, so danke ich dennoch Sky - war doch aufschlussreich !

Nun weiß ich das es "if ("wla".equals(e.getActionCommand())){" heißt und hab es entsprechend editiert ^^

 

[OutOfBounds]

So, ich habe nun versucht JFrame f welches nur in der methode

public static void createGUI () vorliegt nun als Attribut in der Klasse Interface anzugeben -

private JFrame f;

Wenn ich dies mache verschwindet der Fehler bei actionPerformed
this.saveFile(f) - doch es erscheint im Gegenzug dann folgende Fehlermeldung:

Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: null owner window
at java.awt.Window.ownedInit(Unknown Source)
at java.awt.Window.<init>(Unknown Source)
at java.awt.Dialog.<init>(Unknown Source)
at java.awt.FileDialog.<init>(Unknown Source)
at Interface.saveFile(Interface.java:133)
at Interface.actionPerformed(Interface.java:157)
at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
at

...restliche Verweise ausgelassen

bei Interface.saveFile(Interface.java:133)
zeigt es auf
FileDialog fd = new FileDialog(f, "Save File",FileDialog.SAVE);

und bei

at Interface.actionPerformed(Interface.java:157) zeigt es auf

this.saveFile(f)

 

[Mag1c]

Hi,

mach die Variable static und ändere in createGUI():

f = new JFrame("Analyzer");

Gruß
Mag1c

 

[OutOfBounds]

Heiliger Strohsack ...und ich hab die variable auf private gestellt *hand_vor_stirn_knall* herzlichen dank nochmal - dies ist eines dieser Fehler gewesen die ich nicht so schnell vergessen werde ^^