how to add 2 panels in a panel inside a frame

[clydedoris]

i have to make a jframe which contains a jpanel "try" which would contain some components, and 2 jpanels, "green" and "blue". here is my code

import java.awt.*;
import javax.swing.*;

public class Try extends JPanel
{
    GreenPanel green = new GreenPanel();
    BluePanel blue = new BluePanel();
    
    public Try()
    {
        setLayout(null);
        green.setBounds(10, 10, 250, 200);
        blue.setBounds(10, 280, 250, 200);
        
        add(green);
        add(blue);
    }
    
    /*public void paint(Graphics g)
    {
        g.setColor(Color.yellow);
        g.fillRect(getX(), getY(), getWidth(), getHeight());
    }*/
    
    public static void main(String args[])
    {
        Try t = new Try();
        JFrame j = new JFrame();
        j.setSize(300, 700);
        j.setVisible(true);
        j.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        j.getContentPane().add(t);
    }
}


class GreenPanel extends JPanel
{
    @Override
    public void paintComponents(Graphics i)
    {
        i.setColor(Color.green);
        i.fillRect(0, 0, getWidth(), getHeight());
    }
}

class BluePanel extends JPanel
{
    @Override
    public void paintComponents(Graphics s)
    {
        s.setColor(Color.blue);
        s.fillRect(0, 0, getWidth(), getHeight());
    }
}

the problem is, i can't see the 2 jpanels!!huhu

[doWhile]

Try adding the component to the content pane prior to making the JFrame visible, or otherwise call validate on the JFrame or revalidate on the JPanel

[camickr]

You should NOT be overriding the paintComponents() method. To change the background color of a panel you just use:

panel.setBackground( Color.GREEN );

Generally it is NOT a good idea to use a null layout. Give you panels a "preferred size" and then add then to a panel that uses a layout manager.

[clydedoris]

okay, gotta try these later..thanks BTW

[clydedoris]

You should NOT be overriding the paintComponents() method. To change the background color of a panel you just use:

panel.setBackground( Color.GREEN );

Generally it is NOT a good idea to use a null layout. Give you panels a "preferred size" and then add then to a panel that uses a layout manager.

nice one..my problem is solved..whoa thanks!!!! +10000 rep!!!1lol