4. einen mit beginthread begonnen thread schließen

einen mit beginthread begonnen thread schließen

  • ?

    sorry ich muss das nochmal auffassen.
    Hier mal mein derzeitiger Code:

    SOCKET connectSocket(unsigned port)
{
	WSADATA wsa;
	SOCKADDR_IN addr;
	SOCKET acceptSocket, connectedSocket;
	if(WSAStartup(MAKEWORD(2,0), &wsa) != 0)
	{
		return SOCKET_ERROR;
	}
	acceptSocket = socket(AF_INET,SOCK_STREAM,0);
	if(acceptSocket==INVALID_SOCKET)
	{
		return SOCKET_ERROR;
	}
	memset(&addr,0,sizeof(SOCKADDR_IN));
	addr.sin_family=AF_INET;
	addr.sin_port=htons(port);
	addr.sin_addr.s_addr=INADDR_ANY;
	if(bind(acceptSocket,(SOCKADDR*)&addr,sizeof(SOCKADDR_IN)) == SOCKET_ERROR)
	{
		return SOCKET_ERROR;
	}
	if(listen(acceptSocket,10)==SOCKET_ERROR)
	{
		return SOCKET_ERROR;
	}
	if(socketoption == TRUE)
	{
		connectedSocket=accept(acceptSocket,NULL,NULL);
		if(connectedSocket==INVALID_SOCKET)
		{
			return SOCKET_ERROR;
		}
		return connectedSocket;
	}
	else
		return 1002;
}

void Thread1 (PVOID pvoid)
{
	while(1)
	{
		s1 = connectSocket(450);
		if(s1 == SOCKET_ERROR)
		{
			SetDlgItemText(FindWindow(0, ""), IDC_BUTTON2, "Start Server");
			MessageBox(0, "Konnte nicht verbinden!", szAppName, MB_ICONERROR);
		}
		else if(s1 == 1002)
		   	;
		else
		{
			break;
		}
		Sleep(100);
	}
    _endthread () ;
}

void Thread2 (PVOID pvoid)
{
	while(1)
	{
		s2 = connectSocket(451);
		if(s2 == SOCKET_ERROR)
		{
			SetDlgItemText(FindWindow(0, ""), IDC_BUTTON2, "Start Server");
			MessageBox(0, "Konnte nicht verbinden!", szAppName, MB_ICONERROR);
		}
		else if(s2 == 1002)
			;
		else
		{
			break;
		}
		Sleep(100);
	}
    _endthread () ;
}

void Thread3 (PVOID pvoid)
{
	while(1)
	{
		s3 = connectSocket(452);
		if(s3 == SOCKET_ERROR)
		{
			SetDlgItemText(FindWindow(0, ""), IDC_BUTTON2, "Start Server");
			MessageBox(0, "Konnte nicht verbinden!", szAppName, MB_ICONERROR);
		}
		else if(s3 == 1002)
			;
		else
		{
			SetDlgItemText(FindWindow(0, "Watch Out"), IDC_BUTTON2, "Connected");
			break;
		}
		Sleep(100);
	}
    _endthread () ;
}

    Die 3 Threads werden gleich bei Programmstart gestartet.
    natürlich ist socketoption aber dann noch FALSE da man noch nicht auf Start Server geklickt hat. und wenn es FALSE ist macht es nichts und geht gleich die schleife nochmal durch. Hierbei kommt natürlich wieder der connect code aber man kann ja nur 1 mal pro port binden. das heißt das es beim 2. durchlauf gar nicht mehr klappen würde. wie würdet ihr das lösen?

    0
  • ?

    ich hab die abfrage jetzt einfach über den ganzen connectteil gemacht.

    Noch ne andere Frage:
    Wenn der Client beendet wird, welchen wert hat dann Socket_error damit ich abfragen kann ob es ein fehler war oder ob der client beendet wurde?

    0
  • ?

    psuh

    0
  • J

    Das einfachste ist immer noch in die Doku zu schauen...
    z.B. bei "recv":

    If the socket is connection oriented and the remote side has shut down the connection gracefully, and all data has been received, a recv will complete immediately with zero bytes received. If the connection has been reset, a recv will fail with the error WSAECONNRESET.

    0
  • ?
     

    recvBack = recv(s1, (char *)&length, sizeof(length), 0);
