Winsock API - send an receive data simultaneously

[freeman11777]

I'm trying to build a client and a server in the same program. For example, user 1 sends a packet of data to user 2, user 2 after receiving the packet sends back a different packet to user 1. The problem is, after running the program neither user receives the packets.

#pragma comment(lib,"ws2_32.lib")
#include <WinSock2.h>
#include <iostream>
 
static char buffer[8096 + 1];
char a[256] = {};
char MOTD[256];
 
int main()
{
	//WinSock Startup
	WSAData wsaData;
	WORD DllVersion = MAKEWORD(2, 1);
	if (WSAStartup(DllVersion, &wsaData) != 0) //If WSAStartup returns anything other than 0, then that means an error has occured in the WinSock Startup.
	{
		MessageBoxA(NULL, "WinSock startup failed", "Error", MB_OK | MB_ICONERROR);
		return 0;
	}
 
 
 
	SOCKADDR_IN addr; //Address that we will bind our listening socket to
	int addrlen = sizeof(addr); //length of the address (required for accept call)
	addr.sin_addr.s_addr = inet_addr(INADDR_ANY); //Broadcast locally
	addr.sin_port = htons(1111); //Port
	addr.sin_family = AF_INET; //IPv4 Socket
 
 
 
 
 
	SOCKET sListen = socket(AF_INET, SOCK_STREAM, NULL); //Create socket to listen for new connections
	bind(sListen, (SOCKADDR*)&addr, sizeof(addr)); //Bind the address to the socket
	listen(sListen, SOMAXCONN); //Places sListen socket in a state in which it is listening for an incoming connection. Note:SOMAXCONN = Socket Oustanding Max Connections
 
    SOCKET newConnection;
    newConnection = accept(sListen, (SOCKADDR*)&addr, &addrlen);
    if (newConnection == 0){
        std::cout << "Failed to accept the client's connection." << std::endl;
    }else{
        std::cout << "Client Connected!" << std::endl;
    }
 
 
 
    for( int i=0;i< 10;i++ ){
        if (connect(newConnection, (SOCKADDR*)&addr, sizeof(addr)) != 0) //If we are unable to connect...
        {
            MessageBoxA(NULL, "Failed to Connect", "Error", MB_OK | MB_ICONERROR);
             std::cout << WSAGetLastError();
 
        }else{
            std::cout << 123;
        }
    }
 
 
 
	  while (true){
 
            std::cin >> a;
            send(newConnection, a, sizeof(a), NULL);
 
 
            recv(newConnection, MOTD, sizeof(MOTD), NULL);
            std::cout << "MOTD:" << MOTD << std::endl;
	}
 
	system("pause");
	return 0;
}

#pragma comment(lib,"ws2_32.lib")
#include <WinSock2.h>
#include <iostream>

static char buffer[8096 + 1];
char a[256] = {};
char MOTD[256];

int main()
{
	//WinSock Startup
	WSAData wsaData;
	WORD DllVersion = MAKEWORD(2, 1);
	if (WSAStartup(DllVersion, &wsaData) != 0) //If WSAStartup returns anything other than 0, then that means an error has occured in the WinSock Startup.
	{
		MessageBoxA(NULL, "WinSock startup failed", "Error", MB_OK | MB_ICONERROR);
		return 0;
	}



	SOCKADDR_IN addr; //Address that we will bind our listening socket to
	int addrlen = sizeof(addr); //length of the address (required for accept call)
	addr.sin_addr.s_addr = inet_addr(INADDR_ANY); //Broadcast locally
	addr.sin_port = htons(1111); //Port
	addr.sin_family = AF_INET; //IPv4 Socket





	SOCKET sListen = socket(AF_INET, SOCK_STREAM, NULL); //Create socket to listen for new connections
	bind(sListen, (SOCKADDR*)&addr, sizeof(addr)); //Bind the address to the socket
	listen(sListen, SOMAXCONN); //Places sListen socket in a state in which it is listening for an incoming connection. Note:SOMAXCONN = Socket Oustanding Max Connections

    SOCKET newConnection;
    newConnection = accept(sListen, (SOCKADDR*)&addr, &addrlen);
    if (newConnection == 0){
        std::cout << "Failed to accept the client's connection." << std::endl;
    }else{
        std::cout << "Client Connected!" << std::endl;
    }



    for( int i=0;i< 10;i++ ){
        if (connect(newConnection, (SOCKADDR*)&addr, sizeof(addr)) != 0) //If we are unable to connect...
        {
            MessageBoxA(NULL, "Failed to Connect", "Error", MB_OK | MB_ICONERROR);
             std::cout << WSAGetLastError();

        }else{
            std::cout << 123;
        }
    }



	  while (true){

            std::cin >> a;
            send(newConnection, a, sizeof(a), NULL);


            recv(newConnection, MOTD, sizeof(MOTD), NULL);
            std::cout << "MOTD:" << MOTD << std::endl;
	}

	system("pause");
	return 0;
}

To copy to clipboard, switch view to plain text mode 

[high_flyer]

you while loop is only tending to the newConnection socket.
the sListen socket gets no CPU time.