Thread: How to change the background colour by clicking buttons?

Hi guys

Below code mean to change the background colour by clicking the redButton or blueButton, but the code doesn't work, (only can show the frame and two buttons), does any anyone know why?

Thanks heaps

package brainydraw;
 
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
 
public class PictureButton extends JFrame
{
 
    private JButton redButton;
    private JButton blueButton;
 
    public PictureButton()
    {
        setSize(200, 200);
        setLocation(200, 200);
 
        redButton = new JButton("RED");
        blueButton = new JButton("BLUE");
 
        Container content = getContentPane();
        content.setLayout(new FlowLayout());
 
        content.add(redButton);
        content.add(blueButton);
 
        //register the current panel as listener for the buttons
        ActionListener rBListener = new RedAndBlueButtonListener();
        redButton.addActionListener(rBListener);
        blueButton.addActionListener(rBListener);
    }
 
       public class RedAndBlueButtonListener implements ActionListener
       {
           public void actionPerformed(ActionEvent ae)
           {
                Color color = getBackground();  // color will be set
                Object source = ae.getSource();
                if (source == redButton) color = Color.red;
                else if (source == blueButton) color = Color.blue;
                setBackground(color);
                repaint();
            }
       }
 
    public static void main(String[] args)
    {
        JFrame f = new PictureButton();
 
        f.addWindowListener(new WindowAdapter()
        {
 
            public void windowClosing(WindowEvent we)
            {
                System.exit(0);
            }
        });
        f.setVisible(true);
    }
}

Can you elaborate on what you mean by 'code doesn't work'?

You're setting the background of a JFrame. But the background of a JFrame is covered by its contentPane, which is by default opaque. You should be setting the background of the contentPane.

Oh, and it's never necessary to explicitly call repaint() after changing the background of a Swing component.

db

Originally Posted by copeg

Can you elaborate on what you mean by 'code doesn't work'?

Hi copeg

I got a problem that I when I clicked the buttons, the colours of the background is not changed accordingly as it suposed to be, db suggest the colour is set on the frame which is hidden behind the panel, so is there any way I can dispaly colour without using panel?

Thanks heaps

frame.getContentPane().setBackground(...)

db

Thanks db
I have added getContentPane(). in front of setBackground(color) asp below code, then ran it, but got a compiling error:

Exception in thread "main" java.lang.IllegalArgumentException: adding a window to a container
at java.awt.Container.checkNotAWindow(Container.java: 431)
at java.awt.Container.addImpl(Container.java:1039)
at java.awt.Container.add(Container.java:365)
at brainydraw.PictureButton.main(PictureButton.java:7 8)
Java Result: 1

Any more thought?

class RedAndBlueButtonListener implements ActionListener
       {
 
           public void actionPerformed(ActionEvent ae)
           {
                Color color = getBackground();  // color will be set
                Object source = ae.getSource();
                if (source == redButton) color = Color.red;
                else if (source == blueButton) color = Color.blue;
                getContentPane().setBackground(color);
                repaint();
            }
       }

If you want help, you'll have to post an SSCCE that demonstrates the problem. The code you posted doesn't seem to have anything to do with the error you posted.

Hi Kevinworkman

Thanks for the advice, yes I need the help if you can give some. My whole code is asp below, it doesn't compile after I added " getContentPane" into the method "public class RedAndBlueButtonListener implements ActionListener". All I wanted to do is to click the buttons to change background colour.

Any idea?

package brainydraw;
 
import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
 
public class PictureButton extends JFrame
{
 
    private JButton redButton;
    private JButton blueButton;
 
    public PictureButton()
    {
        setSize(200, 200);
        setLocation(200, 200);
 
        redButton = new JButton("RED");
        blueButton = new JButton("BLUE");
 
        Container content = getContentPane();
        content.setLayout(new FlowLayout());
 
        content.add(redButton);
        content.add(blueButton);
 
        //register the current panel as listener for the buttons
        ActionListener rBListener = new RedAndBlueButtonListener();
        redButton.addActionListener(rBListener);
        blueButton.addActionListener(rBListener);
    }
 
       public class RedAndBlueButtonListener implements ActionListener
       {
           public void actionPerformed(ActionEvent ae)
           {
                Color color = getBackground();  // color will be set
                Object source = ae.getSource();
                if (source == redButton) color = Color.red;
                else if (source == blueButton) color = Color.blue;
                getContentPane.setBackground(color);
                repaint();
            }
       }
 
    public static void main(String[] args)
    {
        JFrame f = new PictureButton();
 
        f.addWindowListener(new WindowAdapter()
        {
 
            public void windowClosing(WindowEvent we)
            {
                System.exit(0);
            }
        });
        f.setVisible(true);
    }
}

Hi guys
Thank you all, I just solved this problem. hehe...