Eigenschaften eines Buttons per Setter verändern

[nobsche]

Ich möchte ein Programm erstellen in dem die meisten Buttons deaktiviert sind, damit diese nicht angeklickt werden können, wenn noch nicht alle Daten erfasst sind. So soll z. B. geprüft werden ob alle Textfelder gefüllt sind. Erst wenn dies der Fall ist, sollte der Button "Daten erfassen" aktiviert werden.
Die Prüfung soll in einer anderen Klasse in einem anderen Paket stattfinden. Die Buttons sind Private deklariert und zum aktivsetzen des Buttons ist ein entsprechender Setter vorhanden. Um nicht zu viel überflüssigen Code posten zu müssen, habe ich ein Programm geschrieben, welches die Problematik exakt wiedergibt. Es besteht im Grunde nur aus zwei Buttons. Der erste ist deaktiviert und soll aktiviert werden, wenn der zweite angeklickt wird.

Hier der code:

import java.awt.EventQueue;

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

public class buttons extends JFrame {

    /**
     *
     */
    private static final long serialVersionUID = 1L;
    private JPanel contentPane;
    private JButton btn_button;
    private JButton btn_button2;
    
    
    

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

    /**
     * Create the frame.
     */
    public buttons() {
        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);
        
        btn_button = new JButton("Button");
        btn_button.setEnabled(false);
        btn_button.setBounds(29, 12, 114, 25);
        contentPane.add(btn_button);
        
        btn_button2 = new JButton("Button 2");
        btn_button2.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                new pressButton().activateButton();
            }
        });
        btn_button2.setBounds(29, 60, 114, 25);
        contentPane.add(btn_button2);
    }

    
    public void setBtn_button2(boolean b) {
        System.out.println("erreicht"); //Zum Testen ob die Methode erreicht wird.
        this.btn_button.setEnabled(b);
        
    }

}

public class pressButton {

    public void activateButton() {
        buttons ab = new buttons();
        ab.setBtn_button2(true);
        
    }
}

In der Klasse "pressButton" wird nur der Setter angesprochen, welcher den ersten Button setEnabled(true) schalten soll.
Das die Settermethode erreicht wird, lässt sich anhand einer kleinen Konsolentextausgabe nachvollziehen. Die Zeile "this.btn_button.setEnabled(b)" scheint jedoch nicht berücksichtigt zu werden. Es erscheint keine Fehlermeldung und der Button wird nicht aktiv geschaltet. Der übergebene Boolean b hat zu diesem Zeitpunkt den wert "true". Das habe ich im Debugging-Modus sehen können.

Vermutlich ist es nur eine Kleinigkeit die ich übersehen habe. Aber ich brüte nun schon seit einigen Tagen an diesem Problem und finde keine Lösung. Google und die Forensuche haben mir bisher leider auch nicht weitergeholfen. Könnt ihr mir sagen was ich falsch mache?

Ich danke euch schon mal für eure Unterstützung.

Viele Grüße
nobsche

 

[mrBrown]

Du erstellst in activateButtons ein neues buttons-Objekt, und in diesem neuen Fenster aktivierst du dann den Button. An dem ursprünglichen, sichtbaren Fenster änderst du aber nichts.

 

[nobsche]

Vielen Dank für die schnelle Antwort mrBrown.

OK, ich erzeuge ein neues Objekt. Da ich somit nicht auf den bestehenden Button zugreife, kann ich diesen auch nicht verändern. Ich verändere also ein unsichtbares Objekt. Das kann ich nachvollziehen.

Wie aber kann ich den Button verändern? Hier fehlt mir noch eine kleine Idee.
Ich könnte mir den Button mit einem Getter holen. Um den Getter ansprechen zu können, müsste ich aber auch ein neues Objekt erzeugen. Das würde bedeuten, dass ich den Button wieder nicht verändern kann.

Ich glaube hier fehlt mir der richtige Ansatz.

 

[Blender3D]

Wie aber kann ich den Button verändern? Hier fehlt mir noch eine kleine Idee.

Hier ein Beispiel. Teste es und überlege dir die Funktionsweise.

import javax.swing.JFrame;

