Serialisierung/Deserialisierung ein struct mit einem char* in C

Können Sie eine Klasse reprensenting das Objekt, das Sie verwenden in Ihrer software mit allen Feinheiten und Mitglied func und was auch immer Sie brauchen. Dann haben Sie 'serialisiert' struct, das mehr eine Beschreibung von dem, was am Ende auf das Netzwerk.

Zur Gewährleistung der compiler tun, was Sie ihm sagen zu tun, müssen Sie zu belehren, dass es "pack' die Struktur. Die Richtlinie habe ich hier, ist für den gcc finden Sie in Ihrem compiler-doc, wenn Sie nicht mit gcc.

Dann das serialisieren und Deserialisieren von routine konvertieren zwischen den beiden, die Gewährleistung der byte-Reihenfolge und details wie die.

#include <arpa/inet.h>   /* ntohl htonl */
#include <string.h>      /* memcpy */

class Packet {
    int senderId;
    int sequenceNumber;
    char data[MaxDataSize];
public:
    char* Serialize();
    void Deserialize(char *message);
};

struct SerializedPacket {
    int senderId;
    int sequenceNumber;
    char data[MaxDataSize];
} __attribute__((packed));

void* Packet::Serialize() {
    struct SerializedPacket *s = new SerializedPacket();
    s->senderId = htonl(this->senderId);
    s->sequenceNumber = htonl(this->sequenceNumber);
    memcpy(s->data, this->data, MaxDataSize);
    return s;
}

void Packet::Deserialize(void *message) {
    struct SerializedPacket *s = (struct SerializedPacket*)message;
    this->senderId = ntohl(s->senderId);
    this->sequenceNumber = ntohl(s->sequenceNumber);
    memcpy(this->data, s->data, MaxDataSize);
}

InformationsquelleAutor 246tNt

int senderId;
int sequenceNumber;
...    
char *message = new char[MaxMailSize];
message[0] = senderId;
message[1] = sequenceNumber;

Sind Sie überschreiben die Werte hier. senderId und sequenceNumber sind beide int-Werte und auch mehr als sizeof(char) bytes auf den meisten Architekturen. Versuchen Sie so etwas wie dieses:

char * message = new char[MaxMailSize];
int offset = 0;
memcpy(message + offset, &senderId, sizeof(senderId));
offset += sizeof(senderId);
memcpy(message + offset, &sequenceNumber, sizeof(sequenceNumber));
offset += sizeof(sequenceNumber);
memcpy(message + offset, data, MaxDataSize);

BEARBEITEN:
fixed-code, geschrieben in einem stupor. Auch, wie bereits im Kommentar, jeder wird ein solches Paket ist nicht beweglich durch endian Unterschiede.

InformationsquelleAutor Jherico

Wie erwähnt in anderen Beiträgen, senderId und sequenceNumber sind beide vom Typ int, die wahrscheinlich größer ist als der char, so werden die Werte abgeschnitten.

Wenn das akzeptabel ist, dann ist der code OK. Wenn nicht, dann müssen Sie zu Spalten Sie in Ihre Bestandteile bytes. Gegeben, dass das Protokoll Sie verwenden, wird die genaue Angabe der byte-Reihenfolge von multi-byte-Feldern, die meisten tragbaren und am wenigsten zweideutige Art und Weise, dies zu tun ist durch eine Verschiebung.

Zum Beispiel, sagen wir mal, dass senderId und sequenceNumber sind beide 2 bytes lang ist, und das Protokoll verlangt, dass das höherwertige byte zuerst geht:

char* Serialize() {
    char *message = new char[MaxMailSize];

    message[0] = senderId >> 8;
    message[1] = senderId;

    message[2] = sequenceNumber >> 8;
    message[3] = sequenceNumber;

    memcpy(&message[4], data, MaxDataSize);

    return message;
}

Ich würde auch empfehlen das ersetzen der for Schleife mit memcpy (wenn verfügbar), da es unwahrscheinlich ist, weniger effizient zu sein, und es macht den code kürzer.

Schließlich alle davon ausgegangen, dass char ist ein byte lang. Wenn nicht, dann alle Daten müssen maskiert werden, z.B.:

    message[0] = (senderId >> 8) & 0xFF;

InformationsquelleAutor Steve Melnikoff