Je débute en C en je veux pour m'améliorer créer une mini calculatrice par mes propres moyens, pour m'entrainer.
Voila mon programme (+; -; *; pour l'instant), je ne comprend pas pourquoi le résultat est égal a 0.
#define _CRT_SECURE_NO_WARNINGS
#define _CRT_SECURE_NO_DEPRECATE
#define _CRT_NONSTDC_NO_DEPRECATE
#include <stdio.h>
#include <stdlib.h>
int main()
{
printf("Saisissez un premier nombre : "); // premier nombre
int premierNombre = 0, secondNombre = 0, resultat = 0; // on déclare les variables
scanf("%d", &premierNombre);
printf("Saisissez un second nombre : "); // second nombre
scanf("%d", &secondNombre);
printf("Saisissez une operation : "); // signe opération
int operation = 'qqch';
scanf("%d", &operation);
if (operation=='+'); //si le signe est +
{
int resultat=premierNombre + secondNombre;
}
if (operation == '-');// si le signe est -
{
int resultat=premierNombre - secondNombre;
}
if (operation == '*');// si le signe est *
{
int resultat=premierNombre * secondNombre;
}
printf("Le resultat est %d", resultat);
return 0;
}
Voila si ça inspire quelqu’un
Merci !
- Edité par CodeLearner1 18 janvier 2020 à 12:01:30
Le message qui suit est une réponse automatique activée par un membre de l'équipe. Les réponses automatiques leur permettent d'éviter d'avoir à répéter de nombreuses fois la même chose, ce qui leur fait gagner du temps et leur permet de s'occuper des sujets qui méritent plus d'attention. Nous sommes néanmoins ouverts et si vous avez une question ou une remarque, n'hésitez pas à contacter la personne en question par Message Privé. Pour plus d'informations, nous vous invitons à lire les règles générales du forum
Merci de colorer votre code à l'aide du bouton Code
Les forums d'Openclassrooms disposent d'une fonctionnalité permettant de colorer et mettre en forme les codes source afin de les rendre plus lisibles et faciles à manipuler par les intervenants. Pour cela, il faut utiliser le bouton de l'éditeur, choisir un des langages proposés et coller votre code dans la zone prévue. Si vous utilisez l'éditeur de messages en mode Markdown, il faut utiliser les balises <pre class="brush: cpp;">Votre code ici</pre>.
Mauvais titre
Le titre est un élément important qui ne doit pas être négligé. N'oubliez pas cette règle simple : le titre idéal résume la question que vous allez poser en une petite phrase. Il doit permettre aux visiteurs de se repérer facilement dans le forum visité et d'identifier le sujet à sa seule lecture.
Vous pouvez utiliser divers préfixes comme [Erreur], [MySQL], [Compatibilité], etc... Aussi, pensez à consulter les règles propres à chaque forum (visibles dans les topics épinglés en haut des sections).
De plus, choisir un bon titre permet de rendre plus faciles les recherches des autres membres.
Les titres de type "besoin d'aide" ou "problème" ne sont pas tolérés.
Pour modifier votre titre, éditez le premier message de votre sujet.
(La ligne que tu as écrit déclare une variable appelée 'resultat'. Or tu en as déjà déclarée une plus haut (ligne 10). Du coup ce n'est pas la même, c'est une nouvelle variable 'resultat', qui n'existe que dans le bloc où elle est déclarée − le bloc est la zone délimitée par des accolades. En sortie du if, ce deuxième 'resultat', qui contient le résultat de l'opération, est effacé (puisqu'on sort du bloc). Quant à l'affichage final (ligne 30), il affiche la valeur de 'resultat' : le premier 'resultat', celui que tu as initialisé à 0 et qui n'a pas été modifié.)
Avant 1999, il fallait déclarer comme 'local' les variables déclarées localement dans un bloc.
Tu confonds sans doute avec autre chose.
La règle depuis C89 (le premier standard) c'était que les variables devaient être déclarées en _début de bloc_. Le faire un peu plus loin, c'est depuis C99.
dans le K&R, (standard ANSI) la définition de "compound statement' (bloc), c'est
J'ai rajouté la division et un menu a mon projet et je suis perplexe...ça ne marche plus ! Toute les opération qui sont faite sont mtn des divisions
#define _CRT_SECURE_NO_WARNINGS
#define _CRT_SECURE_NO_DEPRECATE
#define _CRT_NONSTDC_NO_DEPRECATE
#include <stdio.h>
#include <stdlib.h>
int main()
{
printf("Choisissez une option\n1 : Calculette\t2 : Ensemble des diviseurs d'un entier naturel\n3 : Qqch\t4 : Qqcch\n : ");/*Menu*/
int Choix=0;/*declare les varables des menus*/
scanf("%d", &Choix);
if (Choix == 1);
{
printf("Saisissez un premier nombre : "); // premier nombre
int premierNombre = 0, secondNombre = 0, resultat = 0, reste = 0; // on déclare les variables
scanf("%d", &premierNombre);
printf("Saisissez un second nombre : "); // second nombre
scanf("%d", &secondNombre);
printf("Saisissez une operation : "); // signe opération
int operation = 'qqch';
scanf("%d", &operation);
if (operation == '+'); //si le signe est +
{
resultat = premierNombre + secondNombre;
}
if (operation == '-');// si le signe est -
{
resultat = premierNombre - secondNombre;
}
if (operation == '*');// si le signe est *
{
resultat = premierNombre * secondNombre;
}
if (operation == '/');//si le signe est /
{
resultat = premierNombre / secondNombre;
if (resultat != premierNombre * secondNombre);/*si la division nest pas euclidienne*/
{
reste = premierNombre - resultat * secondNombre;
}
}
printf("Le resultat est %d, avec un reste de %d", resultat, reste);/*on annonce le résultat*/
}
return 0;
}
Maintenant que j'ai fait ça, le programme donne tt le temps 0...
#define _CRT_SECURE_NO_DEPRECATE
#define _CRT_NONSTDC_NO_DEPRECATE
#include <stdio.h>
#include <stdlib.h>
int main()
{
printf("Choisissez une option\n1 : Calculette\t2 : Ensemble des diviseurs d'un entier naturel\n3 : Qqch\t4 : Qqcch\n : ");/*Menu*/
int Choix=0;/*declare la variable du menu*/
scanf("%d", &Choix);
if (Choix == 1);
{
printf("Saisissez un premier nombre : "); // premier nombre
int premierNombre = 0, secondNombre = 0, resultat = 0, reste = 0, operation = 'qqch'; // on déclare les variables
scanf("%d", &premierNombre);
printf("Saisissez un second nombre : "); // second nombre
scanf("%d", &secondNombre);
printf("Saisissez une operation : "); // signe opération
scanf("%d", &operation);
if (operation == '+') //si le signe est +
{
resultat = premierNombre + secondNombre;
}
if (operation == '-')// si le signe est -
{
resultat = premierNombre - secondNombre;
}
if (operation == '*')// si le signe est *
{
resultat = premierNombre * secondNombre;
}
if (operation == '/')//si le signe est /
{
resultat = premierNombre / secondNombre;
if (resultat != premierNombre * secondNombre)/*si la division nest pas euclidienne*/
{
reste = premierNombre - resultat * secondNombre;
}
}
printf("Le resultat est %d, avec un reste de %d", resultat, reste);/*on annonce le résultat*/
}
return 0;
}
- Edité par CodeLearner1 19 janvier 2020 à 14:46:25
Essaie de comprendre pourquoi le résultat est toujours 0. Par exemple fais afficher 'resultat' à l'intérieur de chaque 'if'. Tu découvriras alors dans quel 'if' il est passé (nan, je ne spoilerai pas !), ce qui expliquera pourquoi la valeur initiale est restée inchangée.
(J'ajoute que la façon dont tu as initialisé 'operation' n'a pas dû plaire au compilateur. Il n'y a pas eu de message ? Indication : c'est cette variable qui est à l'origine du bug.)
Si j'affiche les résultats dans les ifs, la valeur est toujours la meme soit 0
Je sais. Mon but n'était pas de donner la solution mais de suggérer une méthode pour la trouver.
Normalement, grâce à ces 'printf', tu as dû remarquer que le programme ne passe dans aucun 'if'. Autrement dit, aucune des conditions n'est réalisée. Ceci doit te suggérer qu'il y a un problème avec la variable 'operation', que tu pourrais alors contrôler par un 'printf'. Et là, constatant que 'operation' n'a pas la valeur prévue, tu vas forcément aller voir comment cette variable a été saisie : c'est la ligne citée par PierrotLeFou, que tu sauras alors corriger.
Merci de m'aider ainsi a progresser. Voici le code, mais depuis que j'ai fait des modifs, une erreur qui s'affiche...
Le code :
#define _CRT_SECURE_NO_WARNINGS
#define _CRT_SECURE_NO_DEPRECATE
#define _CRT_NONSTDC_NO_DEPRECATE
#include <stdio.h>
#include <stdlib.h>
int main(int argc, char* argv[])
{
printf("Choisissez une option\n1 : Calculette\t2 : Ensemble des diviseurs d'un entier naturel\n3 : Qqch\t4 : Qqcch\n : ");/*Menu*/
int Choix=0;/*declare la variable du menu*/
scanf("%d", &Choix);
if (Choix == 1)
{
printf("Saisissez un premier nombre : "); // premier nombre
int premierNombre = 0, secondNombre = 0, resultat = 0, reste = 0; // on déclare les variables
scanf("%d", &premierNombre);
printf("Saisissez un second nombre : "); // second nombre
scanf("%d", &secondNombre);
printf("Saisissez une operation : "); // signe opération
char operation = '+';
scanf("%c/n", &operation);
if (operation == '+') //si le signe est +
{
resultat = premierNombre + secondNombre;
}
if (operation == '-')// si le signe est -
{
resultat = premierNombre - secondNombre;
}
if (operation == '*')// si le signe est *
{
resultat = premierNombre * secondNombre;
}
printf("Le resultat est %d, avec un reste de %d et %s", resultat, reste, operation);/*on annonce le résultat*/
}
return 0;
}
Tu pourrais poster l'erreur ? Il suffit de copier-coller le message d'erreur. (L'image que tu as inséré dans ton dernier message n'a visiblement rien à voir.)
En mettant un %s à laplace d'un %c, la fonction printf a essayé de déréfencer une petite adresse genre 0x0000 002B si l'opération était +. Ou peut être Ox0000 000A si c'était le caractère \n
D'où accès dans une zone protégée.
Incidemment, ça ne serait pas compilé comme du C++ au lieu du C ? Parce que tous ces templates, bon...
Tiens ligne 20, il y a /n au lieu de \n probablement (keskifout dans un scanf ?)
- Edité par michelbillaud 26 janvier 2020 à 13:10:17
Yep mais du coup j'ai fait les modifs mais il ya toujours le même erreur...
#define _CRT_SECURE_NO_WARNINGS
#define _CRT_SECURE_NO_DEPRECATE
#define _CRT_NONSTDC_NO_DEPRECATE
#include <stdio.h>
#include <stdlib.h>
int main(int argc, char* argv[])
{
printf("Choisissez une option\n1 : Calculette\t2 : Ensemble des diviseurs d'un entier naturel\n3 : Qqch\t4 : Qqcch\n : ");/*Menu*/
int Choix=0;/*declare la variable du menu*/
scanf("%d", &Choix);
if (Choix == 1)
{
printf("Saisissez un premier nombre : "); // premier nombre
int premierNombre = 0, secondNombre = 0, resultat = 0, reste = 0; // on déclare les variables
scanf("%d", &premierNombre);
printf("Saisissez un second nombre : "); // second nombre
scanf("%d", &secondNombre);
printf("Saisissez une operation : "); // signe opération
char operation = '+';
scanf("%c", &operation);
if (operation == '+') //si le signe est +
{
resultat = premierNombre + secondNombre;
}
if (operation == '-')// si le signe est -
{
resultat = premierNombre - secondNombre;
}
if (operation == '*')// si le signe est *
{
resultat = premierNombre * secondNombre;
}
printf("Le resultat est %d, avec un reste de %d et %c", resultat, reste, operation);/*on annonce le résultat*/
}
return 0;
}
l'erreur : Une exception non gérée a été levée : violation d'accès en lecture. **it** a été 0x2B. a eu lieu
- Edité par CodeLearner1 26 janvier 2020 à 13:55:49
Je viens de compiler le programme. Je n'ai pas d'erreur. Le seul problème, c'est que la saisie de l'opérateur est « sautée » : je crois qu'il lit le dernier '\n', cas classique du tampon à vider avant un 'scanf'. Tiens, d'ailleurs vidons le tampon pour voir... Oui, cette fois ça marche !
Est-ce que tu as vérifié la suggestion de michelbillaud (que la compilation se fait bien en C et non en C++) ?
Le message d'erreur que tu as eu n'a rien à voir avec le vidage de tampon. D'où ma question : as-tu vérifié la suggestion de michelbillaud (que la compilation se fait bien en C et non en C++) ?
Curieusement, 0x2B, c'est le code ascii du caractère '+'.
Comme le monde est petit.
Quand je fait tourner ton programme, j'obtiens ceci
$ ./a.out
Choisissez une option
1 : Calculette 2 : Ensemble des diviseurs d'un entier naturel
3 : Qqch 4 : Qqcch
: 1
Saisissez un premier nombre : 12
Saisissez un second nombre : 34
Saisissez une operation : Le resultat est 0, avec un reste de 0 et
Il ne melaisse pas le temps de taper le caractère pour l'opération.
Rafistolage (assez laid)
do {
scanf("%c", &operation);
} while (operation != '+'
&& operation != '-'
&& operation != '*');
Une autre solution (que je préfère) serait de lire l'operation sous forme d'une chaine.
Qui plus est on pourrait même faire
printf("Saisissez une expression, exemple 3+2\n");
int a, b;
char op[5];
scanf("%d%s%d", &a, op, &b")
- Edité par michelbillaud 26 janvier 2020 à 18:53:00
× Après avoir cliqué sur "Répondre" vous serez invité à vous connecter pour que votre message soit publié.
× Attention, ce sujet est très ancien. Le déterrer n'est pas forcément approprié. Nous te conseillons de créer un nouveau sujet pour poser ta question.
Le Tout est souvent plus grand que la somme de ses parties.
Le Tout est souvent plus grand que la somme de ses parties.
Le Tout est souvent plus grand que la somme de ses parties.
Le Tout est souvent plus grand que la somme de ses parties.
Le Tout est souvent plus grand que la somme de ses parties.
Le Tout est souvent plus grand que la somme de ses parties.