setText aus anderer class heraus

[StepByStepButSlow]

Einen schönen Abend miteinander,

vorweg gesagt ist dies mein erster Post in diesem Forum, da ich mich erst seit einigen Wochen explizit mit Java beschäftige. Da ich über Suchen bei google nicht das Gewünschte gefunden habe, hoffe ich, dass jemand in diesem Forum mir möglicherweise helfen kann.

In meinem Beispielprojekt besitze ich mehrere Textfelder, die über verschiedene Buttons überschrieben werden sollen
Bei einzelnen Textfeldern habe ich dies noch über <textfield>.setText(""); gelöst. Da sich die Anzahl der Textfelder jetzt jedoch mannigfaltig erhöht hat, würde ich gerne wissen, ob es möglich ist dies mithilfe einer separaten Class zu lösen, welche in den z.B. Button integriert wird.

Anbei eine stark verkürzte Herangehensweise meinerseits, die jedoch bislang noch keinen erfolg gebracht hat, da ich das Textfeld in meinem Mainframe nicht angesprochen bekomme:

package guiPrivatePractice;

import java.awt.BorderLayout;
import java.awt.EventQueue;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;
import javax.swing.JTextField;
import javax.swing.JButton;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;

public class LoeschenPerClassMain extends JFrame {

    public JPanel contentPane;
    public JTextField textfield1;
    public JButton button1;

    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    LoeschenPerClassMain frame = new LoeschenPerClassMain();
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }

    public LoeschenPerClassMain() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(100, 100, 450, 300);
        contentPane = new JPanel();
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        contentPane.setLayout(new BorderLayout(0, 0));
        setContentPane(contentPane);
       
        textfield1 = new JTextField();
        contentPane.add(textfield1, BorderLayout.NORTH);
        textfield1.setColumns(10);
       
        button1 = new JButton("New button");
        button1.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
               
                // Ausführen der Class
                LoeschenPerClassSub = new LoeschenPerClassSub();
            }
        });
        contentPane.add(button1, BorderLayout.CENTER);
    }

}

Meine Class zum Überschreiben des Textfeldes:

package guiPrivatePractice;

public class LoeschenPerClassSub {
    LoeschenPerClassMain.textfield1.setText("");
}

Da ich vermutlich völlig auf dem Holzweg bin oder mir einfach noch grundlegende Informationen zu Java fehlen, wäre ich über jeden Tipp oder Hinweis froh.

Vielen Dank und weiterhin einen schönen Abend.

StepByStepButSlow

 

[Luggees]

probier es mal damit:

public static JTextField textfield1;

Damit solltest du das Textfeld auch von anderen Klassen ansprechen können.

 

[Harry Kane]

Gewöhne dir bloss nicht an, ohne Not public static Variablen einzuführen.
Mein Alternativvorschlag:
- Instanziieren der Objekte in einem Loop, sowohl die Objekte mit denen gearbeitet werden soll (in deinem Fall JTextFields) als auch die Objekte die arbeiten sollen.
- In demselben Loop, alle Aktionen durchführen, die mit den Objekten durchgeführt werden müssen, also z. B. die Gui bauen oder die Objekte miteinander bekannt machen, die zusammenarbeiten sollen.
Hier mal ein Beispiel:

public class ParametrizedActionListener {

    public static void main(String[] args) {
        JFrame main = new JFrame("Demo");
        JPanel grid = new JPanel();
        grid.setLayout(new GridLayout(4, 3));
        grid.add(new JLabel("Von hier"));
        grid.add(new JLabel(""));
        grid.add(new JLabel("nach hier"));
        for (int i = 0; i < 3; i++) {
            JTextField source = new JTextField(10);
            JTextField target = new JTextField(15);
            String s = String.valueOf(i+1);
            grid.add(source);
            grid.add(new JButton(new TextContentSetter(s, source, target)));
            grid.add(target);
        }
        main.add(grid);
        main.pack();
        main.setVisible(true);
        main.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }

    static class TextContentSetter extends AbstractAction {

        private JTextComponent source;
        private JTextComponent target;

        public TextContentSetter(String label, JTextComponent source, JTextComponent target) {
            super.putValue(AbstractAction.NAME, label);
            this.source = source;
            this.target = target;
        }

        public void actionPerformed(ActionEvent ae) {
            target.setText(source.getText());
        }
    }
}

 

[StepByStepButSlow]

