problem mit timer/timertask, Farbverlauf

[Bibi]

Hallo,

ich habe eine Klasse die einen Farbverlauf darstellen soll. Einzelne Farben(RGB, jeweils 0-255) sind in dem Array color abgespeichert, bei Aufruf der Klasse verlauf soll durch Veränderung der RGB werte von einem Farbpunkt zum anderen "gewandert" werden.

Wichtig hierbei ist, dass die Abläufe der drei Werte synchron geschehen.

Hierfür habe ich timer eingebaut, die gleichzeitig starten und eigentlich, durch geeignete Wahl der Intervalle, auch gleichzeitig beendet werden sollten. Mein Problem ist dass die timer eben nicht gleichzeitig fertig sind und kann mir nicht erklären woran das liegt.

durch

Thread t1 = new Thread(new verlauf(anzeigeGross));
			t1.start();

wird die Klasse

public class verlauf implements Runnable{
		
		JTextField tf = new JTextField();
		private long zeitR;
		private long zeitG;
		private long zeitB;
		private long startTime;
		 
	    public verlauf(JTextField jtf)
	    {
	        this.tf = jtf;
	    }

	    @Override
	    public void run()
	    {
	        i=0;
			tf.setBackground(color[i]);
			for (i=0 ; ((color[i+1]!=null));i++){
				System.out.println();
				System.out.println();
				System.out.println("i:"+i);
								
				int diffR = color[i].getRed()-color[i+1].getRed();
				if (diffR<0) diffR = diffR * (-1);
				System.out.println("diffR: "+diffR);
				int diffG = color[i].getGreen()-color[i+1].getGreen();
				if (diffG<0) diffG = diffG * (-1);
				System.out.println("diffG: "+diffG);
				int diffB = color[i].getBlue()-color[i+1].getBlue();
				if (diffB<0) diffB = diffB * (-1);
				System.out.println("diffB: "+diffB);
				zeit = 30;
				
				if(diffR==0)	zeitR = 0;
				else			zeitR = (long) ((zeit*1000)/diffR);
				
				if(diffG==0)	zeitG = 0;
				else			zeitG = (long) ((zeit*1000)/diffG);
				
				if(diffB==0)	zeitB = 0;
				else			zeitB = (long) ((zeit*1000)/diffB);
				
				System.out.println("zeitR: "+zeitR);
				System.out.println("zeitG: "+zeitG);
				System.out.println("zeitB: "+zeitB);
				System.out.println();
				
				Timer timerR = new Timer();
				Timer timerG = new Timer();
				Timer timerB = new Timer();
				changeRed cr = new changeRed(tf, color[i], color[i+1]);
				changeGreen cg = new changeGreen(tf, color[i], color[i+1]);
				changeBlue cb = new changeBlue(tf, color[i], color[i+1]);
				
				startTime = System.currentTimeMillis();
				cr.setStartTime(startTime);
				cg.setStartTime(startTime);
				cb.setStartTime(startTime);
				System.out.println("startTime: "+startTime);
				if(zeitR!=0)	timerR.schedule(cr,0, zeitR);
				if(zeitG!=0)	timerG.schedule(cg,0, zeitG);
				if(zeitB!=0)	timerB.schedule(cb,0, zeitB);
				
								
				try {
					Thread.sleep(zeit*1000);								
				} catch (InterruptedException e) {
					// TODO Auto-generated catch block
					e.printStackTrace();
				}
				
			}
	    }
	}

aufgerufen.

Die TimerTask Klasse:

public class changeRed extends TimerTask{

	JTextField tf = new JTextField();
	int red1;
	int red2;
	private Color farbe1;
	private Color farbe2;
	private long stopTime;
	private long startTime;
	private long runTime;
	
	
	public changeRed(JTextField tf, Color farbe, Color farbe2)
    {
		 	this.tf = tf;
	        this.farbe1=farbe; 		//in Farbe1 wird die Anfangsfarbe gespeichert
	        this.farbe2=farbe2;		//in Farbe2 wird die Endfarbe gespeichert
    }
	
	@Override
	public void run() {
		farbe1 = tf.getBackground();
		red1=farbe1.getRed();
		red2=farbe2.getRed();

		
		if(red1<red2){
			red1++;
			if(red1>=red2) {							//Abbruchfunktion
				stopTime = System.currentTimeMillis();
				System.out.println("ROT stopTime: "+stopTime);
				runTime = (stopTime - startTime);
				System.out.println("ROT runTime: "+runTime);
				
	            this.cancel();
	       }
		}
		
		if(red1>red2){
			red1--;
			if(red1<=red2) {							//Abbruchfunktion
				stopTime = System.currentTimeMillis();
				System.out.println("ROT stopTime: "+stopTime);
				runTime = (stopTime - startTime);
				System.out.println("ROT runTime: "+runTime);
				
	            this.cancel();
	       }
		}
		
		if (red1==red2){
			this.cancel();
		}
		
		farbe1 = new Color(red1,farbe1.getGreen(),farbe1.getBlue());
		tf.setBackground(farbe1);

	}

