Thread will einfach nicht stoppen

[BlueLamp]

Hallo ihr,

folgendes Problem :

Ich will gerade ein kleines grafisches Ampelprogramm schreiben, darin möchte ich auch ein Thread einbauen der einen unendlichen Durchlauf der normalen Ampelstufen (rot zu gelb+rot zu gruen zu gelb usw.) ermöglicht.

Das hab ich auch soweit geschafft, der Thread startet auch wunderbar und macht genau das was er soll, nur schaffe ich es nicht ihn zu beenden

private boolean run_runs = false;            // wenn true dann läuft der Thread

private void run_handling () {                 // wird per button aufgerufen
    if(!run_runs) {
      bu_run.setText("Stop!");
      run_runs = true;
      lauf.start();
    }
    else {
      bu_run.setText("Run!");
      run_runs = false;
      lauf.interrupt();                                  //soll den Thread beenden
    }
  }
  
  public class alone extends Thread {
    public void run() {
      while(!lauf.isInterrupted()) {                // wenn interrupt dann break
        set_nextstat();
        try {
          Thread.sleep(1500); // 1 sec schlafen
        } catch(Exception e) { }
      }
    }
  }

 

[BlueLamp]

sorry, hab vergessen zu sagen das der Thread "lauf" heißt

private alone lauf = new alone();

 

[SlaterB]

API lesen bildet:

public void interrupt()

..

If this thread is blocked in an invocation of the wait(), wait(long), or wait(long, int) methods of the Object class, or of the join(), join(long), join(long, int), sleep(long), or sleep(long, int), methods of this class, ---> then its interrupt status will be cleared <--- and it will receive an InterruptedException.

If this thread is blocked in an I/O operation upon an interruptible channel then the channel will be closed, the thread's interrupt status will be set, and the thread will receive a ClosedByInterruptException.

frag (zumindest nicht mich) nicht warum

aber du bekommst die Exception, darauf kannst du ja reagieren und ein eigenes boolean setzen

 

[Marcel_Handball]

Warum schreibst du nicht:

try {Thread.sleep(1500);}
  catch(InterruptedException e) {break; }

 

[BlueLamp]

Danke für die Hilfe erstmal, also Marcels code hat funktioniert, jetzt hab ich aber noch eine frage:
Wie starte ich den Thread danach wieder ?

 

[SlaterB]

gar nicht, neues Objekt + start(),
oder den einen Thread nicht beenden, sondern auf Langzeitsleep setzen

 

[BlueLamp]

Langzeitsleep ?

 

[SlaterB]

Thread.sleep(999999999);
macht nix bis du wieder auf Start drückst und den Thread mit dem nächsten Interrupt wieder loslegen lässt,
ist aber nicht schön,
wenn der Zustand gerettet werden muss, dann diesen vielleicht in ein separates Objekt legen und dieses Zustand-Objekt einem neu angelegten Thread mitgeben

 

[Leroy42]

Thread.sleep(999999999);
macht nix bis du wieder auf Start drückst und den Thread mit dem nächsten Interrupt wieder loslegen lässt,
ist aber nicht schön

Das ist nicht nur nicht schön,
das ist Scheisse!

(und das von dir; SlaterB)

 

[SlaterB]

wenn du keine Alternative nennst darfst du nicht meckern

 

[BlueLamp]

So, habs jetz so gemacht wie du gesagt hast SlaterB - Danke

 

[jPat]

So könnte dein ThreadKlasse aussehen:

  public class alone extends Thread {
  private boolean isRunning = true;    
public void beendeThread(){ // von außen aufrufen 
     isRunning=false;
}
public void warteThread(){ // von außen aufrufen
   this.wait();
   
} 
public void geheWeiter() // von außen aufrufen
{
   this.notify();
}
 public void run() {
     isRunning = true; 
      while(isRunning) {               
        set_nextstat();
        try {
          Thread.sleep(1500); // 1 sec schlafen
        } catch(Exception e) { }
      }
    }
  }

wie du dein Button Ereignis setzt, mußt du dir selbst überlegen, was du erreichen möchtest.
Thread.sleep(1500); // 1 sec schlafen ?? es sind 1,5 sec.

Gruß

 

[SlaterB]

ach ja klar, wait + notify gibts natürlich
aber im Prinzip wie sleep + interrupt