TCP/IP клиент -сервер

[zhenya.ya]

Проблема: мне нужно зациклить так, чтобы сервер работал не с одним клиентом, а со многими. Что я сделал не так?

сервер:

void main()
 {
	 WORD wVersionRequested;
	WSADATA wsaData;
	wVersionRequested=MAKEWORD(2,2);
	WSAStartup(wVersionRequested,&wsaData);

SOCKET s = socket(AF_INET,SOCK_STREAM,IPPROTO_TCP);

	struct sockaddr_in sin;
	sin.sin_family = AF_INET;
	sin.sin_port = htons(80);
	sin.sin_addr.s_addr = INADDR_ANY;

	 int err = bind( s, (LPSOCKADDR)&sin, sizeof(sin) );
	 err = listen( s, SOMAXCONN);

for(;;)
{	
		 char str[22];
    sockaddr_in from;
	int fromlen=sizeof(from);
	SOCKET  s1 = accept(s,(struct sockaddr*)&from, &fromlen);

  err = recv(s1, str, sizeof(str), 0 ); 
  cout<<str<<endl;
 err = send( s1, "Yes!", 5, 0 ); 
 closesocket(s1);
	}
	 	WSACleanup();
system("pause");
 }

клиент:

int main()
{
	WORD wVersionRequested;
	WSADATA wsaData;
	wVersionRequested=MAKEWORD(2,2);
	WSAStartup(wVersionRequested,&wsaData);

int rc;
SOCKET s1 = socket(AF_INET,SOCK_STREAM,IPPROTO_TCP);
 
        struct sockaddr_in anAddr;
        anAddr.sin_family = AF_INET;
        anAddr.sin_port = htons(80);
        anAddr.sin_addr.S_un.S_addr = inet_addr("127.0.0.1");
 
        connect(s1, (struct sockaddr *)&anAddr, sizeof(struct sockaddr));

		for(;;)
		{
		char str[22];
	cout<<"\n input str:"<<endl;
	cin>>str;

		      rc = send( s1, str,sizeof(str), 0 ); 
			  rc = recv(s1,str, sizeof(str), 0 ) ; 
			  cout<<str;
		}
	 closesocket(s1);
			  	WSACleanup();
                system("pause");
                return 0;
}

[veniside]

Сервер и так работает со многими клиентами, просто не одновременно, т.к. у вас клиентское соединение закрывается сразу же после получения первой порции данных. Создавайте для каждого нового клиента поток, в котором будет крутиться этот код:

 err = recv(s1, str, sizeof(str), 0 ); 
  cout<<str<<endl;
 err = send( s1, "Yes!", 5, 0 );

до тех пор, пока общение с клиентом имеет смысл.

[russian-stalker]

del
To delete

[veniside]

Так, как у ТС сейчас написано, цикл будет тупо висеть внутри акцепт(), в ожидании новых клиентов. Можно и без потоков, но тогда перед каждым блокирующим вызовом нужен как минимум select().

[zhenya.ya]

цикл будет тупо висеть внутри акцепт(), в ожидании новых клиентов

А разьве так не должно быть? Я думаю должно быть так: отработал сервер с одним клиентом и все, теперь может с другим и так должен мочь работать до бесконечности.

[profi]

В коде первого поста все не верно. Для работы со многими клиентами одновременно нужно использовать асинхронный режим сокетов. Почитай книгу М.Е.Фленова "C++ глазами хакера". Там это есть.

[veniside]

> разьве так не должно быть?

От задачи зависит. Для веб-сервера, например, это будет не очень удачное решение.

> теперь может с другим и так должен мочь работать до бесконечности

А сейчас разве не может? Просто вы не объяснили толком, что вам нужно. "чтобы сервер работал не с одним клиентом, а со многими", я так понял, что нужно несколько клиентов одновременно обслуживать.

Для работы со многими клиентами одновременно нужно использовать асинхронный режим сокетов

не обязательно, хотя такой подход лучше масштабируется.

[profi]

veniside ну тебе виднее .

[zhenya.ya]

Короче, как я понял, в моем варианте возможно работа со многими клиентами, если не одновременно?
Хм... мне просто сказал преподаватель, что вот так, как в моем примере, нужно зациклить.... таким образом типа правильно... удавлетворяет работе с клиентами, если они не одновременно?

[profi]

Если не одновременно, то да. Все правильно с кодом.