Calculatrice de base

Version imprimable

Bonjour,

En ce moment j'essaie de faire une petite calculatrice C++ en mode console.
Voici comment s'articule mon code :

Main.cpp

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
 
#include <iostream>
#include "fcalcul.h"
 
using namespace std;
 
 
int main()
{
    /*Calculatrice*/
 
    int argUn (0);
    int argDeux (0);
    int resultat1 (0);
    char operateur ('+');
 
    cout << "Bienvenue sur votre calculatrice basique" << endl ;
    cout << "Que voulez-vous calculer ? : " << endl ;
    cout << "Valeur 1 = " ;
    cin >> argUn;
    cout << "Operateur = " ;
    cin >> operateur;
    cout << "Valeur 2 = " ;
    cin >> argDeux;
 
    resultat1 = calcul (argUn, operateur, argDeux);
 
    cout << argUn << operateur << argDeux << " = " << resultat1 << endl ;
 
    return 0;
}

La Fonction fcalcul.cpp

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
 
#include <iostream>
#include "fcalcul.h"
 
 
using namespace std;
 
 
/*Fonction Calcul*/
 
int calcul (int arg1 , int arg2 , char operat)
{
    int resultat2;
    switch (operat)
        {
        case '+' : return ('arg1 + arg2 = resultat2');
            break;
        case '-' : return ('arg1 - arg2 = resultat2');
            break;
        case '*' : return ('arg1 * arg2 = resultat2');
            break;
        case '/' : return ('arg1 / arg2 = resultat2');
            break;
        default : cout << "Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !" ;
            break;
        }
 
    return resultat2;
 
}

fcalcul.h

Code:

1
2
3
4
5
6
7
 
#ifndef FCALCUL_H_INCLUDED
#define FCALCUL_H_INCLUDED
 
int calcul(int arg1, int arg2, char operat);
 
#endif // FCALCUL_H_INCLUDED

Voici le problème que j'ai (image):

Pièce jointe 204771

Je pense que j'ai un problème au niveau de la variable char (opérateur).
C'est comme s'il ne voyait pas que c'est un opérateur.

Si vous avez une idée.

Merci à vous pour votre aide.

Cordialement,

Alaz

23/03/2016, 13h11
Médinoc

Quand je clique sur ta pièce jointe, j'obtiens ceci:

Citation:

Pièce jointe spécifié(e) non valide. Si vous suivez un lien valide, veuillez notifier l'administrateur

Autre chose, ton code:

Citation:

Code:

return ('arg1 + arg2 = resultat2');

Ça compile, ça?
23/03/2016, 13h25
Bousk

Citation:

Envoyé par main.cpp

calcul (argUn, operateur, argDeux);

Citation:

Envoyé par fcalcul.cpp

int calcul (int arg1 , int arg2 , char operat)

:weird:

Ca compile surement, mais ça défie toutes les lois de la logique, et j'ai aucune idée de ce que ça retourne, mais ça doit pas être beau 8O
23/03/2016, 13h34
Alazban

Ah mince alors je vais vous écrire la compilation. (Oui ça compile bien) :) .

Dans mon terminal j'ai :

Bienvenue sur votre calculatrice basique
Que voulez-vous calculer ? :
Valeur 1 = 2
Operateur = +
Valeur 2 = 2
Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !2+2 = 0

Process returned 0 (0x0) execution time : 11.852 s
Press ENTER to continue.

EDIT 1 : Lol Bousk, j'ai déclarer 2 fois les variables, 1 fois dans main.cpp et une autre dans fcalcul.cpp. Je suppose donc que je doit corriger ma fonction fcalcul. Medinoc je vais essayer, Merci à tout les deux :) .
23/03/2016, 14h35
jo_link_noir

Relit les messages de ton compilateur, notamment les warning, parce que return 'arg1 + arg2 = resultat2', c'est du n'importe quoi.
23/03/2016, 14h39
ternel

c'est surtout une constante multicaractère, de type int, dont la valeur est dépendante du compilateur.
Ca n'a rien à voir avec l'évaluation.
Ajoutons que resultat2 n'est pas initialisé
23/03/2016, 15h38
Bousk

Citation:

Envoyé par Alazban

EDIT 1 : Lol Bousk, j'ai déclarer 2 fois les variables, 1 fois dans main.cpp et une autre dans fcalcul.cpp. Je suppose donc que je doit corriger ma fonction fcalcul. Medinoc je vais essayer, Merci à tout les deux :) .

