Discussion :

[space-cowboy]

salut camarades;

j'ai un exercice qui consiste a creer une structure comportant le matricule ,le nom et le salaire d'un employé;ce petit programme permet d'ajouter autant d'employe sans étre obligé de donner leurs nombre au début. et retourne a la fin le nombre d'employé entrer par l'utilisateur;

voici le code dont jé rencontré 1 probléme:leprogramme saute l'étape pour entrer le nom de l'employé+ ne retourne pas leurs nombres(employés) a la fin;merci

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
#include<stdio.h>
#include<conio.h>
 
struct employe
{int mat;
char nom[20];
int sal;};
 
main()
{
  struct employe E[100];
  int n,i=0;
  do
  {
  printf("voulez vous ajouter un employe ? entrer (0/1) avec 0=non et 1=oui:");
  scanf("%d",&n);
  if(n==0)  break;
 
 
  printf("entrer le marticule :");
  scanf("%d",&E[i].mat);
  printf("entrer le nom de l'employe:");
  gets(E[i].nom);
  printf("entrer le salaire de l'employe:");
  scanf("%d",&E[i].sal);
  i++;
  }
  while(n==1);
  return i;
}

[LinuxUser]

Je ne conseille pas d'utiliser gets, comme tu as commencé avec scanf, tu peux encore l'utiliser :

Code :

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

scanf("%s",E[i].nom);

Ensuite le break dans le while à éviter!!!

[space-cowboy]

Je ne conseille pas d'utiliser gets, comme tu as commencé avec scanf, tu peux encore l'utiliser :

Code :

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

scanf("%s",E[i].nom);

Ensuite le break dans le while à éviter!!!

merci juve pour ta réponce;
mais je vois pas quel est le probléme pour gets;parcontre je l'ai utilisé dans d'autre prorammeg é ça a bien marché.

[Emmanuel Delahaye]

Je ne conseille pas d'utiliser gets, comme tu as commencé avec scanf, tu peux encore l'utiliser :

Code :

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

scanf("%s",E[i].nom);

Et en quoi c'est mieux ? De plus, on ne peux plus entrer plusieurs mots...

[orfix]

Salut,
Il faut d'abord vider le buffer sinon ton gets lira toujours la touche entrée utilisée pour valider le premier scanf ( gets n'est pas à conseiller ) ...pour plus de détails ( vider le buffer proprement + lecture sécurisée avec fgets ) je te conseil de lire la FAQ

petite remarque le #include<conio.h> est inutile pour ce bout de code ...

[Franck.H]

Il ne faut jamais utiliser gets, ni même scanf si on la maîtrise pas, préférez fgets

[space-cowboy]

j'ai remplacé le gets par un scanf;ça a bien marché .mais j'ai encore deux problémes:

-le programme ne retourne pas le ombre des employé a la fin?

-je suis pas encore convaincu de l'inefficacité de gets.je veux avoir le gets dans mon programme.

[Pacorabanix]

-je suis pas encore convaincu de l'inefficacité de gets.je veux avoir le gets dans mon programme.

FAQ C
La fonction gets() ne permet pas de spécifier la taille de la chaîne réservée à la lecture, ce qui peut entraîner de graves problèmes si la saisie est plus longue que la taille réservée.

Si quelqu'un a un nom de plus de 20 caracteres il y aura un problème, c'est pas suffisant comme raison ?

[Pacorabanix]

-le programme ne retourne pas le ombre des employé a la fin?

ben ... si mais ça ne veut rien dire ! tel quel le return de main() est utilisé pour renvoyer un code d'erreur (ou 0 si tout va bien) à l'appelant du programme (c'est-à-dire ton système). Il n'affiche rien du tout.

Pourquoi ne pas simplement afficher i à la fin ?

[space-cowboy]

ok;tout marche bien maintenat.merci bien

[Emmanuel Delahaye]

-je suis pas encore convaincu de l'inefficacité de gets.je veux avoir le gets dans mon programme.

gets() ne permet pas de limiter le nombre de caractères saisies. Je conseille d'utiliser uniquement fgets() pour les saisies suivi de fclean() (fonction 'maison' dont le code se trouve sur ce forum en cherchant un peu[1]) et de la fonction de conversion qui va bien si nécdessaire (sscanf(), strtol() etc.).


------------------------
[1] Comme c'est le début de l'année, je la republie :

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
 
#include "fclean.h"
 
#include <stdio.h>
#include <string.h>
 
int fclean (char *s_buffer, FILE * stream)
{
   int err = 0;
   if (s_buffer != NULL && stream != NULL)
   {
      char *pc = strchr (s_buffer, '\n');
 
      if (pc != NULL)           /* La saisie n'a pas ete tronquee */
      {
         /* On remplace '\n' par le caractere nul '\0' */
         *pc = 0;
      }
      else
      {
         /* La saisie a ete tronquee, on purge le flux d'entree */
         int c;
         while ((c = fgetc (stream)) != '\n' && c != EOF)
         {
         }
         err = 1;
      }
   }
   return err;
}

Pose des questions si tu ne comprends pas.

A toi de la mettre dans un fichier séparé que tu ajoutes à ton projet selon les besoins. Je te laisse écrire le header...