struct als array an funktion übergeben.

Mathias G. (motze) 06.03.2015 23:26 #4042568

Lesenswert?

•

hallo liebe Gemeinde,

ich möchte gerne ein struct an einer Funktion übergeben. Leider 
funktioniert das nicht so wie geplant

struct LED
  uint8_t gruen;
  uint8_t rot;
  uint8_t blau;
struct LED led[60];

void output_send_led(struct LED _led[], uint8_t _anzahl_led)
  uint8_t i;
  uint8_t a;
  for (a = 0; a < _anzahl_led; a++)
    for (i = 0; i < 8; i++)
      if(_led[a].gruen & 0x80)
        output_eins();
      else
        output_null();
      _led[a].gruen<<=1;
    for (i = 0; i < 8; i++)
      if(_led[a].rot & 0x80)
        output_eins();
      else
        output_null();
      _led[a].rot<<=1;
    for (i = 0; i < 8; i++)
      if(_led[a].blau & 0x80)
        output_eins();
      else
        output_null();
      _led[a].blau<<=1;
int main(void)
  DDRC |= (1<<PC0);
  led[2].blau = 0xFF;
  led[2].rot = 0xFF;
  led[2].gruen = 0xFF;
    while(1)
    output_send_led(led, 10);
    output_reset();


vielen dank für eure Hilfe.

Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Beitrag melden

Falk B. (falk) 06.03.2015 23:33 #4042576

Lesenswert?

•

▲
▼

@ Mathias G. (motze)

>ich möchte gerne ein struct an einer Funktion übergeben. Leider
>funktioniert das nicht so wie geplant.

Kann man machen, ist aber eher unüblich. Solche großen Massen an Daten 
will man nicht kopieren, sondern übergibt Zeiger.

Da fehlt wohl ein typedef. Eher so.

typedef struct
  uint8_t gruen;
  uint8_t rot;
  uint8_t blau;
LED_t led[60];
void output_send_led(LED_t *_led, uint8_t _anzahl_led)
  uint8_t i;
  uint8_t a;
  for (a = 0; a < _anzahl_led; a++)
    for (i = 0; i < 8; i++)
      if(_led[a].gruen & 0x80)
        output_eins();
      else
        output_null();
      _led[a].gruen<<=1;
    for (i = 0; i < 8; i++)
      if(_led[a].rot & 0x80)
        output_eins();
      else
        output_null();
      _led[a].rot<<=1;
    for (i = 0; i < 8; i++)
      if(_led[a].blau & 0x80)
        output_eins();
      else
        output_null();
      _led[a].blau<<=1;
int main(void)
  DDRC |= (1<<PC0);
  led[2].blau = 0xFF;
  led[2].rot = 0xFF;
  led[2].gruen = 0xFF;
    while(1)
    output_send_led(led, 10);
    output_reset();

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Mathias G. (motze) 06.03.2015 23:41 #4042581

Lesenswert?

•

▲
▼

erstmal ganz herzlichen dank für deine Hilfe leider bekomme ich noch 
eine Fehlermeldung:

array type has incomplete element type

was möchte mir den der Compiler damit sagen?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Falk B. (falk) 06.03.2015 23:45 #4042583

Lesenswert?

•

▲
▼

Möglicherweise fehlt da ein

#include <stdint.h>

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Mathias G. (motze) 06.03.2015 23:48 #4042586

Lesenswert?

•

▲
▼

nein leider nicht die Fehlermeldung bleibt die gleiche.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Falk B. (falk) 06.03.2015 23:50 #4042592

Lesenswert?

•

▲
▼

Dann sind die Dinge oben im Quelltext nicht so in deinem ECHTEN 
Quelltext, sondern in anderer Reihenfolge oder verteilt auf Dateien?

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Patrick D. (oldbug) 06.03.2015 23:53 #4042596

Lesenswert?

•

▲
▼

Irgendwas in diese Richtung wird es aber sein!

OT: wie kommt man eigentlich auf die Idee, irgend einen Namen mit 
Unterstrich[en] am Anfang zu versehen? Lass das...

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Mathias G. (motze) 07.03.2015 00:03 #4042610

Lesenswert?

•

▲
▼

erstmal danke für eure Antwort. Ihr hattet recht es war nicht richtig in 
Quelltext. Jetzt wird es ohne Fehler Compiliert.

vielen dank dafür

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Mathias G. (motze) 07.03.2015 01:43 #4042674

Lesenswert?

•

▲
▼

jetzt habe ich leider ein anderes Problem der folgende Code Funktioniert 
genau einmal danach nicht mehr woran kann das den liegen?

void output_reset()
  _delay_us(60);
void output_alle(uint8_t _led[])
  uint8_t i;
  uint8_t k;
  for (i = 0; i < 180; i++)
    for (k = 0; k < 8; k++)
      if(_led[i] & 0x80)
        output_eins();
      else
        output_null();
      _led[i]<<=1;  
uint8_t Led_60[180] =
  0x00, 0xFF, 0xFF, //1
  0x00, 0xFF, 0xFF, //2
  0x00, 0xFF, 0xFF, //3
  0x00, 0xFF, 0xFF, //4
  0x00, 0xFF, 0xFF, //5
  0x00, 0xFF, 0xFF, //6
  0x00, 0xFF, 0xFF, //7
  0x00, 0xFF, 0xFF, //8
  0x00, 0xFF, 0xFF, //9
  0x00, 0xFF, 0xFF, //10
  0xFF, 0xFF, 0x00, //11
  0xFF, 0xFF, 0x00, //12
  0xFF, 0xFF, 0x00, //13
  0xFF, 0xFF, 0x00, //14
  0xFF, 0xFF, 0x00, //15
  0xFF, 0xFF, 0x00, //16
  0xFF, 0xFF, 0x00, //17
  0xFF, 0xFF, 0x00, //18
  0xFF, 0xFF, 0x00, //19
  0xFF, 0xFF, 0x00, //20
  0xFF, 0xFF, 0xFF, //21
  0xFF, 0xFF, 0xFF, //22
  0xFF, 0xFF, 0xFF, //23
  0xFF, 0xFF, 0xFF, //24
  0xFF, 0xFF, 0xFF, //25
  0xFF, 0xFF, 0xFF, //26
  0xFF, 0xFF, 0xFF, //27
  0xFF, 0xFF, 0xFF, //28
  0xFF, 0xFF, 0xFF, //29
  0xFF, 0xFF, 0xFF, //30
  0xFF, 0xFF, 0xFF, //31
  0xFF, 0xFF, 0xFF, //32
  0xFF, 0xFF, 0xFF, //33
  0xFF, 0xFF, 0xFF, //34
  0xFF, 0xFF, 0xFF, //35
  0xFF, 0xFF, 0xFF, //36
  0xFF, 0xFF, 0xFF, //37
  0xFF, 0xFF, 0xFF, //38
  0xFF, 0xFF, 0xFF, //39
  0xFF, 0xFF, 0xFF, //40
  0xFF, 0xFF, 0xFF, //41
  0xFF, 0xFF, 0xFF, //42
  0xFF, 0xFF, 0xFF, //43
  0xFF, 0xFF, 0xFF, //44
  0xFF, 0xFF, 0xFF, //45
  0xFF, 0xFF, 0xFF, //46
  0xFF, 0xFF, 0xFF, //47
  0xFF, 0xFF, 0xFF, //48
  0xFF, 0xFF, 0xFF, //49
  0xFF, 0xFF, 0xFF, //50
  0xFF, 0xFF, 0xFF, //51
  0xFF, 0xFF, 0xFF, //52
  0xFF, 0xFF, 0xFF, //53
  0xFF, 0xFF, 0xFF, //54
  0xFF, 0xFF, 0xFF, //55
  0xFF, 0xFF, 0xFF, //56
  0xFF, 0xFF, 0xFF, //57
  0xFF, 0xFF, 0xFF, //58
  0xFF, 0xFF, 0xFF, //59
  0x00, 0xFF, 0x00 //60  
int main(void)
  DDRC |= (1<<PC0);
    while(1)
    output_alle(Led_60);
    output_reset();
    _delay_ms(5000);


ich weiß das mit den unterstrichen ist doof ich werde es mir auch 
abgewöhnen.

Also die Leds leuchten genau 5 Sekunden auf und danach geht nix mehr.
Output_reset hat bei anderen Sachen immer hingehauen mit Timing.

vielen dank schonmal

Antwort Zitieren Bearbeiten Löschen Beitrag melden

foobar Gast 07.03.2015 07:33 #4042730

Lesenswert?

•

▲
▼

> Also die Leds leuchten genau 5 Sekunden auf und danach geht nix mehr.

>     _led[i]<<=1;


Kein Wunder, nach einem Durchlauf von output_alle sind alle Werte in 
deinem led-Array 0.

Wenn du die Werte erhalten möchtest:

void output_alle(uint8_t led[])
    uint8_t i, k;
    for (i = 0; i < 180; i++)
        for (k = 0x80; k; k >>= 1)
            if(led[i] & k)
                output_eins();
            else
                output_null();

Antwort Zitieren Bearbeiten Löschen Beitrag melden

(prx) A. K. (prx) 07.03.2015 07:49 #4042744

Lesenswert?

•

▲
▼

Falk Brunner schrieb:
> Solche großen Massen an Daten
> will man nicht kopieren, sondern übergibt Zeiger.

Genau das tut er. Es gibt in C keine Array-Übergabe per Wert, nur per 
Referenz. Sein

> void output_send_led(LED_t _led[], uint8_t _anzahl_led)

ist nur eine andere Schreibweise von deinem

> void output_send_led(LED_t *_led, uint8_t _anzahl_led)

Antwort Zitieren Bearbeiten Löschen Beitrag melden

(prx) A. K. (prx) 07.03.2015 07:59 #4042752

Lesenswert?

•

▲
▼

Falk Brunner schrieb:
> Da fehlt wohl ein typedef. Eher so.

Ob man
  struct LED { ... }
schreibt und im weiteren Code stets
  struct LED
verwendet, oder erst
  typedef struct { ... } LED
und dann dafür
  LED
ist gleich. Seine Variante wirkt nur etwas altbacken. Man kann auch
  typedef struct LED { ... } LED
schreiben, und dann beides verwenden.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Falk B. (falk) 07.03.2015 11:58 #4042928

Lesenswert?

•

▲
▼

@ A. K. (prx)


>ist nur eine andere Schreibweise von deinem

>> void output_send_led(LED_t *_led, uint8_t _anzahl_led)

Hmm, stimmt. Es war schon zu spät.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

Mathias G. (motze) 07.03.2015 12:31 #4042962

Lesenswert?

•

▲
▼

Vielen dank für die Hilfe jetzt geht es so wie ich es will. Manchmal ist 
man echt blind. Klar wenn ich die alle Rausschifte das nur noch nullen 
übrig geblieben sind.

vielen dank nochmal.

Antwort Zitieren Bearbeiten Löschen Beitrag melden

struct als array an funktion übergeben.

Antwort schreiben