Problem mit Countdown Timer

[laurin_e]

Hallo alle

Ich versuche einen Countdown Timer mit einem Arduino Mega zu realisieren. Ziel ist, dass ich mit einem Rotary Switch eine Anzahl Minuten setzen kann. Nach einem Tastendruck soll dann auf dem Display die Minuten und Sekunden abgezählt werden und bei 00:00 eine Aktion ausgeführt werden. Leider scheitere ich bereits bei dem millis(). Wieso zählt der nachfolgende Code von 26 Sekunden rückwärts? Vorgesehen wären 20 Minuten.

unsigned long seconds;
unsigned long minutes;

void setup() {

Serial.begin(9600);
}

void loop() {

static unsigned long startTime = millis();
static unsigned long TargetTime = startTime + (20*60*1000);  // plus 20 minutes

seconds = (TargetTime - millis()) / 1000;
minutes = seconds / 60;
seconds = seconds % 60;
minutes = minutes % 60;

Serial.println("minutes");
Serial.println(minutes);
Serial.println("seconds");
Serial.println(seconds);

}

[combie]

@laurin_e

(20*60*1000)

Testprogramm:

#include <Streaming.h> // die Lib findest du selber ;-)
Print &cout = Serial; // cout Emulation für "Arme"

void setup() 
{
  Serial.begin(9600);
  cout << F("Interval: ") << (20*60*1000) << endl;
}

void loop() 
{

}

Ausgabe:

Interval: 20352

Ich nehme an, dass du was anderes beabsichtigt hast.

Ach ja eine Meldung kommt da auch noch:

E:\Programme\arduino\portable\sketchbook\sketch_feb08d\sketch_feb08d.ino: In function 'void setup()':
E:\Programme\arduino\portable\sketchbook\sketch_feb08d\sketch_feb08d.ino:8:36: warning: integer overflow in expression of type 'int' results in '20352' [-Woverflow]
    8 |   cout << F("Interval: ") << (20*60*1000) << endl;
      |                               ~~~~~^~~~~

[Rentner]

Hallo,
@combie hat Dir ja bereits gezeigt woran es liegt. Du musst dafür sorgen das die Berechnung
(20*60*1000)
nicht als eine Integer Berechnung gemacht wird. Das kannst Du erreichen in dem Du mindestens einen Wert als 32 bit Wert angibst.

static unsigned long TargetTime = startTime + (20*60*1000UL); 

[combie]

Nicht nur das!
Auch verursacht die Addition prinzipiell irgendwann einen Überlauf, welcher nicht kompensiert wird.

[Doc_Arduino]

Hallo,

@Rentner

static unsigned long TargetTime = startTime + (20*60*1000UL);

Ich würde mir angewöhnen immer das erste Literal zu casten. Sonst gehts irgendwann doch wieder schief.

[combie]

Oder alle mit dem Suffix versehen, denn kostet ja nix und sorgt dafür, dass man nicht auf die Operatoren Rangfolge achten muss.
(ein + ist schnell übersehen)

[laurin_e]

Irgendwie ist wirklich der Wurm drin. Wenn ich den Countdown auf z.B. 20Min stelle, zählt er in der Sekundenstelle jede zweite Minute in 10er Schritten von 100 runter, statt von 59 auf 0.

[combie]

Der Kopf ist rund, damit....

constexpr unsigned long interval = 20ul * 60ul * 1000ul;
unsigned long startTime;

void setup()
{
  Serial.begin(9600);
  unsigned long startTime = millis();
}

void loop()
{
  unsigned long difTime = millis() - startTime;
  if(difTime >= interval)
  {
    Serial.println("Zielzeit erreicht.");
    Serial.println("Restart");
    startTime = millis();
    difTime = 0;
  }

  unsigned long seconds = (interval - difTime) / 1000;
  unsigned long minutes = seconds / 60;
  seconds %= 60;
  minutes %= 60;

  Serial.print("minutes ");
  Serial.println(minutes);
  Serial.print("seconds ");
  Serial.println(seconds);
  Serial.println("..............");
  delay(500);
}

[laurin_e]

Danke combie für deinen Code! Der Interval soll ja variabel sein. Wie kann ich einer unsigned long Variabel das Suffix "ul" anhängen? Also damit alle Werte 32 Bit haben.

[combie]

@laurin_e

Wie kann ich einer unsigned long Variabel das Suffix "ul" anhängen?

Gar nicht!
Gibts auch keinen Grund für.

Das funktioniert nur an Literalen.
So stehts im Grundlagen Buch!
Und so ist es auch richtig.

[Doc_Arduino]

Hallo,

@laurin_e

Wie kann ich einer unsigned long Variabel das Suffix "ul" anhängen?

