Repaint()

[Malta]

Hallo Zusammen,

wir wollen einen Punkt auf einem Kreis laufen lassen. Hierzu haben wir eine Klasse abgeleitet von Canvas, als Membervariablen den x und y Wert unseres Punktes. Diesen setzen wir in der Methode animation() neu und rufen repaint() auf. Zum managen der Zeit verwenden wir Thread.sleep(200).
Nun ist es so, dass der Punkt zu Beginn des Programms gezeichent wir, sich nicht Bewegt und dann verschwindet. Die x und y Werte ändern sich jedoch. Woran kann das liegen???
Vielen Dank.

 

[XHelp]

Am Code
Diesen solltest du vlt auch mal zeigen.

 

[Malta]

public class Kreis extends Canvas{
	
	private static final long serialVersionUID = 1L;
	int x =150;
	int y=0;
public Kreis(int w, int h)
	{
		super.setBounds(0,0,w,h);
}

public void paint(Graphics g)
	{
		g.translate(175, 175);
		g.setColor(Color.black);
		g.drawLine(0,175, 0, -175);
		g.drawLine(175, 0, -175, 0);
		g.drawOval(x, y, 8,8);
	}
public void animation(int i)
	{
		double dx;
		double dy;
		if(i<=180)
		{
			dx = Math.cos(i)*150;
			dy = Math.sin(i*Math.PI/180)*150;
		}
		else
		{
			dx = Math.cos(2*Math.PI-i*Math.PI/180)*150;
			dy = Math.sin(2*Math.PI-i*Math.PI/180)*150*(-1);	
		}
		x = (int) dx;
		y = (int) dy;
	}
}

animation wir in einer externen Klasse welche ein Objekt von Kreis enthält aufgerufen igeht von 0 bis 360.

 

[Volvagia]

Allererste Vermutung: EDT blockiert.
Zeig mal den Aufrufer von animation.

 

[Malta]

Kreis k =new Kreis(300,300);
for(int i =0; i<=360; i++)
	{
		k.animation(i);
		try{ 
				Thread.sleep(200); 
				}
			catch (InterruptedException e)
			{}
						
	}

 

[Volvagia]

Weiter. Der Punkt, ab dem der gesamte Vorgang beginnt.

 

[Malta]

Kreis k= Kreis(300,300);
public void actionPerformed(ActionEvent e)
			{
				String label = e.getActionCommand();
				if(label.equals("Start"))
				{
					for(int i=0; i<360; i++)
                                         {k.animation();
                                           k.repaint();
                                        try{ 
                                                Thread.sleep(200); 
                                             }
                                       catch (InterruptedException e)
                                      {}
                                      }
					
			}

so?
wir haben hier gerade ein bisschen ein durcheinader produzier, tut mir leid...

 

[Gast2]

Wie oben schon erwähnt blockierst du dir den EDT. Langwierige Aktionen immer in nem separaten Thread ausführen.

 

[Volvagia]

Genau. Hier liegt das Problem. Zeichenvorgänge werden in den so genannten Event Dispatcher Thread ausgeführt. Dieser besitzt eine Warteschlange in die Vorgänge hauptsächlich von anderen Threads eingereiht werden und er abreitet sie der Reihe nach ab. So weit kein Problem, Events werden aber ebenfalls im EDT abgearbeitet. Kann einfach geprüft werden:

System.out.println(EventQueue.isDispatchThread());

Wenn du jetzt auf eine Taste drückst (ich nehme an der Auslöser ist ein Button) beginnt der EDT den ActionListener abzuarbeiten. Jedes mal, wenn er ein repaint aufruft schickt er die Anweißungen zum Zeichnen an die Warteschlange. (EventQueue) Da der EDT in der Abarbeitung des Listeners steckt stauen sich die Anweißungen bis die Schleife abgearbeitet ist, und erst am Ende kann neu gezeichnet werden.

Kreis k= Kreis(300,300);
public void actionPerformed(ActionEvent e)
{
	String label = e.getActionCommand();
	if(label.equals("Start"))
	{
		new Thread() {
			public void run() {
				for(int i =0; i<=360; i++) {
					k.animation();
					
					try { 
						Thread.sleep(200); 
					}
					catch (InterruptedException e)
					{}
				}
			}
		}.start();
	}
}

Das sollte dein Problem beheben. Damit startet der EDT nur kurz einen neuen Thread und hat dann beide Hände frei neu zu zeichnen, da er nicht ständig blockiert ist.

 

[Malta]

Vielen Dank, dass sieht echt hilfreich aus danke

 

[GUI-Programmer]

Nur würde ich für diesen speziellen Fall nicht mit der Klasse Thread arbeiten, sondern mit SwingWorker, da der repaint()-Aufruf ja schließlich auch erfolgen muss, und zwar auf dem EDT. Oder den Aufruf von EventQueue.invokeLater mit einbauen:

Kreis k= Kreis(300,300);
public void actionPerformed(ActionEvent e)
{
    String label = e.getActionCommand();
    if(label.equals("Start"))
    {
        new Thread() {
            public void run() {
                for(int i =0; i<=360; i++) {
                    k.animation();
                    // HIER der Teil der auf dem EDT ausgeführt wird:
                    EventQueue.invokeLater(new Runnable() {
                        public void run() {
                            k.repaint();
                        )
                    });
                    ///////////////////////////////////////////////////////////
                    try { 
                        Thread.sleep(200); 
                    }
                    catch (InterruptedException e)
                    {}
                }
            }
        }.start();
    }
}

 

[Volvagia]

Threads and Swing

The following JComponent methods are safe to call from any thread: repaint(), revalidate(), and invalidate(). The repaint() and revalidate() methods queue requests for the event-dispatching thread to call paint() and validate(), respectively. The invalidate() method just marks a component and all of its direct ancestors as requiring validation.

 

[Spacerat]

Nur würde ich für diesen speziellen Fall nicht mit der Klasse Thread arbeiten, sondern mit SwingWorker...

Mal ganz am Rande. Der TO verwendet einen Canvas, also AWT. Funktionieren da SwingWorker überhaupt?

 

[bERt0r]

Und nochmal am Rande: repaint() und revalidate() sind threadsafe.

 

[GUI-Programmer]

Ups, da hab ich mich wohl etwas vertan. Ok, rapaint() und desgleichen sind threadsave. Allerdings Methoden, die eine Komponente andersartig verändern, wie etwa setText() (z.B. bei einem JLabel) sind nicht threadsave und sollten daher vom EDT aufgerufen werden.

Mal ganz am Rande. Der TO verwendet einen Canvas, also AWT. Funktionieren da SwingWorker überhaupt?

Ok, stimmt wohl, dennoch sollte man schließlich möglichst bald (nach dem Einstieg in die Grafik-Programmierung) möglichst bald Swing verwenden. Und da lohnt sich die Klasse SwingWorker dann doch!