Array in Struktur

Mehr Code!

Aber:

Das hier

1

struct LEDstruct {

2

  uint8_t Rot[61];

3

  uint8_t Gruen[61];

4

  uint8_t Blau[61];

5

};

ist höchst wahrscheinlich verkehrt rum aufgezogen. Denn der logische 
Zusammenhang zwischen Rot/Grün/Blau ist stärker als die Tatsache, dass 
es 61 Stück davon gibt. D.h. das sollte so sein

1

struct LEDstruct {

2

  uint8_t Rot;

3

  uint8_t Gruen;

4

  uint8_t Blau;

5

};

Diese Struktur beschreibt 1 RGB-Led. Und davon, also von den RGB-Leds, 
hast du 61 Stück.

1

struct LEDstruct myLeds[61];

myLeds[5] wären dann zb die Daten der 6ten Led. Jede einzelne LED 
besteht aus einer Rot-Komponente, einer Gruen Komponente und einer Blau 
Komponente. Die Rot-Komponente der 6-ten LED ist dann myLeds[5].Rot 
Jetzt liest sich das auch von links nach rechts in einer logisch 
sinnvollen Art und Weise vom Groben zur Detailinformation.

    myLeds            das sind alle Leds
    myLeds[5]         das ist eine einzelne aus dem Pool aller Leds
    myLeds[5].Blau    das ist eine Farbe einer einzelnen Led aus
                      dem Pool aller Leds.

Stukturen muss man auch sinnvoll einsetzen. Sonst bringen sie nichts.

Günter G. schrieb:
> Hallo Peter,
>
> das wird es vermutlich sein:

Nein.

Hör mit deiner Salamitaktik auf und zeig alles.

Günter G. schrieb:

> Wird durch struct LEDstruct* LEDPointer eine neue Kopie erzeugt?

Nein. Du brauchst ganz dringend ein C-Buch

Und gewöhn dich daran, dass wir in C

  for(i = 1;i<61;i++){

bei 0 anfangen zu zählen!

Ein Array mit 3 Elementen

int arr[3];

besteht aus den Elementen

  arr[0]
  arr[1]
  arr[2]

zähl nach, sind genau 3 Stück.

Günter G. schrieb:
> for(i = 1;i<61;i++){
>     LEDPointer->Rot[i] = 0;
>     LEDPointer->Gruen[i] = 0;
>     LEDPointer->Blau[i] = 0;
>   }

Macht also was du ihm sagst?

Und dein gezeigter Code hilft immer noch nicht zu erklären, was du unter

> Komischerweise werden alle nicht in der Schleife gesetzten
> LEDs gelöscht. Warum?

eigentlich genau verstehst, bzw. wie man dieses 'gelöscht' deuten soll. 
Was sind 'nicht alle'?

Günter G. schrieb:
> Komischerweise werden alle nicht in der Schleife gesetzten LEDs
> gelöscht. Warum?

Weil du vor der "Setzen-Schleife" eine "Komplett-Löschen-Schleife" hast.
Was also erwartest du?

Das ergibt sich doch ganz klar alles aus dem Code. Vielleicht. Also aus 
dem Teil des Codes, den es wohl nur nach Unterzeichnung eines NDA gibt. 
Was schlecht ist, wenn man etwas will.

> Bei 1 fange ich deshalb an, weil es bei den LEDs auch mit eins
> losgeht und ich diese Stolperfalle weghaben wollte.

Na wart mal ab.
Dieses 'bei 1 anfangen' ist die viel größere Stolperfalle.

Im übrigen bin ich raus. Ratespielchen kann ich mit meinem eigenen Code 
auch veranstalten. Den seh ich wenigstens komplett, auch wenn ich mir 
manchmal wünschen würde es wäre weniger.

Günter G. schrieb:
> Dann wird LED 30 und 31 hochgedimmt. Danach 29 und 32, da soll aber 30
> und 31 anbleiben, geht aber wieder aus.

Dann zeige die Definition von 'Helligkeit'. Vermutlich hast du da bei 
der Initialisierung nicht an deinen "Index-Offset" gedacht.

Günter G. schrieb:
> ja, im Code werden alle LEDs erstmal auf 0 gesetzt.

Okay, auch wenn ich die Verwendung von memset() erwägen würde. Oder 
clearAll()? Zumindest klingt das so.

> Dann wird LED 30 und 31 hochgedimmt. Danach 29 und 32, da soll aber 30
> und 31 anbleiben, geht aber wieder aus.

Und dafür soviel Rumgeschwätze im Code?
Hättest du wie Karl-Heinz schon sagte ein Array von Strukturen, könntest 
du auch das mit memset() erledigen. Und anstatt zwei Variablen 
unabhängig voneinander zu zählen, könnte dir auffallen, das etwas wie

1

for (i = 0; i < 30; ++i) {

2

  LEDPointer->Rot[30 - i] = ...

3

  LEDPointer->Rot[31 + i] = ...

4

}

dein Ansinnen ohne jeden weiteren Kommentar und zweiten Blick 
dokumentiert hätte.

> Bei 1 fange ich deshalb an, weil es bei den LEDs auch mit eins losgeht
> und ich diese Stolperfalle weghaben wollte.

Um dich nicht am Kaffee zu verbrennen, trinkst du ihn auch immer kalt? 
Das ist doch Käse, mal davon abgesehen, dass du so pro Farbei ein Byte 
(Index 0) verschwendest.

Apropos verschwenden: wieviel SRAM hat denn dein Controller? Passen da 3 
mal 61 Byte plus Stack plus was noch alles im geheimen Restcode 
verwendet wird rein? Oder wird vielleicht das Array durch eine der 
unzähligen lokalen Variablen überschrieben - wobei die wegoptimiert 
werden sollten, aber deine Compilerflags kennen wir ja auch nicht.