I open 2 clients, 1 first makes a game room, then the other client joins this room.

10 Game rooms @ 2 Players each

Each player represents a line controlled by keyboard inputs, this line data is sent to the server, stored on the server, then the server returns the other user's line data.

The communication is flawlessly synchronized for about 5 seconds, then the host client freezes and is unresponsive.

The looping communication once the game room is entered is this for the client:

Client:

 
serverUpdate(){
prepare information into data structure
 
//send this data structure to server for server-side storage
iResult = send( ConnectSocket, sendbuf, (int)strlen(sendbuf)+2, 0 );
 
//get other players data
iResult = recv(ConnectSocket, recvbuf, recvbuflen, 0);
 
update other player movement with this received data
}
 
serverUpdate() is called every 30ms
 

Server:

in thread function:

 
//after a socket connection is establish and game room joined,
while(true){
 
iResult = recv(((ClientParams*)lpParam)->getSocket(), recvbuf, recvbuflen, 0);
 
Receive and store user data
 
iSendResult = send( ((ClientParams*)lpParam)->getSocket(), temp.c_str(), DEFAULT_BUFLEN, 0 );
 
Send other user's data
 
}
 

These are all blocking-sockets if that makes a difference with no other settings (ie: TCP_NODELAY)

The last time i did any socket programming was 5 years ago, but your code would block if there is no data on the socket (freeze) as you said you are using blocking sockets.

Ideally you should poll (with select) and wait for data before calling recv.

I've changed my sockets to non-blocking and it no longer freezes, but the delay grows exponentially over time while also giving me the winsock 10035 error(buffer given too many requests i believe?)

So now i will implement the select(), which i have read the documentation on but im still hazy on the function.

Does the select() basically halt winsock's recv buffer from accepting anymore data until the timeout has been reached?

I am assuming you are sending small packets over TCP. So in time, TCP would be stacking the packets until enough data is available.

You should set TCP_NODELAY.

In the last network engine I worked on (general purpose), I had thread dedicated for socket polling, getting the data and transmitting the data message to the game thread.

If you have a turn based game, TCP is fine, but if you need to have a realtime game, you need to consider UDP (http://gafferongames.com/networking-for-game-programmers/udp-vs-tcp/)

Yeah, unfortunately misinformation led me to believe that TCP was for real-time games. However, i have a small amount of people per room(2-4) so it's not "that" bad for my set-up and provides very playable latency caused by TCP. I don't feel like rewriting the 500 lines i have into the TCP networking portion of my game ill keep useing TCP for now for this project.

I'm not sure if im using select() entirely correctly on the client side:

Should i be using select() on reading and writing to my server connection?

while(true){
// clear the set ahead of time
FD_ZERO(&readfds);
FD_ZERO(&writefds);
 
// add our descriptors to the set
FD_SET(ConnectSocket, &readfds);
FD_SET(ConnectSocket, &writefds);
 
// wait until either socket has data ready to be recv()d (timeout 10 secs)
tv.tv_sec = 10;
 
//select
int rv = select(ConnectSocket+1, &readfds, &writefds, NULL, &tv);
 
if (rv == -1) {
    perror("select"); // error occurred in select()
}else if (rv == 0){
    printf("Timeout occurred! No data after 10 seconds.\n");
}else{
//perror("success");
// one or both of the descriptors have data
 
if (FD_ISSET(ConnectSocket, &writefds)) {
    //receive and store user data
    iResult = send( ConnectSocket, sendbuf, (int)strlen(sendbuf)+2, 0 );
}
if (FD_ISSET(ConnectSocket, &readfds)) {
    //receive and store user data
    iResult = recv(ConnectSocket, recvbuf, recvbuflen, 0);
    
    std::stringstream ss(recvbuf);
    ss >> otherUser.posx1;
    ss.ignore();
    ss >> otherUser.posx2;
    ss.ignore();
    ss >> otherUser.posy1;
    ss.ignore();
    ss >> otherUser.posy2;
}
}