Discussion :

[psyphi]

Bonjour,
Je voudrai développé ma propre fonction de fermeture de socket.
Ma fonction s'appelle close_socket_s et fait appel à close (linux) ou closesocket(windows).
Mon problème est le type de paramètre qu'il faut lui donner:
- close_socket_s(SOCKET * sock)
ou
- close_socket_s(SOCKET sock)
D'après moi un SOCKET et un int donc il faudrait logiquement passer en paramètre un pointeur de socket.
Mais d'après les tests que j'ai fait les deux ont l'air de fonctionner.
Je n'ai fait mes tests que sous windows xp.

close_socket_s(SOCKET sock):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
BOOL close_socket_s(SOCKET socket){
	BOOL ret = TRUE;
#if defined (WIN32)
		if(closesocket(socket) == SOCKET_ERROR)
		{
			if( DISPLAY_ERROR_MESSAGE )
				display_socket_error("Error.close_socket_s().closesocket()");
			ret = FALSE;
		}
#else
		if(close(socket) == SOCKET_ERROR)
		{
			if( DISPLAY_ERROR_MESSAGE )
				display_socket_error("Error.close_socket_s().close()");
			ret = FALSE;
		}
#endif
	}
	return ret;	
}

close_socket_s(SOCKET * sock):

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
BOOL close_socket_s(SOCKET * socket){
	BOOL ret = TRUE;
	if( socket == NULL )
	{
		if ( DISPLAY_ERROR_MESSAGE )
			fprintf(stderr, "Error.close_socket_s(): socket is NULL.\r\n");
		ret = FALSE;
	}
	else
	{
#if defined (WIN32)
		if(closesocket(*socket) == SOCKET_ERROR)
		{
			if( DISPLAY_ERROR_MESSAGE )
				display_socket_error("Error.close_socket_s().closesocket()");
			ret = FALSE;
		}
#else
		if(close(*socket) == SOCKET_ERROR)
		{
			if( DISPLAY_ERROR_MESSAGE )
				display_socket_error("Error.close_socket_s().close()");
			ret = FALSE;
		}
#endif
	}
	return ret;	
}

Sous windows le type de SOCKET est bien aussi un int? J'ai cherché dans les msdn sans résultats.

[hegros]

Pourquoi un pointeur ?

SOCKET ce ne serait pas plutot un unsigned int (ca devrait suffir non ?) j'ai pas trouvé dans msdn non plus...
Tu peux regarder sinon dans le fichier winsock32.h mais bon...

[psyphi]

Je pense que OSCKET doit être transmis par référence (en tant que pointeur) afin de pouvoir agir sur la variable d'origine et fermé la socket.
Quand on transmet un paramètre directement on ne modifie pas la variable d'origine:
ex:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
void inc(i){
i++;
}
 
int main(){
int i=0;
inc(i);
printf("%d",i);
return 0;
}

affichera toujours 0.
C'est pourquoi je pense si je fais un close_socket_s(SOCKET sock)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
BOOL close_socket_s(SOCKET sock){
...
closesocket(sock);
...
}
 
int main(){
...
SOCKET sock;
...
close_socket_s(sock);
...
}

Je ne fermerai pas vraiment la SOCKET? Si?

[hegros]

Beh tu ne veux pas modifier la socket (d'ailleurs quel sens cela aurait ?) tu veux juste l'utiliser.

[psyphi]

Ba si je veux le fermer c'est que je le modifie non?

[Emmanuel Delahaye]

Ba si je veux le fermer c'est que je le modifie non?

Tu demandes au système de faire les modifs. Toi, tu ne modifies rien.

quand on fait close (sock), on ne passe pas une adresse... C'est un scalaire (entier), c'est tout.

Après, rien ne t'empêche, à-la-main, de modifier la valeur de la variable socket en INVALID_SOCKET (Windows) ou -1 (Unixoide). Mais ça va laisser le système totalement indifférent...

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

   close (sock), sock = INVALID_SOCKET;

Par contre, ça peut avoir un rôle fonctionnel dans l'application.