c++ unsigned char Speicher kopieren

Ich übergeben zu müssen, ein unsigned char-array aus einer Methode zu einer anderen, und ich habe versucht mit diesem code:

{
     unsigned char *lpBuffer = new unsigned char[182];
 ReceiveSystemState(lpBuffer);

}

BOOL ReceiveSystemState(unsigned char *lpBuffer)
  {
      unsigned char strRecvBuffer[182] = { 0 };
      //strRecvBuffer construction

      memcpy(lpBuffer, strRecvBuffer, sizeof(strRecvBuffer));

      return TRUE;
 }

Keiner dieser 3 Methoden (verwendet in ReceiveSystemState) gearbeitet, als ich erwartet hatte. Nach der Verwendung von jedem einzelnen von Ihnen allen, dass es kopiert wird, ist der erste char von strRecvBuffer und nichts mehr. Die strRecvBuffer leer chars von element zu element, aber ich brauche diese, wie Sie sind, denn string ist eine Nachricht, die von einem hardware-Gerät und die Nachricht wird anallysed über ein Protokoll. Was vermisse ich hier? Muss ich initialisieren lpBuffer falsch?

EDIT: ich habe eine einfache memcpy Methode, um den job zu erledigen. Immer noch das gleiche Ergebnis: alle, die es kopiert wird, ist der erste char von strRecvBuffer.

EDIT2: Arbeiten code:

{
     unsigned char *lpBuffer = new unsigned char[182];
     ReceiveSystemState(lpBuffer);
     for (int i = 0; i < 144; i++)
     {
         memcpy(&c_dateKG[i], lpBuffer + i * sizeof(unsigned char), sizeof(unsigned char) );
     }

}    
  BOOL ReceiveSystemState(unsigned char *lpBuffer)
  {
     unsigned char strRecvBuffer[182] = { 0 };
     //strRecvBuffer construction

     memcpy(lpBuffer, strRecvBuffer, sizeof(strRecvBuffer));

     return TRUE;
  }
  • glaube nicht, dass es einen Unterschied machen wird, da unsigned char hat 1 byte-Größe. also im Grunde 182 * sizeof(unsigned char) ist die gleiche wie 182.
  • Ich habe diese nicht erhalten. Wo kommen die Daten her? Sie definieren strRecvBuffer enthalten eine 0 und Müll danach. Andere als die, memcpy kopiert das array, wird Ihre Schleife.
  • = { 0 } ist die syntax, um alle Elemente auf 0
  • memcpy( lpBuffer, strRecvBuffer, 182 ) sollte den job tun. Es ist etwas anderes bist du nicht zu uns zu zeigen. Auch: warum tun Sie gewirkt void manchmal, und andere nicht? Warum weisen Sie NULL auf eine variable, die sofort gehen aus dem Rahmen, und warum willst du auf null prüfen, bevor delete[] (wobei ein null-Zeiger ist legal), und nicht früher? Für diese Angelegenheit, warum solltest du auf null prüfen, da new[] garantiert nie wieder einen null-Zeiger? Und vielleicht am wichtigsten: warum sind Sie mit new[] statt std::vector< unsigned char >?
  • die Daten stammen aus einem seriellen Anschluss, mit Hilfe einer anderen Methode, und die Erklärung ist für initializind alle Elemente mit 0.
  • was ich nicht zeigen, Sie ist eine Methode, die liest die Daten von einem seriellen port und zurück strRecvBuffer. Das sollte nicht im Konflikt mit dem rest des Codes.
  • Vielleicht, aber es gibt keinen Grund, in den code, den Sie anzeigen für memcpy nicht zu funktionieren.
  • Debug legen Sie einen Haltepunkt in memcopy und überprüfen Sie den Inhalt lpbuffer ersten.
  • Tat und den Vergleich von 2 arrays.
  • in Bezug auf "eine Methode liest die Daten aus einer seriellen Schnittstelle und gibt strRecvBuffer." im obigen code strRecvBuffer ist eine lokale variable, deshalb können wir davon ausgehen, dass Sie auf der Durchreise sind es die unsichtbaren Methode? Der code oben ist nicht der code von der Quelle verursacht das Problem. Ich habe keine Zweifel, Sie haben ein Problem, aber in den Prozess der trimmen, was Sie als unwichtig-code vor dem posten hier, das Sie getrimmt das problem auch. Alle Zweifel, die ich schlage vor, Sie erstellen ein stand-alone-Datei mit nur den code oben zu laufen. sowohl das dyn-alloc " und "statisch" 182 bytes. funktioniert.
  • diese Methode war nicht das problem in der Tat. Ich habe bearbeitet Sie meine Frage mit den funktionierenden code.

