Aide sur mon Code !

Version imprimable

Aide sur mon Code !

Bonjour,
Vous pouvez m'aider de corriger mon code !. :oops:

Je suis debutant en programmation en C et C++
c'est tout mon premier programme ( Calcule de la remise qui depend de la categorie et le montant brut) .

Je rencontré ce probleme avec Visual studio C++ express :cry:
et en plus le programme me demande pas de definir la valeur de codecat (cette partie du code scanf("%S",&codecat);)

Code:

Run-Time Check Failure #2 - Stack around the variable 'codecat' was corrupted.

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
#include <stdio.h>
#include <stdlib.h>
 
int main (void)
{
  float tauxremise=0,remise=0; 
int net=0,brut=0;
char numcom;
char codecat;
 
 
 
printf("%Entrer la Valeur du numcom \n");/*Affihcer le message explicatif pour l'utilisateur*/
 
scanf("%c\0",&numcom);
/*Lire la valeur du variable , s Designe le type char*/
 
printf("%Entrer la Valeur brut \n");
 
 
scanf("%E",&brut);
/*Lire la valeur du variable , e Designe le type float */
 
printf("%Entrer la Valeur du codecat \n");
 
scanf("%S",&codecat);
/*Lire la valeur du variable , s Designe le type char*/
 
 
/*Boucle If , Verifier la valeur du variable CodeCat(categorie) , pour definir le taux du remise , cad la variable tauxremise */
 
if (codecat=='G'){
if (brut >= 50000){
tauxremise = 0.1;
}
}
 
if (codecat=='D'){
if (brut>=10000){
tauxremise=0.05;
}
}
 
if (codecat=='P'){
if (brut>=2000){
tauxremise=0.01;
}
}
 
/* Calcule de la remise */
 
remise=(brut*tauxremise);
 
/* Calculer le net */
 
net=(brut-remise);
 
/* Afficher le resultat */
 
printf("Le net egal a %d \n",net);
printf("du  montant brut %E \n",brut);
printf("De la categorie %c\0 \n",codecat);
 
}

:aie:

06/12/2007, 14h39
cmoibal
Code:

1 2 3 4 char codecat; ... scanf("%S",&codecat);
tou d'abord, la variable codecat est un caractère et non pas une chaine de caractères.....8O

En suite le %S -> %s :roll:

bon courage, il faut voire des exemples avant de débuter a programmer !!!!:king:
06/12/2007, 14h45
Emmanuel Delahaye
Citation:
Envoyé par b.soufiane

Code:

Run-Time Check Failure #2 - Stack around the variable 'codecat' was corrupted.

Code:

1 2 3 4 5 6 char codecat; <...> scanf("%S",&codecat); /*Lire la valeur du variable , s Designe le type char*/ }
Bah, tu définis une variable de 1 char. Tu espères mettre combien de caractères dedans ? Tu ne penses pas qu'un tableau de N char serait mieux ?

D'autre part, c'est "%s" et non "%S".

Enfin, cette saisie constitue un trou de sécurité. Il vaut mieux utiliser exclusivement fgets() suivi de fclean() (fonction 'communautaire' publiée maintes fois sur ce forum) et les fonction de conversion nécessaires pour construire des fonctions de saisie correctes.

corriger du code !

Citation:

Envoyé par Emmanuel Delahaye

Bah, tu définis une variable de 1 char. Tu espères mettre combien de caractères dedans ? Tu ne penses pas qu'un tableau de N char serait mieux ?

D'autre part, c'est "%s" et non "%S".

Enfin, cette saisie constitue un trou de sécurité. Il vaut mieux utiliser exclusivement fgets() suivi de fclean() (fonction 'communautaire' publiée maintes fois sur ce forum) et les fonction de conversion nécessaires pour construire des fonctions de saisie correctes.

jj'ai chang" %s en %c , pck je pense enregistrer un seul caractère dans la variable

mais j'ai toujours le même probleme !

merci pour votre aide
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 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 #include "stdafx.h" int _tmain() {/* Declaration des variable */ char codecat; float tauxremise=0,remise=0; int net=0,brut=0; char numcom; printf("%Entrer la Valeur du numcom \n");/*Affihcer le message explicatif pour l'utilisateur*/ scanf("%c",&numcom); /*Lire la valeur du variable , s Designe le type char*/ printf("%Entrer la Valeur brut \n"); scanf("%d",&brut); /*Lire la valeur du variable , e Designe le type int */ printf("%Entrer la Valeur du codecat \n"); scanf("%c",&codecat); /*Lire la valeur du variable , s Designe le type char*/ /*Boucle If , Verifier la valeur du variable CodeCat(categorie) , pour definir le taux du remise , cad la variable tauxremise */ if (codecat=='G'){ if (brut >= 50000){ tauxremise = 0.1; } } if (codecat=='D'){ if (brut>=10000){ tauxremise=0.05; } } if (codecat=='P'){ if (brut>=2000){ tauxremise=0.01; } } /* Calcule de la remise */ remise=(brut*tauxremise); /* Calculer le net */ net=(brut-remise); /* Afficher le resultat */ printf("Le net egal a %d \n",net); printf("du montant brut %d \n",brut); printf("De la categorie %c\0 \n",codecat); return 0; }

06/12/2007, 15h25
Emmanuel Delahaye
Citation:

Envoyé par b.soufiane

jj'ai chang" %s en %c , pck je pense enregistrer un seul caractère dans la variable

