NEED HELP FAST please

[SpecialKay]

i have a menu bar, when i run the code the first time everything is fine. If i click on the screen i print a Circle or a Square, store the shape in an arraylist and repaint the frame. When the frame repaints the menuBar gets painted again under the orignal.
Any ideas?

public class X2Frame extends JFrame {
   private JMenuBar menuBar = new JMenuBar();
   private JMenu menuTool = new JMenu();
   private JMenuItem menuToolSquare = new JMenuItem();
   private JMenuItem menuToolCircle = new JMenuItem();
   private BorderLayout borderLayout1 = new BorderLayout();
   private CanvasPanel canvasPanel = new CanvasPanel();
 
   public X2Frame() {
      try {
         jbInit();
      } catch (Exception e) {
         e.printStackTrace();
      }
   }
 
   private void jbInit() throws Exception {
      this.setJMenuBar( menuBar );
      this.getContentPane().setLayout(borderLayout1);
      this.setSize( new Dimension(400, 300) );
      menuTool.setText("Tool");
      menuTool.setMnemonic('T');
      menuToolSquare.setText("Square");
      menuToolSquare.setMnemonic('S');
      menuToolSquare.addActionListener( new ActionListener() { public void actionPerformed( ActionEvent e ) {
                  menuToolSquare_ActionPerformed(e);
               } } );
      menuToolCircle.setText("Circle");
      menuToolCircle.setMnemonic('C');
      menuToolCircle.addActionListener(new ActionListener() {
               public void actionPerformed(ActionEvent e) {
                  menuToolCircle_actionPerformed(e);
               }
            });
      menuTool.add( menuToolSquare );
      menuTool.add(menuToolCircle);
      menuBar.add(menuTool);
      this.getContentPane().add(canvasPanel, BorderLayout.CENTER);
   }
 

[sukatoa]

Ahhh!!! so you are using JDeveloper ...am i right?
What jdk did you use?
it is recommended that you use jdk 5.0.... try it again, and update us....

sukatoa

[SpecialKay]

Whats a JDK, and how do i change it?

[SpecialKay]

Update to problem:
Not only is the menubar being repainted, but if i draw a square, then change tools and draw something else, the first square and the menubar gets doubled directly under the first. If i resize the window it goes back to normal. Paint again and it doubles again.

[sukatoa]

Whats a JDK, and how do i change it?

You can download it at Sun Microsystems... .

jdk(Java development kit) 5.0.... i can't remember the procedure... that was last 3 month's ago when i tried to use it.... but seems, the code generator fail sometimes....
I prefer to use netbeans if making GUI in a hurry.....

So, i came back to JCreator Lite....


I like to test your code, can you post the whole code?

sukatoa...

[SpecialKay]

its split up in 6 classes,
not sure that posting it is the easiest way.

import java.awt.Graphics2D;
import java.awt.geom.Ellipse2D;
 
public class Circle extends Shape {
 
   public Circle(int x, int y) {
      super.setX(x);
      super.setY(y);
   }
   public void draw( Graphics2D g2 )
   {
      int CircleX = super.getX() - (super.SHAPE_WIDTH/2);
      int CircleY = super.getY() - (super.SHAPE_WIDTH/2);
      //Problem Area
      Ellipse2D.Double circle = new Ellipse2D.Double(CircleX,CircleY,super.SHAPE_WIDTH,super.SHAPE_WIDTH);
      g2.draw(circle);
   }
}
 
import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
 
import javax.swing.JFrame;
import javax.swing.JMenu;
import javax.swing.JMenuBar;
import javax.swing.JMenuItem;
 
public class X2Frame extends JFrame {
   private JMenuBar menuBar = new JMenuBar();
   private JMenu menuTool = new JMenu();
   private JMenuItem menuToolSquare = new JMenuItem();
   private JMenuItem menuToolCircle = new JMenuItem();
   private BorderLayout borderLayout1 = new BorderLayout();
   private CanvasPanel canvasPanel = new CanvasPanel();
 
   public X2Frame() {
      try {
         jbInit();
      } catch (Exception e) {
         e.printStackTrace();
      }
   }
 
   private void jbInit() throws Exception {
      this.setJMenuBar( menuBar );
      this.getContentPane().setLayout(borderLayout1);
      this.setSize( new Dimension(400, 300) );
      this.setTitle( "Kevin Cameron" );
      menuTool.setText("Tool");
      menuTool.setMnemonic('T');
      menuToolSquare.setText("Square");
      menuToolSquare.setMnemonic('S');
      menuToolSquare.addActionListener( new ActionListener() { public void actionPerformed( ActionEvent e ) {
                  menuToolSquare_ActionPerformed(e);
               } } );
      menuToolCircle.setText("Circle");
      menuToolCircle.setMnemonic('C');
      menuToolCircle.addActionListener(new ActionListener() {
               public void actionPerformed(ActionEvent e) {
                  menuToolCircle_actionPerformed(e);
               }
            });
      menuTool.add( menuToolSquare );
      menuTool.add(menuToolCircle);
      menuBar.add(menuTool);
      this.getContentPane().add(canvasPanel, BorderLayout.CENTER);
   }
 
