Discussion :

[MarvinBosst]

Bonjours a tous,
Je viens de débuter un cours de programmation C++ et étant très autodidacte j'ai penser gros pour un début (j'ai lu peu de pages)
J'ai dériver du cours pour construire un petit programme tout ce qu'il y a de plus simple pour mémoriser le langage C++
Ma calculatrice fonctionne bien pour l'addition et la soustraction mais arriver a la multiplication et la division j'ai un sérieux problème,
ma calculatrice est fausse ! Je n'arrive malheureusement pas a trouver le problème de syntaxe dans mon code..
Je vous demande si vous le voulez bien de m'expliquer ce qui ne marche pas car mon cours ne l'explique pas..

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
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
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
 
#include <iostream>
#include <string>
 
using namespace std;
 
int addition(double n1, double n2)
{
    double resultat(0);
    resultat = n1+n2;
    return resultat;
}
 
int soustaction(double n1, double n2)
{
    double resultat(0);
    resultat = n1-n2;
    return resultat;
}
 
int multiplication(double n1, double n2)
{
    double resultat;
    resultat = n1*n2;
    return resultat;
}
 
int division(double n1, double n2)
{
    double resultat;
    resultat = n1/n2;
    return resultat;
}
 
int main()
{
    double nombre1(0), nombre2(0), nombre3(0);
    char signe;
    int menu(-1);
    cout << "Calculatrice" << endl;
    cout << "Menu" << endl;
    cout << "1- Addition" << endl;
    cout << "2- Soustraction" << endl;
    cout << "3- Multiplication" << endl;
    cout << "4- Division" << endl;
    cout << "0- Quitter" << endl;
    cin >> menu;
    switch (menu)
    {
        case 1:
 
    cout << "Entrer le premier chiffre a calculer :" << endl;
    cin >> nombre1;
    cout << "Maintenant le deuxieme :" << endl;;
    cin >> nombre2;
    nombre3 = addition(nombre1, nombre2);
    cout << "le resultat est : " << nombre3 << endl;
        break;
 
        case 2:
 
    cout << "Entrer le premier chiffre a calculer :" << endl;
    cin >> nombre1;
    cout << "Maintenant le deuxieme :" << endl;
    cin >> nombre2;
    nombre3 = soustaction(nombre1, nombre2);
    cout << "le resultat est : " << nombre3 << endl;
        break;
 
        case 3:
 
    cout << "Entrer le premier chiffre a calculer :" << endl;
    cin >> nombre1;
    cout << "Maintenant le deuxieme :" << endl;;
    cin >> nombre2;
    nombre3 = addition(nombre1, nombre2);
    cout << "le resultat est : " << nombre3 << endl;
        break;
 
        case 4:
 
    cout << "Entrer le premier chiffre a calculer :" << endl;
    cin >> nombre1;
    cout << "Maintenant le deuxieme :" << endl;;
    cin >> nombre2;
    nombre3 = addition(nombre1, nombre2);
    cout << "le resultat est : " << nombre3 << endl;
 
        break;
    }
 
    return 0;
}

[ZeRedDiamond]

Bonjour,
Il y a déjà des erreurs dans le 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
case 3:
 
    cout << "Entrer le premier chiffre a calculer :" << endl;
    cin >> nombre1;
    cout << "Maintenant le deuxieme :" << endl;;
    cin >> nombre2;
    nombre3 = addition(nombre1, nombre2);
    cout << "le resultat est : " << nombre3 << endl;
        break;
 
        case 4:
 
    cout << "Entrer le premier chiffre a calculer :" << endl;
    cin >> nombre1;
    cout << "Maintenant le deuxieme :" << endl;;
    cin >> nombre2;
    nombre3 = addition(nombre1, nombre2);
    cout << "le resultat est : " << nombre3 << endl;
 
        break;
    }

Lorsque l'utilisateur choisit 3 ou 4, vous appelez la méthode addition, c'est donc normal que la calculatrice donne un mauvais résultat. Il faut comme bien indiqué dans le programme, appeler la méthode multiplication dans case 3 et la méthode division dans case 4. Sinon, je ne trouve aucune erreur dans le code.

N'hésitez pas à rappeler en cas de problème!!!

P.S : Essayez de commenter le code, vous en sortirez mieux...

[xflr6]

Bonjour,

Il y a aussi une maladresse dans le typage des fonctions :
en int, alors que les opérandes sont double, et que le renvoi des fonctions est déclaré double aussi.

Cordialement,
xflr6

PS: vous pourriez rajouter aussi le cas par défaut pour gérer les erreurs de saisie par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
switch(menu)
{
case 1 :
 [...]
case 2 :
 [...]
default:
 [...]
}