J'ai pas compris ton histoire sur les variables déclarées 2 fois. Mais si tu relisais un peu ton code, que j'ai mis en avant, tu ne passes pas les paramètres correctement surtout.. :calim2:
Et tu as certainement des warnings qui t'en informent. Que tu tronques un int en char.
Bon ça changera pas le résultat absurde que tu auras surement avec ta syntaxe par-dessus ça, mais ce serait un bon début.

:) Bon ce matin j'ai relu le code et j'ai fais quelques modifications.

main.cpp, j'ai modifié cette ligne resultat1 = fcalcul (argUn, argDeux, operateur); j'ai inversé argDeux et operateur et enfin la fonction récupère l'opérateur. Il doit y avoir un sens à respecter je pense car a chaque fois dans le code j'ai déclaré valeur1 valeur2 puis operateur sauf à cet endroit.
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 #include <iostream> #include "fcalcul.h" using namespace std; int main() { /*Calculatrice*/ int argUn (0); int argDeux (0); int resultat1 (0); char operateur ('+'); cout << "Bienvenue sur votre calculatrice basique" << endl ; cout << "Que voulez-vous calculer ? : " << endl ; cout << "Valeur 1 = " ; cin >> argUn; cout << "Operateur = " ; cin >> operateur; cout << "Valeur 2 = " ; cin >> argDeux; resultat1 = fcalcul (argUn, argDeux, operateur); cout << argUn << " " << operateur << " " << argDeux << " = " << resultat1 << endl ; return 0; }
Autre chose, j'ai renommé mes variable correctement. :) ça fonctionne mieux.
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 #include <iostream> #include "fcalcul.h" using namespace std; /*Fonction Calcul*/ int fcalcul (int argUn , int argDeux , char operateur) { int resultat1 (0); switch (operateur) { case '+' : return ('argUn + argDeux = resultat1'); break; case '-' : return ('argUn - argDeux = resultat1'); break; case '*' : return ('argUn * argDeux = resultat1'); break; case '/' : return ('argUn / argDeux = resultat1'); break; default : cout << "Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !" ; break; } return resultat1; }
Code:

1 2 3 4 5 6 7 8 #ifndef FCALCUL_H_INCLUDED #define FCALCUL_H_INCLUDED int fcalcul(int argUn, int argDeux, char operateur); #endif // FCALCUL_H_INCLUDED
Par contre j'ai toujours un problème, car maintenant il me fait le calcul mais me renvoie toujours le même résultat genre
2+3=1987643693
C'est surement un problème avec resultat1, un truc que je pige pas bien.... resultat1 dans main.cpp et resultat1 dans fcalcul.cpp ... comment se fait la communication entre les deux fichiers ? j'ai du la déclarer 2 fois . Bizarre j'ai le cerveau bloqué la xD.

Bref je vais regarder dans les warnings :), merci pour votre précieuse aide.
J’espère avoir été clair.

24/03/2016, 11h20
Médinoc

Citation:

Envoyé par Alazban

Par contre j'ai toujours un problème, car maintenant il me fait le calcul mais me renvoie toujours le même résultat genre
2+3=1987643693
C'est surement un problème avec resultat1, un truc que je pige pas bien.... resultat1 dans main.cpp et resultat1 dans fcalcul.cpp ... comment se fait la communication entre les deux fichiers ? j'ai du la déclarer 2 fois . Bizarre j'ai le cerveau bloqué la xD.

C'est le problème de return ('argUn + argDeux = resultat1');, le truc dont je suis surpris que ça compile et qui sur un compilateur correctement configuré, devrait au moins te donner un warning.

Que cherches-tu à faire avec un tel code, pour commencer? Que signifient les simple quotes pour toi?
24/03/2016, 12h35
Bousk

Plus je te lis, plus j'ai l'impression que tu mets la charrue avant les boeufs.
Avant de courrir, on apprend à marcher.

Citation:

Envoyé par Alazban

Il doit y avoir un sens à respecter je pense car a chaque fois dans le code j'ai déclaré valeur1 valeur2 puis operateur sauf à cet endroit.

Bien sur qu'il y a un sens.. Comment crois-tu que ça fonctionne sinon ? :weird: Si la fonction attend un string, int et Objet, il faut les lui fournir à l'emplacement attendu, par l'argument attendu. Elle ne devinera rien.

