Arduino - Syntax geändert?

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.

Frickelhauptquartier

Der Name ist wohl Programm.

Dein Array in der Schleife ist zu klein.

Mit wievielen Elementen ist das Array leds[] deklariert?

Ich tippe auf 96.

Und Deine Schleife beschreibt 97.

Das war es wohl, danke euch!

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

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.

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

Thomas G. schrieb:

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

Magic Numbers sollte man vermeiden.

Dirk B. schrieb:

Magic Numbers sollte man vermeiden.

also besser?

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

Joachim B. schrieb:

also besser?

Nein!

So geht das:

1

void prg0() //alles aus

2

{ 

3

  for(auto &led:leds) led = CRGB::Black;

4

  FastLED.show();

5

}

Merke, ohne Index, kein Indexfehler!

Joachim B. schrieb:

#define MAXLED 96

Define ist hier unnötig.

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?

Frank M. schrieb:

Ja, natürlich.

Nöö.. constexpr int anzahlLed {96};