public class start {
    public static void main(String[] args) {
        JFrame frame = new JFrame();
        frame.getContentPane().add(new ButtonPropertyChanger(500, 500));
        frame.setTitle(" Change Button poperties");
        frame.pack();
        frame.setVisible(true);
    }
}

import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JColorChooser;
import javax.swing.JOptionPane;
import javax.swing.JPanel;

@SuppressWarnings("serial")
public class ButtonPropertyChanger extends JPanel implements ActionListener {
    private JButton button = new JButton("Test");

    public ButtonPropertyChanger(int width, int height) {
        setPreferredSize(new Dimension(width, height));
        setBackground(Color.DARK_GRAY);
        CtrlBar bar = new CtrlBar(this);
        bar.setPreferredSize(new Dimension(width, 30));
        add(bar, BorderLayout.NORTH);
        add(button, BorderLayout.CENTER);
        button.setPreferredSize(new Dimension(200, 40));
        button.setFocusable(false);
    }

    @Override
    public void actionPerformed(ActionEvent e) {
        String cmd = e.getActionCommand();
        if (cmd.equals(CtrlBar.getCmd(CtrlBar.TEXT)))
            changeText();
        if (cmd.equals(CtrlBar.getCmd(CtrlBar.VISIBLE)))
            button.setVisible(!button.isVisible());
        if (cmd.equals(CtrlBar.getCmd(CtrlBar.ENABLED)))
            button.setEnabled(!button.isEnabled());
        if (cmd.equals(CtrlBar.getCmd(CtrlBar.RESIZE)))
            changeSize();
        if (cmd.equals(CtrlBar.getCmd(CtrlBar.COLOR)))
            changeColor();
    }

    private void changeColor() {
        Color color = JColorChooser.showDialog(this, "Farbe wählen", button.getBackground());
        if (color != null)
            button.setBackground(color);
    }

    private void changeSize() {
        Dimension dim = inputDimension();
        if (dim != null) {
            button.setPreferredSize(dim);
            setVisible(false);
            setVisible(true);
        }
    }

    private void changeText() {
        String text = JOptionPane.showInputDialog(this, "Neuer Text", button.getText());
        if (text != null)
            button.setText(text);
    }

    private Dimension inputDimension() {
        String dim = JOptionPane.showInputDialog(this, "Weite, Höhe", button.getWidth() + "," + button.getHeight());
        if (dim != null) {
            int[] values = strToDim(dim);
            if (values == null)
                JOptionPane.showConfirmDialog(this, "Bitte 2 ganze Zahlen durch Beistrich getrennt eingeben!");
            else
                return new Dimension(values[0], values[1]);
        }
        return null;
    }

    private int[] strToDim(String dim) {
        int[] values = new int[2];
        String[] strValues = dim.split(",");
        if (values == null || values.length != 2)
            return null;
        try {
            for (int i = 0; i < values.length; i++) {
                values[i] = Integer.parseInt(strValues[i].trim());
            }
        } catch (NumberFormatException e) {
            return null;
        }
        return values;
    }

}

import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JToolBar;

@SuppressWarnings("serial")
public class CtrlBar extends JToolBar {
    private static String[] cmd = { "resize", "visible", "enabled", "text", "color" };
    public final static int RESIZE = 0;
    public final static int VISIBLE = 1;
    public final static int ENABLED = 2;
    public final static int TEXT = 3;
    public final static int COLOR = 4;

    public CtrlBar(ActionListener listener) {
        setRollover(true);
        setFloatable(false);
        for (int i = 0; i < cmd.length; i++) {
            addSeparator();
            addSeparator();
            JButton b = new JButton(cmd[i]);
            b.setFocusable(false);
            add(b);
            b.addActionListener(listener);
        }
    }

    public static final String getCmd(int i) {
        if (i < 0 || i >= cmd.length)
            throw new IllegalArgumentException();
        return cmd[i];
    }

}

 

[nobsche]

Vielen Dank für eure Hilfe. Habe es nun hinbekommen.
Ähnlich wie in deinem Beispiel Blender3D.

 

[Blender3D]

Gern geschehen.