[stendhal666]

Si tu as des penchants autodidactes et que le C++ te plaît, je te conseille le livre Programmation : Principes et pratique avec C++ de Bjarne Stroustrup (l'auteur du langage C++). Les premiers chapitres sont consacrés justement à la programmation d'une calculette, un peu plus puissante que la tienne puisqu'elle lit des expressions ("1+2*3-4").

C'est un bon sujet de programmation pour commencer, qui pourra t'accompagner quelques temps, et qui te permet aussi de t'initier à plein de sujets:
- interaction avec l'utilisateur
- validation des saisies de l'utilisateur
- gestion des erreurs
- concepts et techniques de base des compilateurs / interpréteurs

Pour le code que je vois et qui n'est pas mal du tout, j'ai quelques suggestions au-delà de celles qui ont déjà été faites:
- déclare moins de variables:

Code :

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

return arg1 - arg2;

est tout aussi valable que:

Code :

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

1
2
3
double res(0);
res = arg1 - arg2;
return res;

- déclare celles que tu es obligé d'utiliser le plus près possible de l'endroit où tu les utilises (tu verras que ça aide vraiment quand le programme s'agrandit).
- je te conseille aussi de ne pas utiliser "using namespace std" et de préfixer plutôt les fonctions de std que tu utilises, c'est une bonne habitude à prendre dès le début, qui t'incitera à créer tes propres namespaces au fur et à mesure.

[MarvinBosst]

Un grand merci pour vos conseils, et pour mes fautes énormes d’inattention ^^
Pour ce qui es du code je viens juste de m'y mettre donc les namespaces sont ceux conseiller dans mon cours, disons que j'ai juste pris en compte les commandes de bases (cout, cin, int, double, string, les variables ect) mais je n'ai pas attaquer encore les .h ni même les namespaces, mais je tiendrais en compte vos conseils quand j'aurais fini mon cours C++, pour voir les autres alternatives et merci pour le livre je vais trouver ça pour le lire après celui que j'ai entamer.

[Iradrille]

Il y a beaucoup de répétitions, ça peut se factoriser.

Par exemple

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
40
41
42
43
44
#include <iostream>
#include <string>
 
double addition(double n1, double n2) { return n1+n2; }
double soustaction(double n1, double n2) { return n1-n2; }
double multiplication(double n1, double n2) { return n1*n2; }
double division(double n1, double n2) { return n1/n2; }
 
int main() {
	double nombre1, nombre2;
	int menu;
	cout << "Calculatrice" << endl;
	do {
		std::cout << "Menu" << endl;
		std::cout << "1- Addition" << endl;
		std::cout << "2- Soustraction" << endl;
		std::cout << "3- Multiplication" << endl;
		std::cout << "4- Division" << endl;
		std::cout << "0- Quitter" << endl;
		std::cin >> menu;
	} while(menu > 4 || menu < 0);
 
	std::cout << "Entrer le premier chiffre a calculer :" << endl;
	std::cin >> nombre1;
	std::cout << "Maintenant le deuxieme :" << endl;;
	std::cin >> nombre2;
 
	switch (menu) {
	case 1:
		std::cout << "le resultat est : " << addition(nombre1, nombre2) << endl;
		break;
	case 2:
		std::cout << "le resultat est : " << soustaction(nombre1, nombre2) << endl;
		break;
	case 3:
		std::cout << "le resultat est : " << multiplication(nombre1, nombre2) << endl;
		break;
	case 4:
		std::cout << "le resultat est : " << division(nombre1, nombre2) << endl;
		break;
	}
 
	return 0;
}

[ternel]

Et encore, j'irai utiliser une map<operation_id_type, operation_function> pour lier les identifiants (1, 2, 3, 4) aux identifiants de fonctions, de façon à me passer complêtement du switch.
Ca peut être une idée pour continuer le projet.

[Iradrille]

Pas con ouais.

Le projet peut aussi être continué pour gérer les expressions du type "3 + 2 / 8", un bon moyen de voir a quoi ressemble les ASTs.

[Basile le disciple]

Bonjour à tous,

un bon moyen de voir a quoi ressemble les ASTs.

En parlant d'arbres binaires, je tiens juste à dire qu'il existe un cours sur developpez.net ici qui permet de créer une calculatrice gérant les priorités(ce que tu veux ici) mais aussi les parenthèses si tu veux aller plus loin. Je l'ai d'ailleurs étudié et il est très bien fait. Certes, le langage utilisé est du pascal(sous delphi) mais le but du cours est plutôt de nous présenter la logique...