Problem making buttons visible

Question

Hi there, I have a poker game that all worked fine until I was told to restructure it in a MVC style. I never had this problem before I changed it but I'm having issues with making my buttons visible when it's the players turn. At first I setVisible to false which works fine, then I call my method to display them from my model and they display fine, so seemings as all is good to here I think it's the next bit that's causing my issue, I have actionListeners attached that when fired setsVisible on all the buttons to false again (which works) but the next time I call my method to make certain buttons visible they don't appear. I've tested that the method is being called with the correct parameter passed and it is. Now I must apologise because my knowledge of swing is very poor but I have searched and read alot and tried several approaches with no joy, I will definitly be studying up more on it in the future but time is against me at the mo. The only thing that seems to half work is if I use this.paint(g) the buttons display as I want them but my program crashes cos to be honest I don't know what g should be. Is this the right approach or is there a more simple way I can update my GUI to display my buttons?

Answer 1

Works for me but you could try to add a revalidate(); repaint(); on the buttons parent.
import java.awt.BorderLayout;
import java.awt.EventQueue;
import java.awt.event.ActionEvent;
import javax.swing.AbstractAction;
import javax.swing.JButton;
import javax.swing.JFrame;
public class TestVisible {
    public static void main(String[] args) {
        EventQueue.invokeLater(new Runnable() {
            public void run() {
                final JButton button = new JButton("I do nothing");
                final JFrame frame = new JFrame("Test");
                frame.setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
                frame.getContentPane().add(button);
                frame.getContentPane().add(new JButton(new AbstractAction("Toggle") {
                    @Override
                    public void actionPerformed(ActionEvent e) {
                        button.setVisible(!button.isVisible());
//                        frame.getRootPane().revalidate();
//                        frame.getRootPane().repaint();
                }), BorderLayout.PAGE_START);
                frame.pack();
                frame.setLocationRelativeTo(null);
                frame.setVisible(true);
}