Citation:

Envoyé par Alazban

Par contre j'ai toujours un problème, car maintenant il me fait le calcul mais me renvoie toujours le même résultat genre
2+3=1987643693

Ca on te le dit depuis le premier post : tu as écrit n'importe quoi, donc le résultat est n'importe quoi.
return ('arg1 + arg2 = resultat2'); a un sens pour ton compilateur, et il te retourne la valeur qu'il en tire. Qui n'est clairement pas le sens que tu espères lui donner, et donc forcément pas la valeur que tu attends.

Citation:

Envoyé par Alazban

resultat1 dans main.cpp et resultat1 dans fcalcul.cpp ... comment se fait la communication entre les deux fichiers ? j'ai du la déclarer 2 fois . Bizarre j'ai le cerveau bloqué la xD.

Les noms des variables n'ont aucune signification une fois le programme compilé. Des variables au nom identique dans des scopes différents n'ont strictement aucun lien entre elles.

Tu devrais commencer par te renseigner et lire un minimum de cours, et je ne parle pas que de C++.
La portée des variables et l'appel des fonctions en passant des arguments se retrouve dans le B.A.BA de quasiment tous les langages.
24/03/2016, 12h53
Alazban

Je n'ai jamais lu de cours, ça m'est venu comme ça :) surtout que je ne suis pas un intello qui retient tout et qui ne sait même pas mettre en pratique ce qu'il a appris ... je suis plutôt l'inverse, j’apprends par la pratique (en me basant sur divers cours) je ne crois pas me tromper en disant qu'un élève ne fait jamais d'erreur ????? .... Un peu de positif ça fait pas de mal !

je vais donc chercher à modifier return = argUn *+-/ argDeux = resultat1

Merci pour votre aide
24/03/2016, 13h24
ternel
Ca n'est pas être négatif que de dire qu'il faut apprendre sérieusement.
A tenter au pif, tu ne risques qu'une seule chose: croire sans savoir.

return est une instruction, qui met fin à l'exécution d'une fonction en définissant la valeur que doit prendre l'expression constituée par son appel.

C'est à dire que dans le code suivant, la valeur de l'expression f(2,3) est 5:
Code:

1 2 3 4 5 6 7 8 int f(int a, int b) { return a+b; } int main() { int i = f(2,3); return 0; }
La valeur de retour de l'appel de main sera systématiquement 0.
Pour la question des variables, c'est pareil, il faut comprendre la notion de portée (scope).
24/03/2016, 13h35
Alazban

Je pense la même chose que vous, il me semblait bien avoir posté dans le sous-forum débutant.

Je vous remercie pour vos conseils.

Alaz
24/03/2016, 15h35
Bousk

Citation:

Envoyé par Alazban

je vais donc chercher à modifier return = argUn *+-/ argDeux = resultat1

t'as fait 99% du truc, pourquoi chercher compliqué ? return argUn + argDeux; et basta :calim2:
04/04/2016, 09h02
Alazban

Merci pour vos réponses, cependant je tiens à faire quelque chose qui tient la route.
J'ai testé ce que m'a dit Bousk mais j'ai toujours le même type de résultat.
Donc si la structure de contrôle ne convient pas ... je peux changer et faire appel a if ...else.... .

Merci je vais continuer a lire mes cours en attendant.

Alaz
04/04/2016, 09h11
ternel

fais voir ton code actuel?

Pour l'instant il n'a pas vraiment changé..

main.cpp

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
 
#include <iostream>
#include "fcalcul.h"
 
using namespace std;
 
 
int main()
{
    /*Calculatrice*/
 
    int argUn (0);
    int argDeux (0);
    int resultat1 (0);
    char operateur ('+');
 
    cout << "Bienvenue sur votre calculatrice basique" << endl ;
    cout << "Que voulez-vous calculer ? : " << endl ;
    cout << "Valeur 1 = " ;
    cin >> argUn;
    cout << "Operateur = " ;
    cin >> operateur;
    cout << "Valeur 2 = " ;
    cin >> argDeux;
 
    resultat1 = fcalcul (argUn, argDeux, operateur);
 
    cout << argUn << " " << operateur << " " << argDeux << " = " << resultat1 << endl ;
 
    return 0;
}

