arrays und struct in funktionen

[Tobias N. (silberkristall)]

Hallo,

wie kriege ich die Inhalte von struct und arrays innerhalb einer 
funktion möglichst unumständlich geändert!?

Als beispiel wollte ich für meine LEDs (ws2812b) die Lightweight Library 
verwenden. Meine eigene Lib läuft zwar aber nicht so schön ;)

Soweit so gut. Nun da ich es gerne "einfach" habe wollte ich die Lib um 
eine weitere Funktion erweitern.

In der Lib wird folgendes genutzt:

struct CRGB { uint8_t g; uint8_t r; uint8_t b; };
struct CRGB led[255];

Nun lässt sich das ganze ja wie folgt bearbeiten:

led[1].r = 100;
led[1].g = 100;
led[1].b = 100;

Soweit sogut. Das ist mir aber zu umständlich. Ich hätte gerne zwei 
weitere Funktionen wie folgt:

#define LEDANZAHL 255

struct CRGB { uint8_t g; uint8_t r; uint8_t b; };
struct CRGB led[LEDANZAHL];

// Und nun folgende Funktionen:


// Einmal für eine LED

void writeoneledcolor(int lednummer, int rot, int gruen, int blau)
{

lednummer--;

led[lednummer].r = 100;
led[lednummer].g = 100;
led[lednummer].b = 100;
}

// Und für alle LEDs

void writeallledcolor(int rot, int gruen, int blau)
{
   for (i=0; i<LEDANZAHL; i++)
   {
      led[i].r = rot;
      led[i].g = gruen;
      led[i].b = blau;
   }
}

Nur leider will das nicht so ganz. Die Funktionen können weder auf das 
led struct/array zugreifen noch auf LEDANZAHL.

Pointer etc übergeben ist genau so blöd da man dann wieder mehr angeben 
muss. Es muss doch eine Möglichkeit geben das ganze so umzusetzen.

Danke euch.

[Uwe (Gast)]

Hallo Tobias,

Zeiger auf einen Datenbereich ist nicht unüblich !
Man kann sich etwas Scheibarbeit durch geschickte MACRO Definitionen 
sparen.

Aber sonst, eher weniger.

[Tobias N. (silberkristall)]

Nunja, nur so einen aufruf wie

writeallledcolor(100,100,100,&r, &g, &b);

will ich ja vermeiden.

Also klar das C nicht/nur schwer mit anderen Sprachen vergleichbar ist. 
Aber bei PHP z.b. würde das so ohne Probleme funktionieren.

Also da auf ein Array innerhalb einer funktion zuzugreifen ist standart.

[holger (Gast)]

>Nur leider will das nicht so ganz.

Und was will da nicht? Eigentlich müsste das gehen.

Auch wenn du da ordentlich Murkskram machst.

>void writeallledcolor(int rot, int gruen, int blau)

Was soll die int Scheisse da?
In deinem struct ist alles uint8_t.
Dann mach die Parameter auch uint8_t.

[Easylife (Gast)]

Es sind halt 3 kleine Fehler drin:

- Die Deklaration von i in den Funktionen fehlt jeweils
- writeoneledcolor() setzt .r, .g, .b auf 100, 100, 100 statt auf rot, 
grün, blau

Ansonsten ist alles korrekt und funktioniert auch.
Die Funktionen können auf das globale Array zugreifen.
Was funktioniert denn bei dir nicht?

[Tobias N. (silberkristall)]

Wenn ich das so compilieren will dann mekkert er rum das led in der 
funktion nicht deklariert ist und sowas

[Rufus Τ. F. (rufus)]

Tobias N. schrieb:
> Wenn ich das so compilieren will dann mekkert er rum das led in der
> funktion nicht deklariert ist und sowas

"Er" meckert, weil Du die Dinge, die Du da verwenden willst, 
irgendwoanders deklariert haben wirst. Und wo hast Du das?

In irgendeiner anderen Quelltextdatei, richtig?

Die Themen wo deklariert man Dinge und wie funktionieren include - 
Dateien haben wir hier praktisch jede Woche.

[Easylife (Gast)]

Beweis:

#include <stdio.h>
typedef unsigned char uint8_t;

#define LEDANZAHL 255

struct CRGB { uint8_t g; uint8_t r; uint8_t b; };
struct CRGB led[LEDANZAHL];

// Und nun folgende Funktionen:
// Einmal für eine LED

void writeoneledcolor(uint8_t lednummer, uint8_t rot, uint8_t gruen, uint8_t blau)
{
  int i;
  lednummer--;
  
  led[lednummer].r = rot;
  led[lednummer].g = gruen;
  led[lednummer].b = blau;
}

// Und für alle LEDs

void writeallledcolor(uint8_t rot, uint8_t gruen, uint8_t blau)
{
  int i;
  for (i=0; i<LEDANZAHL; i++)
  {
    led[i].r = rot;
    led[i].g = gruen;
    led[i].b = blau;
  }
}

int main()
{
  int i;
  writeallledcolor(1,2,3);
  writeoneledcolor(2,100,150,200);

  for(i=0; i<5; i++) 
  {
    printf("%d: %d, %d, %d\n", i, led[i].r, led[i].g, led[i].b);
  }
}

Ausgabe:

0: 1, 2, 3
1: 100, 150, 200
2: 1, 2, 3
3: 1, 2, 3
4: 1, 2, 3

[Tobias N. (silberkristall)]

Nein. Ich habe das alles in die c und h datei der Lightweight lib 
geschrieben wo auch alle anderen Funktionen der lib sind.

[holger (Gast)]

>Wenn ich das so compilieren will dann mekkert er rum das led in der
>funktion nicht deklariert ist und sowas

Kompletten Sourcecode zeigen. Und zwar das gesamte Projekt.
Sonst reden sich hier wieder viele den Mund fusselig.

[Tobias N. (silberkristall)]

Ok, moment.

[Tobias N. (silberkristall)]

So habe ich das derzeit compiliert. Die funktionen wie oben geschrieben 
hatte ich in der lightws.c und lightws.h eingefügt und dann klappt es 
halt nicht.

Das ist jetzt das gesamte programm was zurzeit läuft.

[holger (Gast)]

>Das ist jetzt das gesamte programm was zurzeit läuft.

Und was soll man jetzt damit? Ich will das beschissene
Programm das NICHT läuft. Und bitte als ZIP
und nicht alle einzeln.

[Easylife (Gast)]

Da finde ich deine Funktionen jetzt aber nicht...
Naja,

mach einfach folgendes:

im Headerfile:

#define LEDANZAHL 255
typedef struct { uint8_t g; uint8_t r; uint8_t b; } CRGB_t;

in der Quelle, in der du das Array beschreiben/lesen willst:

#include "header.h"
(...)
CRGB_t led[LEDANZAHL];
(...)

[Nop (Gast)]

Mir fehlt irgendwie ein typedef der struct...

[Tobias N. (silberkristall)]

Ich hatte die Funktionen wie oben in die lightws.c/h geschrieben und 
dann versucht in der main.c aufzurufen

[holger (Gast)]

>Ich hatte die Funktionen wie oben in die lightws.c/h geschrieben und
>dann versucht in der main.c aufzurufen

Dann pack mal ein

extern  struct cRGB led[];

in lightws.h. Woher soll lightws.c denn wissen das dein
Array in main.c deklariert wurde?