2 StoppUhren mit Threads

[GimmeGimme09]

Hi, ich als Anfänger tue mich recht schwer damit 2 StoppUhren die parallel laufen müssen zu programmieren. Nach einigen stunden
ist mir dann immerhin etwas gelungen^^. Jetzt fallen mir aber schon 2 probleme auf.

1. die uhr ist ungenau. die dauer zwischen start und stop geht um 1 sek nach.
2. ich kann, wenn ich einmal a oder b gestartet und gestoppt habe, nicht nochmal a oder halt b starten ( ist mir schon klar warum^^ ist müsste ja jetzt den thread neu erzeugen oder verhindern das run() beendet wird) nur weiß ich nicht wie ich das in meinem code machen soll.

Wenn ihr noch mehr fehler sehen solltet immer raus damit, das würde mir sehr helfen

import java.text.DateFormat;
import java.text.SimpleDateFormat;
//import java.util.Date;
import java.util.*;

public class Stoppuhr {

    public static void main(String[] args) throws InterruptedException {

        UhrA a = new UhrA();
        UhrB b = new UhrB();

        Scanner sc = new Scanner(System.in);

        do {
            System.out.print("Eingabe: ");
            String e1 = sc.next();
            if (e1.equals("A") | e1.equals("a")) {
                //System.out.println("WEgpunkt2");
                if (a.isAlive() == true) {

                    a.interrupt();

                    Date d4 = new Date(System.currentTimeMillis());
                    DateFormat df = SimpleDateFormat.getTimeInstance(DateFormat.MEDIUM);
                    System.out.println("Es ist " + df.format(d4) + " Uhr.");

                } else {

                    try {

                        a.start();

                        Date d = new Date(System.currentTimeMillis());
                        DateFormat df = SimpleDateFormat.getTimeInstance(DateFormat.MEDIUM);
                        System.out.println("Es ist " + df.format(d) + " Uhr.");
                    } catch (IllegalThreadStateException e) {
                        System.out.println("Nochmal die Uhr starten geht nicht");
                    }
                }
            } else if (e1.equals("B") | e1.equals("b")) {
                // System.out.println("WEgpunkt3");
                if (b.isAlive() == true) {

                    b.interrupt();

                    Date d3 = new Date(System.currentTimeMillis());
                    DateFormat df = SimpleDateFormat.getTimeInstance(DateFormat.MEDIUM);
                    System.out.println("Es ist " + df.format(d3) + " Uhr.");
                } else {

                    try {

                        b.start();

                        Date d2 = new Date(System.currentTimeMillis());
                        DateFormat df = SimpleDateFormat.getTimeInstance(DateFormat.MEDIUM);
                        System.out.println("Es ist " + df.format(d2) + " Uhr.");
                    } catch (IllegalThreadStateException e) {
                        System.out.println("Nochmal die Uhr starten geht nicht");
                    }
                }
            }
            else {
                System.out.println("Ungültige StoppUhr: Bitte nur A oder B");
            }
            Thread.sleep(1);
        } while (a.isAlive() || b.isAlive());
    }
}

und nochmal eine der Uhren A und B. Die andere sieht genau so aus.

public class UhrA extends Thread {
    long millis;
//long StartZeit, StoppZeit;

    @Override
  public void run() {
      millis = System.currentTimeMillis();

    while (true) {
      if (isInterrupted()) {
          
          millis = System.currentTimeMillis() - millis;
          millis = (millis/1000);
          //Date d4 = new Date(millis);

        

        System.out.println("Dauer: "+millis+" Sekunden");
       break;

      }
    }
  }

 

[Java-Freak]

das ganze gibts auch schon fertig, da musst du nicht mit System.currentTimeMillis() rumrechnen....
Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 16.27 Zeitliches Ausführen mit dem javax.swing.Timer

 

[Marco13]

Auch wenn das die Antwort ist, die man auf so eine Frage hin vermutlich am wenigsten hören will: Fang nochmal neu an.

Die andere sieht genau so aus.
Darüber solltest du nochmal nachdenken...

Allgemein wäre es, wenn die Uhren mehrmals gestartet und gestoppt werden sollen, vermutlich hilfreich, beim stoppen den Thread nicht zu killen. Warum genau sie in einem eigenen Thread laufen ist auch noch nicht ganz klar: Bei start werden sie gestartet, und bei stop wird die Zeit seit "start" berechnet. Ob in der Zwischenzeit noch ein Thread für die Uhr lief, ist eigentlich wurscht (sofern dieser Thread nicht in Zukunft z.B. jede Sekunde eine Ausgabe in der Konsole oder so machen soll).

Ganz pragmatisch:

class Stoppuhr 
{
    public void starteUhr() { ... }
    public void stoppeUhr() { ... }

    // Vielleicht noch....
    public void pauseUhr() { ... } 
    

}

Ansonsten solltest du dir mal ansehen, wie die Methoden Object#wait und Object#notify funktionieren.

 

[slawaweis]

@GimmeGimme09
ich weis jetzt nicht, ob die Aufgabe darin liegt eine Stoppuhr zu haben oder eine selber zu schreiben, aber in den Apache Commons (Sammlung von hilfreichen freien Werkzeugen) gibt die Klasse StopWatch:

StopWatch (Commons Lang 2.5 API)

die genau das macht. Wenn man es selber schreiben muss, kann man sich am API orientieren oder den Quelltext der StopWatch studieren. Um die StopWatch zu bekommen, muss man die Apache Commons Lang JAR runterladen, daneben findet man auch den Quelltext:

Lang - Home

Slawa