Problème scanf() pour lire plusieurs variables dans une boucle

Version imprimable

Bonjour,

mon pgm se plante, message : fichier.exe a cessé de fonctionner.

Je pense c'est lié à la fonction scanf() utilisée plusieurs fois dans une même boucle. Quand j'utilise une seule fois scanf() ça marche. J'ai prévu de vider le buffer mais ça n'a rien donné.

Code:

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
 
#include<stdio.h>
#include<string.h>
 
void vider_stdin(void)
{
    scanf("%*[^\n]");
    getchar();
}
 
 
int main()
{
	int const n=4;
 
        struct produit {
                            char libelle[50];
                            float prix;
                            float tva;
                           }; 
 
     struct produit tabProduit[n]; 
 
     int i; 
     char nom_type_produit[30]; 
 
      for (i=0; i<=n-1; i++) 
        {
		printf("Libellé  : "); scanf("%s", tabProduit[i].libelle); 
		vider_stdin();
 
                printf("Prix  : "); scanf("%f", &tabProduit[i].prix); 
		vider_stdin();
 
                printf("TVA  : "); scanf("%f", tabProduit[i].tva);      
		vider_stdin();
        }
		return 0;     
}

Quelqu'un peut m'aider.

Merci par avance.

06/01/2023, 13h59
nabil23

Mon programme se plante après le 1ère itération (c.à.d après avoir saisi les 3 champs une première fois ) ; puis il se bloque, il ne me donne pas la main pour saisir le "Libelle" du deuxième produit.

Merci.
06/01/2023, 14h53
Sve@r
Bonjour

Citation:

Envoyé par nabil23

Je pense c'est lié à la fonction scanf() utilisée plusieurs fois dans une même boucle. Quand j'utilise une seule fois scanf() ça marche. J'ai prévu de vider le buffer mais ça n'a rien donné.

Je pense moi que c'est lié à la différence dans l'écriture de la ligne 32 et 35. Deux instructions à priori faites pour effectuer la même action mais écrites différemment ; donc fatalement il y en aura au-moins une incorrectement écrite. Or le C ce n'est pas de la roulette, on ne peut pas miser sur le noir et en même temps sur le rouge pour se dire qu'ainsi on ne perdra rien.

Ensuite en ce qui concerne ta fonction vider_stdin(), ne connaissant pas l'astuce que tu as utilisée avec scanf() moi je l'aurais écrite ainsi
Code:

1 2 3 4 void vider_stdin(void) { int c; while ((c=fgetc(stdin)) != '\n' && c != EOF); }
Elle est d'ailleurs écrite de façon similaire dans la faq.
06/01/2023, 15h32
nabil23

Merci de m'avoir répondu.

Les lignes 32 et 35 sont différentes, il s'agit de deux champs distincts (Prix, TVA).

Pour votre code conceranant vider le buffer, le pgm ne fonctionne toujours pas.

J'ai ajouter l'opérteur d'adressage par tout dans toutes les scnaf(), il sort à la saisie de "Libellé" à la troisième itération.

Merci.
06/01/2023, 16h31
dalfab

Citation:

Envoyé par nabil23

Merci de m'avoir répondu.

Les lignes 32 et 35 sont différentes, il s'agit de deux champs distincts (Prix, TVA).

Pour votre code conceranant vider le buffer, le pgm ne fonctionne toujours pas.

J'ai ajouter l'opérteur d'adressage par tout dans toutes les scnaf(), il sort à la saisie de "Libellé" à la troisième itération.

Non, relis bien la ligne 35. Il y a un bug!

Le vidage buffer n'est pas en cause, d'ailleurs ta fonction est meilleure. Le 2 ont exactement le même comportement en mode interactif, mais si on essaie de transmettre les réponses en utilisant un pipe, la tienne seule fonctionnera.

D'ailleurs ici, on n'a pas besoin d'utiliser vider_stdin() qui est une bidouille inutile, surtout ici. Tu peux ôter ces fonctions.

Et attention, ton code fonctionne seulement si le libellé est composé d'un unique mot. Dans le cas contraire, il te faut reprendre la ligne 29. Tu peux utiliser la syntaxe du type de la ligne 7, à moins que tu ne l'ai recopié sans la comprendre.
06/01/2023, 16h41
nabil23

1 pièce(s) jointe(s)

oui, je l'ai remarqué, j'ai ajouté l'opérateur d'adressage au niveau de la ligne 35. C'est mieux mais toujours ça ne marche pas .

Le pgm fait 2 itérations sur 4 et il sort au moment de la saisie du prix à la troisième itération (voir image attachée).

Merci.

1 pièce(s) jointe(s)

Hello,

Je n'ai aucun problème avec ce code:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include<stdio.h>
 
int main(void) {
	int const n=4;
	struct produit {
		char libelle[50];
		float prix;
		float tva;
	};
	struct produit tabProduit[n];
 
	for (int i=0; i<=n-1; i++) {
		printf("Libellé  : ");
		scanf("%s", tabProduit[i].libelle);
		printf("Prix  : ");
		scanf("%f", &tabProduit[i].prix);
		printf("TVA  : ");
		scanf("%f", &tabProduit[i].tva);
	}
	return(0);
}

Pièce jointe 632022

06/01/2023, 19h21
Sve@r

Citation:

Envoyé par nabil23

Les lignes 32 et 35 sont différentes, il s'agit de deux champs distincts (Prix, TVA).

Super la remarque. Quand le sage montre la lune, l'imbécile regarde le doigt. Je n'ai pas parlé de lignes identiques mais de lignes similaires !!! Evidemment que ce sont des champs distincts, n'importe qui le voit de suite.
Mais ce sont des champs similaires, donc qui doivent être saisis de la même façon, ce qui n'est pas le cas.

Citation:

Envoyé par nabil23

Pour votre code conceranant vider le buffer, le pgm ne fonctionne toujours pas.

Normal, c'était juste une remarque sur les différentes façons de vider stdin. Le souci principal étant dans la syntaxe de la saisie. Tant que le souci n'est pas réglé, les choses annexes ne réparent rien.

Citation:

Envoyé par nabil23

J'ai ajouter l'opérteur d'adressage par tout dans toutes les scnaf(),

Il y a du mieux mais pas beaucoup plus. L'opérateur d'adressage ne se met pas tout le temps. Par exemple quand le champ à saisir est déjà une adresse. Je n'avais d'ailleurs absolument pas parlé de la ligne 29 !!!
06/01/2023, 19h27
nabil23

Bonsoir edgarjacobs

Effectivement ça marche, j'ai téléchargé Dev-C++ sur un autre ord et j'ai exécuté le pgm.

Je ne comprends pas, est ce que c'est une probmème de segmentation de mémoire, ...?

En tout cas merci beaucoup, on termine la journée sur une note positive.

MERCI.