	public long getRunTime() {
		return runTime;
	}

	public void setStartTime(long startTime) {
		this.startTime = startTime;
	}

die Klassen changeGreen und changeBlue sind genauso aufgebaut.

Kann es sein dass der timer nicht in millisekunden intervallen arbeiten kann?

Kann mir jemand bei meinem Problem helfen bitte?

 

[Michael...]

Hab ich das richtig verstanden, Du willst per Zwischenstufen von einer Farbe in eine andere wechseln? Warum berechnest Du die einzelnen Farbanteile nicht in einem Thread bzw TimerTask? Wäre doch viel einfacher.
Ein Problem in Deinem Fall könnte die mangelnde Synchronisation sein. z.B. liest Du die Hintergrundfarbe vom Textfeld aus bearbeitest den Rotanteil und weist dann dem Textfeld die neue Farbe mit geändertem Rotanteil und altem Grün/Blauanteil zu

farbe1 = new Color(red1,farbe1.getGreen(),farbe1.getBlue());tf.setBackground(farbe1);

in der Zwischenzeit könnte aber der Grün bzw. Blauanteil von einem anderen Timertask ebenfalls geändert worden sein.

 

[Bibi]

Hab ich das richtig verstanden, Du willst per Zwischenstufen von einer Farbe in eine andere wechseln?

Ja

 

[Bibi]

Hab ich das richtig verstanden, Du willst per Zwischenstufen von einer Farbe in eine andere wechseln?

Ja, bei zB ein Farbverlauf von Rot (R=255,G=0,B=0) nach Türkis(R=0,G=255,B=167) in 3 Sekunden muss

der Rotanteil um 255 Schritte verändert werden in 3 sek
der Grünanteil um 255 Schritee in 3 Sek
der Blauanteil um 167 Schritte in 3 sek

 

[Bibi]

Warum berechnest Du die einzelnen Farbanteile nicht in einem Thread bzw TimerTask? Wäre doch viel einfacher.

die Klasse ist doch ein Thread, und die Ablaufe passieren in einem TimerTask. Erklär mal bitte was du damit meinst

 

[faetzminator]

Wieso nicht in 3*100 Schritten, so dass du durchschnittlich 0.85 abziehst? Ich würde gleiche Schrittmenge eines exakten Verlaufs vorziehen.

 

[Bibi]

Ein Problem in Deinem Fall könnte die mangelnde Synchronisation sein. z.B. liest Du die Hintergrundfarbe vom Textfeld aus bearbeitest den Rotanteil und weist dann dem Textfeld die neue Farbe mit geändertem Rotanteil und altem Grün/Blauanteil zu ...

...in der Zwischenzeit könnte aber der Grün bzw. Blauanteil von einem anderen Timertask ebenfalls geändert worden sein.

Das müsste dann aber nur Einfluss auf die angezeigte Farbe haben, warum werden dadurch die Laufzeiten verändert?

 

[Bibi]

Wieso nicht in 3*100 Schritten, so dass du durchschnittlich 0.85 abziehst? Ich würde gleiche Schrittmenge eines exakten Verlaufs vorziehen.

ich kann eine Farbe nur in integerWerten angeben. Bei einer kleinen Farbänderung zB. 4 Werte in 3 sekunden müsste ich bei jedem der 100 Schritte um 0.013 verändern

 

[faetzminator]

Ich würde (wie bereits erwähnt) jeweils gemeinsame Berechnungen vorschlagen. Hier ein kleines Beispiel:

public static void main(String[] args) throws InterruptedException {
    print(new Color(255, 0, 0), new Color(0, 255, 167), 3000);
}

public static void print(Color c1, Color c2, int ms) throws InterruptedException {
    int sleep = 10;
    int times = ms / sleep;
    for (int i = 0; i < times; i++) {
        int r = calc(c1.getRed(), c2.getRed(), i, times);
        int g = calc(c1.getGreen(), c2.getGreen(), i, times);
        int b = calc(c1.getBlue(), c2.getBlue(), i, times);
        print(new Color(r, g, b));
        Thread.sleep(sleep);
    }
    print(c2);
}

public static int calc(int s, int e, int t, int p) {
    return s + (e - s) * t / p;
}

public static void print(Color c) {
    System.out.println(c);
}

Am Schluss rufe ich print() lediglich ausserhalb der Klammer auf, damit nicht noch ein Mal sleep() aufgerufen wird. Aber natürlich kann man das auch weglassen und [c]i <= times[/c] schreiben.

Edit: Zu deiner Antwort: Natürlich hast du hier (als Beispiel bei 10ms sleep Time) eine maximale Abweichung von 5ms. Muss es denn genauer sein? Gerundet muss sowieso werden, sei es die Zeit auf ganze ms.

 

[Bibi]

Danke erstmal, ich werde das mal versuchen

 

[Bibi]

Vielen Dank, das funktioniert. Ich hatte mich da wohl etwas verannt.