hallo, habe ein Problem mit dem pro. eines Threads

[Tassimo]

folgende aufgabe zu erfüllen:
Die Klasse Treppe soll animiert werden. Alle 1000ms soll die Farbe der Treppe geändert werden. hierzu ist das Implementieren eines Threads erforderlich. folgender Code ist gegeben:
public Class Treppe_________________________{
private Listy<Color> farbeL = Arrays.asList(Color.red, Color.yellow, Color.green);
int Color = farbeL.get(0);

public void paintComponent(Graphics g){
g.setColor(farbe);
//darf nicht verändert werden
}

public _____________________________
______________________
farbe = farbeL.get((int)(Math.random()*3));
repaint();




hier mein erstellter Code dazu:
package GUIJPANEL;

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Paint;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.WindowEvent;
import java.awt.event.WindowListener;
import java.util.*;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class TGUIJP extends JPanel{

JFrame frame;
private List<Color> farbeL = Arrays.asList(Color.red, Color.yellow, Color.green);
treppeThread t2;
Color farbe = farbeL.get(0);


public void paintComponent (Graphics g){
super.paintComponent(g);
g.setColor(farbe);
int width = 40;
int height = 150;
int y = 50;
for(int i = 50; i <= 300; i += width)
{
g.drawRect(i, y, width, height);
g.fillRect(i, y, width, height);

y += 20;

height -= 20;
}
}




public static void main(String[]args){

TGUIJP tgp = new TGUIJP();
tgp.setUpScreen();
new treppeThread().start();
}

public void setUpScreen(){
frame = new JFrame();
frame.setSize(420, 230);
frame.setVisible(true);
frame.add(this);
frame.setDefaultCloseOperation(frame.EXIT_ON_CLOSE);
}

public class treppeThread extends Thread{
TGUIJP tgth = new TGUIJP();
public treppeThread(){
super();
}

public void run(){
while(true){
tgth.farbe = tgth.farbeL.get((int)(Math.random()*3));
repaint();
try{
sleep(1000);
}catch (InterruptedException e){}
}
}

}
}


bitte euch das mal durch zu schauen, mein problem liegt beim erstellen des Threads und verbinden mit der zeichnung, sodass ich dann mit dem Thread automatisch die farben wechseln kann.

 

[Gucky]

Das Ändern der Farbe ginge einmal aktiv, durch das Aufrufen von setColor() durch den Thread und anschließendes repaint() oder passiv durch setzen eines Flags und anschließendes repaint().
Verbinden mit der Zeichnung: der Thread braucht Zugriff auf eine Instanz des zeichnenden Objektes. (Graphics oder so).
Erstellen des Threads:

Farbfred farbfred = new Farbfred(this.g); farbfred.start()

oder was meinst du?

 

[Tassimo]

also mit dem Thread funktioniert das nicht so ganz wie ichs mir vorstelle,
muss ich dem Thread die Klasse TGUIJP zuweisen? also ein objekt davon?
und das erstellen eines Threads erkennt eclipse wie folgt nicht an:
treppeThread tth = new treppeThread();
und "this.g" geht auch nicht

 

[dzim]

Gucky hat hier Recht. Du verwurschtelst hier ein paar Sachen... Ich wüsste jetzt keine ideale Stele, aber stell es dir so vor:

public class TreppeThread extends Thread{

 private final TGUIJP tgth = null;
 private boolean interrupt = false;

 public TreppeThread(final TGUIJP tgth){
  this.tgth = tgth;
 }

 public void setInterrupt(boolean b) {
  this.interrupt = b;
 }

 public void run(){
  while(!interrupt){
   tgth.farbe = tgth.farbeL.get((int)(Math.random()*3));
   repaint();
   try{
    sleep(1000);
   }catch (InterruptedException e){}
  }
 }
}

und in deiner repaint-Methode von TGUIJP machst du z.B. so etwas

private TreppeThread t = null;

public void paintComponent (Graphics g){
 super.paintComponent(g);
 g.setColor(farbe);
 int width = 40;
 int height = 150;
 int y = 50;
 for(int i = 50; i <= 300; i += width) {
  g.drawRect(i, y, width, height);
  g.fillRect(i, y, width, height);
  y += 20;
  height -= 20;
 }
 if (t==null) {
  t = new TreppeThread(this);
  t.start();
 }
}

That's pretty much it...

Den Rest musst du wohl selbst hinbekommen, oder aber da müssen dann die "Swinger" des Forums ran

 

[Gucky]

Für das setInterrupt gibt es schok was. Nämlich isInterrupted(). Dann kann man den Thread mit interrupt() beenden. Und , falls noch nicht geschehen, musst du noch die preferredSize einstellen. Mehr dazu gibt es bei Google.
Aber sonst stimmt alles, was mein Vorredner geschrieben hat.