Correction de mon code par Bérith - page 1

a un moment dans le cour C que je suis en train de suivre, mateo demande de réaliser une calculatrice pour s'entrainer. J'ai essayé et voila ce que sa donne pourtant sa ne marche pas. Pouvez vous me trouver ma faute svp

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
int main(int argc, char *argv[])
{
long Calcul;
printf("== Calculatrice ==\n\n"); //Page d'acceuil
printf("Quelle operation voulez vous realiser?\n");
printf("1. Addition\n");
printf("2. Soustraction\n");
printf("3. Multiplication\n");
printf("4. Division\n");
printf("\nQuel est votre choix?\n");
scanf("%ld", & Calcul);
printf("\n\n");
switch (Calcul) //Après avoir choisit
case 1: //addition
long resultat = 0, nombre1 = 0, nombre2 = 0;
ptintf("Vous avez choisit l'addition\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld", &nombre2);
resultat = nombre1 + nombre2;
printf ("%ld + %ld = %ld\n", nombre1, nombre2, resultat);
break;
case 2: //soustraction
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit la soustraction\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 - nombre2;
printf("%ld - %ld = %ld", nombre1, nombre2, resultat);
break;
case 3: //multiplication
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit la soustraction\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 * nombre2;
printf("%ld X %ld = %ld", nombre1, nombre2, resultat);
break;
case 4: //division
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit la division\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 / nombre2;
printf("%ld / %ld = %ld", nombre1, nombre2, resultat);
break;
default: //si erreur
printf("Vous n'avez pas choisit un nombre correct.\n");
break;
system("PAUSE");
return 0;
}

est ce que sa pourait venir de mes décalages qui sont un peu mal foutu?

oui je l'ai remarqué aussi juste avant de lire le message
alors voici le rapport

Citation : Rapport d'erreur

1>------ Début de la génération : Projet : Calculatrice, Configuration : Debug Win32 ------
1>Compilation en cours...
1>main.c
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(15) : warning C4996: 'scanf': This function or variable may be unsafe. Consider using scanf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files\microsoft visual studio 9.0\vc\include\stdio.h(306) : voir la déclaration de 'scanf'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(20) : error C2143: erreur de syntaxe : absence de ';' avant 'type'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(23) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(25) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(27) : error C2065: 'resultat' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(27) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(27) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(28) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(28) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(28) : error C2065: 'resultat' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(32) : error C2143: erreur de syntaxe : absence de ';' avant 'type'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(35) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(37) : error C2059: erreur de syntaxe : ')'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(37) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(39) : error C2065: 'resultat' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(39) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(39) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(40) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(40) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(40) : error C2065: 'resultat' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(44) : error C2143: erreur de syntaxe : absence de ';' avant 'type'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(47) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(49) : error C2059: erreur de syntaxe : ')'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(49) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(51) : error C2065: 'resultat' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(51) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(51) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(52) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(52) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(52) : error C2065: 'resultat' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(55) : error C2143: erreur de syntaxe : absence de ';' avant 'type'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(58) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(60) : error C2059: erreur de syntaxe : ')'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(60) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(62) : error C2065: 'resultat' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(62) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(62) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(63) : error C2065: 'nombre1' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(63) : error C2065: 'nombre2' : identificateur non déclaré
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(63) : error C2065: 'resultat' : identificateur non déclaré
1>Le journal de génération a été enregistré à l'emplacement "file://c:\Users\Pierre\Documents\Visual Studio 2008\Projects\Calculatrice\Calculatrice\Debug\BuildLog.htm"
1>Calculatrice - 39 erreur(s), 1 avertissement(s)
========== Génération : 0 a réussi, 1 a échoué, 0 mis à jour, 0 a été ignoré ==========

pour le code:

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
int main(int argc, char *argv[])
{
long Calcul;
printf("== Calculatrice ==\n\n"); //Page d'acceuil
printf("Quelle operation voulez vous realiser?\n");
printf("1. Addition\n");
printf("2. Soustraction\n");
printf("3. Multiplication\n");
printf("4. Division\n");
printf("\nQuel est votre choix?\n");
scanf("%ld", & Calcul);
printf("\n\n");
switch (Calcul) //Après avoir choisit
{
case 1 : //addition
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit l'addition\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld", &nombre2);
resultat = nombre1 + nombre2;
printf ("%ld + %ld = %ld\n", nombre1, nombre2, resultat);
break;
case 2 : //soustraction
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit la soustraction\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 - nombre2;
printf("%ld - %ld = %ld", nombre1, nombre2, resultat);
break;
case 3 : //multiplication
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit la soustraction\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 * nombre2;
printf("%ld X %ld = %ld", nombre1, nombre2, resultat);
break;
case 4 : //division
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit la division\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 / nombre2;
printf("%ld / %ld = %ld", nombre1, nombre2, resultat);
break;
default: //si erreur
printf("Vous n'avez pas choisit un nombre correct.\n");
break;
}
system("PAUSE");
return 0;
}

je m'occupe du deplacement des variables tout de suite (mais je laisse long pour une calculatrice il faut des gros chifres^^)