Hallo Luggees und Harry Kane,

vielen Dank für eure schnellen Antworten.

@Harry Kane:
Deine Vorgehensweise werde ich einmal ausprobieren. Derweilen bin ich auf folgende Lösung gekommen:

Mainframe:

package guiPrivatePractice;

import java.awt.BorderLayout;
import java.awt.EventQueue;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.border.EmptyBorder;
import javax.swing.JTextField;
import javax.swing.JButton;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;

public class LoeschenPerClassMain extends JFrame {

    public JPanel contentPane;
    public static JTextField textfield1;
    public JButton button1;

    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                try {
                    LoeschenPerClassMain frame = new LoeschenPerClassMain();
                    frame.setVisible(true);
                } catch (Exception e) {
                    e.printStackTrace();
                }
            }
        });
    }

    public LoeschenPerClassMain() {
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setBounds(100, 100, 450, 300);
        contentPane = new JPanel();
        contentPane.setBorder(new EmptyBorder(5, 5, 5, 5));
        contentPane.setLayout(new BorderLayout(0, 0));
        setContentPane(contentPane);
      
        textfield1 = new JTextField();
        contentPane.add(textfield1, BorderLayout.NORTH);
        textfield1.setColumns(10);
      
        button1 = new JButton("New button");
        button1.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                LoeschenPerClassSub nc = new LoeschenPerClassSub();
                nc.UeberschreibeText();
            }
        });
        contentPane.add(button1, BorderLayout.CENTER);
    }

}

Subframe:

package guiPrivatePractice;

public class LoeschenPerClassSub {
    public void UeberschreibeText() {
        LoeschenPerClassMain.textfield1.setText("");
    }
}

Spricht aus eurer Sicht etwas gegen diese Vorgehensweise? Wie gesagt, so gut kenne ich mich mit der Sprache Java noch nicht aus, um so etwas zu beurteilen.

Noch einen schönen Abend

StepByStepButSlow

 

[Luggees]

Ich würde die Variante mit public static als "quick and dirty" bezeichnen. Geht zwar schneller aber wie Harry Kane schon sagte nicht angewöhnen aus allem schnell public static zu machen

 

[Flown]

Klassen bilden in Objekt-Orientierten Sprachen Modelle zum Abbilden der Wirklichkeit. Was eine Klasse auszeichnet, ist das bekannte Umfeld (der State) und die Akteure (Methoden) um das Umfeld zu beeinflussen.
Wenn man sich jetzt das mal auf der Zunge zergehen lässt, dann wird einem klar, dass LoeschenPerClassSub zwar Akteure besitzt, aber das Umfeld nicht kennt. Das Umfeld wird dann über Konstruktoren oder setter der Klasse gesetzt, oder eben hard-coded direkt in der Klasse.
Hard-Coded Sachen, sind schwer zu ersetzen (was denn Sinn einer Wiederverwendung einer Klasse nichtig macht) und schwer testbar.

public class LoeschenPerClassSub {
  private JTextField textField;

  public LoeschenPerClassSub(JTextField textField) {
    this.textField = textField;
  }

  public void loescheText() {
    textField.setText("");
  }
}

Kleiner Hinweis:
Wenn du eine Klasse in Java vererbst (oder generell bei Vererbung) hat deine Klasse zu deiner erbenden Klasse eine "ist-eine" Beziehung. Das braucht man nur, wenn man die geerbte Klasse um Funktionalität erweitern will. Heißt: Du solltest wie @Harry Kane dein JFrame (was du ja erbst) lokal oder als Feld halten. Denn es gilt grundsätzlich Komposition vor Vererbung.

 

[Harry Kane]

Spricht aus eurer Sicht etwas gegen diese Vorgehensweise?

Ja, sie ist unnötig kompliziert. Der anonyme ActionListener, den du button1 hinzufügst, is quasi eine lokale Variable. Er kennt alle lokalen und Instanzvariablen von LoeschenPerClassMain und damit auch das JTextField, kann darauf zugreifen, und kann deshalb in seiner actionPerformed direkt setText darauf aufrufen. So wie du es gelöst hast, instanziierst du in einer Klasse (A) eine andere Klasse (B), die dann wieder auf eine Variable von A zugreift. Das wäre nur dann statthaft, wenn B so umfangreich wäre, dass man den entsprechenden Code nicht in A reinklatschen möchte.