Komponente (Frame) in anderer Klasse füllen

[Reinhard]

Hallo,

ich habe eine MainForm.java, dort werden mir alle Komponenten einer Form erstellt. Zusätzlich habe ich einen MouseAdapter:

public class CMouseAdapter extends MouseAdapter
{
  JTree tree;
  JFrame frame;
  
  public CMouseAdapter(JTree tree, JFrame frame) {
    this.tree = tree;
    this.frame = frame;
  }
  
  @Override
  @SuppressWarnings("boxing")
  public void mouseClicked(MouseEvent me) {

    // Abfrage linke Maustaste (Doppelklick)
    if(me.getButton() == MouseEvent.BUTTON1 & me.getClickCount() == 2) {      
      //frame.getContentPane().getComponent(0). ...
    }
  }

Der MouseAdapter wird in der MainForm.java eingebunden:

tree.addMouseListener(new CMouseAdapter(tree, frame));

Ich möchte jetzt nachdem ein Doppelklick mit der linken Maustaste auf einen Knoten (Tree) ausgeführt wird z.B. ein JTextField (textField) füllen.

Die Struktur meiner Oberfläche sieht so aus:

frame
- contentPane
---- center - tree
---- east - panel
------ label
------ textField

Bin ich mit folgendem Code auf der richtigen Spur bzw. kann mir jemand helfen in einer Komponente einen Text zu füllen:

frame.getContentPane().getComponent(0). ...

Oder ist es falsch den Frame an die andere Klasse zu übergeben?

Danke,
Reinhard

 

[SamHotte]

Grundsätzlich gibt es viele Wege, aber deine Frage schreit nach einem MVC-Pattern. Such mal hier im Forum das WindController-Beispiel, daran kann man es gut sehen, wie das Zusammenspiel funzt.

 

[Reinhard]

Ok hab in der FAQ was dazu gefunden. Ich les mir den Eintrag mal durch. Ich dachte das geht auf die schnelle einfacher

 

[SamHotte]

Wenn du es einfach willst, übergib einfach deine Referenzen so, wie du sie benötigts. Bei kleineren Progrämmchen kein Problem, es wird nur irgendwann unübersichtlich, und da kann dann so ein Pattern helfen.

 

[Reinhard]

Es handelt sich hier eigentlich im Moment nur um ein kleines Programm.

Mir ist aber im Moment nicht klar, was ich als "Referenz" übergeben soll? Ist das dann der Frame oder gleich das JTextField?

Wie oben geschrieben, habe ich ja bereits den Frame übergeben, aber leider komme ich nicht damit klar, wie ich auf die Komponente richtig zugreifen soll.

Danke,
Reinhard

 

[Gast]

Übergib doch ein Referenz von MainForm an den Adapter und lege in dieser entsprechende Methoden zum Beschreiben an (a la den üblichen getter und setter Methoden)

 

[Reinhard]

Hallo,

kann mir jemand ein Beispiel geben für die Übergabe der "MainForm" an eine andere Klasse.
Ich habe das jetzt ne Stunde probiert, aber Swing wirft mir die lustigsten Exceptions.

Außerdem ist mir auch nicht klar, wie ich auf die "MainForm" dann zugreifen kann.

Das ganze schaut bei mir so im groben aus:

public calss MainForm {
  MainForm window;

  ...

  public static void main(String args[]) {
    window = new MainForm();

    ...

   tree.addMousListener(new CMouseAdapter(window);
  }
}

Ich denke da ist vom Ansatz her noch ein Fehler drin.
Die genaue Exception habe ich leider gerade nicht da.


In CMouseAdapter wollte ich dann etwa so auf die Komponente zugreifen:

window.panel.textField.setText("text");

Bin für jeden Tip dankbar,
Reinhard

 

[Reinhard]

Guten Morgen,

ich habe das ganze jetzt so implementiert:

public class MainForm
{
  private JTextField textField;
  static MainForm window;

  public static void main(String args[])
  {
  }

  public MainForm()
  {
    try {
      //MainForm window = new MainForm();
      window.frame.setVisible(true);
    } catch (Exception e) {
    }
  }

  private void initialize()
  {    
    frame = new JFrame();

    tree = new JTree(model);
    tree.setCellRenderer(new CCellRenderer());
    tree.addMouseListener(new CMouseAdapter(tree, window));
     
    final JPanel panel = new JPanel();
    panel.setLayout(new FormLayout(
      new ColumnSpec[] {
        FormFactory.RELATED_GAP_COLSPEC,
        new ColumnSpec("96px"),
        FormFactory.RELATED_GAP_COLSPEC,
        new ColumnSpec("11px"),
        FormFactory.DEFAULT_COLSPEC,
        FormFactory.DEFAULT_COLSPEC},
      new RowSpec[] {
        FormFactory.RELATED_GAP_ROWSPEC,
        new RowSpec("19px"),
        FormFactory.RELATED_GAP_ROWSPEC,
        FormFactory.DEFAULT_ROWSPEC}));
    frame.getContentPane().add(panel, BorderLayout.EAST);

    final JLabel selektierterKnotenLabel = new JLabel();
    selektierterKnotenLabel.setText("Selektierter Knoten:");
    panel.add(selektierterKnotenLabel, new CellConstraints("2, 2, 1, 1, fill, center"));

    textField = new JTextField();
    panel.add(textField, new CellConstraints("2, 4, 1, 1, fill, fill"));
  }

  public String getTextField()
  {
    return textField.getText();
  }

  public void setTextField(String aText)
  {
    this.textField.setText(aText);
  }

}

Im "addMouseListener" hab ich das ganze so implementiert:

public class CMouseAdapter extends MouseAdapter
{
  JTree tree;
  MainForm window;
  
  public CMouseAdapter(JTree tree, MainForm window) {
    this.tree = tree;
    this.window = window;
  }
  
  @Override
  @SuppressWarnings("boxing")
  public void mouseClicked(MouseEvent me) {

    TreePath treepath;
    int lPathCount = 0;

    treepath = tree.getClosestPathForLocation(me.getX(), me.getY());

    if (treepath != null) {
      lPathCount = treepath.getPathCount();
    }     
        
    // Abfrage linke Maustaste (Doppelklick)
    if(me.getButton() == MouseEvent.BUTTON1 & me.getClickCount() == 2) {
      if(lPathCount == AppDefs.gTreeGruppe) {
        // 
        window.setTextField(treepath.getLastPathComponent().toString());
      }
    }
    me.consume();
  }

    private boolean createPopup(String aDesc, int mouseX, int mouseY) {   
    ...    
    return true;
  }
  
}

Folgende Zeile funktioniert aber so leider nicht:

window.setTextField(treepath.getLastPathComponent().toString());

Ich bekomme dann folgende Exception:

Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at de.bmw.mox.forms.tree.CMouseAdapter.mouseClicked(CMouseAdapter.java:65)
	at java.awt.AWTEventMulticaster.mouseClicked(AWTEventMulticaster.java:212)
	at java.awt.Component.processMouseEvent(Component.java:5491)
	at javax.swing.JComponent.processMouseEvent(JComponent.java:3126)
	at java.awt.Component.processEvent(Component.java:5253)
	at java.awt.Container.processEvent(Container.java:1966)
	at java.awt.Component.dispatchEventImpl(Component.java:3955)
	at java.awt.Container.dispatchEventImpl(Container.java:2024)
	at java.awt.Component.dispatchEvent(Component.java:3803)
	at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:4212)
	at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3901)
	at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3822)
	at java.awt.Container.dispatchEventImpl(Container.java:2010)
	at java.awt.Window.dispatchEventImpl(Window.java:1774)
	at java.awt.Component.dispatchEvent(Component.java:3803)
	at java.awt.EventQueue.dispatchEvent(EventQueue.java:463)
	at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:242)
	at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:163)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:157)
	at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:149)
	at java.awt.EventDispatchThread.run(EventDispatchThread.java:110)