   private void menuToolSquare_ActionPerformed(ActionEvent e) {
      canvasPanel.setCircle(false);
   }
 
   private void menuToolCircle_actionPerformed(ActionEvent e) {
      canvasPanel.setCircle(true);
   }
}
 
 
import java.awt.Graphics2D;
 
public class Square extends Shape {
 
   public Square(int x, int y) {
      super.setX(x);
      super.setY(y);
   }
 
   public void draw( Graphics2D g2 )
   {
         g2.drawRect(super.getX(), super.getY(), super.SHAPE_WIDTH, super.SHAPE_WIDTH);
   }
}
 
 

[SpecialKay]

import java.awt.Graphics2D;
 
public abstract class Shape {
   static int SHAPE_WIDTH = 20;
   int x;
   int y;
 
   abstract public void draw( Graphics2D g2 );
 
   public Shape() {
   }
 
   public void setX(int x) { this.x = x; }
   public int getX() { return x; }
   public void setY(int y) { this.y = y; }
   public int getY() { return y; }
}
 
 
import java.awt.Graphics2D;
import java.awt.geom.Ellipse2D;
 
public class Circle extends Shape {
 
   public Circle(int x, int y) {
      super.setX(x);
      super.setY(y);
   }
   public void draw( Graphics2D g2 )
   {
      int CircleX = super.getX() - (super.SHAPE_WIDTH/2);
      int CircleY = super.getY() - (super.SHAPE_WIDTH/2);
      //Problem Area
      Ellipse2D.Double circle = new Ellipse2D.Double(CircleX,CircleY,super.SHAPE_WIDTH,super.SHAPE_WIDTH);
      g2.draw(circle);
   }
}
 
 
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.event.MouseAdapter;
 
import java.awt.event.MouseEvent;
 
import java.util.ArrayList;
 
import javax.swing.JPanel;
 
public class CanvasPanel extends JPanel {
 
   private ArrayList shapeList = new ArrayList();
   private boolean circle = false;
 
   public CanvasPanel() {
      try {
         jbInit();
      } catch (Exception e) {
         e.printStackTrace();
      }
   }
 
   public void setCircle(boolean circle) {
      this.circle = circle;
   }
 
   private void jbInit() throws Exception {
      this.addMouseListener(new CanvasPanel_this_mouseAdapter(this));
   }
 
   void this_mouseClicked(MouseEvent e) {
      if(circle == false) {
         Square square = new Square(e.getX(), e.getY());
         shapeList.add(square);
      }
      else {
         Circle circle = new Circle(e.getX(), e.getY());
         shapeList.add(circle);
      }
   repaint();
   }
   public void paintComponent ( Graphics g) {
      for ( int i = 0; i < shapeList.size(); i++)
      {
         Graphics2D g2 = (Graphics2D)g;
         Shape shape = (Shape)shapeList.get(i);
         shape.draw(g2);
      }
   }
 
}
final class CanvasPanel_this_mouseAdapter extends MouseAdapter {
   private CanvasPanel adaptee;
 
   CanvasPanel_this_mouseAdapter(CanvasPanel adaptee) {
      this.adaptee = adaptee;
   }
 
   public void mouseClicked(MouseEvent e) {
      adaptee.this_mouseClicked(e);
   }
}
 
 

[sukatoa]

store the shape in an arraylist and repaint the frame. When the frame repaints the menuBar gets painted again under the orignal.

Where is it? sorry if i can't find it....

Is this your complete code? only to choose shape then click to show those shapes? what about the quotes above?

I've test your code, no problem.... but i have doubt the quotes above.....
can you please add more details?

You didn't even set the close operation of your frame.... if you are using Windows, have a look at your task manager... many java process might still running.... they consumes your temp memory..... maybe anytime your system will crash....

sukatoa

[SpecialKay]

Nope nothing is running, yes that is all my code.
It is suspost to be 6 classes. All the code does is allow you to chose a tool (Square, Circle) then paint that shape on the frame. It is all working fine except the menubar, and the shapes are being doubled under the orignal. If i re-size the window the doubles disapear, untill i draw another shap, then everything on the frame is being doubled.

[SpecialKay]

i have set the exit
frame.setDefaul tCloseOperation ( JFrame.EXIT_ON_ CLOSE );