InformationsquelleAutor MRM | 2012-10-05
  1. 6

    Ihren code absolut Müll. Einige Hinweise:

    Verwenden, sizeof

    Verwenden, sizeof(static_massive_name); oder count_of_arr_elements * sizeof(arr_type); Beispiel:

    unsigned char src[255];    
unsigned char dst[255];

//fill src with random data
for (int i = 0; i < 255; ++i) {
    src[i] = static_cast<unsigned char> (rand() % 255);
}

memcpy(src, dst, sizeof(dst));


//now dst will have copyed values from src (random numbers)

    UPDATE:

    Vollständige source-code zum testen:

    #include <iostream>
#include <string.h>
#include <time.h>
#include <stdlib.h>

using namespace std;

void print(unsigned char* arr, size_t size) {
    for (size_t i = 0; i < size; ++i) { 
        //see type casting (to int)!!!
        cout << "arr[" << i << "] = " << (int)arr[i]<< endl;
    }
}

int main() {
    srand(time(0));

    //unsigned char type hold values from 0 to 255
    unsigned char src[15];
    unsigned char dst[15];

    for (int i = 0; i < 15; ++i) {
        src[i] = rand() % 255;
    }

    memcpy(src, dst, sizeof(dst));

    print(src, 15);
    print(dst, 15);

    return 0;
}

    Ergebnis

    arr[0] = 34
arr[1] = 80
arr[2] = 183
arr[3] = 112
arr[4] = 18
arr[5] = 120
arr[6] = 183
arr[7] = 0
arr[8] = 0
arr[9] = 0
arr[10] = 0
arr[11] = 57
arr[12] = 137
arr[13] = 4
arr[14] = 8
arr[0] = 34
arr[1] = 80
arr[2] = 183
arr[3] = 112
arr[4] = 18
arr[5] = 120
arr[6] = 183
arr[7] = 0
arr[8] = 0
arr[9] = 0
arr[10] = 0
arr[11] = 57
arr[12] = 137
arr[13] = 4
arr[14] = 8
    • Ich habe versucht auch, das Ergebnis ist das gleiche: nur der erste char ist kopiert.
    • dein Beispiel funktioniert Prima, die Anwendung der Methode auf mein code funktioniert nicht.
    • 1. Wenn mein code funktioniert, als das system memcpy, funktioniert einwandfrei 2. Wenn Sie die Druck-unsigned char - sicher zu machen cast auf int (wenn der code in c++) oder printf (C-code): unsigned char var = 12; printf("var = %d\n", (int) var);
    • ich bin sicher, memcpy, funktioniert einwandfrei, es ist noch etwas da, in meinem code, die ich nicht richtig benutzen. Und es macht wirklich keinen Sinn, da ich die syntax sieht jetzt gut.
    • nach ein paar mehr schaut mein code, den ich realisiert, dass:BOOL ReceiveSystemState(unsigned char *lpBuffer), lpBuffer übergeben Sie einen Zeiger auf ein array von chars, also das, was der debugger zeigte meiner als "nur der erste char" war tatsächlich der erste char von dem ganzen memory zone lpBuffer hinwies. (meine Erklärung ist vielleicht nicht allzu genau). So gab es keine andere problem als ... nun, mir! Ich brauchte nur zu verwenden memcpy(&c_dateKG[i], lpBuffer + i * sizeof(unsigned char), sizeof(unsigned char) ); kopieren Sie die Werte aus dem array wieder aus dem Speicher. Ich habe meinen Beitrag aktualisiert mit den korrekte Verwendung von code. Danke!
    • Der Beispiel-Code ist falsch! "src" und "dst" in memcpy Funktionen ausgelagert werden müssen, richtig: memcpy(dst, src, sizeof(src)); überprüfen Sie die manual. Der Beispielcode gegeben, scheint zu funktionieren, weil die nicht initialisierte "Sommerzeit" auch enthält zufällige Zeichen. Bessere Tests, ich benutze anstelle von rand() % 255 in das src-Initialisierung-loop.
    • Fehler in meinem vorherigen Kommentar, das ist richtig: memcpy(dst, src, sizeof(dst));

    InformationsquelleAutor Maxim
Schreibe einen Kommentar