ゲームの通信プログラムについてなんですが

質問

ボードゲームで対戦用にとTCP通信プログラムを導入したいのですがどうにもうまくいきません。
ゲームのところでつながっているにはつながっているんですが無限ループのように通信しているようなんです。
VisualStudio2010とwinsocketを使っております。

サーバーのプログラムはこうです。
void COMMU::SERWSA()
{
port = 6543;

//WinSock初期化
rtn = WSAStartup(
MAKEWORD(2,2),
&wsaData);
}

void COMMU::SERSOCK()
{
listen_s = socket(
AF_INET,
SOCK_STREAM,
0);
if(listen_s < 0)
　　　 {
WSACleanup();
return;
}

//0で初期化
memset(&saddr, 0, sizeof(SOCKADDR_IN));

saddr.sin_family = AF_INET;
saddr.sin_port = htons(port);
saddr.sin_addr.s_addr = INADDR_ANY;

if(bind(listen_s,
(struct sockaddr *)&saddr,
sizeof(saddr)) == SOCKET_ERROR)
　　　 {
closesocket(listen_s);
WSACleanup();
return;
}
}

void COMMU::SERLIS()
{
//クライアントからの接続待ち状態にする
if(listen(listen_s, 0) == SOCKET_ERROR)
{
closesocket(listen_s);
WSACleanup();
return;
}

fromlen = (int)sizeof(from);
s = accept(listen_s, (SOCKADDR *)&from, &fromlen);
if(s == INVALID_SOCKET)
　　　 {
closesocket(listen_s);
WSACleanup();
return;
}else{
inet_ntoa(from.sin_addr);
}

//リスンソケットはもう不要
closesocket(listen_s);

MessageBox(NULL,"繋がった","読み込みエラー", MB_OK |MB_ICONERROR);
}
COMMU::~COMMU()
{
if(shutdown(s, SD_BOTH) != 0){
perror("シャットダウンに失敗しました\n");
}

//ソケットのクローズ
closesocket(s);

//ソケットのリソース解放
rtn = WSACleanup();
if(rtn == SOCKET_ERROR){
fprintf(stderr, "WSACleanup失敗\n");
return;
}
printf("WSACleanup成功\n");

printf("サーバ終了\n");
}

続いてクライアントです。
void Client::CLWSA()
{
//ポート番号の入力
port = 6543; //数値に変換

//WinSock初期化
rtn = WSAStartup(
MAKEWORD(2,2),//上位と下位を指定してWORDデータを返すマクロ
&wsaData);
}

void Client::CLSOCK()
{
//ソケットを開く
s = socket(
AF_INET, //アドレスファミリ
SOCK_STREAM, //TCP(接続型通信)で通信
0); //プロトコル(0:WinSockが自動的に判断)
if(s < 0){
//perror("ソケットをオープンできません\n");
WSACleanup();
return;
}

//ホスト名からホスト情報を取得
lpHost = gethostbyname("PC○○");
if(lpHost == NULL)
　　　 {
addr = inet_addr(szServer);
//ネットワークアドレスからホスト情報を取得
lpHost = gethostbyaddr((char*)&addr,4,AF_INET);
}
if(lpHost == NULL)
　　　{
printf("ホスト情報取得エラー\n");
closesocket(s);
WSACleanup();
getchar();
return;
}
}

void Client::CLCONNE()
{
//0で初期化
memset(&saddr, 0, sizeof(SOCKADDR_IN));

saddr.sin_family = lpHost->h_addrtype; //ホストのアドレスファミリ
saddr.sin_port = htons(port);
saddr.sin_addr.s_addr = *((u_long *)lpHost->h_addr);

//接続要求する
if(connect(s, (SOCKADDR *)&saddr,
sizeof(saddr)) == SOCKET_ERROR){
printf("connectエラー\n");
closesocket(s);
WSACleanup();
getchar();
return;
}else{
MessageBox(NULL,"つながりました","読み込みエラー", MB_OK |MB_ICONERROR);
}
}

Client::~Client()
{
if(shutdown(s, SD_BOTH) != 0){
perror("シャットダウンに失敗しました\n");
}

//ソケットのクローズ
closesocket(s);

//ソケットのリソース解放
rtn = WSACleanup();
if(rtn == SOCKET_ERROR){
fprintf(stderr, "WSACleanup失敗\n");
return;
}
printf("WSACleanup成功\n");

printf("クライアント終了\n");
}

です。
わかりにくいかと思いますがかなり困ってます・・・
回答よろしくお願いします。

回答No.3

Wr5 · ベストアンサー

回答じゃありませんが……

回答への補足だとフィルタ効かないんですかねぇ…。
ココでは個人情報のやりとりは削除対象かと。
質問や回答でメアドが含まれると判定されると「事務局確認中」みたいになるんですけどねぇ…。アウトだと削除される。
# というワケでそのうち事務局の目に留まってメールアドレスは削除されるかと思われます。

どっかのフリーのアップローダを使うとか、自分のWebページにファイルを置いてリンク貼るとか……。
質問中(あとは補足欄とか)で完結できるようにするのが望ましいと思われますが。
# コードを削っていって現象が確認できる最低限のものにして…というのがベター。

しかし…フリーのアップローダだとしばらくすると削除されるでしょうから、後からこの質問見た人(同じような問題に直面して検索してきた人)の役には立たないでしょうねぇ。
まぁ、個人のWebスペースでも削除されたら同じことなんですが。
# 回答に参考ページとして検索して見つけたURL貼ることはありますが…そっちの場合は個人情報(まあ、他人ですけど)のやりとりとは判断されない…んですかね。
# 元々そういう意図でのURL掲示ではありませんけどどこで判断するのか…と。(事務局判断…なんでしょうけどね)

回答No.2

Wr5

既に指摘されているように…掲示されているコードだけではどうにも……。

まぁ、妄想で回答(指摘)してみると………。

recv()がブロッキングモードで止まってて処理が進まないのを無限ループと勘違いしていませんか？
# さらにrecv()で指定したサイズが1回で受信できる。という保証もありませんので受信処理にはご注意を。
サーバ／クライアント双方でWireSharkなどでパケットキャプチャしてみて、期待するデータの送受信が正しく行われているか確認してみてはどうでしょうか。

補足

わかりました。試してみます。
どうもありがとうございます。

回答No.1

fresh_homepie

「無限ループのように通信している」とのことですが、

・呼び出し元のコードも記載されていないので、このクラスがどう使われているかも不明。ループしている箇所が（少なくともここには）記載されていないのでなぜ無限ループのようだと思ったのか…。

・そもそも通信している部分（sendとかrecvとか）が記載されていない。すでに「つながっている」とのことなので、成功している部分（connectとか）だけをここに記載されても…。

といったところで、この情報だけでは判断できないです。

「つながっている」というのはサーバの「繋がった」、クライアントの「つながりました」メッセージボックスは出てる、ということですよね。それなら、問題はその先のコードにあるということなので、それを提示してもらわないと。

補足

申し訳ございません。
たりないことだらけでしたね・・・

できたら直接見てくれた方が大変助かりますのでメールアドレス置いときます。

samurainovel@yahoo.co.jp

返信してくれた方にデータ送ります。
お願い致します。