C＋＋で構造体をUDP通信

Question

以下のサイトのプログラムをベースに構造体をUDP通信での送受信をしようとしたのですがちょっと詰まってしました。
https://qiita.com/srs/items/c9286b5cff99e741b993

一応送る構造体は以下のものを想定しています。

typedef struct {
  int x;
  int y;
  int z;
}msg;

msg msg_test = { 0, 1 , 2};

構造体を宣言してsendtoの部分を書き換えればいけると思ったのですが，どうやらうまくいかないようで...。
多分凄い初歩的な部分だと思われますが，解決策を教えていただけるとありがたいです。

Comments

[int32_t]

> どうやらうまくいかないようで...。

どのように書き換えて、期待に反してどのような結果になったのですか?

[thkana]

書き換える前(記事そのまま)であれば「うまくいく」のですね?

[AbeTakashi]

なにをやってどう上手くいかないのかを書かないとアドバイスが出てこないと思います。実際に書いたソースコードとビルド方法、実行方法、上手くいかないとは具体的に何が起きてるのか？（エラーメッセージがでるならそれをまるっと表記する）、動作環境を質問文に追記して頂ければ、回答が出てくる可能性がかなり高くなると思います。

[Naben]

返信遅くなってすみません...。書き換える前はsendtoの部分を構造体に適した形にしたつもりでしたが，回答者さんのを見るにコード自体が結構違ってたみたいでした

Answer 1

文字列の代わりに構造体を書けばよいのでは？

#include <sys/types.h>
#include <sys/socket.h>  // socket, bind, sendto, recv
#include <netinet/in.h>  // sockaddr_in
#include <arpa/inet.h>   // inet_addr
#include <unistd.h>      // close
#include <stdio.h>       // printf
#include <stdlib.h>      // atoi

struct msg { int x, y, z; };

class simple_udp {
    int sock;
    struct sockaddr_in addr;
public:
    simple_udp(const char *address, int port) {
        sock = socket(AF_INET, SOCK_DGRAM, 0);
        addr.sin_family = AF_INET;
        addr.sin_addr.s_addr = inet_addr(address);
        addr.sin_port = htons(port);        
    }
    void udp_send(const msg &data) {
        sendto(sock, &data, sizeof(msg), 0, (struct sockaddr *)&addr, sizeof(addr));
    }
    void udp_bind(){
        bind(sock, (const struct sockaddr *)&addr, sizeof(addr));
    }
	void udp_recv(msg &data){
        recv(sock, &data, sizeof(msg), 0);
    }

    ~simple_udp(){
        close(sock);
    }
};

void send(int argc, char **argv)
{
	msg data = { atoi(argv[1]), atoi(argv[2]), atoi(argv[3]) };
	simple_udp udp0("127.0.0.1", 4001);
	udp0.udp_send(data);
}

void recv()
{
	simple_udp udp0("0.0.0.0", 4001);
	udp0.udp_bind();
	while (1) {
		msg data;
		udp0.udp_recv(data);
		printf("recv: %d %d %d\n", data.x, data.y, data.z);
	}
}

int main(int argc, char **argv)
{
	if (argc == 4)
		send(argc, argv);
	else if (argc == 1)
		recv();
}

エラーチェックをしていないので、原理を知るためだけのコードです。

Comments

[Naben]

ありがとうございます．C++の初歩的な部分だと思われるかもしれませんが凄い助かりました！

Answer 2

質問に挙げられたサイトでは、任意の文字列(std::string)を送受できるサンプルがあります。
...ということは、

typedef struct {
  int x;
  int y;
  int z;
}msg;

と文字列との相互変換(msg⇔std::string)関数を用意すればいい。

std::string msg2string(const msg& m) { ... }
msg string2msg(const std::string&) { ... }

を用意しておけば、
送信： msg2string で文字列化して送信
受信 : 受信した文字列を string2msg で復元

Comments

[Naben]

ありがとうございます．こちらの回答も非常に参考になります！