Discussion :

[nabil23]

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 :

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
 
#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.

[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.

[Sve@r]

Bonjour

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 :

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

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.

[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.

[dalfab]

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.

[nabil23]

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.

[edgarjacobs]

Hello,

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

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
#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);
}

[Sve@r]

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.

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.

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 !!!