【C言語】配列から構造体へコピーしたい

Question

#やりたいこと
配列データuart_bufferを構造体にコピーするときにmemcpyを使用すると左(NG)のように、payload1に0x44332211とセットされてしまいます。
これをpayload1に0x11223344とセットしたい(右のOK)のでビットシフト処理で(下記コード)無理やり格納しました。他に良い方法があれば知りたい。
セットしたい

    /*パソコンデータ受信用構造体にデータセット*/
    //memcpy(&Pc_to_mst_data_frame_struct.slv_id, UART_struct->uart_buffer,size);
    Pc_to_mst_data_frame_struct.slv_id          =       UART_struct->uart_buffer[0];
    Pc_to_mst_data_frame_struct.command_id      =       UART_struct->uart_buffer[1];
    Pc_to_mst_data_frame_struct.payload1        |=       UART_struct->uart_buffer[2] << 24;
    Pc_to_mst_data_frame_struct.payload1        |=       UART_struct->uart_buffer[3] << 16;
    Pc_to_mst_data_frame_struct.payload1        |=       UART_struct->uart_buffer[4] <<  8;
    Pc_to_mst_data_frame_struct.payload1        |=       UART_struct->uart_buffer[5];
    Pc_to_mst_data_frame_struct.payload2        |=       UART_struct->uart_buffer[6] << 24;
    Pc_to_mst_data_frame_struct.payload2        |=       UART_struct->uart_buffer[7] << 16;
    Pc_to_mst_data_frame_struct.payload2        |=       UART_struct->uart_buffer[8] <<  8;
    Pc_to_mst_data_frame_struct.payload2        |=       UART_struct->uart_buffer[9];   

#しりたいこと
上記コード以外でスマートなやり方があればご教示お願いします。
無知で申し訳ありません。宜しくお願い致します。

Comments

[otn]

structの定義を書いてください。

[hachimitu]

情報不足で申し訳ありません。

こちらになります↓
typedef struct Pc_to_mst_data_frame_handle_typedef
{
uint8_t slv_id;
uint8_t data_size;
uint8_t command_id;
int32_t payload1;
int32_t payload2;
int32_t payload3;
int32_t payload4;
}__PACKED Pc_to_mst_data_frame_handle_typedef;

[rubato6809]

その構造体定義をここに返答するのではなく、質問を編集して書き加えてくださいな。

Answer 1

データ側を変更出来ないのであれば
ntohlを使う事になります

---

追記

あくまでも、ビッグエンディアン(ネットワークバイトオーダー)からホストバイトオーダーへの変換であることに留意が必要

// MSVCの場合、以下どちらか
#include <winsock2.h>
#include <winsock.h>

// Linuxの場合、以下どちらか
#include <netinet/in.h>
#include <arpa/inet.h>

memcpy(&Pc_to_mst_data_frame_struct.slv_id, UART_struct->uart_buffer,size);
Pc_to_mst_data_frame_struct.payload1 = ntohl(Pc_to_mst_data_frame_struct.payload1);

Answer 2

多分、その方法が妥当だと思います。
少し、気になったのは、

    Pc_to_mst_data_frame_struct.payload1        |=       UART_struct->uart_buffer[2] << 24;
    Pc_to_mst_data_frame_struct.payload1        |=       UART_struct->uart_buffer[3] << 16;
    Pc_to_mst_data_frame_struct.payload1        |=       UART_struct->uart_buffer[4] <<  8;
    Pc_to_mst_data_frame_struct.payload1        |=       UART_struct->uart_buffer[5];
    Pc_to_mst_data_frame_struct.payload2        |=       UART_struct->uart_buffer[6] << 24;
    Pc_to_mst_data_frame_struct.payload2        |=       UART_struct->uart_buffer[7] << 16;
    Pc_to_mst_data_frame_struct.payload2        |=       UART_struct->uart_buffer[8] <<  8;
    Pc_to_mst_data_frame_struct.payload2        |=       UART_struct->uart_buffer[9];   

でのコピー順。UART_struct->uart_buffer[5] からの逆順とし、最初は、|= での OR演算でなく、単なる = での代入としましょう。(ゼロクリアされてない) 他の payload? も同様ですね。

また、memcpy()で失敗したとありますが、データサイズは一致してますか?
Pc_to_mst_data_frame_handle_typedefは、 unit8_t が 3つでその後、int32_t ちょっと違うように見えます。
また、この手のコピーは、unionが定番とも思いましたが、payload1 が奇数アドレス。ここ、偶数するのが一般的と思いました。(従って unionは使えない)

Comments

[hachimitu]

回答ありがとうございます。指摘されたところは修正しました。助かりました。

Answer 3

通信データは数値の並び順が決まっているため、memcpyでコピーしてしまうと逆順になってしまいます
こういう場合は数値取得のためのマクロを定義しておきます

// 16bit数値の取得
#define  GETWORD(p)  (((uint8_t*)(p))[0]<<8 | ((uint8_t*)(p))[1])
// 32bit数値の取得 
#define  GETDWORD(p) (GETWORD(p)<<16 | GETWORD(((uint16_t*)(p))+1)) 


...
 Pc_to_mst_data_frame_struct.payload1 =GETDWORD(& UART_struct->uart_buffer[2]);

プログラマたるもの、いかに手を抜くかを考えましょう