Shiftoperation sinnvoll?

[makrox (Gast)]

Hallo,
im Anhang habe ich ein CAN protocoll eingestellt.
Ich möchte die Daten passend auf jeweils 1 byte splitten.

Bisher meine Idee:

Byte1 = (unsigned char) (Data1 >> 6);
Byte2 = (unsigned char)((Data1 << 2) + (Data2 >> 12));
Byte3 = (unsigned char)(Data2 >> 4);
Byte4 = (unsigned char)(Data2 << 4) + (Data3 >> 10);
Byte5 = (unsigned char)(Data3 >> 2);
//...

Man erkennt schon die Problematik, was die Übersichtlichkeit des Codes 
angeht.

Gibt es für solche Dinge eine geschikte Lösung, die dann auch noch 
halbwegs lesbar ist?
Mir fällt gerade nichts ein, deshalb mein post.

Grüße
mx

[Mani (Gast)]

Schau Dir mal structs bzw. union an...

[makrox (Gast)]

Hi da habe ich auch schon daran gedacht, nur irgendwie muss ich die 
Daten da ja auch rein bringen. Ein byte kann ja bits von 2 unabhängigen 
Daten enthalten.
Geht das mit einem Struct ohne Shiften, ich weiß nicht wie?

Kannst du mir da ein Beispiel zeigen?

[Vlad T. (vlad_tepesch)]

makrox schrieb:
> Kannst du mir da ein Beispiel zeigen?


union CanMsg
{
  struct RawData{
    uint8_t bytes[8];
  }rawData;

  struct MyData
  {
    unsigned dummy: 8; // Byte 0 not specified
    unsigned data1:14;
    unsigned data2:14;
    unsigned data3:14;
    unsigned data4:14;
  }structuredData;
}

must halt schauen, ob die Bits korrekt auf die Daten abgebilded werden.
Das ist höchst architektur- und compilerabhängig, deswegen eigentlich 
nicht zum Datenaustausch zu empfehlen.

Besser ist was generisches, was eine abstrakte Nachrichtenbeschreibung 
nimmt und diese in einen anderen Typen umwandelt, bzw eine 
Struktdefinition und  Konvertierungsroutine erstellt