ANALYSEUR D'EXPRESSION AVEC L'API WINDOWS ( VISUAL C++ )

Commentaire de MuPuF le 19/06/2005 00:52:35

... il plante au démarage en mode release, encore une différence de gestion de la mémoire, rrr... essais de le corriger, j'ai 2 warnings si tu veux:
C:\Documents and Settings\Progr@mmeur Fou\Bureau\analyseur d expression complexe\fonctions.h(1405) : warning C4172: returning address of local variable or temporary
C:\Documents and Settings\Progr@mmeur Fou\Bureau\analyseur d expression complexe\fonctions.h(1414) : warning C4172: returning address of local variable or temporary

Il doit y avoir un rapport

Commentaire de cosmobob le 19/06/2005 03:56:59

salut,
déja j'utilise vs7, et le code compile pas : les operateurs comme += doivent - dans le standard c++ que vc6 ne respecte pas - retourner une valeur. Il suffit de leur faire renvoyer un Complexe& et de mettre return *this; a la fin de chaque implementation de tes operateurs +=, -=, etc.

sinon moi ton programme plante au démarrage en release ET en debug.
En debug il me met que la pile autour d'une variable dans la fonction Liste interpretation (char* expression) a été corrompu.
En gros ca veut dire que t'as mal géré les pointeurs.
regarde ca : char expr[5]; et jamais dans la suite de ta fonction tu ne vérifies que ton indice j utilisé dans  expr[j] ne dépasse 4 (et j finit par dépasser 4; ton code est mal concu ...). D'ou un buffer overflow...
Ceci ne m'etonne pas trop car le fait de renvoyer a d'autres endroits un pointeur qui désignait un tableau déclaré en local ne t'as pas ennuyé (le fameux warning  "returning address of local variable or temporary"). Pour le corriger, rend la variable que tu renvoies static !! static dit que ta variable continue d'exister tout le temps meme quand tu es sorti de la fonction; toi tu renvoyais un pointeur vers une variable locale qui n'etait donc plus valide quand tu sors de la fonction appelée !!!!!

Bon il te manque certaines bases du C qd meme ;)





moi aussi ca plante en release au démarrage

Commentaire de MuPuF le 19/06/2005 10:51:10

Moi vc++ 6, le mode release plante, mais va savoir ou, puisque en debug il marche, je trouve ça vraiment con que ce soit pas le meme code en release qu'en debug (outre le fait qu'il ait des infos de debug)...
Sinon, je pense que le fait de corriger ça devrait enlever le prob du release.
Good luck, préviens quand tu as fini, car je l'utilise ton prog, en plus les varibles sont tjs accessible entre 2 instances ... (stocké dans un fichier sous forme de texte)

Commentaire de cosmobob le 19/06/2005 11:50:50

enfin la ca devrait plutot etre ASSERT(j >=0 && j < 5);

Commentaire de MuPuF le 19/06/2005 12:19:42

lol, pas poss ces matt ... je me perds ....

Commentaire de cosmobob le 20/06/2005 00:26:26

ben moi je reussis toujours a le faire bugger :-)
primo, si tu fais un new char[ ... ] quelque part, fais un delete [] par la suite (la t'as des fuites de memoire)
Note qu'une connaissance meme basique de la STL te ferait pencher pour l'utilisation des std::string plutot que de faire toi meme un new char[taille] ( et comme ca t'aurais rien besoin de desallouer)
sinon le code est tres mal organisé (tu as qd meme un SetWindowText a l'interieur de ta fonction calcul !!!! il y avait moyen de renvoyer le calcul, puis de faire tout ce qui est affichage a part)

enfin pour faire boguer le truc, des le debut je reste appuyé sur une touche, et il me dit qu'il y a un pb de gestion de la memoire, je cite:
(---------------------------
Microsoft Visual C++ Debug Library
---------------------------
Debug Assertion Failed!

Program: ...lyseur d expression complexe\Debug\analyseur d expression.exe
File: dbgheap.c
Line: 1100

Expression: _CrtCheckMemory()

For information on how your program can cause an assertion
failure, see the Visual C++ documentation on asserts.

(Press Retry to debug the application)
---------------------------
Abandonner   Recommencer   Ignorer  
---------------------------
)

enfin voila, je suis pas du meme avis que MuPuf, c'est loin d'etre parfait (tes problemes sont liés a la mauvaise organisation du code !!!!), mais ca reste cela dit pas trop mal.

a+

Commentaire de cosmobob le 20/06/2005 00:40:26

tiens, en plus, a partir du moment ou ca reussit a calculer des expressions mal parenthesees on peut penser que le parsage de l'expression est mauvais !!!

Commentaire de cosmobob le 20/06/2005 01:36:57

aléatoire = ya du mal a donner d'exemple.
La seule chose a dire est que la memoire est mal gerée, donc si ca plante pas, c'est de la chance ...
et sinon prends pas ca pour du cassage, j'ai essayé de faire en sorte le plus possible que tu puisses corriger tes erreurs; apres c'est les limites d'un message sur un site, une phrase brute peut avoir l'air plus crue que dans l'esprit de celui qui l'écrit.

un conseil si tu veux améliorer ton truc : reflechis a comment re organiser ton code et repars de zero. Sachant que ta stratégie de décomposition d'expression est déja faite (ce qui a  du etre le plus long)

a++ ;)

Commentaire de Pamaury le 20/06/2005 18:51:24

"unsigned long double"(fonction.h)
->çà existe çà ?????? pas à ma connaissance en tout cas çà compile pas et je comprend bien pourquoi . un double ne peux pas être signé ou non signé .

ensuite faudrait faire un
using namespace std;

Commentaire de Arnaud16022 le 21/06/2005 20:32:54

qui est l'abr***** qui a mis 5?
10/10 génial

Commentaire de gagah1 le 22/06/2005 15:41:13

Par contre il gère pas d'erreur de saisi. Exemple si on fait 10m10 il donne comme resultat 1010, si tu fais 12=5 le resultat est 125. A verifier?