Paint wird 2 mal ausgeführt (?)

[Guest]

hi,

ich hab da ein sehr komisches problem. Evtl. sehe ich den Fehler einfach nicht....

Hier mal der code

public class MyImage extends Component {
	Image img;
	int x;
	int y;
	MyImage(String fileName){
		try{
			img = ImageIO.read(new File("..."));  
		}catch(Exception e) {e.printStackTrace();}
		setSize(16,16);
		
	}
	public void paint(Graphics g){
		try{Thread.sleep(500);}catch(Exception e){}
		setSize(16,16);
		((Graphics2D)(g)).drawImage(img,x,y,null);
		setLocation(x,y);
		System.err.println("Painted Component");
}
}

public class MyFrame extends Frame implements MouseMotionListener,MouseListener 
{ 
	private Image dbImage;
	private Graphics dbg;
    MyImage img;
  public void init() 
  { 
     setSize(600,600);
	 img = new MyImage("");
     img.addMouseListener(this);
     img.x=50;
     img.y=50;
     add(img);
     setVisible(true);
  } 
  public void update (Graphics g)
  {
        // Initialisierung des DoubleBuffers
        if (dbImage == null)
        {
              dbImage = createImage (this.getSize().width, this.getSize().height);
              dbg = dbImage.getGraphics ();
        }

        // Bildschirm im Hintergrund löschen
        dbg.setColor (getBackground ());
        dbg.fillRect (0, 0, this.getSize().width, this.getSize().height);
        // Auf gelöschten Hintergrund Vordergrund zeichnen
        dbg.setColor (getForeground());
        //paint (dbg);
        img.paint(dbg);
        // Nun fertig gezeichnetes Bild Offscreen auf dem richtigen Bildschirm anzeigen
        g.drawImage (dbImage, 0, 0, this);
  }
(...)
  public void mousePressed(MouseEvent arg0) {
	// TODO Auto-generated method stub
	 img.x=(int) (Math.random()*500+20);
	 img.y=(int) (Math.random()*500+20);
	 repaint();
	}
}

Komischerweise wird die paint - Methode 2x aufgerufen und ausgeführt. Aber warum den das ???
Hat jemand vielleicht eine Idee was hier passiert??

 

[Leroy42]

Gegenfrage: Was stört dich daran? ???:L

Die paint-Methoden müssen so designed werden, das sie
beliebig oft aufgerufen werden können.

Jedesmal nachdem ein Teil deiner Zeichnung wieder sichtbar wird
(Fenster wurde überlagert, war im Hintergrund, minimiert, ...) ruft
die GUI die Methode von sich aus auf und wie genau der erste Aufbau
eines Fensters vonstatten geht, darfst du ihr auch nicht vorschreiben

Die paint-Methode darf einfach nicht so konzipiert werden, daß irgendeine
Programmlogik in ihr davon abhängt, wieoft sie aufgerufen wurde.

 

[L-ectron-X]

Diese Zeile gehört meines Erachtens nicht in die update()-Methode.
Ein ziemlich häufig gemachter Fehler im Zusammenhang mit DoubleBuffering...

img.paint(dbg); //zu entfernen

update() ist die Reaktion auf repaint(). Die paint()-Methode wird automatisch im Anschluss aufgerufen.
Und in der paint()-Methode sollte auch auf nichts gewartet werden.
sleep() hat dort nichts zu suchen.

 

[gast]

hi,

ich versteh nur nich , warum die paint methode 2x aufgerufen wird.

selbst wenn ich es so mache:

public class MyImage extends Component {
   Image img;
   int x;
   int y;
   MyImage(String fileName){
      try{
         img = ImageIO.read(new File("...")); 
      }catch(Exception e) {e.printStackTrace();}
      setSize(16,16);
      
   }
   public void paint(Graphics g){
      setSize(16,16);
      ((Graphics2D)(g)).drawImage(img,x,y,null);
      setLocation(x,y);
      System.err.println("Painted Component");
}
} 
public class MyFrame extends Frame implements MouseMotionListener,MouseListener
{
   private Image dbImage;
   private Graphics dbg;
    MyImage img;
  public void init()
  {
     setSize(600,600);
    img = new MyImage("");
     img.addMouseListener(this);
     img.x=50;
     img.y=50;
     add(img);
     setVisible(true);
  }
  public void paint (Graphics g)
  {
        img.paint(dbg);

   }
(...)
  public void mousePressed(MouseEvent arg0) {
   // TODO Auto-generated method stub
    img.x=(int) (Math.random()*500+20);
    img.y=(int) (Math.random()*500+20);
    repaint();
   }
}

wenn ich auf das image klicke, führt er die paint-Methode 2x aus!
eigentlich sollte sie aber nur 1x ausgeführt werden.
Ist meiner meinung nach einfach ein Performance-Problem. Mich Persönlich stört es ja nicht.(Jedenfalls nicht in diesem beispiel, da es ja nur ein 16x16 bild ist.)
Wenn ich aber etwas aufwendigere Zeichnungen machen möchte, dann ist es schon ein wenig blöd, daß java hier 2x das selbe zeichnet!
Weiß jemand woran das liegen kann?? (Es muß ein Fehler im Code sein..)
Evtl. führt java beim Mouse-press und beim Mouse-release die Methode mouse-Pressed auf?? ???:L

 

[Gast]

sorry...

muss so heißen:

public void paint (Graphics g)
{
img.paint(g);

}