Problem mit einem Timer (Aktualisierung etc.)

[SlippyToad]

Hallöchen,

ich programmiere gerade ein Spiel und wollte einen Timer erstellen. Nun habe ich allerdings ein Problem - hier mal der Quelltext:

public class PicrossTimer extends Canvas implements PropertyChangeListener {
    private long startTime;
    private int currentTime;

    // constructor
    public PicrossTimer() {
        startTime = Calendar.getInstance().getTimeInMillis();
    }

    // methods
    public void paint(Graphics g) {
        g.drawString("" + currentTime, 0, 10);
    }

    public void refreshTime() {
            currentTime = (int)( Calendar.getInstance().getTimeInMillis() - startTime );
    }

    public void addTime(int value) {
            currentTime += value;
    }

    public void propertyChange(PropertyChangeEvent event) {
        if (event.getPropertyName().equals("picross_refresh_timer")) {
            refreshTime();
            repaint();
        } else if (event.getPropertyName().equals("picross_button_mistake")) {
            addTime(100000);
            repaint();
        }

    // ...
    // Canvas zeugs
    }

Das aktualisieren geschieht in einem Thread.

public class PicrossTimerRefresh extends Thread {
    private PropertyChangeSupport propChange;

    // constructor
    public PicrossTimerRefresh() {
        propChange = new PropertyChangeSupport(this);
    }

    public void run() {
        while (true) {
            try {
                Thread.sleep(500);
                propChange.firePropertyChange("picross_refresh_timer", null, null);
            } catch (InterruptedException e) {}
        }
    }

    // ...
    // listener adden und so...

Was da in addTime übergeben wird, wie oft der Timer aktualisiert wird und das es Millisekunden sind ist erstmal völlig egal.

Das Problem ist folgendes:
Die Zeit wird regelmäßig hochgezählt. Wenn jetzt allerdings "picross_button_mistake" ausgelöst wird, wird zwar der Wert draufaddiert, allerdings ist das Update beim nächsten "picross_refresh_timer" event wieder weg.
Es könnte also folgendermaßen aussehen:
0
500
1000
1500
> picross_button_mistake
101500
2000
2500
...

Jemand ne Idee, wie ich das Problem umgehen kann. Hatte es davor schon auf anderem Wege probiert, aber es kam das Gleiche raus.
Muss ich eventuell currentTime mit Semaphoren sperren?

Danke im Voraus - Tom

 

[wayne0101]

klar, daß die änderung durch picross_button_mistake später wieder rückgängig gemacht wird. bei einem aufruf deiner methode refreshTime() wird ja der wert von currentTime überschrieben durch das ergebnis der subtraktion der aktuellen zeit und deinem startwert. also entweder merkst du dir alles, was noch draufzurechnen ist in einer extra variablen, die du bei refreshTime() immer hinzufügst oder du änderst den wert von startTime.

übrigends hätte ich an deiner stelle das ganze mit System.currentTimeMillis() gelöst, da dies dann, meiner meinung nach, besser lesbar ist. davon abgesehen hätte ich es mir so gespart die klasse Calendar einzubinden und immer Calendar.getInstance() aufzurufen.

viele grüße

wayne

 

[SlippyToad]

Ach ich seh schon - da hab ich wohl nicht ganz zu Ende gedacht
Und System.currentTimeMillis() kannte ich noch nicht - danke.

Ich erweiterte PicrossTimer also einfach um:

private int value;

und änder die Methoden in:

public void refreshTime() {
    currentTime = (int)( System.currentTimeMillis() - startTime ) + value;
}

public void addTime(int value) {
    this.value += value;
}

Nun geht's - danke.