fonction fcalcul.cpp
Si je dois modifier quelque chose je pense que c'est dans la fonction ... au niveau du return dans le switch, à la vue de ce que l'on m'a déjà expliqué même si je ne saisis pas vraiment pourquoi je ne peux pas utiliser switch pour retourner un résultat. Si je ne peut pas utiliser return alors avec quoi je renvois mon résultat. Si la condition switch ne convient pas je peux faire autrement.

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
 
#include <iostream>
#include "fcalcul.h"
 
using namespace std;
 
 
/*Fonction Calcul*/
 
int fcalcul (int argUn , int argDeux , char operateur)
{
    int resultat1 (0);
 
    switch (operateur)
        {
        case '+' : return ('argUn + argDeux');
            break;
        case '-' : return ('argUn - argDeux');
            break;
        case '*' : return ('argUn * argDeux');
            break;
        case '/' : return ('argUn / argDeux');
            break;
        default : cout << "Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !" ;
            break;
        }
 
    return resultat1;
 
}

fcalcul.h

Code:

1
2
3
4
5
6
7
8
 
#ifndef FCALCUL_H_INCLUDED
#define FCALCUL_H_INCLUDED
 
int fcalcul(int argUn, int argDeux, char operateur);
 
 
#endif // FCALCUL_H_INCLUDED

Reprenons les bases.