Was mache ich denn mit den Getter / Setter Methoden falsch?

Danke,
Reinhard

 

[SamHotte]

Als erstes solltest du Konstruktionen wie

catch (Exception e) {
}

aus deinem Hirn und deinem Code entfernen! Du beraubst dich sämtlicher Debug-Möglichkeiten damit.

Fange die relevanten Exceptions dort auf, wo sie auftreten können, und generiere eine aussagekräftige Fehlermeldung (mit Eclipse kannst du try-catch-Blöcke sehr komfortabel erzeugen lassen).

 

[SamHotte]

Zweitens: die getter/setter sind ok, der Fehler lässt sich durch debuggen finden (breakpoint vor der entsprechenden Zeile setzen und schrittweise durch das Programm gehen - du siehst die jeweiligen Werte deiner Variablen dann (in Eclipse oben rechts).

Wenn du dann nichts erkennen kannst, dann löse deine Konstruktion 'treepath.getLastPathComponent().toString());' auf in sowas:

Object lastPathComponent = treepath.getLastPathComponent();
String s = lastPathComponent.toString();

Damit kannst du im Debugger sehen, wo die NullPointerException genau auftritt.

 

[Reinhard]

@SamHotte:

Ich habe die Logging-Funktionalität aus dem Exception-Handling entfernt, um den Code hier im Forum übersichtlicher darzustellen. Bei mir im Programm wird das natürlich geprüft

 

[SamHotte]

Sehr gut

 

[Reinhard]

Wenn ich "window" an den CMouseAdapter übergebe, hat es immer den Wert "null". Darum wohl die Exception

  public CMouseAdapter(JTree tree, MainForm window) {
    this.tree = tree;
    this.window = window;
  }

Der MouseAdapter bekommt hier zwei Werte. Bei tree passt alles, diese Werte sehe ich auch im Debugger, aber window ist immer NULL.

Ist die Übergabe evtl. noch zu früh bzw. mir ist eigentlich nicht klar, wann die Übergabe denn dann passieren sollte?

 

[SamHotte]

wann rufst du denn überhaupt die MainForm.initialize() auf?

 

[Reinhard]

Das ganze sieht bei mir so aus. Das initialize() wird im Konstruktor von MainForm aufgerufen:

public class MainForm
{
  private JTextField textField;
  private JTree tree;
  private MainForm window;
  
  public static void main(String args[])
  {
    Logging.LogInfoMessage("Singletons created");

    try {
      MainForm window = new MainForm();
      window.frame.setVisible(true);
    } catch (Exception e) {
      Logging.LogErrorMessage(e.toString());
    }
  }

  public MainForm()
  {
    initialize();
  }

  private void initialize()
  {    
     
    frame = new JFrame();
    frame.setIconImage(Toolkit.getDefaultToolkit().getImage("./img/Achat_small.gif"));
    frame.setTitle("JDBCTree");
    frame.setBounds(100, 100, 500, 375);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);    
     
    // ProjektID auf 1 setzen (hier muss die ID z.B. bei einem Auswahldialog übergeben werden)
    int lintProjektID = 1;
    
    try
    {        
      tree = new JTree(model);
      tree.setCellRenderer(new CCellRenderer());
      tree.addMouseListener(new CMouseAdapter(tree, window));
      frame.getContentPane().add(tree, BorderLayout.CENTER);
    }
    catch ( SQLException e )
    {
      Message.ShowErrorMessage(e.toString());
      return;
    }     

    final JPanel panel = new JPanel();
    panel.setLayout(new FormLayout(
      new ColumnSpec[] {
        FormFactory.RELATED_GAP_COLSPEC,
        new ColumnSpec("96px"),
        FormFactory.RELATED_GAP_COLSPEC,
        new ColumnSpec("11px"),
        FormFactory.DEFAULT_COLSPEC,
        FormFactory.DEFAULT_COLSPEC},
      new RowSpec[] {
        FormFactory.RELATED_GAP_ROWSPEC,
        new RowSpec("19px"),
        FormFactory.RELATED_GAP_ROWSPEC,
        FormFactory.DEFAULT_ROWSPEC}));
    frame.getContentPane().add(panel, BorderLayout.EAST);

    final JLabel selektierterKnotenLabel = new JLabel();
    selektierterKnotenLabel.setText("Selektierter Knoten:");
    panel.add(selektierterKnotenLabel, new CellConstraints("2, 2, 1, 1, fill, center"));

    textField = new JTextField();
    panel.add(textField, new CellConstraints("2, 4, 1, 1, fill, fill"));
  }

  public String getTextField()
  {
    return textField.getText();
  }

  public void setTextField(String aText)
  {
    this.textField.setText(aText);
  }

}

Mein Problem ist bestimm nur eine minimale Sache, aber ich komm einfach nicht drauf

 

[SamHotte]

Dann isses klar - solange der Konstruktor nicht fertig ist, wird das Objekt noch nicht an irgendwen anders übergeben. Daher kann dein Maushörer auch die Referenz nicht auflösen.

Lösungsvorschlag: du hast tree ja schon als Attribut deklariert. Verschiebe den 'tree.addMouseListener(...)' in die main (hinter MainForm ... = new ... ), dann sollte es funzen.

 

[Reinhard]

Hast recht, jetzt funktioniert es. Ich muss mir das ganze wohl noch ein bisschen anschauen. Vielen dank für deine geduldige Hilfe.

Schöne Grüße,
Reinhard

 

[Gast]

Hallo warum benötigt die Klasse MainFrame eine Instanz (window) von sich selbst?

mach doch mal folgendes:

tree.addMouseListener(new CMouseAdapter(tree, this));

 

[Reinhard]

Hallo Gast,

stimmt so geht es auch