mais j'ai toujours le même probleme !

merci pour votre aide

Tu n'as toujours pas compris comment on mettait les balises de code ... Et si tu lisais les règles du forum ?

Citation:

Code:

printf ("%Entrer la Valeur du numcom \n");

C'est quoi ce "%E" ? Il n'y a pas de variables correspondante. Le comportement est indéfini.

Citation:

Code:

printf ("De la categorie %c\0 \n", codecat);

Pourquoi un \0 (marche de fin de chaine) au milieu de la chaine ? Ce qui est après sera ignoré. C'est voulu ?
Citation:
Code:

1 2 3 char codecat; <...> scanf ("%c", &numcom);
Tu saisies un caractère. OK. Mais n'oublie pas que tu dois 'valider' avec <enter>, ce qui génère un '\n'. Qu'est-ce qui lit ce '\n' ? Si personne ne le lit, l'appel suivant de n'importe quelle fonction d'entrée ne sera pas bloquant.

Je recommande d'utiliser exclusivement fgets() (C standard) suivie de fclean() (code 'maison' à chercher sur le forum) pour réaliser les entrées quelle qu'elles soient.
06/12/2007, 15h27
Médinoc

Pourquoi fais-tu commencer tes printf() par "%" ?

Probleme non resoulu encore

Bonjour,

Je ne comprend pas pourquoi le programme me permet pas d'enregistrer la valeur du codecat
il n'execute pas cette partie du code

Code:

scanf("%c",&codecat);

alors quand je change ce code en mettant %s

Code:

scanf("%s",&codecat);

tout marche bien sauf qu'à la fin j'ai ce probleme

Code:

Run-Time Check Failure #2 - Stack around the variable 'codecat' was corrupted.

et sa je sais que c'est dû au compretement de la variable !
mais ..

et voilà mon code source :

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
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
 
 
#include "stdafx.h"
 
 
int _tmain()
{/* Declaration des variable */
char codecat;
float tauxremise=0,remise=0; 
int net=0,brut=0;
char numcom;
 
 
 
 
printf("Entrer la Valeur du numcom \n");/*Affihcer le message explicatif pour l'utilisateur*/
 
scanf("%c",&numcom);
/*Lire la valeur du variable , s Designe le type char*/
 
printf("Entrer la Valeur brut \n");
 
 
scanf("%d",&brut);
/*Lire la valeur du variable , e Designe le type int */
 
printf("Entrer la Valeur du codecat\n");
 
scanf("%c",&codecat);
/*Lire la valeur du variable , c Designe le type char*/
 
 
/*Boucle If , Verifier la valeur du variable CodeCat(categorie) , pour definir le taux du remise , cad la variable tauxremise */
 
if (codecat=='G'){
if (brut >= 50000){
tauxremise = 0.1;
}
}
 
if (codecat=='D'){
if (brut>=10000){
tauxremise=0.05;
}
}
 
if (codecat=='P'){
if (brut>=2000){
tauxremise=0.01;
}
}
 
/* Calcule de la remise */
 
remise=(brut*tauxremise);
 
/* Calculer le net */
 
net=(brut-remise);
 
/* Afficher le resultat */
 
printf("Le net egal a %d \n",net);
printf("du  montant brut %d \n",brut);
printf("De la categorie %c \n",codecat);
 
return 0;
}

Et merci pour votre aide

06/12/2007, 19h29
Emmanuel Delahaye

Citation:

Envoyé par b.soufiane

Je ne comprend pas pourquoi le programme me permet pas d'enregistrer la valeur du codecat
il n'execute pas cette partie du code

Code:

scanf("%c",&codecat);

Pourquoi tu t'obstines avec scanf() alors que d'autres solutions universellement adoptées ont été proposées.
06/12/2007, 22h25
b.soufiane

Parce que tout simplement je veux arriver de resoudre mon probleme et apprendre à utiliser scanf() bien .

après je pourrais tenter de changer le code .

merci pour votre aide
c'est très gentil

Citation:

Envoyé par Emmanuel Delahaye

Pourquoi tu t'obstines avec scanf() alors que d'autres solutions universellement adoptées ont été proposées.
06/12/2007, 23h53
Thierry Chappuis

Citation:

Envoyé par b.soufiane

Parce que tout simplement je veux arriver de resoudre mon probleme et apprendre à utiliser scanf() bien .

après je pourrais tenter de changer le code .

merci pour votre aide
c'est très gentil

C'est une fonction relativement complexe et difficile à utiliser correctement. Beaucoup de programmeurs expérimentés décident ne pas l'utiliser, simplement parce qu'elle n'a pas été prévue pour réaliser des saisies non formatées. Si toutefois ton but est absolument de maîtriser cette fonction, voici une lecture indispensable: scanf démystifiée.

Thierry
07/12/2007, 12h38
b.soufiane

merci

Citation:

Envoyé par Thierry Chappuis

C'est une fonction relativement complexe et difficile à utiliser correctement. Beaucoup de programmeurs expérimentés décident ne pas l'utiliser, simplement parce qu'elle n'a pas été prévue pour réaliser des saisies non formatées. Si toutefois ton but est absolument de maîtriser cette fonction, voici une lecture indispensable: scanf démystifiée.

Thierry

Merci pour le cours sur scanf , c'est vrai qu'elle génére beaucoup d'erreurs mais je veux maîtriser cette fonction et faire avec mon petit programme , qui ne veut pas se terminer :aie: .

Merci pour votre aide

Bonne journée