if(recvBack  == WSAECONNRESET) MessageBox(0, "", "", 0);

    so funktioniert es jedenfalls nicht. wenn der client geschlossen ist tut sich nix

    0
  • J

    Dann ist die Frage, wie der Client geschlossen wird...
    Aber "recv" kommt schon sofort zurück, oder nicht? Wenn nicht, dann schliess der Client die Verbindung nicht korrekt; stört aber auch nicht, da denn irgendwan der Timeout kommt.

    0
  • ?

    server:

    recvBack = recv(s1, (char *)&length, sizeof(length), 0);
		if(recvBack  == WSAECONNRESET)
		{
			MessageBox(0, "Client has been closed!", szAppName, 0); 
			SendMessage(FindWindow(0, "progname"), WM_COMMAND, IDCANCEL, 0); 
		}

    client:

    case WM_DESTROY:
		  closesocket(s);
		  WSACleanup();
          PostQuitMessage (0) ;
          return 0 ;
    0
  • J

    markusxxx schrieb:

    recvBack = recv(s1, (char *)&length, sizeof(length), 0);
		if(recvBack  == WSAECONNRESET)

    Ähh... Du hast aber schon die Doku zu "recv" gelesen, oder???

    Falls nicht, dann hol das mal schleunigst nach!

    0
  • ?
     

    recvBack = recv(s1, (char *)&length, sizeof(length), 0);
		if(recvBack == SOCKET_ERROR && GetLastError() == WSAECONNRESET)
		{
			MessageBox(0, "Client has been closed!", szAppName, 0); 
			SendMessage(FindWindow(0, "progname"), WM_COMMAND, IDCANCEL, 0); 
		}

    funktioniert nicht 😞

    0
  • B

    MSDN schrieb:

    If no error occurs, recv returns the number of bytes received. If the connection has been gracefully closed, the return value is zero. Otherwise, a value of SOCKET_ERROR is returned, and a specific error code can be retrieved by calling WSAGetLastError.

    0
  • ?

    hm jeder sagt was anderes. Soll ich jetzt 0 oder den Error code von Jochen abfragen?

    0
  • ?

    heul langsam nervts nichts klappt -.-

    recvBack = recv(s1, (char *)&length, sizeof(length), 0);
		if(recvBack == 0)
		{
			MessageBox(0, "Client has been closed!", szAppName, 0); 
			SendMessage(FindWindow(0, "name"), WM_COMMAND, IDCANCEL, 0); 
		}

    geht natürlich auch nicht. Es scheint so als würde recv die kontrolle nicht mehr zurückgeben

    0
  • ?

    hat keiner eine Idee?

    0
  • T

    geht natürlich auch nicht. Es scheint so als würde recv die kontrolle nicht mehr zurückgeben

    Ja, recv ist blockierend. Lies die MSDN.
    Simon

    0
  • ?

    du weißt leider nicht mal um was es überhaupt geht

    0
  • B

    markusxxx schrieb:

    hm jeder sagt was anderes. Soll ich jetzt 0 oder den Error code von Jochen abfragen?

    Beides. Wenn recv 0 zurückgibt, dann wurde die Verbindung "normal" geschlossen, bei SOCKET_ERROR ist ein Fehler aufgetreten, den du mit WSAGetLastError ermittelst. edit: Und du bist dir auch sicher, dass closesocket auch aufgerufen wird?

    theta schrieb:

    recv ist blockierend.

    Je nach dem, ob der Socket blockierend ist, oder nicht? Wenn der Socket nicht blockierend ist und keine Daten vorliegen, müsste recv mit SOCKET_ERROR und WSAGetLastError()==WSAEWOULDBLOCK zurückkommen.

    0
  • ?

    hier nochma der Code:

    server:
C/C++ Code:
        recvBack = recv(s1, (char *)&length, sizeof(length), 0);
        if(recvBack  == 0)
        {
            MessageBox(0, "Client has been closed!", szAppName, 0);
            SendMessage(FindWindow(0, "progname"), WM_COMMAND, IDCANCEL, 0);}
Client
     case WM_DESTROY:
          closesocket(s);
          WSACleanup();
          PostQuitMessage (0) ;
          return 0 ;
    0
  • T

    Badestrand schrieb:

    theta schrieb:

    recv ist blockierend.

    Je nach dem, ob der Socket blockierend ist, oder nicht? Wenn der Socket nicht blockierend ist und keine Daten vorliegen, müsste recv mit SOCKET_ERROR und WSAGetLastError()==WSAEWOULDBLOCK zurückkommen.

    Korrekt. Nur habe ich im Thread nirgends was von nicht blockierenden Sockets gesehen. Habe das deshalb angenommen. Ist nicht ok, ich gebs zu.

    Simon

    0

  • markusxxx,
    du musst sämtliche Returnwerte von recv behandeln. Für > 0 ist das kein Problem, dann hast du Daten bekommen.
    0 bedeutet "Gegenstelle hat brav aufgelegt" und
    < 0 bedeutet "Irgendein Problem"

    Wenn du non-blocking Sockets verwendest musst du bei < 0 noch checken ob der Fehlercode den du von WSAGetLastError bekommst WSAEWOULDBLOCK war, denn das wäre ein Fehler der "recoverable" ist. Alles andere kannst du als "non-recoverable" ansehen, und die Verbindung schliessen.

    0
  • ?

    hell hab <= und es geht immernoch nicht ich hab die schnauze voll

    0
Anmelden zum Antworten