GUI-Komponente in separate Klassen auslagern?

Hallo zusammen,

ich habe eine GUI programmiert. Leider ist der Code ziemlich unübersichtlich. Deswegen dachte ich, ich lagere die Erstellung der Komponenten aus. Nur leider verschwinden dann die ausgelagerten Komponenten aus der GUI.

Beispiel:

Erste Klasse:

package Verknüpfung;

import java.awt.BorderLayout;

public class Teil1 extends JFrame {
  private JPanel contentPane;

  static int f;

  /*
   * Launch the application.
   */
  public static void main(String[] args) {
    Teil2 blubb = new Teil2();
    Teil2.textfeld();

    EventQueue.invokeLater(new Runnable() {
      public void run() {
        try {
          Teil1 frame = new Teil1();
          frame.setVisible(true);
        }
        catch (Exception e) {
          e.printStackTrace();
        }
      }
    });
  }

  /*
   * Create the frame.
   */
  public Teil1() {
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setBounds(100, 100, 450, 300);
    contentPane = new JPanel();
    contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
    setContentPane(contentPane);
    contentPane.setLayout(null);

    JLabel lbl1 = new JLabel("Test");
    lbl1.setBounds(45, 68, 69, 20);
    contentPane.add(lbl1);

    JLabel lbl2 = new JLabel("Test");
    lbl2.setBounds(45, 151, 69, 20);
    contentPane.add(lbl2);
  
    JButton btnNewButton = new JButton("New button");
    btnNewButton.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent arg0) {
        int i;
        i = 5;
        f = i + 6;
      }
    });
    btnNewButton.setBounds(209, 199, 115, 29);
    contentPane.add(btnNewButton);
  }
}

Zweite Klasse:

package Verknüpfung;

import java.awt.BorderLayout;

public class Teil2 extends JFrame {
  private JPanel contentPane;

  private JTextField txt1;

  private JTextField txt2;

  /*
   * Launch the application.
   */
  public static void main(String[] args) {
    EventQueue.invokeLater(new Runnable() {
      public void run() {
        try {
          Teil2 frame = new Teil2();
          frame.setVisible(true);
        }
        catch (Exception e) {
          e.printStackTrace();
        }
      }
    });
  }
                                                                                                  
  /*
   * Create the frame.
   */
  public Teil2() {
    setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    setBounds(100, 100, 450, 300);
    
    contentPane = new JPanel();
    contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
    setContentPane(contentPane);
    contentPane.setLayout(null);

    txt1 = new JTextField();
    txt1.setBounds(267, 74, 146, 26);
    contentPane.add(txt1);
    txt1.setColumns(10);
    
    txt2 = new JTextField();
    txt2.setBounds(267, 149, 146, 26);
    contentPane.add(txt2);
    txt2.setColumns(10);
    txt2.setText("Hi");
  }

  public static void textfeld() {
  }                                                                               }

Answer 1

[regex9]

Vermutlich hängst du sie nicht richtig an.

Befasse dich einmal mit dem MVC-Pattern. Mit diesem kannst du eine Trennung von Logik und grafischer Oberfläche schaffen.

Deinen Code schau ich mir später nochmal an und setze dann einen Kommentar unter diese Antwort. Schon jetzt würde ich dir allerdings raten, mit Layout Managern zu arbeiten.

Comments

[denni9]

danke regex9!

Answer 2

[NatanInfoPhilo]

Habe in der GUI nur GUI Elemente.

Habe alles kommentiert.

Dann sollte es übersichtlich sein.

Answer 3

[regex9]

Ich schreibe nun doch einfach eine neue Antwort. Deinen Code habe ich mir inzwischen genauer angeschaut.

So etwas:

Teil2 blubb = new Teil2();
Teil2.textfeld();

Solltest du nicht tun. Kapsel alles was mit deiner View zu tun hat in der Klasse.

Zum Beispiel so:

class View extends JFrame {
  public View() {
    init();
  }

  private void init() {
    // set frame settings ...

    contentPane.add(getTextField(10));
    JTextField textField2 = getTextField(10);

    textField2.setText("hello");
    contentPane.add(textField2);
  }

  private static JTextField getTextField(int columns) {
    // create and return textfield ...
  }
}

Mit der Methode getTextField kannst du den Vorgang, ein Textfeld zu erzeugen, abstrahieren. Dazu kannst du dir auch eine Helferklasse anlegen.

Wenn dein Frame aus sehr vielen Komponenten besteht, kann es durchaus Sinn machen, aufzuteilen. Beispiel: Du baust dir einen Editor und hast darin einen Objekt-Dialog, eine Preview-Box, ein Code-Fenster, etc.. Dann würde ich tatsächlich für die einzelnen Parts eigene Klassen anlegen bzw. eigene Swing-Komponenten erstellen (die bspw. von JPanel erben).

SettingsPanel settingsPanel = new SettingsPanel();
ScriptPanel scriptPanel = new ScriptPanel();
// ...
frame.add(settingsPanel);
frame.add(scriptPanel);
// ...

Jede Komponente sollte dabei für sich stehen - einen eigenen Aufgabenbereich erfüllen und möglichst unabhängig von anderen Komponenten sein.

Comments

[denni9]

danke für deine hilfe!!! das hilft mir gut!!!