uint16_t auf uint8_t

[Robert M. (xertno)]

Hi, ich will ein 16Bit Datenwort auf zwei 8Bit Datenworte aufteilen.
Habe folgendes geschrieben.

uint8_t Data1, Data2;
uint16_t Temp;

  Data2=Temp;
  Temp>>=8;
  Data1=Temp;

die Frage ist geht das so.

[Johannes M. (johnny-m)]

Wieso schiebst Du an dem Ausgangswert rum?

dataL = temp;
dataH = temp >> 8;

[Ulrich (Gast)]

Hi,
ich habe eine Variable: uint16_t id
und ein Array: uint8_t array[10]

Nun habe ich dass hier geschrieben:

*(uint16_t*)&array[1]= id;

damit weiße ich array[1] das lowbyte und array[2] das highbyte zu. Laut 
*.lss wird dass ohne irgendeinen Overhead wie gewünscht richtig 
umgesetzt.


Die Frage ist jetzt nur kann man das auch lesbarer schreiben?

[Εrnst B. (ernst)]

Ulrich wrote:

> Nun habe ich dass hier geschrieben:
>

*(uint16_t*)&array[1]= id;

> Die Frage ist jetzt nur kann man das auch lesbarer schreiben?

Statt einem Array eine Struct verwenden, in der die einzelnen 
Speicherstellen aussagekräftige Namen haben. Evtl mit anonymen Unions 
drinnen, wenn sowohl 16- als auch 8-Bit Zugriffe nötig sind.

z.B.

struct wasweisich {
  uint8_t befehl;
  union {
    uint16_t argument;
    uint8_t argument_byte[2];
  }
}

...


struct wasweisich a;
a.befehl=42;
a.argument=5678;
if (a.argument_byte[0]==23) {

Wenn das Speicherlayout wichtig ist, noch ein __attribute(packed) an die 
struct-definition anhängen.

[Ulrich (Gast)]

aso ich habe noch was vergessen:

An vielen verschiedenen stellen im code habe ich solche zuweisungen in 
vorhandenen arrays mit unterschiedlichem aufbau. Deswegen kann ich das 
nur so wie oben schreiben. Die frage ist nur kann man den cast 
verkürzen? Kann man da irgendwie den referenzierungs und 
dereferenzierungsoperator weglassen?

[Εrnst B. (ernst)]

Nein.

Du kannst das ganze ge-caste aber in ein Macro packen,

Ungetestet:

#define INT(x) (*(uint16_t*)&(x))


INT(array[42])=444;
uint16_t z=INT(array[23]);

[Ulrich (Gast)]

schade, habe gedacht da gibts noch einen Trick. Jetzt muss ich halt 
aufpassen das ich bei den ganzen casts usw. keinen Fehler machen.

Trotzdem Danke für deinen Hilfe.

[Ulrich (Gast)]

Entweder du hast deinen Post noch editiert oder ich habs überlesen.

Ich mache es jetzt mit #define:

#define cast16 *(uint16_t*)&

cast16 array[1]= id;

funktioniert perfekt und vermeidet sicherlich den einen oder anderen 
bösen castfehler.