Qt C++ QByteArray.constdata() copy

In der Zeile:

1

writedata = (MyDataClass*) ba.constData();

speicherst du einen Pointer auf deine Daten irgendwo hin und erwartest 
dann hier

1

SendData(protocolclass)

das deine Daten in einem bis jetzt komplett unbeteiligten Objekt stehen?

Versuch noch einmal deinen Code etwas aufgeräumter zu gestalten, so wird 
dir wahrscheinlich keiner helfen können. Generell gilt auch, dass casts 
praktisch nie eine gute Idee sind.

Hallo, ja ganz glücklich bin ich auch noch nicht, aber dazu muss erst 
mehr Erfahrung sammeln.

1

const MyClass d = *reinterpret_cast<MyClass*>(byteArr.data());

Ähnlich hatte ich es auch am Anfang gedacht, aber das gibt die 
Fehlermeldung

Semantic Issue
reinterpret_cast from type 'const char *' to MyClass* casts away 
qualifiers

>das deine Daten in einem bis jetzt komplett unbeteiligten Objekt stehen?

1

void *databytes;

2

// set id and size

3

int id = 10;

4

int size = 2; 

5

databytes = protocolclass.Adddata(id,size);

6

MyDataclass* wdata = reinterpret_cast<MyDataclass*>(databytes);

Unbeteiligt ist es doch nicht nach meinem Verständnis. Mein Verständnis 
ist, dass der void* databytes Pointer auf den Speicherbereich der 
MyProtocolclass zeigt, welches wiederum ein Buffer ist. Der 
Speicherbereich wird auf die Struktur (MyDataclass) gecastest und ich 
kann die Werte dementsprechend ändern.

Oder ist mein Verständnis falsch?

>Vielleicht erklärst du erstmal was du eigentlich erreichen willst, weil
>das ist alles irgendwie ein bisschen wirr was du da machst. Bei dem Code
>oben fehlt irgendwo ein const oder es ist wo eins zu viel, du kannst dir
>ziemlich aussuchen welches du weg oder hin machst ;)


Ok, zweiter Versuch:

1

void MyClass::handlemessage(const QByteArray &ba)

2

{

3

4

    MyProtocolclass protocolclass;

5

    protocolclass.Init();

6

    void *databytes;

7

    // set id and size

8

    int id = 10;

9

    int size = 2; 

10

11

    databytes = protocolclass.Adddata(id,size);

12

    MyDataclass* wdata = reinterpret_cast<MyDataclass*>(databytes);

13

    wdata->value0 = 1;

14

    wdata->value1 = 1;

15

   SendData(protocolclass)

16

}

17

18

void MyClass::SendData(MyProtocolclass &myprotolclass)

19

{

20

  // ... 

21

22

}

In dem Beispiel setze ich Werte im Struct MyDataclass manuell und sende 
Sie per Ethernet an ein Gerät. Die Klasse MyProtoclass ist gegeben und 
beinhaltet den kompletten Protokolloverhead und die Nutzdaten für das 
Gerät.

Das Beispiel orientiert sich am Hersteller des Gerätes und Protokolls 
zum Daten senden, klappt auch wunderbar.

Ich bekomme nun von anderen Applikationen nur die Nutzdaten als 
QByteArray und möchte die einfach übergeben und an das Gerät senden.

Das hier klappt auch wunderbar und es werden die Werte aus dem 
QByteArray an das Gerät gesendet:

1

void MyClass::handlemessage(const QByteArray &ba)

2

{

3

4

    MyProtocolclass protocolclass;

5

    protocolclass.Init();

6

    void *databytes;

7

    // set id and size

8

    int id = 10;

9

    int size = 2; 

10

11

    databytes = protocolclass.Adddata(id,size);

12

    MyDataclass* wdata = reinterpret_cast<MyDataclass*>(databytes);

13

    wdata->value0 = ba[0];

14

    wdata->value1 = ba[1];

15

   SendData(protocolclass)

16

}

17

18

void MyClass::SendData(MyProtocolclass &myprotolclass)

19

{

20

  // ... 

21

22

}

Diesen Teil würde ich gerne ändern und die Daten im QByteArary direkt 
auf die Struktur kopieren und senden, ohne jeden einzeln Index 
anzugeben.
Hier ist auch der Nachteil, wenn die Struktur z.B. ein uint16 beinhaltet 
würde es mit dem Index gar nicht gehen.

