spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

so.. ich hab jetzt den fehler etwas eingrenzen können..

der server blockiert, weil mein m_acceptSocket im fdSetRead NACH select() noch immer enthalten ist..

d.h. bei



wird accept() aufgerufen..

WIESO ist m_acceptSocket noch im fdSetRead enthalten? sollte doch von select() rausgenommen worden sein oder? O_o

ok.. also hier ist der code:

http://rafb.net/p/kokOzp17.html

*edit:
also listen() wird aufgerufen, bevor die hauptschleife beginnt.. m_acceptSocket wird auf listening gesetzt, und dann wird nur noch process() in einer endlosschleife aufgerufen (siehe code)

ich muss listen() doch nur 1mal aufrufen oder? oder muss m_acceptSocket() nach annehmen einer verbindung wieder auf listening gesetzt werden?

*edit2:

Zitat von »"MSDN"«

	C-/C++-Quelltext
1 2 3 4 5 6 7	int select( __in int nfds, __inout fd_set* readfds, __inout fd_set* writefds, __inout fd_set* exceptfds, __in const struct timeval* timeout );

The parameter readfds identifies the sockets that are to be checked for readability. If the socket is currently in the listen state, it will be marked as readable if an incoming connection request has been received such that an acceptis guaranteed to complete without blocking. For other sockets, readability means that queued data is available for reading such that a call to recv, WSARecv, WSARecvFrom, or recvfrom is guaranteed not to block.

demnach wird mein m_acceptSocket im fdSetRead gelassen, sobald eine verbindung aufgebaut werden soll..

aber danach bleibt es aus irgendeinem grund drinnen, obwohl sich nur 1 client verbunden hat, und kein weiterer sich verbinden will

Zitat von »"Nox"«

listen muss nur einmal gestartet werden, aber du rufst in dem Code nie startWinsocket auf.

der constructor der klasse sieht so aus:

Zitat von »"Nox"«

Auch kannst du das vereinfachen:

	C-/C++-Quelltext
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16	Client newConnection; int addrsize = sizeof(SOCKADDR_IN); int freeClientSlot; if(FD_ISSET(m_acceptSocket, &fdSetRead)) { newConnection.socket = accept(m_acceptSocket, (SOCKADDR*)&newConnection.addr, &addrsize); freeClientSlot = getFreeClientSlot(); if(freeClientSlot != -1) { m_Client[freeClientSlot] = newConnection; m_Client[freeClientSlot].used = true; } }

nicht nur vereinfachen, sondern auch logischer abändern.. normal müsste ich ja zuerst überprüfen ob überhaupt ein freier slot zur verfügung steht, und erst danach die verbindung annehmen.. aber der code stammt aus einem tutorial, und hab da noch keine verbesserungen vorgenommen..

Zitat von »"Nox"«

Außerdem würde ich nur auf die Fehlercodes prüfen und solche Sachen wie if(ret < 0) eher verzichten, aber das ist nur eine persönliche Meinung

auf die fehlerbehandlung bin ich noch nicht so genau eingegangen, das hab ich verschoben auf den zeitpunkt, sobald mein server die erste nachricht empfangen kann, und auf den client zurück schicken kann..
also fehlerabfragen usw. kommen noch..

....................

zu meinem problem, wieso m_acceptSocket noch im fdSetRead ist, hast du nichts gefunden?

....................

nein nein.. es läuf t so ab:

process() wird permanent aufgerufen..

--> client connected..

process() wird ausgeführt..
--> nimmt verbindung an..
process() fertig..

process() wird erneut aufgerufen:

whie(1)
{
server.process();
}

und erst genau da geht der in die accept funktion rein und bleibt da..

das hab ich bereits geändert... und das problem ist das gleiche

HIER kann man sich wer will die paar zeilen runterladen und sie compilieren. vllt. findet jemand beim debuggen einen fehler..

--> Visual Studio 2005