SIMPLE SERVEUR ECHO (TCP) MULTITHREAD MULTIPLATFORME (WINDOWS / *NIX)

Commentaire de Cphil51 le 02/03/2007 12:59:55

Je regarde a ca tout de suite.

Commentaire de Cphil51 le 02/03/2007 14:05:19

J'ai cru comprendre sur msdn qu'un CloseHandle() ne terminait pas le thread tout de suite mais le laissait juste "tranquille". J'en conclu donc que c'est à peu prêt équivalent a pthread_detach(). J'ai donc simplifié un peu le code en faisant comme avec les pthreads.

Commentaire de ncoder le 02/03/2007 15:55:05

Deuxième chose :
Pourquoi tu enchaines "CloseHandle()" immédiatement après avoir fait "CreateThread()" ?

Il faudrait attendre que les threads se soient terminés tous seuls avant de faire "CloseHandle()", non ?
Sinon ils sont arrétés immédiatement après leur lancement, non ?

Dans msdn ils utilisent WaitForMultipleObjects(MAX_THREADS, hThread, TRUE, INFINITE);, et ensuite CloseHandle().
Cf : http://msdn2.microsoft.com/en-us/library/ms682516.aspx

Là je ne comprends pas tout ...

Commentaire de ncoder le 02/03/2007 17:06:27

Merci pour tes réponses.

Quand le thread se termine tout seul, je pense qu'il faut quand meme faire CloseHandle().

En effet, TerminateThread() m'a l'air louche. On peut peut-etre risquer moins en faisant TerminateThread() suivi de CloseHandle()... (l'un termine le thread l'autre le ferme?)

Ce que tu me dis, c'est que comme CloseHandle() ne termine pas le thread mais juste "attend", dès que le thread se termine de lui-meme, CloseHandle() s'occupe du reste. Et ainsi tout fonctionne, et c'est pas génant que CreateThread soit immédiatement suivi de CloseHandle(), puisque celui-ci n'interrompt pas le thread durant son lancement.

Est-ce exact ?

Commentaire de ncoder le 02/03/2007 17:20:16

Je connaissais l'utilité de WaitForMultipleObjects()

OK merci pour l'explication de CloseHandle().
Je mélangeais le handle et le thread, en gros.

A+ et bien pour ta source, on comprend facilement ce que tu fais.
Bonne continuation

Commentaire de max12 le 02/03/2007 19:50:17 administrateur CS

Je te conseil d'utiliser uniquement PThread tant qu'a faire du multiplateforme, dans le pire des cas tu vas simplement inclure un DLL avec la version Windows, et lorsque tu voudras faire la synchronisation ce sera plus simple (et sans synchro c'est le crash).

Ensuite il y a le bug du closehandle dans ton code qui a été mensionné, il faut le faire lorsque tu en as terminé avec le thread c'est tout, dans mon cas je le faisait a la fin de la fonction du thread, mais je ne sais pas si c'est politiquement correct même si aucun problème ne survenais. Le read retourne -1 aussi parfois. Tu ne vérifie pas non plus tes memory alloc alors en cas de out of memory ton prog crash.

et finalement je suis rouillé un peu avec pthread mais il y a une commande a mettre a la fin de ta fonction client_manager quelque chose du genre p_thread_exit ou je suis pas certain (sous Nux c'était segmentation fault sinon :S)

bon c'est pas mal ce que j'avais a dire

http://www.cppfrance.com/codes/SERVEUR-MULTITHREAD-SOUS-LINUX-WINDOWS-MYSQL-VCPLUSPLUS-DEVCPLUSPLUS_40044.aspx
C'est de moi et je crois avoir été assez accomodant sur la compilation facile :P

A+

Commentaire de Cphil51 le 02/03/2007 21:05:35

Bon j'ai trouvé la ou ça crash quand le client est killé. Sinon j'ai tout refais en pthread, j'up bientôt la nouvelle version.

Commentaire de amin127 le 16/04/2007 12:11:06

bonjour, je tente de le compiler sous linux et voici ce que j'ai comme erreur :
main.c:27: error: expected specifier-qualifier-list before ‘SOCKADDR_IN’
main.c: In function ‘clientManager’:
main.c:61: error: ‘socket_t’ has no member named ‘clientThread’
main.c:66: error: ‘socket_t’ has no member named ‘data’
main.c:69: warning: incompatible implicit declaration of built-in function ‘strlen’
main.c: In function ‘main’:
main.c:118: error: ‘SOCKADDR_IN’ undeclared (first use in this function)
main.c:118: error: (Each undeclared identifier is reported only once
main.c:118: error: for each function it appears in.)
main.c:131: error: ‘socket_t’ has no member named ‘data’
main.c:132: error: ‘socket_t’ has no member named ‘data’
main.c:133: error: ‘socket_t’ has no member named ‘data’
main.c:135: error: ‘SOCKADDR’ undeclared (first use in this function)
main.c:135: error: expected expression before ‘)’ token
main.c:135: error: too few arguments to function ‘bind’
main.c:142: error: expected expression before ‘)’ token
main.c:142: error: too few arguments to function ‘accept’
main.c:142: error: ‘INVALID_SOCKET’ undeclared (first use in this function)
main.c:145: error: ‘socket_t’ has no member named ‘clientThread’
main.c:145: warning: passing argument 3 of ‘pthread_create’ from incompatible pointer type

bien à toi
bonne journée