Geht nicht, sie ist ja schon Unsigned Long.
Literale sind default vom Datentyp int. Auf unseren 8Bit AVR Controllern int16_t.
Deswegen muss man aufpassen mit Formeln und Literalen nicht den Wertebereich zu überschreiten. Das Suffix legt den Datentyp vom Literal fest. Eben einen anderen als default int. Eine andere Möglichkeit sind cast Operatoren.

[laurin_e]

Dann ersetze ich 20ul mit meiner Minuten-Variable vom Typ unsigned long. Muss das constexpr weg?

[Doc_Arduino]

Hallo,

alles was veränderlich sein soll darf nicht konstant sein.

Man könnte aus:
constexpr unsigned long interval = 20ul * 60ul * 1000ul;

machen:
unsigned long factor {20};
constexpr unsigned long konstanterAusdruck {60ul * 1000ul};
unsigned long interval {factor * konstanterAusdruck};

oder so ähnlich. factor und interval sind damit zur Laufzeit veränderlich.

[laurin_e]

Moin zusammen

Hab den Code entsprechend dem Vorschlag von combie geändert. Es funktioniert soweit, bis auf dass die Anzeige auf dem Display suboptimal ist. Die Minuten und Sekunden "rutschen", d.h. z.B. jeweils von 9 Sekunden bis 0 Sekunden werden 90, 80, 70 usw. angezeigt. Hab schon versucht das Ganze mit sprintf und einem Buffer zu printen, funktioniert aber irgendwie nicht.
Hier die neue Funktion CountdownTime:

void CountdownTime() {

  unsigned long factor{ newPos };
  constexpr unsigned long constExpression{ 60ul * 1000ul };
  unsigned long interval {factor* constExpression};
  
  unsigned long difTime = millis() - startTime;
  if (difTime >= interval) {
    Serial.println("Zielzeit erreicht.");
    Serial.println("Restart");
    tft.setCursor(240, 210);
    tft.print("      ");
    startTime = millis();
    difTime = 0;
    Countdown = false;
  }
  unsigned long seconds = (interval - difTime) / 1000;
  unsigned long minutes = seconds / 60;
  seconds %= 60;
  minutes %= 60;


  tft.setCursor(240, 210);
  Serial.println("Interval:");
  Serial.println(interval);
  Serial.println("Minutes");
  Serial.println(minutes);
  tft.print(minutes);
  tft.print(":");
  Serial.println("Seconds");
  Serial.println(seconds);
  tft.print(seconds);
  //tft.print(".................");
}

[laurin_e]

Ich konnts nun doch lösen mit sprintf. Sorry!

[uwefed]

@laurin_e

Die Minuten und Sekunden "rutschen", d.h. z.B. jeweils von 9 Sekunden bis 0 Sekunden werden 90, 80, 70 usw. angezeigt.

Einfache Lösung ist bei zahlen kleiner 10 ein Leerzeichen vorher drucken.

Serial.print("minutes ");
if(minutes < 10) Serial.println(" ");
Serial.println(minutes);

Grüße Uwe

[combie]

@laurin_e

konnts nun doch lösen

Schön, dass es jetzt tut!

Wichtig war die TargetTime Berechnung.
Einmal die Addition dort, wg. dem drohnden den Überlauf, und die int Multiplikation mit ihrem eigenen Überlauf.

Weniger wichtig, und immer noch vorhanden, ist folgendes:
Die Funktion tut zuviel.

Grundsätzlich sollte eine Funktion nur eine Sache machen.
Dann sollte eine Funktion weder statische Variablen im Bauch haben, noch auf externe bzw. globale Daten zugreifen.
Alles was die Funktion braucht sollte man als Parameter übergeben.
Ergebnisse werden per return übergeben.
Rückgabe über Referenzen ist halbwegs OK, da man das anhand der Funktionssignatur halbwegs gut erkennen kann.

Die Gründe sind:

1. Testbarkeit
2. Übersicht
3. Wartbarkeit
4. Wiederverwendung

[laurin_e]

Hallo Leute
Ich hab nun doch noch ein neues Problem. Wenn ich den rotary encoder von 0 rückwärts, also gegen den Uhrzeiger drehe, steht plötlich 429496 aufm Display. Und diese hinteren Zahlen gehen auch nicht mehr weg, wenn ich wieder über 0 hochdrehe..

[my_xy_projekt]

42 ist immer die Lösung.

[Doc_Arduino]

Hallo,

ganz pauschal, du hast eine Wertebereichsüberschreitung. Das ist erstmal etwas völlig normales. Du musst für dich den gültigen Wertebereich festlegen und alles was drunter oder drüber liegt korrigieren oder begrenzen. Je nachdem was du mit den Werten machen möchtest.

Display. Das Verhalten ist auch normal. Du musst alle Ziffern überschreiben. Du weißt wieviele Digits maximal angezeigt werden. Diese Anzahl musst du immer schreiben. Damit "löschen" sich alte Zahlen wie von selbst. Komme nicht auf die Idee jedesmal das gesamte Display zu löschen.