Arduino - Syntax geändert?

[Thomas G. (Firma: Frickelhauptquartier) (taximan)]

Ich habe hier ein Script, dass ich ca 1,5 Jahre nicht bearbeitet habe. 
Der Compiler meckert bei dieser (und vieler ähnlicher Zeilen), bricht 
aber nicht ab.
void prg0(){  // alles aus
  for(int i=0;i<=96;i++)
{leds[i] = CRGB::Black; }
FastLED.show();
}

C:\Users\TG\Documents\Arduino\lighthouse04\lighthouse04.ino:3058:16: 
note: within this loop
 3058 |   for(int i=0;i<=96;i++)
      |               ~^~~~

Was ist hier los?
Ich bin mir ziemlich sicher, dass diese Meldung(en) früher nicht kamen.
Verwendete IDE 1.8.13

[Wastl (hartundweichware)]

Thomas G. schrieb:
> Der Compiler meckert bei dieser (und vieler ähnlicher Zeilen), bricht
> aber nicht ab.

Vielleicht zeigst du mal die vollständige Meckerzeile. Sonst
gibt's fröhliches Rätselraten.

[MaWin O. (mawin_original)]

>Frickelhauptquartier

Der Name ist wohl Programm.

[Paul B. (paule201)]

Dein Array in der Schleife ist zu klein.

[Harald K. (kirnbichler)]

Mit wievielen Elementen ist das Array leds[] deklariert?

Ich tippe auf 96.

Und Deine Schleife beschreibt 97.

[Thomas G. (Firma: Frickelhauptquartier) (taximan)]

Das war es wohl, danke euch!

[Joachim B. (jar)]

aber warum <= wenn du nur 96 Elemente hast? (int i=0;i<=96)

[Paul B. (paule201)]

Joachim B. schrieb:
> aber warum <= wenn du nur 96 Elemente hast? (int i=0;i<=96)

Sein Array ist LED[96] = LED[0] bis LED[95]

Sein Schleife schreibt aber 97 Elemente. LED[0] bis LED[96]--> das merkt 
der Compiler --> Compiler Warnung.

[Thomas G. (Firma: Frickelhauptquartier) (taximan)]

Wobei die '96' durchgerutscht ist; in vielen anderen Zeilen steht 
richtig '95'.

[Dirk B. (dirkb2)]

Thomas G. schrieb:
> Wobei die '96' durchgerutscht ist; in vielen anderen Zeilen steht
> richtig '95'.

Magic Numbers sollte man vermeiden.

[Joachim B. (jar)]

Dirk B. schrieb:
> Magic Numbers sollte man vermeiden.

also besser?

#define MAXLED 96
for(int i=0;i<MAXLED;i++)

[Arduino F. (Firma: Gast) (arduinof)]

Joachim B. schrieb:
> also besser?

Nein!


So geht das:

void prg0() //alles aus
{ 
  for(auto &led:leds) led = CRGB::Black;
  FastLED.show();
}

Merke, ohne Index, kein Indexfehler!

Joachim B. schrieb:
> #define MAXLED 96
Define ist hier unnötig.

[Frank M. (ukw) (Moderator)]

Joachim B. schrieb:
> also besser?
> #define MAXLED 96
> for(int i=0;i<MAXLED;i++)

Ja, natürlich. Gerade, wenn die Konstante an zig Stellen im Source 
vorkommt. Willst Du jedesmal mehrere Dutzend Zahlen austauschen, wenn Du 
eine LED neu dranlötest?

[Arduino F. (Firma: Gast) (arduinof)]

Frank M. schrieb:
> Ja, natürlich.
Nöö..
constexpr int anzahlLed {96};