Commentaire de amin127 le 19/04/2007 12:15:15

Bonjour,
tjrs des erreurs concernant la compilation sous linux. est ce que tu peux me dire quelles sont les options de compilations que tu utilises.
Merci

Commentaire de amin127 le 20/04/2007 12:26:41

voilà avec : "gcc -o main main.c" c'est ce que j'obtiens:

main.c:29: error: expected specifier-qualifier-list before ‘SOCKADDR_IN’
main.c: In function ‘clientManager’:
main.c:63: error: ‘socket_t’ has no member named ‘clientThread’
main.c:68: error: ‘socket_t’ has no member named ‘data’
main.c:71: warning: incompatible implicit declaration of built-in function ‘strlen’
main.c: In function ‘main’:
main.c:120: error: ‘SOCKADDR_IN’ undeclared (first use in this function)
main.c:120: error: (Each undeclared identifier is reported only once
main.c:120: error: for each function it appears in.)
main.c:133: error: ‘socket_t’ has no member named ‘data’
main.c:134: error: ‘socket_t’ has no member named ‘data’
main.c:135: error: ‘socket_t’ has no member named ‘data’
main.c:137: error: ‘SOCKADDR’ undeclared (first use in this function)
main.c:137: error: expected expression before ‘)’ token
main.c:137: error: too few arguments to function ‘bind’
main.c:144: error: expected expression before ‘)’ token
main.c:144: error: too few arguments to function ‘accept’
main.c:144: error: ‘INVALID_SOCKET’ undeclared (first use in this function)
main.c:147: error: ‘socket_t’ has no member named ‘clientThread’
main.c:147: warning: passing argument 3 of ‘pthread_create’ from incompatible pointer type

Commentaire de dijouxthomas le 22/04/2007 12:32:12

Bonjour,
moi aussi j'ai le même problème à la compilation sous linux et changer "socket_t" par "client_sock_struct" ne résout pas le problème.

main.c:29: error: expected specifier-qualifier-list before ‘SOCKADDR_IN’
main.c: In function ‘clientManager’:
main.c:63: error: ‘client_sock_struct’ has no member named ‘clientThread’
main.c:68: error: ‘client_sock_struct’ has no member named ‘data’
main.c: In function ‘main’:
main.c:120: error: ‘SOCKADDR_IN’ undeclared (first use in this function)
main.c:120: error: (Each undeclared identifier is reported only once
main.c:120: error: for each function it appears in.)
main.c:133: error: ‘client_sock_struct’ has no member named ‘data’
main.c:134: error: ‘client_sock_struct’ has no member named ‘data’
main.c:135: error: ‘client_sock_struct’ has no member named ‘data’
main.c:137: error: ‘SOCKADDR’ undeclared (first use in this function)
main.c:137: error: expected expression before ‘)’ token
main.c:137: error: too few arguments to function ‘bind’
main.c:144: error: expected expression before ‘)’ token
main.c:144: error: too few arguments to function ‘accept’
main.c:144: error: ‘INVALID_SOCKET’ undeclared (first use in this function)
main.c:147: error: ‘client_sock_struct’ has no member named ‘clientThread’
main.c:147: warning: passing argument 3 of ‘pthread_create’ from incompatible pointer type

Commentaire de amin127 le 23/04/2007 15:17:52

navré mais comme tu peux le voir cela ne change rien du tout.

:>gcc -o main main.c
main.c:31: error: expected specifier-qualifier-list before ‘socketaddr_in’
main.c: In function ‘clientManager’:
main.c:65: error: ‘socket_t’ has no member named ‘clientThread’
main.c:70: error: ‘socket_t’ has no member named ‘data’
main.c:73: warning: incompatible implicit declaration of built-in function ‘strlen’
main.c: In function ‘main’:
main.c:122: error: ‘socketaddr_in’ undeclared (first use in this function)
main.c:122: error: (Each undeclared identifier is reported only once
main.c:122: error: for each function it appears in.)
main.c:135: error: ‘socket_t’ has no member named ‘data’
main.c:136: error: ‘socket_t’ has no member named ‘data’
main.c:137: error: ‘socket_t’ has no member named ‘data’
main.c:139: error: ‘sockaddr’ undeclared (first use in this function)
main.c:139: error: expected expression before ‘)’ token
main.c:139: error: too few arguments to function ‘bind’
main.c:146: error: expected expression before ‘)’ token
main.c:146: error: too few arguments to function ‘accept’
main.c:146: error: ‘INVALID_SOCKET’ undeclared (first use in this function)
main.c:149: error: ‘socket_t’ has no member named ‘clientThread’
main.c:149: warning: passing argument 3 of ‘pthread_create’ from incompatible pointer type

Au plaisir

Commentaire de Cphil51 le 22/12/2007 23:27:44

Bon maintenant que j'ai pu installer durablement linux sur ma machine, j'ai pu regarder a vos "problèmes". Pour ceux que ca interesse encore, il suffisais de modifier le début du code dans main.c :

...
#include <string.h>


...



#ifdef WIN32

  ...

#else

  ...
  #define SOCKADDR_IN    struct sockaddr_in
  #define SOCKADDR       struct sockaddr
  #define INVALID_SOCKET -1

  ...

#endif

C'était vraiment très compliqué... Suffisait de lire la doc...

Commentaire de klapaudius le 03/10/2009 00:50:29

Pas beaucoup de commentaire dans ce source pas facile pour les débutant en serveur TCP