Variable von 1. Fenster an 2. Fenster übergeben.

[Eric31]

Hallo zusammen,

ich bin gerade dabei ein Programm zu schreiben bei dem man eine Sinusgrundschwingung mit Oberwellen mit beliebiger Amplitude und Frequenz überlagern und das ganze in einer Datei ausgeben kann. Die Datei soll dann später von einer Spannungsquelle eingelesen werden und die Spannung für Testzwecke nachbilden.

Nun zu meinem Problem:
Ich erzeuge mir mit der Klasse "JmainFrame" Fenster in dem ich die Anzahl der Harmonischen vom User einlese.
Zusätzlich habe ich eine weitere Klasse "JInputFrame" in der ein neues Fenster erzeugt wird in dem man dann die Amplitude und Frequenz der einzelnen Harmonischen eintragen kann. Um das Ganze variabel zu gestalten will ich abhängig von der zuvor eingegeben Anzahl an Harmonischen Textfelder für die Frequenz und Amplitude erstellen. Dafür hab ich mir ein JTextField Feld erstellt und als Index die Variable für die Anzahl der Harmonischen genutzt. Allerdings weiß ich nicht wie ich diese in diesem Fall richtig übergebe.

Wenn ich sie wie nachfolgend als Parameter an den Kontruktor übergebe (double anz) fehlt sie mir "oben" bei der Deklaration der Felder.
Ich könnte natürlich auch die Deklaration der Felder erst nach "Anz_Ha = (int) anz;" machen, aber ich benötige die Felder auch noch in der actionPerformed methode und sie sind dadurch dann nicht bekannt.

public class JInputFrame extends JFrame implements ActionListener
    {
    int Anz_Ha;
    JButton Ausgabe, Plotten;
    JLabel Ha_La[] = new JLabel[Anz_Ha];
    JLabel Amp_La = new JLabel("Amplitude");
    JLabel Fre_La = new JLabel("Frequenz");
    JTextField Amp_Te[] = new JTextField[Anz_Ha];
    JTextField Fre_Te[] = new JTextField[Anz_Ha];
    
    
    JInputFrame(double anz, String Title)
    {   
        super (Title);
        Anz_Ha = (int) anz;
        
        JLabel Gr_La = new JLabel("Grundschwingung");
        JTextField Gr_Te = new JTextField();

 

[KonradN]

Ich verstehe Deinen Text nicht wirklich. Aber die Problematik in der Überschrift ist recht deutlich denke ich mal.

Du hast vermutlich mehrere Werte, die Du dann in den Fenstern austauschen willst. Diese Werte kannst Du in einer Klasse ablegen. Also etwas in der Art:

public class Model {
    private int anzahl;
    // Weitere Werte
    
    // Getter / Setter
}

Davon kannst Du dann z.B. eine Instanz erstellen und diese dann in den anderen Klassen übergeben. Da Du eine Referenz auf die Instanz von Model übergibst, können die Klassen auch Veränderungen nachträglich lesen.

Wenn Du nun noch willst, dass ein Fenster mitbekommt, wenn Du eine Änderung machst, dann wendest Du noch das Observer Pattern an. Das bedeutet, dass Du in der Klasse Model noch einen Liste führst von Elementen, die an Veränderungen Interesse haben und da kann sich dann jeder Eintragen, der sich dafür interessiert. Das sieht dann so aus, wie in der Klasse https://github.com/kneitzel/JAdventure/blob/Lektion-3/src/main/java/org/jadv/framework/Model.java
(Kannst Du 1:1 übernehmen denke ich mal... und die Setter setzen dann nicht nur den Wert sondern rufen auch hasChanged() auf.)

Dann hast Du sowas wie ein EingabeFenster, welches da dann einfach Setter aufruft (Wann immer sich halt die Werte ändern sollen)
Und ein AnzeigeFenster, das die gleiche Referenz hat und sich bei der Model Referenz eingetragen hat als Interessent. Immer, wenn sich da Werte ändern werden diese übernommen.

Das wäre dann ein relativ einfacher Aufbau bei dem die Fenster sich nicht einmal kennen - sie sind also komplett entkoppelt über das Model.

Und noch ein paar generelle Hinweise:
a) FCoI - Favor Composition over Inheritance. Statt von JFrame abzuleiten solltest Du Dir überlegen, eine Klasse zu schreiben, die ein JFrame hat. Das führt in der Regel zu besser lesbaren Code (weshalb diese Regel mal erfunden wurde).
b) So Funktionale Interfaces (Interfaces mit nur einer Methode) sollte man nicht erben, wenn es nicht sein muss. Das führt zu Methoden, die nicht sagen, was sie wieso machen. So bekommst Du in der KLasse durch das erben von ActionListener eine Methode actionPerformed. Aber was die Methode genau macht, besagt das ja nicht. Da ist es besser, wenn die Methode klar besagt, was sie macht. Also wenn da Werte an das Model übergeben werden, dann wäre das eine Methode wie "updateModel" oder so. Und da, wo Du diesen ActionListener setzt, kannst Du dann ein Lambda Ausdruck wie e -> updateModel() übergeben oder wenn updateModel auch das ActionEvent als Parameter bekommt, dann könnte es eine Methodenreferenz sein: this::updateModel

