Discussion :

[icode15]

Bonjour
voici mon code

Code :

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

1
2
3
4
5
6
7
do{
    printf("Entrer Quantités Produits \n");
    scanf("%d",&q_produits);
    printf("Entrer Prix Unitaire du Produit \n");
    scanf("%d",&prix_uni);
 }
while((q_produits<=0) || (&prix_uni<=0));

je voudrais que la boucle se repète tant qu' au moins un des 2 est negatif

Merci

[NeoKript]

Salut,

penses à présenter ton message correctement (balise CODE).
Pour ce qui est de ton problème, je pense que tu mélange variable et adresse de variable.

&prix_uni est l’adresse de la variable prix_uni et non sa valeur (je suppose que prix_uni est un entier ou un flottant).

Du coup il suffit d’enlever le & sur ta seconde condition.

Code :

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

1
2
3
4
5
6
7
do
{
        printf("Entrer Quantités Produits \n");
        scanf("%d", &q_produits);
        printf("Entrer Prix Unitaire du Produit \n");
        scanf("%d", &prix_uni);
} while (q_produits <= 0 || prix_uni <= 0);

NeoKript

[icode15]

Merci pour la remarque sur la balise CODE
pour ce qui est du & c'etait une erreur de frappe
merci

[Obsidian]

Dans ce cas, ça a l'air correct. Quels résultats obtiens-tu ?

[Lucien63]

Je metterais plutôt ça :

Code :

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

1
2
//...
while (q_produits < 0 || prix_uni < 0);

Je ne pense pas que zero soit négatif

[icode15]

resultat souhaité obtenu
on sort de la boucle lorsque les 2 nombres sont positifs .
Maintenant je rame sur ce code
je voudrais creer une boucle qui est
tant que l'utilisateur entre une lettre differente de 'n ' ou 'N' on reste dans la boucle en lui posant tjrs la meme question
Continuer ?(O / N)

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
 
#include<stdio.h>
#include<conio.h>
int prix_uni,q_produits;
long t_achats;
char rep;
main(void){
t_achats=0;
rep='n';
 while(rep!='N' || rep!='n'){
do{
printf("Entrer Quantités Produits \n"); 
scanf("%d",&q_produits); 
printf("Entrer Prix Unitaire du Produit \n"); 
scanf("%d",&prix_uni); 
} 
while ((q_produits <= 0) || (prix_uni <= 0));
t_achats=t_achats + q_produits*prix_uni;
printf("Total=%d% \n",t_achats);
printf("Continuer ?(n ou N pour arreter) \n");                                    
scanf("%c",&rep);                                   
}

Lorsque j'entre dans la boucle la première fois et que j'arrive sur

Code :

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

printf("Continuer ?(n ou N pour arreter) \n");

je n'ai pas la possibilité d'entrer un caractere et la boucle reprend aussitot

[icode15]

Je metterais plutôt ça :

Code :

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

1
2
//...
while (q_produits < 0 || prix_uni < 0);

Je ne pense pas que zero soit négatif

je ne veux pas qu'il entre de prix nulle ou de quantité nulle
donc le code sera

Code :

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

while ((q_produits <= 0) || (prix_uni <= 0));

pas de probleme à ce niveau

[dré kam]

resultat souhaité obtenu

Code :

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

1
2
3
4
5
6
7
 
t_achats=t_achats + q_produits*prix_uni;
printf("Total=%d% \n",t_achats);
printf("Continuer ?(n ou N pour arreter) \n");                                    
scanf("%c",rep);                                   
}
 while(rep!='N' && rep!='n');

La derniere accolade (juste avant le while) n'a jamais été ouverte.
J'aurais fait:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
do
{
printf("Continuer ?(n ou N pour arreter) \n");                                    
scanf("%c",rep);                                   
}
 while((rep!='O') && (rep!='o')&&(rep!='N') && (rep!='n'));
if((rep=='O') || (rep=='o'))
{
//reste du programme
}
return 0;

[diogene]

C'est une conséquence du scanf précédent :

Code :

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

scanf("%d",&prix_uni);

celui-ci laisse dans le tampon clavier au minimum le caractère '\n' qui donc est lu dans la foulée par le scanf("%c",...) sans qu'il soit besoin d'entrer des caractères supplémentaires au clavier.

Solution : vider le buffer clavier après la lecture de prix_uni.
- ne pas utiliser fflush() qui est défini pour des flux de sortie
- voir la FAQ qui propose une fonction pour le faire
- ou modifier le scanf() pour qu'il le fasse :

Code :

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

scanf("%d%*[^\n]%*c",&prix_uni);

[dré kam]

Il y a plus simple mais moins rigoureux: mettre un getchar() après le scanf pour qu'il récupère le \n.

[icode15]

voici ce que j'ai obtenu en bidouillant le code
je reconnais que j'ai du mal à expliquer les lignes 23 et 24
mais ça marche
si quelqu'un pourait expliquer ?

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
 
#include<stdio.h>
#include<conio.h>
int prix_uni,mo,q_produits,somme,monnaie;
long t_achats;
char rep;
main(void){
t_achats=0;
do{
 do{
printf("ENTRER QUANTITES PRODUITS \n"); 
scanf("%d",&q_produits); 
}
while (q_produits <= 0 );
do{
printf("ENTRER PRIX UNITAIRE [un prix logique en CFA]  \n"); 
scanf("%d",&prix_uni); 
}
while ((prix_uni <= 0) ||prix_uni%5!=0 );
t_achats=t_achats + q_produits*prix_uni;
printf("TOTAL ACHATS=%d% \n",t_achats);
printf("CONTINUER ?['n' ou 'N' POUR ARRETER] \n"); 
scanf("%c",&rep); 
scanf("%c",&rep);
}
while(rep!='N'&& rep!='n');

très complexe ce scanf!!

[dré kam]

C'est parce que le scanf juste avant (qui permet d'entrer le prix unitaire) récupère le nombre et le '\n' du caractère "entrée". Donc quand tu rappelle un scanf le \n est toujours dans le buffer. Le 1er scanf récupère ce '\n' et le 2e la valeur. Ca fait toujours ce genre de problèmes quand tu mets un scanf dans une boucle.

[icode15]

C'est parce que le scanf juste avant (qui permet d'entrer le prix unitaire) récupère le nombre et le '\n' du caractère entré. Donc quand tu rappelle un scanf le \n est toujours dans le buffer. Le 1er scanf récupère ce '\n' et le 2e la valeur. Ca fait toujours ce genre de problèmes quand tu mets un scanf dans une boucle.

Merci

[Sve@r]

très complexe ce scanf!!

Petit rappel: scanf() est fait pour récupérer des datas issues d'un flux formaté. Et ce que tape l'utilisateur est tout sauf formaté. Que pasa si l'utilisateur tape "toto" là où tu lui demandes un long ???

Solution plus robuste

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
#include<stdio.h>
int main(void){
    int q_produits
    char saisie[80];
    while(1) {
        printf("ENTRER QUANTITES PRODUITS \n"); 
        fgets(saisie, 80, stdin);
        if (sscanf(saisie, "%d", &q_produits) == 1 && q_produits > 0) break;
        printf("Erreur de saisie - Recommencez\n");
    }
}