problem mit repaint() in Thread

[suziheizer]

Hallo

Habe eine Klasse Display von Canvas abgeleitet in dieser laeuft ein Thread welcher in einer endlosschleife repaint() aufruft. Es wird Double Buffered gezeichnet.

Mein Problem ist das in die update Meyhode nur 2 mal hereingegangen wird. Die Endlosschleife laeuft jedoch permanent d.h ab dem dritten aufruf von repaint() wird nichts mehr gezeichnet.(ausprobiert)

Woran koennte das liegen?

import java.awt.Canvas;
import java.awt.Graphics;
import java.awt.Image;

class Display extends Canvas implements Runnable{
   
   int cnt=0;

   private Image dbImage;
   private Graphics dbGraphics;
   int appletsize_x=800;
   int appletsize_y=600;
   int v_x0,v_x1,v_x2;
   int v_y0,v_y1,v_y2;
   int x_pos = 60;
   int y_pos = 300;
   int radius = 30;
   Thread th;

   public void Display(){};
   public void startAnimation()
   {
      Thread th = new Thread(this);
      th.start();
   }

   public void run()
   {

      while (true) {
         cnt++;
         repaint();
         try {
            Thread.sleep(40);
         } catch (InterruptedException e) {
            //nichts
         }
      }
   }

   public void update(Graphics g)
   {
      System.out.println("test1");
      //Double-Buffer initialisieren
      if (dbImage == null) {
         dbImage = createImage(
            getSize().width,
            getSize().height
         );
         dbGraphics = dbImage.getGraphics();
      }
      //Hintergrund löschen
      dbGraphics.setColor(getBackground());
      dbGraphics.fillRect(0,0,getSize().width,getSize().height
       );
      //Vordergrund zeichnen
      dbGraphics.setColor(getForeground());
      paint(dbGraphics);
      //Offscreen anzeigen
      g.drawImage(dbImage,0,0,this);
   }

   public void paint(Graphics g)
   {
       System.out.println("test2");
       g.drawOval(x_pos-radius+cnt, y_pos-radius, 2*radius, 2*radius);
   }



   }

 

[0x7F800000]

Vorschlag 1:

In truth, the majority of GUI components do not need to do incremental drawing, so most programs can ignore the update() method and simply override paint() to render the component in it's current state. This means that both system-triggered and app-triggered rendering will essentially be equivelent for most component implementations.

mach's dir nicht zu kompliziert, mach dieses update-dings weg, schieb alles in paint und gut iss, würde ich mal spontan vorschlagen :roll:


[edit: ah, okay, wo Schandro mich darauf hinweist: meine vermutung war doch falsch, du zeichnest schon irgendwas, aber imho an irgendeiner nicht ganz passenden stelle [kA was man sich bei AWT früher für kunstgriffe einfallen lassen musste]
Asche auf mein Haupt^^ ]
[edit: Falscher ]Vorschlag 2:
bist du dir sicher, dass die paint methode nicht aufgerufen wird? Imho wird die sehr wohl aufgerufen. Aber die ganzen tollen ergebnisse landen wieder auf dem doublebuffer, wegen dieser anweisung:

paint(dbGraphics);

AWT würd ja sehr gerne was zeichnen, aber du fängst den versuch ja in update ab, und schickst alles auf den unsichtbaren buffer. Oder wie? ???:L

 

[Schandro]

Andrey, mach ihn doch net so fertig^^

wahrscheinlich hast du nie mit AWT gearbeitet: Der code in update() is der Standard DoubleBuffering code, den suziheizer mit Sicherheit irgendwo aus dem Internet rauskopiert hat. (und der übrigens nicht für den Fehler verantwortlich ist )

@suziheizer

Mit dieser main funktioniert das bewegen des Kreises ohne Probleme, verstehe deswegen dein Problem nicht :

   public static void main(String[] args){
	   JFrame window = new JFrame();
	   window.setBounds(100,100,900,900);
	   Display d;
	   window.add(d = new Display());
	   d.startAnimation();
	   window.setVisible(true);
   }

Bitte immer den gesamten Code (oder bessser ein kurzes kompilierbares ausführbares Beispiel) posten

 

[0x7F800000]

Andrey, mach ihn doch net so fertig^^

wahrscheinlich hast du nie mit AWT gearbeitet: Der code in update() is der Standard DoubleBuffering code, den suziheizer mit Sicherheit irgendwo aus dem Internet rauskopiert hat. (und der übrigens nicht für den Fehler verantwortlich ist )

Lol, ich mach niemanden "fertig", ich rede halt so keein stress, aaaalles ruhig


Und mit AWT hab ich in der tat nie gearbeitet.
Und wo ich jetzt sehe, dass die funktionalität von paint komplett ins update rübergezogen wurde, tippe ich spontan auch darauf, dass suziheizer einfach das starten der animation vergessen hat.

Sorry wenn das schon wieder irgendwie anders als gewollt rübergekommen ist
Spät abends werde ich womöglich unbewusst sarkastischer als sonst^^

 

[suziheizer]

Hallo
Danke es geht. Hab die Main Methode von einer anderen Klasse aus aufgerufen. Wenn ich Sie in Display reinmache gehts.

 

[Quaxli]

Die Methode "public void Display()" ist übrigens kein Konstruktor!
Das Ding muß "public Display()" heißen.

 

[suziheizer]

Danke an alle