Makro zum Initialisieren von Werten in struct

[ruud86 (Gast)]

Hallo hab mal ne Frage,

nur so zur persönlichen Weiterbildung

Beispiel:

#define red  ???

typedef struct RGB_struct
{
  uint8_t value_r;
  uint8_t value_g;
  uint8_t value_b;
} RGB_t;

int main(){
RGB_t RGB= red;  // RGB Werte für rot in struct

Wie müsste das define aussehen damit die Farbwerte für rot im struct RGB 
stehen?


Gruß

Adrian

[Marco M. (marco_m)]

Je nach deiner Interpretation der Farbwerte z.B. so:

#define red  {255,0,0}

[ruud86 (Gast)]

Hab ich schon probiert, beim umspeichern bekomme ich dann Probleme

RGB_t RGB={10, 50, 90};
RGB=red;

Fehlermeldung für die zweite Zeile:

> Error  1  expected expression before '{' token

Was mache ich in dem Fall falsch?

[Karl H. (kbuchegg)]

ruud86 schrieb:
> Hab ich schon probiert, beim umspeichern bekomme ich dann Probleme
>
>

> RGB_t RGB={10, 50, 90};
> RGB=red;
>

>
> Fehlermeldung für die zweite Zeile:
>
>> Error  1  expected expression before '{' token
>
> Was mache ich in dem Fall falsch?

Logisch.
Makros sind ja keine Zauberstäbe.
Makros machen Textersetzung. Nicht mehr.

#define red  { 10, 50, 90 }

RGB_t RGB = red;


ist deswegen ok, weil die Syntax zur Initialisierung eines struct nun 
mal so aussieht

RGB_t RGB = { 10, 50, 90 };

Ob du jetzt den Klammerteil direkt hinschreibst, oder ob du den durch 
ein Text Suchen&Ersetzen vom Präprozesser einsetzen lässt, ist dem 
Compiler ja wurscht. Der kriegt nur das Endergebnis (die letzte Form) zu 
sehen und das ist syntaktisch korrekt.

Wohingehend die Zuweisung

  RGB = { 10, 50, 90 };

syntaktisch nicht korrekt ist. Und daran ändert sich auch nichts, wenn 
du den Klammerteil nicht direkt hinschreibst, sondern hinter einer 
Textersetzung in Form eines Makros versteckst.


Makros sind keine Wunderwuzzi. Sie machen nur Textersetzungen. Das was 
bei dieser Textersetzung herauskommt, muss syntaktisch korrekt sein. 
Daran ändert auch ein Makro nichts.

[ruud86 (Gast)]

Ah stimmt danke, da war mal was...

Das kommt davon,wenn zu lange mit Labview arbeitet.

Gibt es eine Möglichkeit das Makro zu schreiben, dass ein umspeichern 
geht?

Danke schon Mal für die Nachhilfe ;)

Gruß

[Marco M. (marco_m)]

ruud86 schrieb:
> Hab ich schon probiert, beim umspeichern bekomme ich dann Probleme
>
>

> RGB_t RGB={10, 50, 90};
> RGB=red;
>

>
> Fehlermeldung für die zweite Zeile:
>
>> Error  1  expected expression before '{' token
>
> Was mache ich in dem Fall falsch?

Initialisieren ist nicht zuweisen. RGB kann über ein Literal (das was in 
{..} steht) initialisiert werden, aber erst ab C99 gibt es die 
Möglichkeit, struct-Literale in Ausdrücken zu verwenden. Du kannst dann 
"RGB=(RGB_t)red;" schreiben, was  "RGB=(RGB_t){255,0,0};" entspricht. Du 
kannst dann übrigends auch "someFunction((RGB_t){255,255,0});" machen 
und so was.

Wenn dein Compiler es kann, dann definiere red so:

#define red  ((RGB_t){255,0,0})

GCC kann diese Schreibweise schon länger, kann also gut sein, dass es 
bei dir auch funktioniert.

[ruud86 (Gast)]

super danke

[Johann L. (gjlayde)]

Marco M. schrieb:
> Wenn dein Compiler es kann, dann definiere red so:
>
> #define red  ((RGB_t){255,0,0})
>
>
> GCC kann diese Schreibweise schon länger,

So wie jeder andere C99-Compiler:

http://gcc.gnu.org/onlinedocs/gcc/Compound-Literals.html

Für Makronamen sollte man eher Großbuchstaben verwenden und für 
Variablen hingegen Kleinschreibung wählen.  Hat sich so eingebürgert und 
damit wird die Quelle besser verständlich.