einzelnes struct an structarray zurückgeben

[Marion (Gast)]

Hallo,

ich habe ein struct Array.

struct cHSV
{
  union {
    struct {
      union {
        uint8_t hue;
      uint8_t h; };
      union {
        uint8_t saturation;
        uint8_t sat;
      uint8_t s; };
      union {
        uint8_t value;
        uint8_t val;
      uint8_t v; };
    };
    uint8_t raw[3];
  };
};
const int NUM_LEDS = 10;
struct cHSV leds[NUM_LEDS];

Nun würde ich gerne mit einer Funktionen eine Led ändern.

leds[0] = setHSV(hue, sat, val);

struct cHSV setHSV( uint8_t hue, uint8_t sat, uint8_t val)
{
  struct cHSV hsv;
  hsv.hue = hue;
  hsv.hue = sat;
  hsv.hue = val;
  return hsv;
}

Aufgerufen werden soll das ganze so, innerhalb einer anderen Funktionen. 
Das geht leider nicht, die Leds bleiben unberührt.

leds[0] = setHSV(hue, sat, val);

[Marion (Gast)]

struct cHSV setHSV(uint8_t hue, uint8_t sat, uint8_t val)
{
  struct cHSV hsv[1];
  hsv[0].hue = hue;
  hsv[0].sat = sat;
  hsv[0].val = val;
  
  return *hsv;
}

So geht es. Kann man das ganze auch ohne, dass ich hsv[1] machen muss. 
Also nur hsv?

[Peter II (Gast)]

Marion schrieb:
> So geht es. Kann man das ganze auch ohne, dass ich hsv[1] machen muss.
> Also nur hsv?
was kommt denn für ein Fehler?

[Karl H. (kbuchegg)]

Marion schrieb:
>

> struct cHSV setHSV(uint8_t hue, uint8_t sat, uint8_t val)
> {
>   struct cHSV hsv[1];
>   hsv[0].hue = hue;
>   hsv[0].sat = sat;
>   hsv[0].val = val;
> 
>   return *hsv;
> }

>
> So geht es. Kann man das ganze auch ohne, dass ich hsv[1] machen muss.
> Also nur hsv?

Natürlich.
Das geht schon so, wie du das im Eröffnungsposting gezeigt hast.
Wenn nicht, liegt der Fehler im Code, den du nicht gezeigt hast.

[Karl H. (kbuchegg)]

Oops

struct cHSV setHSV( uint8_t hue, uint8_t sat, uint8_t val)
{
  struct cHSV hsv;
  hsv.hue = hue;
  hsv.hue = sat;
  hsv.hue = val;
  return hsv;
}

Lauter Zuweisungen an hsv.hue?

Fehler im Code oder Tippfehler im geposteten Ausschnitt, der so gar 
nicht in deinem Programm steht?

[Marion (Gast)]

Das war es, hatte beim c&p die Änderungen für hue vergessen. Somit wurde 
val auch nie gesetzt. val == 0 > Led AUS ;)

Danke

[Peter II (Gast)]

ich halte es aber nicht für sinnvoll stucts zu kopieren.

Warum nicht in die SetFunktion ein zeiger auf die struct mitgeben?

setHSV(leds, hue, sat, val);