2byte以上の数値データ送信

Question

2byte以上のデータ送信について
int 型のデータ4つを送信するためにプログラムを書いたのですがうまく動きません.
ご指摘ください.

送信側

first = 0x00000001
second = 0x00000000
third = 0x00000000
fourth = 0x00000000

msg = [first,second,third,fourth].pack("N*")
socket.write(msg)
#これでネットワークバイトオーダでunsigned long * 4のデータサイズで書き込めるはず

受信側

  struct node *tmp;
        if(recv(socket,tmp,sizeof(char) * 16,0) < 0)
                perror("ID ERROR");
        tmp->n_ntoh();
//受け取ったデータをネットワークバイトオーダからホストオーダに変換して
//値をとり出すことを目標
    

//以下は他の部分でされている定義
   typedef struct node {
    uint32 first;
    uint32 second;
    uint32 third;
    uint32 fourth;
    void n_ntoh(void){
     first = ntoh(first);
     second = ntoh(second);
     third = ntoh(third);
     fourth = ntoh(fourth);
   }
  };

Answer 1

C/C++側への回答となります。

原因１ tmpの実態がどこにもない。
struct node work;
struct node *tmp = work;
とする。でもこれは無いと思う。理由はChironianさんの仰っている通り。
原因２ uint32が32bit整数じゃない。
uint32_tを使う。
原因３ 関数ntohが期待の通りに動いていない。
ntohlではないか？？？？？
原因４ 受信が分割されている。recvは取りこぼすこともある。
int i,r;
char *p = (char *)tmp;
for ( i = 0 ; i < 16 ; ){
if ((r = recv(socket, p+i, 16-i, 0)) < 0) {
perror("ID ERROR");
} else {
i += r;
}
}
として、取りこぼしをなくす。
原因５ 通信できてない
まず、通信できるようにする。問題のコードはもっと前にある。

今の情報ではこのくらいのような気がします。

そもそも論ですが、C/C++はソースを読んで構造体の占有メモリサイズを数える事は出来ないのですよ。
メンバfirstとメンバsecondの間に、アライメント調整の為のゴミがあってもOKな言語ですからね。
上記を考慮すると、cateyeさんが言っておられるように配列に読ませないといけません。
struct node {
uint32_t first[4]; <--- ここに読ませる
void n_ntoh(void){
first[0] = ntohl(first[0]);
first[1] = ntohl(first[1]);
......
}
};
まぁ、
・ターゲットマシンが固定。
・コンパイラが固定。
・構造体での割り付けルールは全て分かっている。
ならば、構造体に読み込んでも良いのかもしれませんが、他の人はどうしてるんでしょうね。

Answer 2

「うまく動かない」では分かりません。どううまく動かないのか書いてください。

Rubyは分からないので、C++で気が付いた事だけですが・・・
struct node *tmp;はポインタですが実態はどこにあるのでしょう？
・・・そもそもソケットは通信できていますか？

Comments

[退会済みユーザー]

すいません
補足すると, 送信側ではサイズを調べてみると16となっており16byte のデータが送信できています.

受信側ではうまくデータを受け取れておらず, データには文字化けした文字が入っているだけです.
構造体にバイトストリームのデータを書き込むのはこの方法であっているのでしょうか？

[cateye]

送受信が出来ているかどうか・・・
通信内容については、サイズの問題もあるので、Wiresharkなどのネットワークモニタで送受信データを確認するほうが確実です。
受信方法はuint32の配列（メモリが連続しているのが保障される）で受けて受信データをn_ntoh()に渡すほうがいいのではないでしょうか？
あと今気が付いたのですが＾＾；
first = ntoh(first);の左辺のfirstは分かりますが右辺のfirstは何なんでしょう？
また、ntoh()は何なんでしょう？・・・ntohl()の事でしょうか？
参考：http://wisdom.sakura.ne.jp/system/winapi/winsock/winSock4.html

Answer 3

こんにちは。

cateyeさんも指摘されてますが、受信バッファの与え方を間違っているように思います。
C/C++言語に慣れない内はよくやってしまうのですが、ドキュメントに

size_t recv(int s, void *buf, size_t len, int flags);

と記載されている時、ついbufに初期化していないポインタを渡してしまうのですよ。
struct node *tmp;ではtmpに値を設定していないので、不定値が入っています。

では、recv()のbuf引数に何を渡すべきかというと、受信バッファへのポインタですね。
例えば、'struct node tmp;でスタック上にメモリが確保されます。 そして、&tmpがtmp`へのポインタになります。

受信側ではうまくデータを受け取れておらず, データには文字化けした文字が入っているだけです.

不定な値をrecv()関数へ渡している場合アクセス違反で落ちることが多いです。
受信バッファの中身を確認できたということは落ちていないはずですから、別の原因があるかも知れません。
そもそも、cateyeさんのご質問通り、受信できているのかどうか確認下さい。
if (recv(...) < 0)では不十分です。何バイト受信できているか確認しましょう。

それと、「文字化け」ですか？ 送っているデータは「文字」ではないですよね？

頑張ってください。

---

最後に、ついでです。sizeof(char) * 16は無意味に複雑な記述です。単純に16としてしまうか、sizeof(struct node)とすると良いです。後者の使い方はまさにsizeof演算子の代表的な使い方です。