repaint während Thread läuft

[poochie89]

Hallo, ich habe ein Problem mit einer Aufgabe, es geht im groben darum ein (simples) billardspiel zu programmieren, dazu habe ich ein JPanel Tisch welches das ganze zeug zeichnet, diese klasse besitzt ein objekt der klasse Ball. Ball implementiert Runnable und führt die bewegung des balles in der run()-methode aus (ändert dessen position forlaufend). Wenn ich nun das ganze ausführe dann seh ich nur wo sich der ball zu beginn und zu ende befindet. Wo mache ich hier was falsch, muss ich da den thread kurz anhalten weil es sonst zu schnell geht?!
Danke schonmal für die Hilfe.

Thread t = new Thread(this.ball);

t.start();

while(t.isAlive()){
this.repaint();
}

 

[SlaterB]

wieso führst du die Bewegung des Balls in einem Thread aus und nicht einfach direkt in einer Schleife dort wo du gerade bist?

edit:
ok, damit nebenher repaint aufgerufen werden kann, meine pädagogische Frage bringt nix,

also es ist so: solange die Methode, die den Thread startet (ActionListener?), nicht beendet wird, wird nichts neu gezeichnet, die GUI ist blockiert,
die repaint-Aufrufe müssen entweder in den ersten Thread rein, an geeignete Stellen,
oder du brauchst einen zweiten Thread, der den ersten beobachtet, und repaint() aufruft,

aber besser nicht so wie bisher, millionenfach pro Sekunde, setze in die Schleife noch ein Thread.sleep(30) ein, um 30 ms zu warten

 

[poochie89]

also wir sollten den ball als runnable implementieren, stand so in der angabe, fand das auch nicht schön dass der ball seine bewegung ausführt.... hoffe ich hab dich richtig verstanden.

 

[SlaterB]

siehe edit oben

 

[poochie89]

jetzt ist mir schon einiges klarer, das ist also die schuld von repaint(), hab eben auch versucht nen string auszugeben in der schleife und der ist öfters ausgegeben worden.
hab jetzt nen 2ten thread erstellt mit repaint in der run-Methode. aber so funktionierts noch nicht, gleicher effekt wie vorher.hab leider mit threads nur ganz wenig gearbeitet bisher des wegen tue ich mich sehr schwer
und danke für die hilfe.

      Runnable r = new Runnable() {
	 public void run() {
	 PoolTable.this.repaint();
	}
	};
		    	    
	Thread ballThread = new Thread(this.ball);
	Thread guiThread = new Thread(r);
			      
	 ballThread.start();
	 guiThread.start();


          while(ballThread.isAlive()&&won==false){
		    		 
		    		  
		    		  try {
						ballThread.sleep(30);
					} catch (InterruptedException e) {
						// TODO Auto-generated catch block
						e.printStackTrace();
					}

 

[SlaterB]

du hast nach wie vor eine Schleife beim Aufrufer, nach wie vor kann dieser nicht fertig werden und solange passiert gar nix anderes,
so wirds in etwa was:

    final Thread ballThread = new Thread(this.ball);
     ballThread.start();

    Runnable r = new Runnable() {
     public void run() {

          while(ballThread.isAlive()){
                    try {
                        ballThread.sleep(30);
                    } catch (InterruptedException e) {
                        e.printStackTrace();
                    }
                    PoolTable.this.repaint();
          }
      } };                    
     Thread guiThread = new Thread(r);
     guiThread.start();
     // Aufrufer ist hier fertig, Threads können solange laufen wie sie lustig sind

 

[poochie89]

ok, also ich muss vom repaint()-Thread den anderen schlafen legen, weil sonst einfach die schleife abgearbeitet wird ohne dass sie ineinander greiden oder? leider hat dein lösungsvorschlag noch nicht geklappt, immer noch gleicher effekt.

 

[SlaterB]

was genau wie schnell ist kann ich an dem Code nicht beurteilen, baue in alle Threads an interessante Stellen System.out.println-Ausgaben ein,
verwende Thread.sleep() um alles beliebig langsam zu machen

wenn der Ball schneller als in 1 sec fertig ist, dann kann es generell möglich sein dass du wenig Zwischenschritte siehst,
unabhängig von den Änderungen hier

mehr Code könnte helfen, ansonsten kann ich nix neues mehr beitragen

 

[poochie89]

kann es sein dass der ballthread einfach viel zu schnell fertig ist?!?!
ich habe jetzt mal in dessen run-methode ein Thread.sleep eingebaut und jetzt sieht ganz gut aus, der ball läuft relativ flüssig..

 

[SlaterB]

sein kann alles, klingt doch gut