Discussion :

[débutant_01]

bonsoir , mon but de ce code est de faire une liste chainée , chaque liste chainée se constitue d'une structure contenant les information des étudiants (numero,nom,prenom,moyenne) et j'ai fait une boucle while qui sera désactivée lorsque je saisis un nombre différent de 1 et ainsi afficher toute la liste chainée des étudiants .
saif que le programme se plante dans l'exécution ,et parfois il saute des scanfs .

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
#include <stdio.h>
#include <stdlib.h>
 
 
 
 
 
int main(int argc , char *agrv[])
{
    int classement=1;
int nb=1;
   typedef struct info{
   char  numero[10] ;
   char nom [20];
   char prenom [20];
   char niveau [20];
   float moyenne ;
   struct info *next;
}etudiant;
etudiant *tete,*p;
  tete=NULL;
  while(nb==1)
{
 
  etudiant *eleve;
eleve=(etudiant*)malloc(sizeof(etudiant));
printf("entrer le numero del'etudiant %d: \n",classement);
 scanf("%s",eleve->numero)  ;
printf("entrer le nom del'etudiant  \n");
 scanf("%2f",eleve->nom)  ;
  printf("entrer le prenom del'etudiant  %d\n",classement);
  scanf("%2f",eleve->prenom)  ;
 printf("entrer le niveau del'etudiant  %d\n",classement);
  scanf("%2f",eleve->niveau)  ;
printf("entrer la moyenne del'etudiant %d\n",classement);
 scanf("%2f",eleve->moyenne)  ;
eleve->next=tete ;
tete=eleve ;
printf("\n\n\nsi vous voulez entrer les information d'un nouvel eleve ,taper 1\n") ;
scanf("%d",&nb)  ;
classement++;
}
p=tete; classement=1;
while(p!=NULL)
{
printf("____________________________________________");
 printf("\nles information de l'eleve numero %d\n",classement );
 printf("%s \n",p->numero) ;
  printf("%s \n",p->nom) ;
 printf("%s \n",p->prenom) ;
 printf("%s \n",p->niveau) ;
 printf("%d \n",p->moyenne) ; 
classement++;
p=p->next;
}
  return 0;
}

[Sve@r]

et parfois il saute des scanfs .

Bonjour

C'est un problème connu. Quand tu veux saisir un nombre et que tu écris scanf("%d", &nb) puis que tu tapes ton nombre (exemple "10"), alors tu tapes "1", "0" suivi de <return>. Or ce <return> n'étant pas numérique, il reste dans stdin. ensuite, au scanf() suivant, le <return> toujours présent est alors lu (surtout si le scanf() suivant attend une chaine).

scanf() n'est pas fait pour les saisies humaines. C'est bien au début pour avoir un truc de TP mais ensuite faut changer.

Si tu veux faire une saisie efficace, il faut passer par fgets()

Code c :

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

1
2
3
4
5
6
7
8
9
10
 
char tampon[100];
int nb;
while (1)
{
    printf("Entrez votre age\n");
    fgets(tampon, 100, stdin);
    if (sscanf(tampon, "%d", &age) == 1) break;
    printf("Saisie incorrecte, recommencez !!!\n");
}

La différence c'est que fgets() récupère tout le contenu de stdin jusqu'au <return> inclus. Donc stdin reste toujours clean.

Concernant ta liste chainée elle semble correcte mais écrite à la "va comme je te pousse" (c'est indenté comme un gros sale et en plus tout dans le main). Va voir ici t'as un exemple de liste chainée écrit un peu plus proprement...

[débutant_01]

d'accord merci , je vais essayer d'appliquer tes recommandation merci encore une fois

[diogene]

Ce n'est pas la cause du problème.

Le problème évoqué par Sve@r se produit lorsqu'on mélange des scanf() où les caractères <blancs> (donc en particulier le return) sont des délimiteurs et sont ignorés lorsqu'ils précèdent le champ à lire (par exemple %s, %f, %d,...) avec des scanf() où les caractères <blancs> sont traités comme n'importe quel autre caractère (par exemple %c). Or ici, on n'a que des scanf() du premier type.

Par contre, il y a deux erreurs dont la première justifie un comportement erratique :

Code :

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

scanf("%2f",eleve->moyenne)  ;

Dans un scanf() on doit préciser l'adresse de destination donc ici l'adresse de eleve->moyenne. Le & d'adressage n'a pas à figurer dans les autres scanf(), car, pour eux, l'argument est un tableau dont l'identificateur représente (dans ce contexte) l'adresse du premier élément.

Deuxième erreur

Code :

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

printf("%d \n",p->moyenne) ;

p->moyenne est un float, pas un entier.

[johnlobs]

Bonjour,

J'aimerais connaitre la logique de lecture:
numero,nom,prenom et niveau sont des chaines de caractères mais le niveau est lu avec %s par contre les autres auront droit à un %2f.

L'utilisation de la fonction scanf() pour des chaines de caractères est déconseillée!

Enfin un dernier point ,tu aura une incohérence d'affichage car tu commencera par afficher le dernier élément de la chaîne mais dans l'affichage avec la variable classement réinitialisée à 1 tu prétendras que c'est le premier élément. Ça aura une répercussion sur l'utilisateur qui à la saisie voit des numéros mais à l'affichage en verra d'autres car le premier élément saisi sera affiché en dernier!

Bon courage