Darstellung dauert zu lange. Wie rest verzögern?

[Arp]

Hallo,

Ich habe eine 2D Simulation programmiert die simuliert was mit kollidierenden Galaxien passiert. Das ganze basiert auf einer N Body Simulation. Nun hab ich ein Problem mit der Darstellung. Ich speicher in der Simulation alle x Zeitschritte die koordinaten aller Teilchen und dann noch zwei Schlüsselwörter um sie zu trennen. Einmal TIME, um zu wissen das jetzt ein Zeitschritt abgelaufen ist, und einmal BLUE um zu wissen das ab da die Partikel zur zweiten "blauen" Galaxie gehören.
Um das darzustellen hab ich einen Player geschrieben der im prinzip nur die Textdatei einliest. Das ganze läuft in einer while schleife. Kommt zwischendurch eine Zeile mit "BLUE", wird die zeilennummer für später gespeichert. Kommt "TIME", dann wird alles was in zwei Vector's steht mittels drawOval gezeichnet. Kommt nichts von beiden, werden die einträge gesplittet und in den x bzw. y vector gespeichert. Danach werden die Vector's auf 0 gesetzt und der ganze spaß beginnt von vorn.

Nun, ich kenn mich mit Swing und co. nicht so wirklich aus, geschweige denn mit Threads. Aber nachdem ich hier einiges rumprobiert hab, glaube ich das die Darstellung der einzelnen Pixel offenbar in einem separaten Thread passiert. Das führt dann nun zum Problem das die for schleife die einfach den Vector durch geht nicht schnell genug fertig wird, und der ursprüngliche Thread schon anfängt diesen Vector zu überschreiben.

So sieht mein Code aus:

import java.io.*;
import java.util.Vector;
import javax.swing.*;
import java.awt.*;

public class player 
{
	static Vector<Double> coordsx = new Vector<Double>();
	static Vector<Double> coordsy = new Vector<Double>();
	static double pc = 30.8566776e15; 
	static double faktor = 200*pc;
	static double zeit = 0;
	static int nr = 0;
	
	public static void main(String args[]) throws IOException
	{
		BufferedReader in = new BufferedReader(new FileReader("out.txt"));
		String zeile = "";
		JFrame MainWindow = new JFrame("Simulator");
		
		MainWindow.setLayout(null);
		MainWindow.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		MainWindow.setSize(900,900);
		MainWindow.setResizable(false);
		MainWindow.setContentPane(new MyComponent());
		MainWindow.setVisible(true);
		
		while((zeile=in.readLine())!=null)
		{
			if(zeile.contains("TIME"))
			{
				zeit = Math.round(Double.parseDouble(zeile.split("\t")[1]));
				MainWindow.repaint();
				coordsx.clear();
				coordsy.clear();
				
			}
			else
			{
				if(zeile.contains("BLUE"))
				{
					nr = coordsx.size();
				}
				else
				{
					coordsx.add(Double.parseDouble(zeile.split("\t")[0]));
					coordsy.add(Double.parseDouble(zeile.split("\t")[1]));
				}
				
			}
		}
	}
	public static class MyComponent extends JComponent
	{
		/**
		 * 
		 */
		private static final long serialVersionUID = 1L;

		public void paint(Graphics g)
		{
			g.setColor(Color.RED);
			g.drawLine(50,50,50,850);
			g.drawLine(50,850,850,850);
	//		g.drawString("zeit: "+zeit+" Myr", 50, 20);
			g.drawString(850*faktor/1000/pc+" kpc" ,20,40);
			g.drawString(850*faktor/1000/pc+" kpc" ,820,835);
			g.drawLine(40,50,60,50);
			g.drawLine(850,840,850,860);
			for(int i=0; i<nr-1;i++)
			{
				if(((coordsx.elementAt(i)/(faktor)+450)>51)&&(coordsy.elementAt(i)/(faktor)+450)<849)
				{
					g.setColor(Color.BLACK);
					g.fillOval((int)(coordsx.elementAt(i)/(faktor)+450),(int)(coordsy.elementAt(i)/(faktor)+450),3,3);		
				}
			}
			for(int i=nr; i<coordsx.size();i++)
			{
				if(((coordsx.elementAt(i)/(faktor)+450)>51)&&(coordsy.elementAt(i)/(faktor)+450)<849)
				{
					g.setColor(Color.BLUE);
					g.fillOval((int)(coordsx.elementAt(i)/(faktor)+450),(int)(coordsy.elementAt(i)/(faktor)+450),3,3);		
				}
			}
		
		}
	}
}

Könnt ihr mir vielleicht sagen wie ich das ändern kann, so dass das Programm immer abwartet bis repaint() fertig ist?
Thx.

 

[SlaterB]

du könntest vor dem repaint eine int oder boolean-Variable setzen
und am Ende von paint wird diese Variable geändert,

auf diese Änderung wartest du:

while (..) {
  Thread.sleep(30); // 30ms waren
}

oder ganz einfach zwischen den Änderungen generell eine bestimmte Zeit wie 30ms warten,
dann ist das Zeichnen vielleicht schon fertig

noch besser:
du könntest die neuen Kooridinaten in einer neuen lokalen Liste speichern und erst ganz am Ende diese neue Liste ins Klassenattribut speichern,
in paint dagegen wird am Anfang eine lokale Variable mit dem Inhalt der änderbaren Klassenattribute gesetzt,
so kann es fast keine Probleme durch Datenänderungen mehr geben,
allenfalls werden Schritte übersprungen wenn es ohne sleep() mehr Durchläufe gibt als gepaintet werden kann

ganz sicher wird es durch Synchronisation bei den Kopiervorgängen, oder Speicherung der Daten in exakt einem höheren Objekt,
ansonsten besteht die Möglichkeit, dass die Schleife gerade einen Teil der Daten aktualisiert hat und paint zeitgleich einen Teil alte + einen Teil neue Daten kopiert und dann anzeigt

edit:
übrigens besser

public void paintComponent(Grpahics g) {
   super.paintComponent(g);
   ..

verwenden

 

[Arp]

Hi,

Die Idee ist eigentlich gut, aber scheinbar hilft das nicht wirklich was.

Habe jetzt am Anfang der paint Methode ein go = false, am ende der Methode ein go = true

und in der Main steht nun nach dem repaint()

MainWindow.repaint();
while(go==false)
{
  	try 
   	{
  		Thread.sleep(0,500); // wartet 500 ns
   	} 
    	catch (InterruptedException e) 
    	{
   		e.printStackTrace();
    	}
}

Die animation läuft zwar, aber auf der console werden trotzdem ständig ArrayIndexOutofBounds Errors angezeigt. Und das kann ich mir jetzt eigentlich nicht erklären. Denn wenn repaint nun im eigenen Thread abläuft, sollte es in der main, sequentiell nicht weiter gehen solang die static boolean go nicht auf true steht. Aber trotzdem scheinen die Daten geändert zu werden.

Ich versuche das mal so umzubauen das erstmal alles eingelesen wird, dann in einem Vector gespeichert wird das pro element die anderen beiden Vector's erhält.

 

[SlaterB]

schreibe doch ein System.out.println()-Log, wann beginnt paint, wann endet es,
wann befindet sich die Schleife im Warten, wann rechnet sie,

theoretisch ist sicher möglich, dass das Warten gerade zu Ende ist und Rechnen wieder losgeht und dennoch gleichzeitig erneut gemalt wird,
richtiger gegenseitiger Ausschluss heißt Synchronisation

 

[Michael...]

Habe jetzt am Anfang der paint Methode ein go = false

Der Vorschlag war vor dem Aufruf von repaint() die Variable auf false zu setzen und innerhalb der paint nur die Varialbe zurückzusetzen - da man letztendlich wenig Einfluss darauf hat wann die paint() aufgerufen wird.

 

[SlaterB]

oh stimmt, zum Glück war meine Idee bisschen besser als das Verständnis des Fehlers danach

 

[Arp]

Ich dachte pain() wird aufgerufen wenn repaint() aufgerufen wird.

 

[SlaterB]

nein, direkt schon gar nicht, sonst wäre es überhaupt nicht nebenläufig,
ansonsten müsste diese Methode ähnlich deinem Code warten bis ein separater Thread paint aufruft, auch kompliziert

nein, das bedeutet nur das irgendwo die Info hinterlegt wird, bei nächster Gelegenheit zu malen,
das kann durchaus noch paar ns dauern, bis nach zufälligen x weiteren Befehlen deiner main-Methode der nächsten Thread überhaupt erst drankommt,
das kann ewig dauern wenn die GUI gerade blockiert ist,
das kann gar keine Auswirkungen haben die GUI nicht zu sehen ist oder paint gerade läuft