muethos76/rheinturm

rheinturm's Issues

ledstrip.ino

https://github.com/MueThoS76/Rheinturm/blob/master/ledstrip.ino#L3
Hier würde ich definitiv mit Variablen arbeiten:

for (int i = LED_SEKUNDEN_EINER_MIN; i <= LED_SEKUNDEN_EINER_MAX; i++) {
    if (i - LED_SEKUNDEN_EINER_MIN < sekunden.einer) {

Das klappt dann bei allen anderen LEDs auch so.

---

https://github.com/MueThoS76/Rheinturm/blob/master/ledstrip.ino#L5
Hier solltest du zentral die Farben speichern:

// Im Hauptprogramm, ACHTUNG: ich weiß nicht ob `int` hier richtig ist.
int LED_ON  = PIXELS_CLOCK.Color(100,80,0);
int LED_OFF = PIXELS_CLOCK.Color(0,0,0);

// In ledstrip.ino
if (i - LED_SEKUNDEN_EINER_MIN < sekunden.einer) {
  PIXELS_CLOCK.setPixelColor(i, LED_ON);
} else {
  PIXELS_CLOCK.setPixelColor(i, LED_OFF);
}

uhr.ino

Rheinturm/uhr.ino

Lines 11 to 12 in fffa252

	} while (((millis() - start) <= (1000 * sec)) && (timeinfo.tm_year < (2016 - 1900)));
	if (timeinfo.tm_year <= (2016 - 1900)) return false; // the NTP call was not successful

Hier würde ich ein Makro schreiben, damit die Gültigkeitsprüfung auf die erhaltene Zeitinfo deutlich wird.

#define TIMEINFO_INVALID (timeinfo.tm_year <= (2016 - 1900))

Und dann:

} while (((millis() - start) <= (1000 * sec)) && TIMEINFO_INVALID);
if (TIMEINFO_INVALID) return false;  // the NTP call was not successful

---

Rheinturm/uhr.ino

Line 19 in fffa252

sekunden.zehner = (timeinfo.tm_sec / 10) % 10;

Das würde ich explizit casten:

sekunden.zehner = (unsigned short)(timeinfo.tm_sec / 10);