Ton main est le suivant:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 int main() { /*Calculatrice*/ int argUn (0); int argDeux (0); int resultat1 (0); char operateur ('+'); // lecture de cin resultat1 = fcalcul (argUn, argDeux, operateur); cout << argUn << " " << operateur << " " << argDeux << " = " << resultat1 << endl ; return 0; }
Supposons que cin produisent les valeurs suivante pour les variables locales de main

argUn = 1;
operateur = '+';
argDeux = 2;

On arrive à l'instruction suivante: resultat1 = fcalcul (argUn, argDeux, operateur);.
Il s'agit d'une seule instruction (il n'y a qu'un ;), constituée uniquement d'une expression.
Une telle instruction-expression est évaluée pour ses "effets de bord", c'est à dire les modifications en mémoire et les entrées sorties.
Je prends le temps de détailler cette instruction pour que tu percoives le problème dans la suite
D'après les règles de priorité, cette expression est ainsi constituée:
application de l'opérateur = avec pour opérande gauche resultat1 et opérande droit l'expression fcalcul (argUn, argDeux, operateur).
C'est à dire l'affectation de la valeur de cette sous-expression à la variable resultat1.

fcalcul (argUn, argDeux, operateur) est un appel de la fonction int fcalcul (int argUn , int argDeux , char operateur).

Tu noteras que le type de l'appel de fonction est le type de retour de la fonction appelée, ici int.
Jusque là, tout va bien, tu affectes un int à un int.

Avant toute autre chose, l'évaluation de l'appel de fcalcul commence par la création de trois variables temporaires (les paramètres de la fonction): chacune est initialisée avec la valeur indiquée dans l'appel. (Aux optimisations près, qui ne changeront pas les effets de ce fonctionnement)
Ainsi, les variables int argUn , int argDeux , char operateur, déclarées par la fonction, sont créées, et initialisées en leur affectant les valeurs des variables de main().
C'est à dire, ici, tout se passe comme si on avait:
Code:

1 2 3 int argUn = 1; int argDeux = 2; char operateur = '+';
C'est précisément ce mécanisme qu'on appelle le passage par copie.

On en arrive à la définition de ta fonction:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 int fcalcul (int argUn , int argDeux , char operateur) { int resultat1 (0); switch (operateur) { case '+' : return ('argUn + argDeux'); break; case '-' : return ('argUn - argDeux'); break; case '*' : return ('argUn * argDeux'); break; case '/' : return ('argUn / argDeux'); break; default : cout << "Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !" ; break; } return resultat1; }
Je commence par quelques remarques "tactiques":
les mots clés continue, break, return et throw créent des instructions qui changent inconditionnellement le flux normal d'exécution. Ces trois mots clés ont un point commun: l'instruction suivante n'est pas celle qui la suit. (goto aussi, mais celui-là permet de faire de grosses bêtises)
Ainsi un break après un return est toujours inaccessible, donc inutile.

Pour plus de lisibilité, si un case ne tient pas sur une seule ligne, je recommande de ne rien mettre sur la ligne du case.

Ainsi, ton code peut être plus lisible:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 int fcalcul (int argUn , int argDeux , char operateur) { int resultat1 (0); switch (operateur) { case '+' : return ('argUn + argDeux'); case '-' : return ('argUn - argDeux'); case '*' : return ('argUn * argDeux'); case '/' : return ('argUn / argDeux'); default : cout << "Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !" ; break; } return resultat1; }
On remarque plusieurs choses:

resultat1 n'est utilisé que par le dernier return: autant remplacer la variable par sa valeur à cet endroit, et retirer la variable ("une variable de moins est une source d'erreur en moins")
tu détectes bien une erreur, mais ta fonction retourne une valeur légitime (0) plutot qu'une impossible ou de lever une exception. Il n'y a pas de valeur impossible pour int (dans ton cas), donc tu devrais avoir une exception. Cela supprime carrément le return du point précédent.
Tu n'as pas lu nos messages précédents.

Pour les premiers points on arrive au code suivant:
Code:

1 2 3 4 5 6 7 8 9 10 int fcalcul (int argUn , int argDeux , char operateur) { switch (operateur) { case '+' : return ('argUn + argDeux'); case '-' : return ('argUn - argDeux'); case '*' : return ('argUn * argDeux'); case '/' : return ('argUn / argDeux'); default : throw std::runtime_error("Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !"); } }
Passons au point qui te pose vraiment problème.
Puisqu'on a supposé que operateur vaut '+', la fonction est exécuté exactement comme si elle était écrite ainsi:
Code:

1 2 3 int fcalcul (int argUn , int argDeux) { return ('argUn + argDeux'); }
J'en reviens encore une fois à la grammaire du langage.
L'instruction return ('argUn + argDeux'); est une "return instruction", constituée du mot-clé return, d'une sous-expression et d'un ;.
En tant que "return instruction", son action est de donner la valeur de la sous-expression à l'expression ayant appelé la fonction.
Il s'agit donc d'attribuer la valeur de ('argUn + argDeux') à une valeur temporaire de type int (le type de retour de la fonction).

Toujours d'après la grammaire, cette sous-expression est constituée de:

une parenthèse ouvrante (
une sous expression constituée de
une apostrophe '
des caractères différents d'une apostrophe argUn + argDeux
une apostrophe '
une parenthèse fermante )

L'opérateur parenthèse vaut la même chose que la sous expression qu'il contient, il est ici inutile.
Tu aurais aussi bien pu écrire return 'argUn + argDeux';. Pour t'en souvenir, saches que return n'est pas une fonction, mais un mot-clé.

Par les apostrophes qui la commencent et la terminent, la sous expression 'argUn + argDeux' est une déclaration de litéral de caractère.
Il s'agit d'une constante de compilation, exactement comme 42, 'a' et "truc".
Sauf que si ces trois dernières sont de types respectifs int, char et char const*, 'argUn + argDeux' est un batard de la norme: un "Multicharacter literal" de type int, et de valeur complètement arbitraire (mais pas aléatoire, car définie par l'implémentation du compilateur). (regarde le point 6 de la documentation sur cppreference.com)

Et c'est précisément à cause de ce type batard, int que Médinoc a été surpris, car l'expression étant à la fois valable et du bon type, le compilateur n'a aucune raison de créer un warning.
Cela dit, il devrait y avoir un warning systématique sur les constantes multicaractères, qui ne sont ni portables, ni gérables, ni vraiment sémantiques.

Maintenant, comprends-tu ton problème?
Il y a quelque chose de bizarre dans ton code, entre return 'argUn + argDeux'; et return resultat1;.
L'un utilise des apostrophes, pas l'autre. Ca manque de cohérence.

J'irai plus loin, avec plus de warning, et compilateur un minimum gentil, tu aurais eu des messages concernant des paramètres de fonction nommés sans être utilisés (argUn et argDeux)

04/04/2016, 11h23
ternel

Citation:

Envoyé par Alazban

Si je dois modifier quelque chose je pense que c'est dans la fonction ... au niveau du return dans le switch, à la vue de ce que l'on m'a déjà expliqué même si je ne saisis pas vraiment pourquoi je ne peux pas utiliser switch pour retourner un résultat. Si je ne peut pas utiliser return alors avec quoi je renvois mon résultat. Si la condition switch ne convient pas je peux faire autrement.

En fait, il faut que tu prennes le temps de comprendre précisément comment fonctionne chaque mot-clé, chaque symbole que tu utilises.
Ici, ton problème, ce n'est ni switch, ni return, mais la paire d'apostrophes (').
04/04/2016, 11h26
Alazban

Merci pour le temps que tu as pris, je vais méditer ça et je reviendrais quand j'aurais avancé, vraiment merci. :) :)
04/04/2016, 11h35
ternel
Pour information, je pensais les "multicharacter literals" totalement inutile, il semblerait que ce ne soit pas totalement le cas.
En fait, switch fonctionne avec, au contraire des char*.
Code:

1 2 3 4 switch( tag ) { case 'TAG1': ...; case 'TAG2': ...; }
Il peut être intéressant de lire ce document.

Yeah,

En faite, j'ai appris que le type de variable char s’appuie sur ASCII c'est donc pour ça que mon programme me renvoyait quand même quelque chose de bizarre au début du genre un gros chiffre (1 + 3 = 1965977522) https://upload.wikimedia.org/wikiped...e-wide.svg.png

Bon, tout fonctionne correctement. Voici le code,

Main.cpp
J'ai modifié le type de variable en double pour avoir des nombres a virgule.

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
 
#include <iostream>
#include "fcalcul.h"
 
using namespace std;
 
 
int main()
{
    /*Calculatrice*/
 
    double argUn (0);
    double argDeux (0);
    double resultat1 (0);
    char operateur ('+');
 
    cout << "Bienvenue sur votre calculatrice basique" << endl ;
    cout << "Que voulez-vous calculer ? : " << endl ;
    cout << "Valeur 1 = " ;
    cin >> argUn;
    cout << "Operateur = " ;
    cin >> operateur;
    cout << "Valeur 2 = " ;
    cin >> argDeux;
 
    resultat1 = fcalcul (argUn, argDeux, operateur);
 
    cout << argUn << " " << operateur << " " << argDeux << " = " << resultat1 << endl ;
 
    return 0;
}

fcalcul.cpp
J'ai enlevé les ' ' autour de return argUn + argDeux mais j'ai laissé les break car dans mon cours c'est conseillé et puis je trouve cela plus propre.

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
 
#include <iostream>
#include "fcalcul.h"
 
using namespace std;
 
 
/*Fonction Calcul*/
 
double fcalcul (double argUn , double argDeux , char operateur)
{
    switch (operateur)
        {
        case '+' : return argUn + argDeux;
            break;
        case '-' : return argUn - argDeux;
            break;
        case '*' : return argUn * argDeux;
            break;
        case '/' : return argUn / argDeux;
            break;
        default : cout << "Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !" << endl ;
            break;
 
        }
}

fcalcul.h

Code:

1
2
3
4
5
6
7
8
 
#ifndef FCALCUL_H_INCLUDED
#define FCALCUL_H_INCLUDED
 
double fcalcul(double argUn, double argDeux, char operateur);
 
 
#endif // FCALCUL_H_INCLUDED

Merci pour votre aide , je vais continuer d'optimiser tout ça.

26/04/2016, 10h01
Bousk

Citation:

Envoyé par Alazban

j'ai appris que le type de variable char s’appuie sur ASCII

Un char c'est, dans 99% des cas, 8 bits, devant représenter [0; 127]
A ne pas confondre avec le signed char [-128; 127] et l'unsigned char [0; 255]
Même si dans la majorité des cas un char sera un signed char, ce n'est pas dans la norme

Citation:

Envoyé par Alazban

c'est donc pour ça que mon programme me renvoyait quand même quelque chose de bizarre au début du genre un gros chiffre (1 + 3 = 1965977522)

Euh.. gni ?! :weird:
On te dit depuis le début que ton problème vient de cette syntaxe wtf return ('argUn + argDeux = resultat1'); qui te retournera tout sauf ton résutlat (ou alors avec une probabilité de... 1/((2^32)-1 ?) :ptdr:

Ton code n'est toujours pas bon
- je peux diviser par 0
- si l'opérateur n'est pas reconnu il n'y a pas de return
- et donc (au moins) un warning à la compilation, un cas indéterminé et un résultat random dans ce cas

Citation:

Envoyé par Alazban

J'ai enlevé les ' ' autour de return argUn + argDeux

ENFIN ! 8O

Citation:

Envoyé par Alazban

mais j'ai laissé les break car dans mon cours c'est conseillé

Quel est ce cours qui semble à jeter en partie ? Les break sont 900% inutiles (voire plus) dans ce cas. :calim2:

break, c'est utile quand il n'y a pas de return.
Ici, tout ce que les break vont te donner, ce sont des avertissements de compilation "unreachable code", à cause du return juste avant.

Voici une version où les break sont utiles:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 double fcalcul (double argUn , double argDeux , char operateur) { double ret = 0; switch (operateur) { case '+' : ret = argUn + argDeux; break; case '-' : ret = argUn - argDeux; break; case '*' : ret = argUn * argDeux; break; case '/' : ret = argUn / argDeux; break; default : cout << "Vous n'avez utilisé aucun signe de calcul, veuillez recommencer !" << endl ; break; } return ret; }

26/04/2016, 14h39
Alazban

Merci pour vos réponses, cependant je me pose maintenant la question :

Logiquement un break (dans le switch) va stopper la lecture de la suite de la condition.??? si oui donc j'ai raison de mettre le break, je vais relire le cours sur le return pour être sur. . ( C'est pour ça que je trouve ça plus propre. Je ne sais pas si vous me suivez)
C'est pas mal d'écrire le code comme ça medinoc je vais peut être le mettre en place.
Même avec un return j'ai pas d’erreur "unreachable code".

Bousk tu ne m'a jamais donné de solution si ce n'est me montrer du doigt les problèmes du codes.
Exemple :

Citation:

Ton code n'est toujours pas bon
- je peux diviser par 0
- si l'opérateur n'est pas reconnu il n'y a pas de return
- et donc (au moins) un warning à la compilation, un cas indéterminé et un résultat random dans ce cas

même quand je dit que j'ai enlevé les ' ' .... lol tu me dit Enfin !! mais pourtant tu ne pas pas dit que je devais. Bref

Avec mon niveau actuel je n’ai aucune carte pour changer la donne. Je pense mettre en place une boucle pour que le programme ne s’arrête pas si il y a une erreur ou si je veux continuer a calculer après un premier calcul.

Je vais me répéter, moi la théorie ça me passe par les trous de nez, jveux du pratique y'a que comme ça que j'apprends. Biensur je suis preneur des principes, notions et autres, mais les grandes école avec leurs enseignements du ça passe ou ça casse ne fonctionne pas avec moi.

Bien cordialement,

Alaz
26/04/2016, 14h49
Alazban

OK donc la définition du return est :
Termine l'exécution d'une fonction et retourne le contrôle à la fonction d'appel (ou au système d'exploitation si vous transférez le contrôle à partir de la fonction main).L'exécution reprend dans la fonction d'appel au point immédiatement après l'appel.

Donc pas besoin de break :)
26/04/2016, 17h38
Bousk

Citation:

Envoyé par Alazban

Logiquement un break (dans le switch) va stopper la lecture de la suite de la condition.??? si oui donc j'ai raison de mettre le break, je vais relire le cours sur le return pour être sur. . ( C'est pour ça que je trouve ça plus propre. Je ne sais pas si vous me suivez)
C'est pas mal d'écrire le code comme ça medinoc je vais peut être le mettre en place.