Das sind aber einfache, allgemeine Hinweise zu Clean Code - auch ohne diese Hinweise funktioniert der Code natürlich.

 

[Eric31]

Vielen Dank für die schnelle und ausführliche Antwort.

Meine Überschrift war wohl etwas irreführend bzw. würde es schon so funktionieren, aber ich will eigentlich nur einmal eine Variable übergeben und dafür ist eine eigene Klasse etwas übertrieben, oder nicht?

Nochmal zum Problem:
ich lese im ersten Fenster einen Wert ein, der mir die Größe eines Feldes für das 2. Fenster festlegen soll. Dadurch will ich die Anzahl an Textfelder im 2. Fenster abhängig von der Usereingabe gestalten.


Ausschnitt Klasse/Fenster1:

    public void actionPerformed(ActionEvent arg0) {
        String str = new String();
        str = Anz_Ha_Te.getText();
        //String in double umwandeln
        double anz = Double.valueOf(str);
        
        //Fenster schließen
        dispose();
        
        try
        {
        if (anz > 0)
        {
            
            JInputFrame InputFrame = new JInputFrame(anz, "Spannungsformgenerator");
            
        }

Ausschnitt Klasse/Fenster2:

public class JInputFrame extends JFrame implements ActionListener
    {
    int Anz_Ha;
    JButton Ausgabe, Plotten;
    JLabel Ha_La[] = new JLabel[Anz_Ha];
    JLabel Amp_La = new JLabel("Amplitude");
    JLabel Fre_La = new JLabel("Frequenz");
    JTextField Amp_Te[] = new JTextField[Anz_Ha];
    JTextField Fre_Te[] = new JTextField[Anz_Ha];
    
    
    JInputFrame(double anz, String Title)
    {   
        super (Title);
        Anz_Ha = (int) anz;
        
        setSize(500,600);
        setLayout(null);

 

[KonradN]

Ja, das geht auch.

Ich glaube, jetzt habe ich das Problem verstanden - Du initialisierst derzeit die Felder direkt bei der Deklaration. Das musst Du dann natürlich auch im Konstruktor machen:

public class JInputFrame extends JFrame implements ActionListener
    {
    int Anz_Ha;
    JButton Ausgabe, Plotten;
    JLabel Ha_La[];
    JLabel Amp_La = new JLabel("Amplitude");
    JLabel Fre_La = new JLabel("Frequenz");
    JTextField Amp_Te[];
    JTextField Fre_Te[];
   
   
    JInputFrame(double anz, String Title)
    {  
        super (Title);
        Anz_Ha = (int) anz;
       
        setSize(500,600);
        setLayout(null);
        Ha_La = new JLabel[Anz_Ha];
        Amp_Te = new JTextField[Anz_Ha];
        Fre_Te = new JTextField[Anz_Ha];

Edit: Beim Konstruktor die eckigen Klammern in der Zuweisung noch entfernt.

 

[Eric31]

Ja das ist mir bewusst, aber ich hab unten dann noch die methode "actionPerformed" und benötige da ebenfalls die Felder. Wenn ich sie im Konstruktor definiere sind sie dort nicht bekannt. Gibt es noch einen anderen Weg das umzusetzen ?

 

[KonradN]

Du deklarierst sie als Instanzvariablen. Du initialisierst sie nur in dem Konstruktor.

An der Deklaration hat sich doch nichts geändert. Daher sind sie weiterhin überall da verfügbar, wo sie verfügbar waren.

 

[KonradN]

Also um es noch einmal deutlich zu machen:

public class JInputFrame extends JFrame implements ActionListener {

    ...
    JLabel Ha_La[];  // Reine Deklaration. Das Feld gibt es daher, aber es hat erst einmal nur null als Inhalt!
    ...
   
    JInputFrame(double anz, String Title) {
       
        ...
        Ha_La = new JLabel[Anz_Ha]; // Hier ist eine reine Zuweisung!
                                    // Also kein JLabel davor was es sonst zu einer
                                    // lokalen Variable machen würde!
        ...

Und aufpassen - mir ist ein Fehler passiert - bei der Zuweisung sind natürlich auch keine [] mehr!

 

[Eric31]

Ach optimal genau so wollt ich es. Hatte es gestern abend schon mal so probiert, aber stand da wohl auf dem Schlauch.

Vielen Dank!!