Trying to create a painting application.

Question

Hello, I am sophomore Computer Science student and I am stuck on my project.
My problem is very basic.
Heres the project question:
Create an application that enables to user to paint a picture. The user should be able to choose the shape to draw, the color in which the shape should appear and whether the shape should be filled with color. Use the graphical user interface components we discussed in this chapter, such as JComboBoxes, JRadioButtons and JCheckBoxes, to allow the user to select various options. The program should provide a JButton object that allows the user to erase the window.
now, I've created the framework of the program. That is, how the program looks and stuff. But there is one annying problem!
How do I paint?!
As soon as I put the paint method inside my program, it completely overwrites the GUI I've created. Copy-paste the following code in your compiler (and read the comments towards the end):
========
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
public class Project extends JFrame {
     public static void main (String args[]) {
          Project app = new Project();
          app.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     // Initiating values
     Container c;
     JPanel p = new JPanel();
     JRadioButton oval=new JRadioButton("Oval");
     JRadioButton line=new JRadioButton("Line");
     JRadioButton rect = new JRadioButton("Rect");
     ButtonGroup b = new ButtonGroup();
     JCheckBox filled = new JCheckBox("Filled");
     JButton clear = new JButton("Clear");
     String colorNames[] = {"Red","Green","Blue","Pink","Magenta","Orange"};
     Color colors[] = {Color.red,Color.green,Color.blue,Color.pink,Color.magenta,Color.orange};
     JList colorList = new JList(colorNames);
     public Project() {
          //Starting stuff..
          super("Microsoft Paint 2004");
          setSize(600,600);
          c=getContentPane();
          c.setLayout(new BorderLayout(5,5));
          c.setBackground(Color.white);
          //Defining ButtonGroup
          b.add(oval);     
          b.add(line);
          b.add(rect);
          // Adding stuff to JPanel object. All JPanel stuff goes in here
          p.setLayout(new FlowLayout()); // Setting Layout for JPanel
          p.add(oval);
          p.add(line);
          p.add(rect);
          p.add(filled);
          p.add(clear);
          p.add(new JScrollPane(colorList));
          p.setBackground(Color.gray);
          // Defining JList
          colorList.setVisibleRowCount(3);
          colorList.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
          // Adding JPanel to Container
          c.add(p,BorderLayout.SOUTH);
          // Adding Mouse Listeners
          addMouseListener(new MyMouseHandler());
          addMouseMotionListener(new MyMouseMotionHandler());
          //Adding listeners for the Radio Buttons
          oval.addItemListener(new RadioButtonListener());
          line.addItemListener(new RadioButtonListener());
          rect.addItemListener(new RadioButtonListener());
          show();
     private class MyMouseHandler extends MouseAdapter {
          public void mouseClicked (MouseEvent e)
               repaint(); // Calling the paint method     
     private class MyMouseMotionHandler extends MouseMotionAdapter {
     private class RadioButtonListener implements ItemListener {
          public void itemStateChanged(ItemEvent e)
     //public void paint (Graphics g) {          // Run the program without the paint method, and then run it with the paint method. You'll see what I mean.
     //     g.drawString("Hello",160,160);

Answer 1

Ok, I understand what you are saying.
Now, based on my understand.. and after further analysing your code; I've created two different versions of my code. However..... the annoying thing is, the same problem persists!!
I JUST CANT GET THE MOUSE TO DRAW!
I am gonna copy-paste both versions of the code, the first one is my original code, the second peice of code is what I created by understanding your code/explaination.
Preferably, PLEASE analyse my code original instead. I am more comfortable with extending JFrame rather than using it as an Object (like how you do, heh).
===== MY ORIGINAL CODE =====
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
public class Project extends JFrame {
     // Initiating values
     Container c;
     JPanel p = new JPanel();
     JRadioButton oval=new JRadioButton("Oval");
     JRadioButton line=new JRadioButton("Line");
     JRadioButton rect = new JRadioButton("Rect");
     ButtonGroup b = new ButtonGroup();
     JCheckBox filled = new JCheckBox("Filled");
     JButton clear = new JButton("Clear");
     String colorNames[] = {"Red","Green","Blue","Pink","Magenta","Orange"};
     Color colors[] = {Color.red,Color.green,Color.blue,Color.pink,Color.magenta,Color.orange};
     JList colorList = new JList(colorNames);
     Drawer d = new Drawer();
     public static void main (String args[]) {
          Project app = new Project();
          app.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     public Project() {
          //Starting stuff..
          setSize(600,600);
          c=getContentPane();
          c.setLayout(new BorderLayout(5,5));
          c.setBackground(Color.white);
          //Defining ButtonGroup
          b.add(oval);     
          b.add(line);
          b.add(rect);
          // Adding stuff to JPanel object. All JPanel stuff goes in here
          p.setLayout(new FlowLayout()); // Setting Layout for JPanel
          p.add(oval);
          p.add(line);
          p.add(rect);
          p.add(filled);
          p.add(clear);
          p.add(new JScrollPane(colorList));
          p.setBackground(Color.gray);
          // Defining JList
          colorList.setVisibleRowCount(3);
          colorList.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
          // Adding JPanel to Container
          c.add(p,BorderLayout.SOUTH);
          //Adding Drawer to Container
          c.add(d,BorderLayout.CENTER);
          // Adding Mouse Listeners
          addMouseListener(new MyMouseHandler());
          addMouseMotionListener(new MyMouseMotionHandler());
          //Adding listeners for the Radio Buttons
          oval.addItemListener(new RadioButtonListener());
          line.addItemListener(new RadioButtonListener());
          rect.addItemListener(new RadioButtonListener());
          show();
     private class MyMouseHandler extends MouseAdapter {
          public void mouseClicked (MouseEvent e)
               repaint();                    
     private class MyMouseMotionHandler extends MouseMotionAdapter {
     private class RadioButtonListener implements ItemListener {
          public void itemStateChanged(ItemEvent e)
       private class Drawer extends JPanel{
           public void paintComponent(Graphics g) {
               g.drawString("Superman is dead",200,200);
}======= MY CODE (2nd version)============
import javax.swing.*;
import java.awt.*;
import java.awt.event.*;
public class Project extends JFrame {
     // Initiating values
     Container c;
     JPanel p = new JPanel();
     JRadioButton oval=new JRadioButton("Oval");
     JRadioButton line=new JRadioButton("Line");
     JRadioButton rect = new JRadioButton("Rect");
     ButtonGroup b = new ButtonGroup();
     JCheckBox filled = new JCheckBox("Filled");
     JButton clear = new JButton("Clear");
     String colorNames[] = {"Red","Green","Blue","Pink","Magenta","Orange"};
     Color colors[] = {Color.red,Color.green,Color.blue,Color.pink,Color.magenta,Color.orange};
     JList colorList = new JList(colorNames);
     Drawer d = new Drawer();
     public static void main (String args[]) {
          Project app = new Project();
          app.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
     public Project() {
          //Starting stuff..
          setSize(600,600);
          c=getContentPane();
          c.setLayout(new BorderLayout(5,5));
          c.setBackground(Color.white);
          //Defining ButtonGroup
          b.add(oval);     
          b.add(line);
          b.add(rect);
          // Adding stuff to JPanel object. All JPanel stuff goes in here
          p.setLayout(new FlowLayout()); // Setting Layout for JPanel
          p.add(oval);
          p.add(line);
          p.add(rect);
          p.add(filled);
          p.add(clear);
          p.add(new JScrollPane(colorList));
          p.setBackground(Color.gray);
          // Defining JList
          colorList.setVisibleRowCount(3);
          colorList.setSelectionMode(ListSelectionModel.SINGLE_SELECTION);
          // Adding JPanel to Container
          c.add(p,BorderLayout.SOUTH);
          //Adding Drawer to Container
          c.add(d,BorderLayout.CENTER);
          show();
     private class Drawer extends JPanel{
            public Drawer() {
            //Adding listeners for the Radio Buttons
          oval.addItemListener(new RadioButtonListener());
          line.addItemListener(new RadioButtonListener());
          rect.addItemListener(new RadioButtonListener());
            // Adding Mouse Listeners
          addMouseListener(new MyMouseHandler());
          addMouseMotionListener(new MyMouseMotionHandler());
          private class MyMouseHandler extends MouseAdapter {
               public void mouseClicked (MouseEvent e)
          private class MyMouseMotionHandler extends MouseMotionAdapter {
          private class RadioButtonListener implements ItemListener {
               public void itemStateChanged(ItemEvent e)
           public void paintComponent(Graphics g) {
                    g.drawString("Superman is dead",200,200);
THANK YOU THANK YOU THANK