1

    MyDataclass* wdata = reinterpret_cast<MyDataclass*>(databytes);

2

    wdata->value0 = ba[0];

3

    wdata->value1 = ba[1];

Ist es jetzt nicht möglich einfach die Daten im QByteArray auf die 
Struktur MyDataClass zu kopieren? Ich würde kein Index benötigen und ich 
könnte auch unterschiedliche Typen z.B. uint16 übergeben.

Habs nun hinbekommen.

1

void MyClass::handlemessage(const QByteArray &ba)

2

{

3

4

    MyProtocolclass protocolclass;

5

    protocolclass.Init();

6

    void *databytes;

7

    // set id and size

8

    int id = 10;

9

    int size = 2; 

10

11

    databytes = protocolclass.Adddata(id,size);

12

    MyDataclass* wdata = reinterpret_cast<MyDataclass*>(databytes);

13

    memcpy(wdata,ba.data(),size);

14

15

16

   SendData(protocolclass)

17

}

Thorben schrieb:
> databytes = protocolclass.Adddata(id,size);
> myDataclass* wdata = reinterpret_cast<MyDataclass*>(databytes);
> memcpy(wdata,ba.data(),size);

Gut, dass du den wdata Pointer zwischen der 2. und 3. Zeile nicht 
dereferenzierst. Das macht die 2. Zeile allerdings auch überflüssig.

>Ne das ist kaputt, das geht so nicht. Überleg doch mal wo das memcpy
>eigentlich hin kopiert.


Läuft seit 18:00 Uhr im Dauerlauftest mit Werteänderungen im QByteArray. 
Die erste Stunde gab es keine Probleme und mehr weiss ich erst am 
Montag.

1

databytes = protocolclass.Adddata(id,size);

2

MyDataclass* wdata = reinterpret_cast<MyDataclass*>(databytes);

3

memcpy(wdata,ba.data(),size);

Das Memcpy kopiert das ByteArray auf den Speicherbereich wdata.

Thorben schrieb:
> Das Memcpy kopiert das ByteArray auf den Speicherbereich wdata.

wdata ist ein Pointer kein Speicherbereich. Dieser Pointer zeigt 
hoffentlich auf einen zulässigen Speicherbereich. Was genau dieses 
Addddata zurück gibt, weißt nur du.

Wenn die Addata Funtkion einen void* zurück gibt, ist der 
reinterpret_cast unnötigt. Liefert sie etwas anderes zurück, hast du 
vermutlich undefined behaviour.

Du hast natürlich recht wdata ist ein Pointer und der Pointer zeigt auf 
den Speicherbereich.

>Wenn die Addata Funtkion einen void* zurück gibt, ist der
>reinterpret_cast unnötigt.

Addata gibt ein void* zurück, welcher auf den Speicherbereich des 
dynamischen Buffers in der Protokolklasse zeigt.

Ich glaube hier hast du ebenfalls vollkommen recht. Ich denke der 
reinterpret_cast wird nur benötigt, wenn man gerne über die Struktur 
darauf zugreifen möchte.

Ich werde es  am Montag probieren direkt die Daten im QByteArray.data() 
mit memcopy zukopieren.

Thorben schrieb:
> Addata gibt ein void* zurück, welcher auf den Speicherbereich des
> dynamischen Buffers in der Protokolklasse zeigt.

Du solltest bei der Gelegenheit auch nochmal über deine Namen 
nachdenken. Ich habe mit Absicht einmal Addata und einmal Addddata 
geschrieben ;-) Bei SendData kommt man nicht so leicht durcheinander.

Früher war das einfacher. Da hat man ein char Array genommen und fertig.

Heute wird das in
const QByteArray &ba
gepackt. Und mit
MyDataclass* wdata = reinterpret_cast<MyDataclass*>(databytes);
gecastet.
Intern passiert wahrscheinlich das gleiche, als hätte man ein char Array 
genommen hätte. Nur durchblicken tut das keiner mehr.
Am besten programmiert man noch einen Iterator anstelle des memcpy.

Mein Fazit: Nehmt die Komplexität raus, dann versteht ihr euren eigenen 
Code wieder.

Ja, anscheinend war mein Fehler wo anders. Der Test über das Wochenende 
ergab keine Probleme.