Winsock Problem!!

[skifteri]

Hello guys, I'm not new to C++ , but recently I jumped over to Network programming and I'm testing some stuff out.
I compiled both codes Client/Server and everything compiled just fine, but after running , I never get any responde from Client connection .. ??
thanks in advance.

//server source code

#define WIN32_LEAN_AND_MEAN

#include <winsock2.h>
#include <Ws2tcpip.h>
#include <stdio.h>
#include <iostream>
#pragma comment(lib, "Ws2_32.lib")
//GENERAL TYPES

#define DEFAULT_PORT 27015
#define CLIENTNR 10
#define CLIENTWAITING 5
char name[] = {0};

void wsacheck(int data);
int socketcheck(int data);

int main()
{
	int ErrorCheck = NULL;


	WSADATA wsa;
	ErrorCheck = WSAStartup(MAKEWORD(2, 2), &wsa);
	wsacheck(ErrorCheck);
	SOCKET srvservice;

	printf("socket()");
	socketcheck(srvservice = socket(AF_INET, SOCK_STREAM, 0));

	struct sockaddr_in srvaddr;

	srvaddr.sin_family = AF_INET;
    srvaddr.sin_addr.S_un.S_addr = INADDR_ANY;// inet_pton(AF_INET, "192.168.0.103", &srvaddr.sin_addr);
	srvaddr.sin_port = htons(DEFAULT_PORT);

	
	if (bind(srvservice, (sockaddr*)&srvaddr, sizeof(srvaddr)) == SOCKET_ERROR)
	{
		printf("bind() Failed");
		closesocket(srvservice);
	}
	printf("Listen()");
	socketcheck(listen(srvservice, CLIENTWAITING));

	SOCKET AcceptSocket;
	printf("Waiting for a client to connect...\n");
	while (1) {
		AcceptSocket = SOCKET_ERROR;
		while (AcceptSocket == SOCKET_ERROR) {
			AcceptSocket = accept(srvservice, NULL, NULL);
		}
		printf("Client Connected.\n");
		srvservice = AcceptSocket;
		break;
	}


	// Send and receive data.
	int bytesSent;
	int bytesRecv = SOCKET_ERROR;
	char sendbuf[32] = "Server: Sending Data.";
	char recvbuf[32] = "";

	bytesRecv = recv(srvservice, recvbuf, 32, 0);
	printf("Bytes Recv: %ld\n", bytesRecv);

		bytesSent = send(srvservice, sendbuf, strlen(sendbuf), 0);
	printf("Bytes Sent: %ld\n", bytesSent);


	system("pause");
	return 0;
}


/////////////////////////Function to Winsock Error checking/////////////////////////
void wsacheck(int data)
{
	if (data != 0)
	{
	printf("Initalizing Winsock Error \n");
	WSACleanup();
}
	else
		printf("Winsock Initializing successful!\n\n");
}


/////////////////////////Function to Socket Error checking/////////////////////////
int socketcheck(int data)
{
	
	int result = data;
	if (data == INVALID_SOCKET)
	{
		printf(" failed / Error Code: %i \n", WSAGetLastError());
	}
	else
	{
		printf(" Started successful\n\n");
	}
	return result;
}

Client

//client source code
#define WIN32_LEAN_AND_MEAN

#include <winsock2.h>
#include <Ws2tcpip.h>
#include <stdio.h>
#include <iostream>
#pragma comment(lib, "Ws2_32.lib")
using namespace std;
//GENERAL TYPES

#define DEFAULT_PORT 27015
#define CLIENTNR 10
#define CLIENTWAITING 5
char name[] = {0};

void wsacheck(int data);
int socketcheck(int data);

int main()
{
	int ErrorCheck = NULL;


	WSADATA wsa;
	ErrorCheck = WSAStartup(MAKEWORD(2, 2), &wsa);
	wsacheck(ErrorCheck);
	SOCKET srvservice;

	printf("socket()");
	socketcheck(srvservice = socket(AF_INET, SOCK_STREAM, 0));

	struct sockaddr_in srvaddr;

	srvaddr.sin_family = AF_INET;
    srvaddr.sin_addr.S_un.S_addr = INADDR_ANY;// inet_pton(AF_INET, "192.168.0.103", &srvaddr.sin_addr);
	srvaddr.sin_port = htons(27015);

	
	if (bind(srvservice, (sockaddr*)&srvaddr, sizeof(srvaddr)) == SOCKET_ERROR)
	{
		printf("bind() Failed");
		closesocket(srvservice);
	}
	printf("Listen()");
	socketcheck(listen(srvservice, CLIENTWAITING));

	SOCKET AcceptSocket;
	printf("Waiting for a client to connect...\n");
	while (1) {
		AcceptSocket = SOCKET_ERROR;
		while (AcceptSocket == SOCKET_ERROR) {
			AcceptSocket = accept(srvservice, NULL, NULL);
		}
		printf("Client Connected.\n");
		srvservice = AcceptSocket;
		break;
	}


	// Send and receive data.
	int bytesSent;
	int bytesRecv = SOCKET_ERROR;
	char sendbuf[32] = "Server: Sending Data.";
	char recvbuf[32] = "";

	bytesRecv = recv(srvservice, recvbuf, 32, 0);
	printf("Bytes Recv: %ld\n", bytesRecv);

		bytesSent = send(srvservice, sendbuf, strlen(sendbuf), 0);
	printf("Bytes Sent: %ld\n", bytesSent);


	system("pause");
	closesocket(srvservice);
	shutdown(srvservice, 2);
	return 0;
}


/////////////////////////Function to Winsock Error checking/////////////////////////
void wsacheck(int data)
{
	if (data != 0)
	{
	printf("Initalizing Winsock Error \n");
	WSACleanup();
}
	else
		printf("Winsock Initializing successful!\n\n");
}


/////////////////////////Function to Socket Error checking/////////////////////////
int socketcheck(int data)
{
	
	int result = data;
	if (data == INVALID_SOCKET)
	{
		printf(" failed / Error Code: %i \n", WSAGetLastError());
	}
	else
	{
		printf(" Started successful\n\n");
	}
	return result;
}

Regards

[2kaud]

while (AcceptSocket == SOCKET_ERROR)

If accept() fails, it returns INVALID_SOCKET, not SOCKET_ERROR.

[2kaud]

SOCKET AcceptSocket;
	printf("Waiting for a client to connect...\n");
	while (1) {
		AcceptSocket = SOCKET_ERROR;
		while (AcceptSocket == SOCKET_ERROR) {
			AcceptSocket = accept(srvservice, NULL, NULL);
		}
		printf("Client Connected.\n");
		srvservice = AcceptSocket;
		break;
	}

This seems a bit cumbersome. Why not something like

SOCKET AcceptSocket;
puts("Waiting for a client to connect...");
while ((AcceptSocket = accept(srvservice, NULL, NULL)) == INVALID_SOCKET);
puts("Client Connected.");
srvservice = AcceptSocket;