EDIT:
apres avoir déplacé mes variables (et rendu le code plus lisible...) voila ce que j'ai

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
int main(int argc, char *argv[])
{
printf("== Calculatrice ==\n\n"); //Page d'acceuil
printf("Quelle operation voulez vous realiser?\n");
printf("1. Addition\n");
printf("2. Soustraction\n");
printf("3. Multiplication\n");
printf("4. Division\n");
printf("\nQuel est votre choix?\n");
scanf("%ld", & Calcul);
printf("\n\n");
switch (Calcul) /Après avoir choisit
{
case 1 : //addition
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit l'addition\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld", &nombre2);
resultat = nombre1 + nombre2;
printf ("%ld + %ld = %ld\n", nombre1, nombre2, resultat);
break;
case 2 : //soustraction
printf("Vous avez choisit la soustraction\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 - nombre2;
printf("%ld - %ld = %ld", nombre1, nombre2, resultat);
break;
case 3 : //multiplication
printf("Vous avez choisit la soustraction\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 * nombre2;
printf("%ld X %ld = %ld", nombre1, nombre2, resultat);
break;
case 4 : //division
printf("Vous avez choisit la division\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld"), &nombre2);
resultat = nombre1 / nombre2;
printf("%ld / %ld = %ld", nombre1, nombre2, resultat);
break;
default: //si erreur
printf("Vous n'avez pas choisit un nombre correct.\n");
break;
}
system("PAUSE");
return 0;
}

et voici le rapport:

Citation : Rapport d'erreur

1>1>------ Début de la génération : Projet : Calculatrice, Configuration : Debug Win32 ------
1>Compilation en cours...
1>main.c
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(15) : warning C4996: 'scanf': This function or variable may be unsafe. Consider using scanf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files\microsoft visual studio 9.0\vc\include\stdio.h(306) : voir la déclaration de 'scanf'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(22) : warning C4996: 'scanf': This function or variable may be unsafe. Consider using scanf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files\microsoft visual studio 9.0\vc\include\stdio.h(306) : voir la déclaration de 'scanf'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(24) : warning C4996: 'scanf': This function or variable may be unsafe. Consider using scanf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files\microsoft visual studio 9.0\vc\include\stdio.h(306) : voir la déclaration de 'scanf'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(32) : warning C4996: 'scanf': This function or variable may be unsafe. Consider using scanf_s instead. To disable deprecation, use _CRT_SECURE_NO_WARNINGS. See online help for details.
1> c:\program files\microsoft visual studio 9.0\vc\include\stdio.h(306) : voir la déclaration de 'scanf'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(34) : error C2059: erreur de syntaxe : ')'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(46) : error C2059: erreur de syntaxe : ')'
1>c:\users\pierre\documents\visual studio 2008\projects\main.c(57) : error C2059: erreur de syntaxe : ')'
1>Le journal de génération a été enregistré à l'emplacement "file://c:\Users\Pierre\Documents\Visual Studio 2008\Projects\Calculatrice\Calculatrice\Debug\BuildLog.htm"
1>Calculatrice - 3 erreur(s), 4 avertissement(s)
========== Génération : 0 a réussi, 1 a échoué, 0 mis à jour, 0 a été ignoré ==========

Bérith

9 mars 2008 à 8:57:45

casper

9 mars 2008 à 9:22:25

Salut,

Alors je pense que tu devrais déclarer tes variables en tout début de fonction ce sera plus lisible pour quelqu'un qui n'a pas participé à ton petit projet ;-).
Alors dans la famille fautes de frappes je dirais déjà à la ligne 20 la fonction printf est nommée "ptintf". Sinon faudrait que tu postes le message d'erreur car je n'ai pas de compilo sous la main...

Ah et aussi tu pourrais utiliser des int à la place des long, ça devrait être suffisant dans le cas présent ;-).

Bye

toto07

9 mars 2008 à 9:27:11

il manque les accolades pour ton switch

kakulupia

9 mars 2008 à 9:40:00

+1 toto, tu as été plus vite que moi

9 mars 2008 à 11:48:29

albin12

9 mars 2008 à 13:25:35

voci le code qui fonctionne:
#include <stdio.h>
#include <stdlib.h>
#include <math.h>

int main(int argc, char *argv[])
{
long Calcul=0;
printf("== Calculatrice ==\n\n"); //Page d'acceuil
printf("Quelle operation voulez vous realiser?\n");
printf("1. Addition\n");
printf("2. Soustraction\n");
printf("3. Multiplication\n");
printf("4. Division\n");
printf("\nQuel est votre choix?\n");
scanf("%ld", & Calcul);
printf("\n\n");
if(Calcul==1)
{ //addition
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit l'addition\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld", &nombre2);
resultat = nombre1 + nombre2;
printf ("%ld + %ld = %ld\n", nombre1, nombre2, resultat);
}
else if(Calcul==2)
{
long resultat = 0, nombre1 = 0, nombre2 = 0; //soustraction
printf("Vous avez choisit la soustraction\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld", &nombre2);
resultat = nombre1 - nombre2;
printf("%ld - %ld = %ld", nombre1, nombre2, resultat);
}
else if(Calcul==3) //multiplication
{
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit la soustraction\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld", &nombre2);
resultat = nombre1 * nombre2;
printf("%ld X %ld = %ld", nombre1, nombre2, resultat);
}
else if(Calcul==4)
{
long resultat = 0, nombre1 = 0, nombre2 = 0;
printf("Vous avez choisit la division\n");
printf("Entrez le nombre 1 : ");
scanf("%ld", &nombre1);
printf("Entrez le nombre 2 : ");
scanf("%ld", &nombre2);
resultat = nombre1 / nombre2;
printf("%ld / %ld = %ld", nombre1, nombre2, resultat);
}
else
{ //si erreur
printf("Vous n'avez pas choisit un nombre correct.\n");
}
system("PAUSE");
return 0;
}

j'ai juste remplacer le switch par if car je n'arrivais pas a le corriger
j'espere que ce code fonctionne pour toi car pour moi avec dev c++ il
fonctionne parfaitement

9 mars 2008 à 13:38:17

ok merci

Varmin

9 mars 2008 à 13:44:45

scanf("%ld", & Calcul);

C'est pas ca
scanf("%ld", &Calcul);

sans le espace entre le & et le nom de ta variable.

Correction de mon code

Correction de mon code

Supprimer

Modérer

Signaler le message aux modérateurs