break arrête l'exécution de la boucle/du switch en cours. Return arrête l'exécution de la fonction en cours.

Citation:

Envoyé par Alazban

Même avec un return j'ai pas d’erreur "unreachable code".

A partir du moment où tu n'as même pas de warning pour "not all control paths return a value", je suis pas choqué qu'un unreachable code n'apparaisse pas. Peut-être qu'un unreachable break ne fera pas sourciller le compilo dans tous les cas, franchement, j'en sais rien, et à vrai dire on s'en moque un peu dans ce cas, et au quotidien je ne vois personne écrire un break derrière un return.
Il faudrait compiler en -wall voire -wextra (ou quelque chose comme ça) pour commencer.

Citation:

Envoyé par Alazban

Bousk tu ne m'a jamais donné de solution si ce n'est me montrer du doigt les problèmes du codes.
même quand je dit que j'ai enlevé les ' ' .... lol tu me dit Enfin !! mais pourtant tu ne pas pas dit que je devais. Bref

Ha bon ? Réponse #14 que vois-je ?

Citation:

Envoyé par Bousk

t'as fait 99% du truc, pourquoi chercher compliqué ? return argUn + argDeux; et basta :calim2:

(de moi-même en plus, j'avais oublié)
Btw, sur ce forum on ne donne que rarement la solution toute faîte, surtout pour des trucs aussi évidents.
27/04/2016, 10h47
Alazban

Yop,

Merci pour ton aide :).