Drawing (mouse events)

[John Lockheart]

I've created six ellipses labelled p1, p2...up to p6. I'm trying to make it so that on a mouse click (if a p object is selected), an unfilled rectangle will be drawn around it. As a way of letting someone know the object was selected. I haven't had much luck. I don't want the rectangle to be drawn until the mouse event, and i've even tried drawing it first and then having it moved when a p object is clicked...still no luck. Any help would be great.

    
private class ENDMouseListener extends MouseAdapter {
public void mousePressed(MouseEvent e) {
 
       // Determine which object (if any) was clicked in,
       //"contains" is inside an abstract class, overidden in ellipse class, and rectangle class. (returns the coordinates).
    Shape s = null;
if (p1 != null && p1.contains(e.getX(), e.getY())){
  s = p1;
}else if (p2 != null && p2.contains(e.getX(), e.getY())){
  s = p2;
}else if (p3 != null && p3.contains(e.getX(), e.getY())){
  s = p3;
}else if (p4 != null && p4.contains(e.getX(), e.getY())){
   s = p4;
}else if (p5 != null && p5.contains(e.getX(), e.getY())){
   s = p5;
}else if (p6 != null && p6.contains(e.getX(), e.getY())){
   s = p6;
}
// was a shape selected?
if (s != null) { //Draw a rectangle around the circle
 
  player = s;
  point_x = e.getX(); //x-point clicked
  point_y = e.getY(); //y-point clicked
 
 Selector r1 = new Selector(Shape.MEDIUM, Color.black, false, point_x, point_y); //rectangle around ellipse
 //r1.draw(g2); //can't call draw??? (not sure how to draw it)
 }
 
        repaint(); //tried drawing it with all the other shapes then repainting it over the ellipse chosen (didn't work).
        }
 
public void mouseReleased(MouseEvent e) {//no player is currently selected
     
     player = null;
        }
 

}

[Ernest Friedman-Hill]

Please don't post the same question to more than one forum. See here for an explanation of why I just deleted the other copy of this thread.

As for your question: you need to do all your painting in your paint() or paintComponent() method. Instead of trying to draw the rectangle here in the event handler, instead have the event handler set a member variable that represents the selection, and have the paint method draw the selection if and only if it finds that member set.

This is a general principle: event handlers should modify a data model of some kind, while your painting method should render that model to the screen.

[John Lockheart]

Thanks, got it working now...Sorry about double posting. I was in a huge rush before I left for work and needed some help. Also sorry about posting in the wrong forum. I'll try to be more considerate and obey the rules.

One more question if you don't mind. Once I create a drawing, like a rectangle on a mouse event (like clicking inside a circle). How would I go about getting rid of the rectangle if a click occurred outside the circle? Would I have to redraw it and set color to background color or can I destroy it somehow?

[Michael Dunn]

it would probably go something like this

    
boolean clickedInCircle = false;//class field
mouseEvent()
{
  if(mouseClickInCircle) clickedInCircle = true;
  else clickedInCircle = false;
  repaint();//edit - added this line
}
paintComponent(..)
{
  super.paintComponent(g);
  g.drawOval(...);//circle
  if(clickedInCircle) g.drawRect(...);
}

[ February 04, 2007: Message edited by: Michael Dunn ]