Werte einem Array zuweisen

[Stefan (Gast)]

Ich habe ein kleines Programm, welches sich werte für das Array aus 
einer Switch-Anweisung holt.

void blinklicht(unsigned long wartezeit, unsigned int modus) {
  unsigned long aktuelleZeit = millis();
  static unsigned long vorherigeZeit;
  static int j;
  int rgbModus[7];

  switch(modus) {
  case 0: 
    rgbModus[] = {
      0, 2, 4, 6, 1, 3, 5, 7                          }; // RB RB RB RB
    break;
  case 1: 
    rgbModus[]  = {
      1, 3, 5, 7, 0, 2, 4, 6                          }; // BR BR BR BR
    break;
  case 2: 
    rgbModus[]  = {
      0, 1, 4, 5, 2, 3, 6, 7                          }; // RR BB RR BB
    break;
  case 3: 
    rgbModus[]  = {
      2, 3, 6, 7, 0, 1, 4, 5                          }; // BB RR BB RR
    break;

  case 4: 
    rgbModus[]    = {
      0, 1, 2, 3, 4, 5, 6, 7                          }; // RR RR BB BB
    break;
  case 5: 
    rgbModus[]    = {
      4, 5, 6, 7, 0, 1, 2, 3                          }; // BB BB RR RR
    break;

  case 6: 
    rgbModus[]    = {
      2, 3, 4, 5, 0, 1, 6, 7                          }; // BB RR RR BB
    break;
  case 7: 
    rgbModus[]    = {
      0, 1, 6, 7, 2, 3, 4, 5                          }; // RR BB BB RR
    break;

  default:
    modus = 0;
  }

  if(aktuelleZeit - vorherigeZeit > wartezeit) {
    j = 1;
    if(aktuelleZeit - vorherigeZeit > wartezeit * 2) {
      j = 0;
      vorherigeZeit = aktuelleZeit;
    }
  }

  if(j == 1)
  {
    for(int k = 0; k < 4; k++) ShiftPWM.SetRGB(rgbModus[k], 255, 0, 0);
    for(int k = 4; k < 8; k++) ShiftPWM.SetRGB(rgbModus[k], 0, 0, 255); 
  }
  else
  {
    for(int k = 0; k < 4; k++) ShiftPWM.SetRGB(rgbModus[k], 255, 0, 0);
    for(int k = 4; k < 8; k++) ShiftPWM.SetRGB(rgbModus[k], 0, 0, 255);   
  }
}

sketch_aug22a:52: error: expected primary-expression before ']' token
sketch_aug22a:52: error: expected primary-expression before '{' token
sketch_aug22a:52: error: expected `;' before '{' token

[Kindergärtner (Gast)]

Warum der Aufwand, jedes mal das komplette Array zu kopieren?!

static const int rgbModus [][] = {{ 0, 2, 4, 6, 1, 3, 5, 7}, {1, 3, 5, 7, 0, 2, 4, 6 }, /* .... */ };
  // ...
  if(j == 1)
  {
    for(int k = 0; k < 4; k++) ShiftPWM.SetRGB(rgbModus[modus][k], 255, 0, 0);
    for(int k = 4; k < 8; k++) ShiftPWM.SetRGB(rgbModus[modus][k], 0, 0, 255); 
  }
  else
  {
    for(int k = 0; k < 4; k++) ShiftPWM.SetRGB(rgbModus[modus][k], 255, 0, 0);
    for(int k = 4; k < 8; k++) ShiftPWM.SetRGB(rgbModus[modus][k], 0, 0, 255);   
  }

[Markus B. (markusborti)]

Grüß dich,

du hast das Array für sieben Elemente definiert, fügst aber immer 8 ein.
Das müsste eigentlich zu einem Overflow führen.

Den Tip von Kindergärtner finde ich prima.

Schöne Grüße

[Kindergärtner (Gast)]

Markus Borti schrieb:
> du hast das Array für sieben Elemente definiert, fügst aber immer 8 ein.
> Das müsste eigentlich zu einem Overflow führen.
Nein, das ist in C schlicht und einfach nicht definiert. Man kann 
keine Arrays zuweisen/kopieren.

[Stefan (Gast)]

void blinklicht(unsigned long wartezeit, unsigned int modus) {
  unsigned long aktuelleZeit = millis();
  static unsigned long vorherigeZeit;
  static int j;

  static int rgbModus[][8] = {
    {
      0, 2, 4, 6, 1, 3, 5, 7        }
    , // RB RB RB RB
    {
      1, 3, 5, 7, 0, 2, 4, 6        }
    , // BR BR BR BR
    {
      0, 1, 4, 5, 2, 3, 6, 7        }
    , // RR BB RR BB
    {
      2, 3, 6, 7, 0, 1, 4, 5        }
    , // BB RR BB RR    
    {
      0, 1, 2, 3, 4, 5, 6, 7        }
    , // RR RR BB BB
    {
      4, 5, 6, 7, 0, 1, 2, 3        }
    , // BB BB RR RR
    {
      2, 3, 4, 5, 0, 1, 6, 7        }
    , // BB RR RR BB
    {
      0, 1, 6, 7, 2, 3, 4, 5        }
    , // RR BB BB RR
  };
  
  if(modus < 8) {
    if(aktuelleZeit - vorherigeZeit > wartezeit) {
      j = 1;
      if(aktuelleZeit - vorherigeZeit > wartezeit * 2) {
        j = 0;
        vorherigeZeit = aktuelleZeit;
      }
    }

    if(j == 1)
    {
      for(int k = 0; k < 4; k++) ShiftPWM.SetRGB(rgbModus[modus][k], 255, 0, 0);
      for(int k = 4; k < 8; k++) ShiftPWM.SetRGB(rgbModus[modus][k], 0, 0, 255); 
    }
    else
    {
      for(int k = 0; k < 4; k++) ShiftPWM.SetRGB(rgbModus[modus][k], 0, 0, 255);
      for(int k = 4; k < 8; k++) ShiftPWM.SetRGB(rgbModus[modus][k], 255, 0, 0);   
    }
  }
  else {
    Serial.print("Falsche Eingabe! Wert '"); 
    Serial.print(modus); 
    Serial.println("' darf nicht hoeher '8' sein!");
  }
}

So passt es nun, die Frage ist, warum "  static int rgbModus[][8] = {" 
nötig ist. Wir hier wieder die Zahl von 1-8 angegeben?

Bei [7] bekomme ich mecker vom Compiler das zuviele Werte vorhanden 
sind.

[Timmo H. (masterfx)]

Stefan schrieb:
> So passt es nun, die Frage ist, warum "  static int rgbModus[][8] = {"
> nötig ist. Wir hier wieder die Zahl von 1-8 angegeben?
Ich weiß zwar jetzt nicht genau was du meinst, aber bei der Deklaration 
des Arrays wird die Größe angegeben (8 Elemente), die Indizierung geht 
aber von 0-7. Wenn du von 1-8 haben willst musst du Basic als 
Programmiersprache nehmen ;)

[Kindergärtner (Gast)]

Stefan schrieb:
> So passt es nun, die Frage ist, warum "  static int rgbModus[][8] = {"
> nötig ist. Wir hier wieder die Zahl von 1-8 angegeben?
Jedes der "Unter-Arrays" muss gleich groß sein, und damit C weiß wie 
groß, muss man es angeben.
Das "static const" sorgt dafür dass die Daten nicht jedes Mal in den 
lokalen Speicher (Stack) kopiert werden müssen, sondern 1x in den Flash 
geschrieben werden und von da direkt verwendet.