IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Réseau C Discussion :

Scruter un tableau de sockets de service


Sujet :

Réseau C

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre averti
    Inscrit en
    Juin 2007
    Messages
    50
    Détails du profil
    Informations forums :
    Inscription : Juin 2007
    Messages : 50
    Par défaut Scruter un tableau de sockets de service
    Bonjour,

    je suis en train de faire un programme Serveur/Client en mode tcp/ip : le serveur doit dialoguer avec plusieurs clients, pour cela je créé dans mon serveur un tableau de sockets de service (une pour chaque nouveau client), et je fais un select sur la socket d'écoute ainsi que sur les sockets de services, et dès qu'il y en a une qui est éligible, je traite ma requête (dialogue read/write avec le serveur).

    Or j'ai un problème : lorsque je lance mon serveur, et que je crée un nouveau client, celui-ci se connecte au serveur (la connexion est bien établie), le tableau de socket de service est bien rempli, alors je fais un test parmi les sockets de service du tableau je cherche celles qui sont prêtes avec un FD_ISSET, mais là, il n'y en a aucune !
    Du coup mon programme attend qu'il y ait un socket éligible, et il attend indéfiniment !

    Est-ce que cela veut dire que parmi les sockets de service, elles ne sont jamais éligibles ???

    Voici un extrait de mon code pour le serveur, j'espère que quelqu'un prendra la peine d'y jeter un coup d'oeil (j'ai mis des commentaires partout), car là, je ne vois vraiment pas où est mon erreur...

    Merci d'avance à ceux qui prendront la peine de me lire.

    Serveur :
    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
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
    75
    76
    77
    78
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    141
    142
    143
    144
    145
    146
    147
    148
    149
    150
    151
    152
    153
    154
    155
    156
    157
    158
    159
    160
    161
    162
    163
    164
    165
    166
    167
    168
    169
    170
    171
    172
    173
    174
    175
    176
    177
    178
    179
    180
    181
    182
    183
    184
    185
    186
    187
    188
    189
    190
    191
    192
    193
    194
    195
    196
    197
    198
    199
    200
    201
    202
    203
    204
    205
    206
    207
    208
    209
    210
    211
    212
    213
    214
    215
    216
    217
    218
    219
    220
    221
    222
    223
    #define NB_MAX_PENDANTES 10
    #define NB_MAX_CLIENTS 10
    #define TAILLE 128
     
     
    int main(int argc, char *argv[])
    {
     
    	char *adrSnom;
    	int nport, soc_ec, i, max_soc, soc_libre;
    	int soc_service[NB_MAX_CLIENTS]; // tableau de sockets de service 
    	int res;
    	fd_set set;
    	int k, n;
     
    	struct sockaddr_in adrC, adrS;
    	struct hostent *hp;
     
    	if(argc != 3)
    	{
    		fprintf(stderr, "Usage : %s adrSnom nport\n", argv[0]);
    		exit(1);
    	}
     
    	adrSnom = argv[1];
    	nport = atoi(argv[2]);
     
     
    	/* Creation d'une socket domaine Internet et mode connecte */
    	soc_ec = socket(AF_INET, SOCK_STREAM, 0);
    	if(soc_ec < 0)
    	{
    		perror("socket ip");
    		exit(1);
    	}
     
    	/* Creation adresse serveur */
    	adrS.sin_family = AF_INET;
    	adrS.sin_port = htons(nport);	/* 0 pour attribution d'un port libre */
     
     
    	/*printf("Resolution adresse serveur...\n");*/
    	hp = gethostbyname(adrSnom);
    	if(hp == NULL)
    	{
    		herror("gethostbyname ip");
    		close(soc_ec);
    		exit(1);
    	}
     
    	memcpy(&adrS.sin_addr.s_addr, hp->h_addr, hp->h_length);
     
     
     
    	/* Attachement de la socket a l'adresse du serveur */
    	if(bind (soc_ec, (struct sockaddr *) adrS, sizeof(struct sockaddr_in))==-1)
    	{
    		perror("bind\n");
    		close(soc_ec);
    		exit(1);
    	}
     
    	/* Recuperation du port sous forme Network */
    	if(getsockname (soc_ec, (struct sockaddr *) adrS, &adrlen)<0)
    	{
    		perror("getsockname ip");
    		close(soc_ec);
    		exit(1);
    	}
    	printf("Port %d ouvert\n", ntohs(adrS.sin_port));
     
    	printf("Serveur : transformation socket en socket d'ecoute...\n");
    	/* Transformation de la socket en socket d'ecoute */
    	if(listen(soc_ec, NB_MAX_PENDANTES) == -1)
    	{
    		perror("listen");
    		close(soc_ec);
    		exit(1);
    	}
     
    	/* Initialisation : pour le moment le descripteur max est soc_ec */
    	max_soc = soc_ec;
    	soc_libre = 0;	//L'indice du 1er emplacement de libre dans le tableau de sockets est 0
     
     
    	/* Initialisation du tableau de sockets de service */
    	for(i=0; i<NB_MAX_CLIENTS; i++)
    		soc_service[i] = -1;
     
     
     
    	while(1)
    	{	
    		printf("Serveur : en attente de connexions...\n");
     
    		FD_ZERO(&set);
    		FD_SET(soc_ec, &set);
     
    		printf("Dans le while\n");
     
    		/* On scrute parmi les sockets de service presentes dans le tableau */
    		for(i=0; i<NB_MAX_CLIENTS; i++)
    			if(soc_service[i] >= 0)
    				FD_SET(soc_service[i], &set);
     
     
    		printf("Avant le select\n");
     
    		res = select(max_soc+1, &set, NULL, NULL, NULL);	
     
    		if(res<0) {perror("select"); goto fin_serveur;}
     
    		if(res>0)
    		{
    			printf("Apres le select\n");
     
    			/* Si la socket d'ecoute est prete et qu'il y a des sockets de service libres : acceptation d'un nouveau client*/
    			if( FD_ISSET(soc_ec, &set) && soc_libre < NB_MAX_CLIENTS )
    			{
    				socklen_t adrlen = sizeof(struct sockaddr_in);
    				soc_service[soc_libre] = accept(soc_ec, (struct sockaddr *) adrC, &adrlen);
     
    				if(soc_service[soc_libre]<0) {perror("accept"); goto fin_serveur;}
     
    				//Mise a jour du fd max parmi les sockets de service (pour le select)
    				if(max_soc < soc_service[soc_libre])
    					max_soc = soc_service[soc_libre];
     
     
    				printf("Serveur : connexion etablie...\n");
     
     
    				//Mise a jour de l'indice soc_libre -> trouver un emplacement de libre dans le tableau 
    				for(i=0; i<NB_MAX_CLIENTS; i++)
    					if(soc_service[i]<0)	//des que un emplacement libre
    						break;		//sortir de ce for 
     
     
    				printf("Apres le 1er for\n");
     
     
    				//Si cet indice vaut NB_MAX_CLIENTS, alors il n'y a plus aucune place de libre
    				if(i == NB_MAX_CLIENTS)
    				{
    					printf("Serveur : pile des clients pleine\n");
    					soc_libre = NB_MAX_CLIENTS;
    					continue;
    				}
     
    				//Recuperation de cet indice 
    				soc_libre = i;
     
    				printf("La valeur de soc_libre = %d\n", soc_libre);
     
     
    				//Recherche de l'indice d'une socket de service prete 
     
    				/*** PROBLEME : IL N'Y A JAMAIS DE SOCKETS DE SERVICE QUI SOIENT ELIGIBLES ! ***/
    				for(i=0; i<NB_MAX_CLIENTS; i++)
    				{
    					printf("---- [i=%d] for additionnel\n",i);
    					if(soc_service[i]!=-1 && FD_ISSET(soc_service[i], &set))
    						printf("FD_ISSET pour i=%d\n", i);
    					else 
    						printf("pas de FD_ISSET i=%d\n", i);
    				}
     
    				printf("Sortie du for additionnel\n");
     
     
    				i=0;
    				while(i<NB_MAX_CLIENTS)
    				{
    					if(soc_service[i]!=-1 && FD_ISSET(soc_service[i], &set))
    						break;
    					else i++;
    				}
     
     
    				/*** DU COUP ICI : i=NB_MAX_CLIENTS ***/
     
    				printf("Apres recherche : i = %d\n", i);
     
    				/* Si aucune n'est prete : on continue (bouclera jusqu'a ce qu'il y en ai une de prete) */
    				if(i==NB_MAX_CLIENTS)
    				{
    					printf("------ Aucune n'est prete\n");
    					continue;
    				}
     
    				/*** ET ALORS LE PROGRAMME REMONTE AU WHILE POUR ATTENDRE QU'IL Y EN AIT UNE QUI SOIT ELIGIBLE : IL SE BLOQUE***/
     
    				printf("Avant le FD_ISSET\n");
     
    				if(soc_service[i]!=-1 && FD_ISSET(soc_service[i], &set))
    				{
    					printf("Serveur : Traitement de la requete...\n");
     
    					(...)
     
     
    					//Une fois la requete traitee pour ce client : liberation de l'emplacement correspondant dans le tableau
    					soc_service[i] = -1;
    					if(soc_libre == NB_MAX_CLIENTS) //Si plus de place libre 
    						soc_libre = i;  //Une place vient de se liberer 
     
    				} 
     
    			} /* Fin du if( FD_ISSET(soc_ec, &set) && soc_libre < NB_MAX_CLIENTS ) */
     
     
    		}/*fin du if(res>0)*/
     
     
    	}/* Fin du while(1) */
     
    	fin_serveur : printf("Serveur : fin du serveur, fermeture sockets...\n");
    		close(soc_ec);
    		/* Fermeture de toutes les sockets de service */
    		for(i=0; i<NB_MAX_CLIENTS; i++)
    			close(soc_service[i]);
    		exit(0);
    }

  2. #2
    Expert éminent
    Avatar de Emmanuel Delahaye
    Profil pro
    Retraité
    Inscrit en
    Décembre 2003
    Messages
    14 512
    Détails du profil
    Informations personnelles :
    Âge : 68
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Retraité

    Informations forums :
    Inscription : Décembre 2003
    Messages : 14 512
    Par défaut
    Citation Envoyé par honeydew Voir le message
    Voici un extrait de mon code pour le serveur, j'espère que quelqu'un prendra la peine d'y jeter un coup d'oeil (j'ai mis des commentaires partout), car là, je ne vois vraiment pas où est mon erreur...
    Moi, je dis que select(), ça fait faire des usines à gaz... Utilise les threads, c'est beaucoup plus simple.

  3. #3
    Expert confirmé

    Profil pro
    Inscrit en
    Janvier 2007
    Messages
    10 610
    Détails du profil
    Informations personnelles :
    Âge : 67
    Localisation : France

    Informations forums :
    Inscription : Janvier 2007
    Messages : 10 610
    Billets dans le blog
    2
    Par défaut
    moi je dis qu'une modularité et une découpe fonctionnelle sont essentielles quand on fait des actions répétitives

  4. #4
    Membre averti
    Inscrit en
    Juin 2007
    Messages
    50
    Détails du profil
    Informations forums :
    Inscription : Juin 2007
    Messages : 50
    Par défaut
    Bonjour et merci pour vos réponses,
    Citation Envoyé par Emmanuel Delahaye Voir le message
    Moi, je dis que select(), ça fait faire des usines à gaz... Utilise les threads, c'est beaucoup plus simple.
    L'ennui c'est que je n'ai pas le choix, le sujet du projet impose d'utiliser une scrutation avec select() ...

    Citation Envoyé par souviron34
    moi je dis qu'une modularité et une découpe fonctionnelle sont essentielles quand on fait des actions répétitives
    "découpe fonctionnelle" = créer plusieurs petites fonctions plutot que de tout faire dans le main, c'est bien ça ? Je vais essayer de découper mon programme en plusieurs fonctions et voir ce que ça donne.

    Par contre, je ne vois pas ce que tu veux dire par "modularité"....

    Encore merci.
    honeydew

  5. #5
    Rédacteur

    Avatar de ram-0000
    Homme Profil pro
    Consultant en sécurité
    Inscrit en
    Mai 2007
    Messages
    11 517
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 62
    Localisation : France, Haute Garonne (Midi Pyrénées)

    Informations professionnelles :
    Activité : Consultant en sécurité
    Secteur : High Tech - Opérateur de télécommunications

    Informations forums :
    Inscription : Mai 2007
    Messages : 11 517
    Par défaut
    Citation Envoyé par honeydew Voir le message
    Par contre, je ne vois pas ce que tu veux dire par "modularité"....
    Tu mets les fonctions du serveur (open, bind, select, accept, close, ...) dans un module/fichier qui s'appelle serveur.c et serveur.h

    Les fonction du client (attente d'une commande, réponse à la commande, changement d'état, fermeture de la socket, ...) dans un module/fichier qui s'appelle client.c et client.h.

    Ton main dans un fichier qui s'appelle main.c et qui appelle OpenServer(...) dans serveur.c
    etc

    Cela fait des unités de programmation (modules) plus faciles à gérer et que tu peux éventuellement sous traiter à quelqu'un d'autre (si les interfaces sont clairement définies.
    Raymond
    Vous souhaitez participer à la rubrique Réseaux ? Contactez-moi

    Cafuro Cafuro est un outil SNMP dont le but est d'aider les administrateurs système et réseau à configurer leurs équipements SNMP réseau.
    e-verbe Un logiciel de conjugaison des verbes de la langue française.

    Ma page personnelle sur DVP
    .

  6. #6
    Rédacteur
    Avatar de 3DArchi
    Profil pro
    Inscrit en
    Juin 2008
    Messages
    7 634
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Juin 2008
    Messages : 7 634
    Par défaut
    Citation Envoyé par Emmanuel Delahaye Voir le message
    Moi, je dis que select(), ça fait faire des usines à gaz... Utilise les threads, c'est beaucoup plus simple.
    Pourrais-tu développer. En effet, j'ai plutôt tendance à préférer select. Bien sûr, il faut faire attention quand on en sort de ne pas avoir une superbe fonction de plusieurs centaines de ligne gérant des micros cas particulier, mais c'est le b.a.ba. Intuitivement, j'avais l'impression qu'un select sur plusieurs sockets sera plus efficace que plusieurs threads sur une socket chacun, mais peut-être je me trompe. L'autre avantage que je vois, c'est que select c'est à priori portable. Ce n'est pas le cas des threads.

  7. #7
    Expert éminent
    Avatar de Emmanuel Delahaye
    Profil pro
    Retraité
    Inscrit en
    Décembre 2003
    Messages
    14 512
    Détails du profil
    Informations personnelles :
    Âge : 68
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Retraité

    Informations forums :
    Inscription : Décembre 2003
    Messages : 14 512
    Par défaut
    Citation Envoyé par 3DArchi Voir le message
    L'autre avantage que je vois, c'est que select c'est à priori portable. Ce n'est pas le cas des threads.
    • Les pthreads sont portables.
    • Sous Windows, select() ne fonctionne que sur les sockets ... (impossible de gérer stdio)

    Le mécanisme qui permet à définir le 1er paramètre de select() est tordu et source de nombreuses erreurs ...

  8. #8
    Rédacteur
    Avatar de 3DArchi
    Profil pro
    Inscrit en
    Juin 2008
    Messages
    7 634
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations forums :
    Inscription : Juin 2008
    Messages : 7 634
    Par défaut
    Citation Envoyé par Emmanuel Delahaye Voir le message
    • Les pthreads sont portables.
    • Sous Windows, select() ne fonctionne que sur les sockets ... (impossible de gérer stdio)

    Le mécanisme qui permet à définir le 1er paramètre de select() est tordu et source de nombreuses erreurs ...
    Pour les pthreads, j'avoue que je ne connais pas. En général, j'utilise l'API spécifique de la plateforme pour cela.
    Hormis ce préambule, j'avoue que je ne suis toujours pas convaincu de cette aversion pour le select. Probablement lié à des expériences différentes

  9. #9
    Expert éminent
    Avatar de Emmanuel Delahaye
    Profil pro
    Retraité
    Inscrit en
    Décembre 2003
    Messages
    14 512
    Détails du profil
    Informations personnelles :
    Âge : 68
    Localisation : France, Paris (Île de France)

    Informations professionnelles :
    Activité : Retraité

    Informations forums :
    Inscription : Décembre 2003
    Messages : 14 512
    Par défaut
    Citation Envoyé par 3DArchi Voir le message
    Pour les pthreads, j'avoue que je ne connais pas. En général, j'utilise l'API spécifique de la plateforme pour cela.
    http://emmanuel-delahaye.developpez.com/pthreads.htm
    Hormis ce préambule, j'avoue que je ne suis toujours pas convaincu de cette aversion pour le select. Probablement lié à des expériences différentes
    Comment tu gères les sockets et les saisies utilisateur sous Windows avec select() ?

Discussions similaires

  1. Envoi/reception tableau byte socket TCP
    Par jhon_milou dans le forum Entrée/Sortie
    Réponses: 2
    Dernier message: 01/06/2007, 11h40
  2. Comment envoyer un grand tableau avec socket UDP
    Par jhon_milou dans le forum Entrée/Sortie
    Réponses: 8
    Dernier message: 29/05/2007, 09h36
  3. Scruter un tableau
    Par jojoc dans le forum C++Builder
    Réponses: 4
    Dernier message: 27/03/2007, 17h35
  4. Comment transmettre un tableau par socket
    Par vodevil dans le forum Programmation et administration système
    Réponses: 11
    Dernier message: 08/04/2006, 14h19
  5. retour tableau d'objets par service web axis jboss
    Par TrollMaster dans le forum XML/XSL et SOAP
    Réponses: 6
    Dernier message: 27/11/2005, 21h45

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo