C++ Résolu/Fermé

Question

Bonjour,

J'utilise Code Guear Borland C++ 2009 pour faire de la programmation. Je travaille avec des applications fiches VCL. Je suis en train de programmer une calculatrice et j'ai un problème.

Tout d'abord mon code:
http://www.cijoint.fr/cjlink.php?file=cj200912/cijpX4OOV8.txt

Lorsque je veut faire plusieurs additions, ça marche par contre pour les soustractions, les multiplications et les divisions, il n'affiche pas la bonne réponse.
Si vous pouviez m'aider à résoudre mon problème, ce serait sympa.

Locki

PS Je suis encore débutant en progammation alors ne soyez pas trop méchant avec mon code...

Pacorabanix · Answer

Bonjour,

je ne connais pas trop la bibliothèque que tu utilises.

Néanmoins, je pense que tu es, et c'est normal puisque tu es débutant, victime de la mauvaise conception de ton code.

Dans l'idéal il faudrait séparer l'affichage dans des cases des calculs. Mais il faut surtout absolument éviter d'avoir des code "doublons" qui sont plus ou moins censés faire la même chose :

void __fastcall TMain::btEgalClick(TObject *Sender)
{
	if (operateurEnCours == 1) // Plus
	{
		edNombre->Text = resultatFinal + edNombre->Text.ToDouble();
		nombre = 0;
		nombre2 = 0;
	}

	if (operateurEnCours == 2) // Moins
	{
		edNombre->Text = resultatFinal - edNombre->Text.ToDouble();
		nombre = 0;
		nombre2 = 0;
	}

	if (operateurEnCours == 3) // Fois
	{
		edNombre->Text = resultatFinal * edNombre->Text.ToDouble();
		nombre = 0;
		nombre2 = 0;
	}

	if (operateurEnCours == 4) // Divisé
	{
		edNombre->Text =resultatFinal / edNombre->Text.ToDouble();
		nombre = 0;
		nombre2 = 0;
	}

}


et juste après :

void __fastcall TMain::btMoinsClick(TObject *Sender)
{
	operateurEnCours = 2 ;
	resultatFinal = resultatFinal + edNombre->Text.ToDouble();
	nombre = edNombre->Text;
	edNombre->Text = "0";
	nombre2 = edNombre->Text;

}
//---------------------------------------------------------------------------
void __fastcall TMain::btFoisClick(TObject *Sender)
{
	operateurEnCours = 3; // Fois
	resultatFinal = resultatFinal + edNombre->Text.ToDouble();
	nombre = edNombre->Text;
	edNombre->Text = "0";
	nombre2 = edNombre->Text;
}
//---------------------------------------------------------------------------
void __fastcall TMain::btDiviseClick(TObject *Sender)
{
	operateurEnCours = 4 ; // Divisé
	resultatFinal = resultatFinal + edNombre->Text.ToDouble();
	nombre = edNombre->Text;
	edNombre->Text = "0";
	nombre2 = edNombre->Text;
}
...

j'ai l'impression que tu fais une addition à chaque fois là... Je peux me tromper bien sûr, mais essaye de tout regrouper en une fonction "Calcul(int numopération)' qui exécute le calcul par exemple, et les clics sur les divers boutons ne font que modifier par exemple les variables nombre et nombre2. Dans l'idéal même : une classe qui s'occupe des calculs et une autre classe (qui pourrait être ta classe principale de ton programme qui s'ocuppe de l"'interface" (gestion des évènements avec les boutons, affichage, etc...) 

Ce n'est qu'une suggestion possible, il y a surement d'encore meilleures conception que cela, mais un truc de ce goût serait un rempart contre les bugs qui t'arrivent. Et cela permet de modifier plus facilement son code pour la suite.

Locki · Answer

ok merci

Locki · Answer

Alors, qqn a une aide pr moi???

Pacorabanix · Answer

Voilà un schéma qui pourrait être utile. Je ne prétends pas qu'il soit le meilleur possible, mais il devrait surement t'aider à clarifier ton programme.

Tu crées une nouvelle classe (Appelons-là Calculette)
tu l'instancieras dans ton programme : 
Calculette taCalculette;

Tes boutons se chargeront d'appeler des méthodes publiques de cette classe, et c'est les méthodes internes (private) de la classe qui se chargeront de faire tout ce qui est "cuisine interne".

Tu as besoin de :
(prototypes des méthodes publiques)

* Pouvoir entrer un nombre (le premier ou le deuxième)
void Calculette::entrerNb1(string)
void Calculette::entrerNb2(string)

* Pouvoir entrer une opération
void Calculette::entrerOp(? un type enum serait le mieux pour le paramètre, déclaré dans la classe)

* Pouvoir exécuter le calcul et recevoir le résultat
string Calculette::resultat(void)

Pour ton problème de signe, qui peut-être cause l'erreur, ne peux-tu pas faire un Conversion string -> double, faire le double *(-1), conversion double -> string  ? Ou éventuellement tu crées une méthode "changerSigne()" dans ta classe, qui s'occupera de mettre une variable bool à true ou false, et le bon signe sera choisi à l'aide de cette variable private dans les méthodes entrerNbX() )

quoiqu'il en soit avec le signe, voici à quoi ressembleront tes évènements de boutons:

Cliquer sur une opération fait exécuter :

TaCalculette.entrerNb1( ce qui est dans la textbox);
TaCalculette.entrerOp( le code de l'opération );

cliquer sur égal fait exécuter :

TaCalculette.entrerNb2(  ce qui est dans la textbox);
double res = TaCalculette.resultat();
//puis tu affiches le résultat dans ta textbox

Ensuite, je te laisse imaginer le contenu de ces méthodes afin de prévoir diverses erreurs et réaliser l'opération.
N'oublie pas de prévoir un petit constructeur pour initialiser tout à zéro.
Tu auras en tout cas ces variables private:
double nombre1, nombre2;
typeenum operation;

Tu pourrais utiliser des variables private bool qui diraient si un nombre valide est bien entré (le premier et le deuxième). Ces variables seraient vérifiées dans la méthode resultat().

Tu peux essayer de prévoir les erreurs éventuelles, selon si tu arrives ou pas à convertir ta string en double dans les méthodes entrerNbX().

Une manière de faire toute bête serait, en cas d'erreur, de mettre le nombre en question à "NaN" en interne. (Not A Number).

Locki · Answer

Merci a toi de m'avoir répondu mais le problème c'est que je n'ai pas encore vu le type enum.
Si tu pouvais m'expliquer ce type ce seait très sympa de ta part vu que sous google c'est des explications auxquelles je n'ai pas compris.

Si tu connaitrais un tuto ou un site ou il explique en détail les différents types.

Merci beaucoup pour ton post, il va surement m'aider.

Locki