I can't show the correct event .

Question

I have check again . I had written Oval, Rectangle and Line JButton.
However, I just show circle.
I think this parts of the code has problem :but I don't clear
And the following has this all code(2 file) , thx a lot !
'************************************************I'm sure problem . I guess
clear.addActionListener(
new ActionListener() {
public void actionPerformed(ActionEvent e){
repaint();
System.out.println("Clear button is clicked");
public void itemStateChanged(ItemEvent event){
if (event.getSource() == oval)
p3 = new Painter(0);
else if (event.getSource() == oval && event.getSource() == filled)
p3 = new Painter(1);
else if (event.getSource() == line)
p3 = new Painter(2);
else if (event.getSource() == rectangle)
p3 = new Painter(3);
else if (event.getSource() == rectangle && event.getSource() == filled)
p3 = new Painter(4);
This is the following my code : thx .
'*************************************************JavaAss2.java
import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;
public class JavaAss2 extends JFrame implements ItemListener {
private JCheckBox filled = new JCheckBox("filled");
private JRadioButton oval = new JRadioButton("Oval");
private JRadioButton line = new JRadioButton("Line");
private JRadioButton rectangle = new JRadioButton("Rectangle");
private JButton clear = new JButton("Clear");
private Painter p3;
public JavaAss2() {
setBackground(Color.gray);
Container c = getContentPane();
c.setLayout(new BorderLayout());
JPanel p1 = new JPanel();
p1.setLayout(new GridLayout(5, 1));
c.add(p1, BorderLayout.EAST);
Painter p2 = new Painter();
p2.setLayout(new BorderLayout());
c.add(p2, BorderLayout.CENTER);
clear.setSize(30, 15);
ButtonGroup btngp = new ButtonGroup();
btngp.add(oval);
btngp.add(line);
btngp.add(rectangle);
p1.add(oval);
p1.add(line);
p1.add(rectangle);
p1.add(filled);
p1.add(clear);
oval.addItemListener(this);
line.addItemListener(this);
rectangle.addItemListener(this);
filled.addItemListener(this);
clear.addActionListener(
new ActionListener() {
public void actionPerformed(ActionEvent e){
repaint();
System.out.println("Clear button is clicked");
public void itemStateChanged(ItemEvent event){
if (event.getSource() == oval)
p3 = new Painter(0);
else if (event.getSource() == oval && event.getSource() == filled)
p3 = new Painter(1);
else if (event.getSource() == line)
p3 = new Painter(2);
else if (event.getSource() == rectangle)
p3 = new Painter(3);
else if (event.getSource() == rectangle && event.getSource() == filled)
p3 = new Painter(4);
if (oval.isSelected())
new Painter(0);
else if (oval.isSelected() && filled.isSelected())
new Painter(1);
else if (line.isSelected())
new Painter(2);
else if (rectangle.isSelected())
new Painter(3);
else if (rectangle.isSelected() && filled.isSelected())
new Painter(4); */
public static void main (String args[]){
JavaAss2 javaAss2 = new JavaAss2();
javaAss2.setTitle("Painter");
javaAss2.setSize(500, 500);
//javaAss2.pack();
javaAss2.setVisible(true);
javaAss2.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
'******************************************Painter.java
import javax.swing.*;
import javax.swing.event.*;
import java.awt.*;
import java.awt.event.*;
public class Painter extends JPanel{
private int Oid;
public Painter() {
setBackground(Color.white);
setSize(500, 500);
public Painter(int id){
Oid = id;
public void paintComponent(Graphics g){
super.paintComponent(g);
switch(Oid) {
case 0: g.drawOval(20, 20, 50, 50);
System.out.println("You have chosen Oval !!!");
break;
case 1: g.fillOval(20, 20, 50, 50);
System.out.println("You have chosen Oval with fill colour !!!");
break;
case 2: g.drawLine(100, 100, 150, 150);
System.out.println("You have chosen Line !!!");
break;
case 3: g.drawRect(30, 120, 150, 75);
System.out.println("You have chosen Rectangle !!!");
break;
case 4: g.fillRect(30, 120, 150, 75);
System.out.println("You have chosen Rectangle with fill colour !!!");
break;
}

Answer 1

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class test extends JFrame implements ItemListener {
  private JCheckBox filled = new JCheckBox("filled");
  private JRadioButton oval = new JRadioButton("Oval");
  private JRadioButton line = new JRadioButton("Line");
  private JRadioButton rectangle = new JRadioButton("Rectangle");
  private JButton clear = new JButton("Clear");
  // we need these references in itemStateChanged()
  private Painter p3;
  Container c;
  public test() {
    setBackground(Color.gray);
//    Container c = getContentPane();
    c = getContentPane();
    c.setLayout(new BorderLayout());
    JPanel p1 = new JPanel();
    p1.setLayout(new GridLayout(5, 1));
    c.add(p1, BorderLayout.EAST);
    // so we can use p3 in itemStateChanged()
//    Painter p2 = new Painter();
    p3 = new Painter();
//    p2.setLayout(new BorderLayout());
    p3.setLayout(new BorderLayout());
//    c.add(p2, BorderLayout.CENTER);
    c.add(p3, BorderLayout.CENTER);
    clear.setSize(30, 15);
    ButtonGroup btngp = new ButtonGroup();
    btngp.add(oval);
    btngp.add(line);
    btngp.add(rectangle);
    p1.add(oval);
    p1.add(line);
    p1.add(rectangle);
    p1.add(filled);
    p1.add(clear);
    oval.addItemListener(this);
    line.addItemListener(this);
    rectangle.addItemListener(this);
    // line below >> class cast exception - because
    // in itemStateChanged we cast all events to JRadioButton
    // we really don't need a listener for the check box
    // we'll check to see if it is selected in our
    // itemStateChanged method
    //filled.addItemListener(this);
    clear.addActionListener(new ActionListener() {
      public void actionPerformed(ActionEvent e){
        System.out.println("Clear button is clicked");
    setLocation(300,25);
  public void itemStateChanged(ItemEvent event){
    JRadioButton button = (JRadioButton)event.getSource();
    String id = button.toString().substring(
                 button.toString().indexOf("text=") + 5,
                 button.toString().lastIndexOf("]"));
    System.out.println("id = " + id);
    c.remove(p3);
    if (button == oval) {
      if(filled.isSelected())
        p3 = new Painter(1); // fill
      else
        p3 = new Painter(0); // draw
    if (button == line)
      p3 = new Painter(2);
    if (button == rectangle) {
      if(filled.isSelected())
        p3 = new Painter(4); // fill
      else
        p3 = new Painter(3); // draw
    c.add(p3, BorderLayout.CENTER);
    c.repaint();
    c.validate();
  public class Painter extends JPanel{
    private int Oid;
    public Painter() {
      setBackground(Color.white);
      setSize(500, 500);
    public Painter(int id){
      Oid = id;
    public void paintComponent(Graphics g){
      super.paintComponent(g);
      System.out.println("Oid = " + Oid);
      switch(Oid) {
        case 0:
          g.drawOval(20, 20, 50, 50);
          System.out.println("You have chosen Oval !!!");
          break;
        case 1:
          g.fillOval(20, 20, 50, 50);
          System.out.println("You have chosen Oval with " +
                             "fill colour !!!");
          break;
        case 2:
          g.drawLine(100, 100, 150, 150);
          System.out.println("You have chosen Line !!!");
          break;
        case 3:
          g.drawRect(30, 120, 150, 75);
          System.out.println("You have chosen Rectangle !!!");
          break;
        case 4:
          g.fillRect(30, 120, 150, 75);
          System.out.println("You have chosen Rectangle " +
                             "with fill colour !!!");
  public static void main(String[] args) {
    test javaAss2 = new test();
//    JavaAss2 javaAss2 = new JavaAss2();
    javaAss2.setTitle("Painter");
    javaAss2.setSize(500, 500);
    //javaAss2.pack();
    javaAss2.setVisible(true);
    javaAss2.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
}