automatischer Farbwechsel

Hallo Leute,

ich würde gerne einen Kreis erstellen, welcher ständig die Farbe wechselt. Solange bis ein Button geklickt wird.
Die Erstellung eines Kreises ist kein Problem, der Farbwechsel an sich auch nicht - nur wird dies nicht angezeigt, wenn ich es beispielsweise in eine Schleife packe und laufen lassen.

In meinem Kopf habe ich eine "Endlosschleife" welche die Farbe des Kreises ändert und der Benutzer dies auch sieht. Sobald er den Button drückt, hört diese "Endlosschleife" auf zu laufen. Wie kann man soetwas realisieren?
Mir ist gerade die Idee von Threads gekommen, dazu habe ich leider noch kein Wissen. Ist dies der richtige Weg?

Ich hoffe jemand hat einen guten Tipp für mich

Danke

Danke für den Hinweis, ich habe es mir ja fast gedacht. Leider klappt der Versuch nicht, die Anwendung bricht mit einem Fehler ab.
Ich habe in einer Subactivity einen Kreis, beim Klick auf einen Button soll dieser anfangen die Farbe zu wechseln.

An dieser Stelle etwas Quellcode:

public class game extends Activity {


	private int pointPlayer1 = 0;
	private int pointPlayer2 = 0;
	private Circle gameCircle;
	private FrameLayout gameField;
	private Thread changeColor;
	
	@Override
	protected void onCreate(Bundle savedInstanceState) {
		// TODO Auto-generated method stub
		super.onCreate(savedInstanceState);
		setContentView(R.layout.game);
		
		gameField = (FrameLayout) findViewById(R.id.circle_view);	
		Display display = getWindowManager().getDefaultDisplay();	
		
		gameCircle = new Circle(this,
								display.getWidth(),
								display.getHeight(),
								Circle.getColor());	
		
		gameField.addView(gameCircle);
		
		// nebenläufiger Prozess für den Farbwechsel
		changeColor = new Thread(){
			public void run(){
				try {
					while(true){
						// Wartezeit zwischen den Farben
						sleep((int)((Math.random()*2500)+1000));
						gameCircle.chanceColor(Circle.getColor());
						// TODO: FrameLayout aktualisieren verbessern
						gameField.removeAllViews();
						gameField.addView(gameCircle);
						
					}
				} catch (InterruptedException e) {
					// TODO Auto-generated catch block
					e.printStackTrace();
				}
				finally{
					finish();
				}
				
			}
		};
		
	}

Nachdem der Button geklickt wurde, soll der Thread gestartet werden, sobald es dazu kommt, stürzt das Programm sofort ab.
Führe ich es nicht parallel aus, also wie folgt klappt alles:

// Wartezeit zwischen den Farben
			try {
				Thread.sleep((int)((Math.random()*2500)+1000));
			} catch (InterruptedException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
			gameCircle.chanceColor(Circle.getColor());
			// FrameLayout aktualisieren
			gameField.removeAllViews();
			gameField.addView(gameCircle);

Kann mir jemand meinen Fehler beschreiben?
Ich habe das Gefühl der Thread startet in der falschen Activity (im Menü vor dieser game-Activity), sprich er ist gar nicht am richtigen Ort, aber dies kann ja nicht sein oder?

Oh nein, Button und Kreis befinden sich natürlich in einer Activity.
Die Fehler entstehen bei der 1. Zeile, nehme ich diese weg bei der 2. Zeile im Folgenden:

gameField.removeAllViews();
gameField.addView(gameCircle);

Wie zeichnet man etwas neu? .repaint() oder .refresh() existiert leider nicht. Gern würde ich daher das FrameLayout neu zeichnen, was ich nur angelegt habe, damit der Kreis einem kleinen Bereich zugeordnet ist.
Der Kreis ist ein Canvas-Objekt. Ich ändere beim Kreis die Eigenschaft der Farbe, doch da dieser bereits gezeichnet ist sieht man dies natürlich nicht. Da der Kreis dem FrameLayout zugeordnet ist, würde ein neues zeichnen des Kreises dazu führen, dass alles korrekt ist.

Mein Versuch war nun alle Views innerhalb des FrameLayout zu löschen und wieder diesen neuen View (den Canvas-Kreis) hinzuzufügen.

//edit:
Mir ist gerade eingefallen, dass ich das Canvas-Objekt je refreshen kann:

gameCircle.refreshDrawableState();
gameCircle.invalidate();

Leider stützt das Programm bei invalidate() ab
Wie macht man es richtig?

Hallo block_,
eine richtige Fehlermeldung bekomme ich leider nicht Es steht nur da, das der Prozess beendet wird.

meine Kreis-Klasse sieht wie folgt aus:

public class Circle extends View {
	private final float x;
	private final float y;
	private final int r;
	private static final int WININDEX = 4;
	private static int tmpindex = 0;
	private Paint mPaint = new Paint(Paint.ANTI_ALIAS_FLAG);
	
	public Circle(Context context, int width, int hight, int color) {
    	super(context);
    	mPaint.setColor(color);
    	
    	this.x = width/2;
    	this.y = hight/2;
    	this.r = (int) ((Math.min(width, hight)*0.8)/2);
	}
	
	@Override
	protected void onDraw(Canvas canvas) {
    	super.onDraw(canvas);
    	canvas.drawCircle(x, y, r, mPaint);
	}
	
	public void chanceColor(int color){
		mPaint.setColor(color);
	}
	
	public boolean isWinColor(){
		return tmpindex == WININDEX;
	}
	


	// Klassenmethoden:
	// gibt eine zufällige Farbe zurück
	public static int getColor(){
		int arrColor[] = new int[10];
		arrColor[0] = Color.rgb(47,79,79);
		arrColor[1] = Color.rgb(0,0,128);
		arrColor[2] = Color.rgb(0,191,255);
		arrColor[3] = Color.rgb(0,255,255);
		arrColor[4] = Color.rgb(0,100,0);	// win
		arrColor[5] = Color.rgb(255,255,0);
		arrColor[6] = Color.rgb(184,134,11);
		arrColor[7] = Color.rgb(178,34,34);
		arrColor[8] = Color.rgb(255,69,0);
		arrColor[9] = Color.rgb(255,20,147);
		
		tmpindex = (int)(Math.random()*arrColor.length);
		return arrColor[tmpindex];
	}
	
}

Im Spiel wird der Kreis wie folgt erstellt:

gameCircle = new Circle(this,
								display.getWidth(),
								display.getHeight(),
								Circle.getColor());

Nun möchte ich gern im Thread die Farbe wechseln:

gameCircle.chanceColor(Circle.getColor());
gameCircle.refreshDrawableState();

getColor sollte natürlich nicht von außen aufrufbar sein, sondern nur private in der Kreis-Klasse selbst!

Danke antifish,

ich verstehe - macht natürlich Sinn. Wenn es so läuft, würde mich eins noch brennend interessieren. Kann man in einem Thread auch auf ein View-Element zugreifen um zu prüfen, ob ein Button geklickt wurde als Beispiel?

Muss ich eine Thread-Klasse erstellen, welche vom Thread erbt und dann überschreibe ich die run-Methode um ein View-Element zu übergeben oder gibt es einen Trick?