JInternalFrame: Warum funktioniert das hier nicht?

[jagdfalke]

Hi, der Titel sagt eigentlich schon alles. Ich habe 3 Klassen:


hier fängt alles an:

package golfrechner;

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;

public class MyDesktop extends JFrame{

  private JDesktopPane desk;
  Dimension dm = Toolkit.getDefaultToolkit().getScreenSize();

  MyDesktop() {
    this.desk = new JDesktopPane();
    desk.setBackground(Color.GRAY);
    desk.setDesktopManager(new DefaultDesktopManager());
    setContentPane(desk);
    setSize(800,600);
    setLocation(((dm.width/2) - 400), ((dm.height/2) - 300));
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setTitle("Golfrechner 1.0");
    setResizable(false);
    
    JMenuBar menubar = new JMenuBar();
    menubar.add(createFileMenu());
    setJMenuBar(menubar);
  }

  public void addChildScoreCard(JInternalFrame  child,  int loc_x, int loc_y, int size_x, int size_y) {
    
    //child.setLocation(loc_x, loc_y);
    child.setSize(size_x,size_y);
    child.setDefaultCloseOperation(JInternalFrame.DISPOSE_ON_CLOSE);
    desk.add(child);
    child.setVisible(true);
    
  }


  public JMenu createFileMenu() {
    //DATEI
    JMenu ret = new JMenu("Datei");
    JMenuItem mi;
    MyActionListener actionlistener = new MyActionListener();
    
       //NEUE SCOREKARTE
       mi = new JMenuItem("Neue Scorekarte", 'n');
       mi.addActionListener(actionlistener);
       mi.setActionCommand("JMenu-NeueScorekarte");
       ret.add(mi);
       
       //NEUER PLATZ
       mi = new JMenuItem("Neuer Platz", 'p');
       ret.add(mi);
       
       ret.addSeparator();
       
       //SCHLIESSEN
       mi = new JMenuItem("Beenden", 'b');
       mi.addActionListener(actionlistener);
       mi.setActionCommand("JMenu-Close");
       ret.add(mi);
  
    return ret;
  }
  


  public static void main(String[] args) {

    SplashWindow slshwnd = new SplashWindow();
    slshwnd.setVisible(true);

    try {
      Thread.sleep(2500);
    } catch (InterruptedException ex) {}
    
    slshwnd.setVisible(false);
    slshwnd.dispose();

    MyDesktop desktop = new MyDesktop();
    desktop.setVisible(true);

  }

}

die Child-Klasse:

package golfrechner;

import javax.swing.*;
import javax.swing.JInternalFrame;
import java.awt.Color;
import java.awt.*;

class ChildFrame extends JInternalFrame {
  public ChildFrame(String title) {
    
    setTitle(title);
    setMaximizable(true);
    setIconifiable(true);
    setClosable(true);
    setBackground(Color.lightGray);
    
    Container contentpane = this.getContentPane();
    contentpane.setLayout(null);
    
    JLabel lbltest = new JLabel("Hello World");
    lbltest.setBounds(10,10,100,20);
    contentpane.add(lbltest);
    
  }
}

der ActionListener:

package golfrechner;

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

public class MyActionListener implements ActionListener {
  
  public void actionPerformed(ActionEvent evt) {
    String cmd = evt.getActionCommand();
    if(cmd.equals("JMenu-Close")) {
      System.exit(0);
    }
    if(cmd.equals("JMenu-NeueScorekarte")) {
      MyDesktop myDesktop = new MyDesktop();
     
      myDesktop.addChildScoreCard(new ChildFrame("test"), 10, 10, 300, 200);

    }
  }
  
}

das Problem ist, dass die beim drücken auf "Neue Scorekarte" keine neues InternalFrame erscheint. Warum denn nicht?

mfg
jagdfalke

 

[spoensche]

du must in der klasse childframe im konstruktor super aufrufen, damit die klasse die elternklasse JinternalFrame instantieren kann. dann sollte es funktionieren.

 

[Roar]

du must in der klasse childframe im konstruktor super aufrufen, damit die klasse die elternklasse JinternalFrame instantieren kann. dann sollte es funktionieren.

ne, das wird automatisch gemacht.
@jagdfalke: natürlich geht das nicht. du erzeugst bei jedem klick eine neue JDesktopPane, die NICHTS mit der pane zu tun hat die auf deinem fenster angezeigt wird, auch wenn die beiden den gleichen variablennamen haben.
deine neue pane sowie dein iframe werden, sobald die methode zurückgekehrt ist recht schnell vom gc gefressen.

 

[jagdfalke]

hä? hab ich etz net ganz kapiert.
Ich wollte, dass bei jedem Klick ein neues InternalFrame gebaut wird. Geht das nicht?

 

[Beni]

Übergib dem "MyActionListener" das Objekt hinter "desk" aus MyDesktop und füge deine Frames dann diesem Objekt hinzu. Wenn du jedes mal wenn du ein Blatt Papier nimmst, einen neuen Schreibtisch (mit "MyDesktop myDesktop = new MyDesktop();") dafür kaufst, kommst du nicht weit ;-)

 

[jagdfalke]

also so oder?:

  MyDesktop() {
    
   
    
    this.desk = new JDesktopPane();
    desk.setBackground(Color.gray);
    desk.setDesktopManager(new DefaultDesktopManager());
    setContentPane(desk);
    setSize(800,600);
    setLocation(((dm.width/2) - 400), ((dm.height/2) - 300));
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setTitle("Golfrechner 1.0");
    setResizable(false);

    MyActionListener actionlistn = new MyActionListener(desk);    

    JMenuBar menubar = new JMenuBar();
    menubar.add(createFileMenu(actionlistn);
    setJMenuBar(menubar);
  }

und ind createFileMenu:

mi.addActionListener(actionlistener);

und etz? sich, dass du desk aus MyDesktop() meinst? und nicht desktop aus public static void main(String[] args) ???
mfg
jagdfalke

 

[Beni]

Ok, MyDesktop wäre sinnvoller.

Also dann:

public class MyActionListener implements ActionListener {
  private MyDesktop desktop;

  public MyActionListener( MyDesktop desktop ){
    this.desktop = desktop;
  }
 
  public void actionPerformed(ActionEvent evt) {
    String cmd = evt.getActionCommand();
    if(cmd.equals("JMenu-Close")) {
      System.exit(0);
    }
    if(cmd.equals("JMenu-NeueScorekarte")) {
//      MyDesktop myDesktop = new MyDesktop();
     
      desktop.addChildScoreCard(new ChildFrame("test"), 10, 10, 300, 200);

    }
  }
 
}

public MyDesktop(){
  ... new MyActionListener( this );
}

 

[jagdfalke]

ahh super !!! danke !!
Ich hatte bis gerade eben alles genauso wie du gschrieben hast, nur in MyActionListener hatte ich das hier stehen:

public class MyActionListener implements ActionListener {
  
  public JFrame desktop;
  
  MyActionListener(JFrame desktop) {
    this.desktop = desktop;
  }
  
  public void actionPerformed(ActionEvent evt) {
    String cmd = evt.getActionCommand();
    if(cmd.equals("JMenu-Close")) {
      System.exit(0);
    }
    if(cmd.equals("JMenu-NeueScorekarte")) {
      //addChildScoreCard(new ChildFrame("1"), 10, 10, 300, 200);
      desktop.addChildScoreCard(new ChildFrame("test"), 10, 10, 300, 200);
      System.out.println("NEUE SCORECARD");
    }
    if(cmd.equals("JMenu-about")) {
      About abt = new About();
    }
  }
  
}

Und hab mich dann gewundert, dass ich in desktop nicht die methoden und variablen finde (man siehts nicht gleich: ich hatte desktop als JFrame deklariert, aus irgendeinem mir unbekannten Grund)

Ach nochwas: findet noch jemand, dass JInternalFrame scheußlich aussehen? So was die Titelleiste und Border angeht. Lässt sich da was drehen?

mfg
jagdfalke

 

[Beni]

Du kannst es mit einem anderen LookAndFeel versuchen (und ja, ich finde JInternalFrames allgemein hässlich... dumme Fensterchenpolitik, die verwirrt nur :wink: )

LookAndFeel.. siehe google, gibts mehr als genug

 

[jagdfalke]

Danke, also das funktioniert jetzt. Allerding bin ich schon wieder auf die nächste Frage gestoßen:
Auf dem JInternalFrame gibts ein JTextField und ein JButton. Wenn ich auf den Button drücke soll eine Funktion aufgerufen, die aus dem JInternalFrame den Inhalt des JTextFields holt und auswertet. Aber wie kann ich sicher gehen, dass wirklich das JTextField aus dem JInternalFrame genommen wird, das die Funktion aufgerufen hat?
mfg
jagdfalke

 

[Wildcard]

Warum übergibst du nicht direkt den Text, anstatt eine Funktion aufzurufen die ihn sich holt? :bahnhof:

 

[jagdfalke]

du meinst, den Text beim Aufruf der Funktion vom ActionListener aus übergeben?
Ja warum nicht. Aber da ist doch immernoch das problem, dass ich nicht weiß wie ich auf das aufrufende InternalFrame zugreifen kann.