Discussion :

[lcfseth]

Bonjour,
Alors voila j'ai un gros probleme.
Je travail sur un mini-Projet en C. Je dois faire une calculatrice.
Bon j'ai termine tout le code de la calculatrice. Et la je travail sur le dessin d'une courbe de dessin. Pour cela j'utilise 3 variable Max,Min,Densite.
Pour le moment je n'ai ecrit aucune fonction qui modifie ces valeurs.
Pourtant la valeur de Max change. J'ai un peu trace le code et il semblerait que la modification se fait au niveau de cette fonction :

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
 
int ResultatFonction(char * Op,DPoint **  Liste ,double Max,double Min,int n)
	{
	double x=0;
	double r=0;
	double pas = 1;//(Max-Min)/n;
	int err=0; 
	int c=0; //Compte le nombre de points valide
	Vider(Liste);
	x=Min;
	while(x<=Max)
		{
		Val_Constantes[4]=x;
		err=CalculerOperation(Op,&r);
		if (err == ERR_SUCCES)
			{
			Push(Liste,x,r);
			c++;
			}
		x+=pas;
		}
	if (c==0) return ERR_NEPEUTDESSINER; // Si nous n'avons aucun point valide alors on la fonction ne peut etre dessiner avec ses parametres
	return ERR_SUCCES;
	}

Liste : c'est une liste chainee. j'ai utilise un code que j'ai trouver sur Devellopez.com
CalculerOperation : Calcul une formule mathematique du style 5+10log(4)...
Val_Constantes[4]=x; : La constante "x" est predefini et remplacé dans la CalculerOperation par Val_Constantes[4]. donc si Val_Constantes[4]=4 et qu'on a 5+x. CalculerOperation renvoi 9.

En fait Max prend la valeur de r dans la derniere iteration. (Si je prend comme fonction f(x)=0 Max = 0, f(x)=2x Max = 2*Max...
Lors de l'appel de la fonction si je met

Code :

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

1
2
 
ResultatFonction(Op,&Liste ,Min,Max,Densite)

au lieu de

Code :

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

1
2
 
ResultatFonction(Op,&Liste ,Max,Min,Densite)

Le probleme disparait (enfin peut etre qu'une autre variable est touche).

Au debut de mon programme je define les trois variables.

Code :

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

1
2
3
4
 
double Max=10;
double Min=-10;
double Densite=50;

Lorsque j'interverti Densite et Max :

Code :

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

1
2
3
4
 
double Densite=50;
double Max=10;
double Min=-10;

c'est Densite qui change.
Alors juste pour tester j'ai cree un variable qui n'est utilise null part et je l'initialise à 100

Code :

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

1
2
3
4
5
 
double Salut=100;
double Densite=50;
double Max=10;
double Min=-10;

Et maintenant c'est Salut qui change.

Si je declare ces variables comme constante, plus de probleme. Seulement je dois les modifer. Au pire je pense les declarer const et ajouter des variable zoom et incDensite. et faire un truc du style

Code :

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

1
2
 
ResultatFonction(Op,&Liste ,Max*zoom,Min*zoom,Densite*IncDensite)

Mais franchement c'est moche. En plus je devrais l'expliquer a mes profs alors.

Je pense que ca doit etre un probleme de chevauchement de variables en memoire mais je ne sais pas trop comment y remedier.
Ps: Mon projet est en C, si vous pouviez me donner que des solutions C et non C++. (Quoique meme en compilant en tant que c++ le probleme reste le meme)

Je travail sous VisualC++(Avec Visual Studio 2005) et je n'utilise que les bibliotheques AINSI et WIN32(Pour l'interface).
Merci pour votre aide.

[Emmanuel Delahaye]

Code :

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

1
2
 
		Val_Constantes[4]=x;

Comment est défini Val_Constantes ? Les globales, çaÿ mal...

[lcfseth]

Comment est défini Val_Constantes ? Les globales, çaÿ mal...

char * Constantes[] = {"pi","e","c","g","x","a","b","c","d","e","m","m1","m2","m3","m4"};//Contient les chaines representant les constantes
double Val_Constantes[] = {CPI,CE,299792458,6.6742e-11,0,0,0,0,0,0,0};

Je suis oblige de les declarer en global car ils sont uilise au cours de 3 fonctions successive. Mais j'ai deja teste le code en mode console et il fonctionne parfaitement. Ce n'est que lorsque j'ai voulu rajoute la partie dessin de courbe(donc passage à du win32) qu'est apparu le probleme.

[Emmanuel Delahaye]

char * Constantes[] = {"pi","e","c","g","x","a","b","c","d","e","m","m1","m2","m3","m4"};//Contient les chaines representant les constantes
double Val_Constantes[] = {CPI,CE,299792458,6.6742e-11,0,0,0,0,0,0,0};

Et tu modifies une constante ?

Code :

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

Val_Constantes[4]=x;

A moins que... tu mélanges dans le même tableau, des valeurs fixes et des variables ? C'est plutôt bizarre comme conception.

Et pour les valeurs constantes, qu'est-ce qui t'empêche d'utiliser des macros (#define) ? Je ne vois pas à quoi sert le tableau de constantes, à part pour compliquer le code inutilement...

[lcfseth]

Si je n'utilise pas de #define c'est parceque ce ne sont pas des constantes que j'utilise dans mon programme. En realite lorsque l'utilisateur entre une operation du style "5e+4pi" Une premiere fonction utilise le premier tableau pour reformuler l'operation de facon plus conventionel en "5*e+4*pi"
et la fonction Calcul va utiliser le second tableau pour remplacer "e" et "pi" par leur valeur.
Utiliser un tableau me permet de m'affranchir des long switch et me permet de rajouter de nouvelles constantes sans modifier le code source.
Il me suffit de rajouter 2 valeur et de redefinir la valeur "NombreDeConstantes". (qui est une constante definie en utilisant #define)

Pour le nom, au depart je voulais simplement utiliser des constantes mais je me suis apercu que je pouvais introduire la notion de variable, En definissant une constante (au sens mathematique et non algorithmique) "x" et on modifiant sa valeur avant chaque appel. La fonction Calcul la considere a chaque fois comme constante. Je peux ainsi utilise une meme operation ("5x" par exemple) pour calculer plusieurs valeurs que je
pourais utilisé pour dessiner la courbe de la fonction f(x)=5*x.


Je sais qu'il y a mieux, mais je prefere faire long et pas optimise mais avoir un principe que je pourrais expliquer simplement ( certain de mes profs sont assez lents ;( )

[Pierre Maurette]

Bonjour,

Votre problème semble effectivement étrange, et il est difficile de rentrer dedans. Vous avez fait plusieurs tests, admettez que vous pouvez peut-être en avoir fait un un peu rapidement dans l'énervement. Deux suggestions:
- Min et Max sont des noms dangereux, ils peuvent éventuellement avoir été définis "ailleurs". Rendez-leur leurs chances d'être uniques, par exemple en les renommant Min_Calc_Canvas et Max_Calc_Canvas.
- Faîtes souvent des clean builds, et en particulier après un refactoring comme évoqué ci-dessus. Et surtout quand vous introduisez win32. Là, vous pouvez carrément créer un autre projet, et copier/coller vos sources.

Si la variable change de façon reproductible, n'hésitez pas à utiliser le débogueur pour la surveiller. Ni à faire des recherches sur son nom sur l'ensemble du projet.

Good luck...

[Climoo]

Vraiment bizarre ton probleme... Meme si je comprend pas grand chose à ce qu'est censé faire ton programme (je suis loin d'être un pro...), il est clair que quand tu passes Max en parametres, il ne peut théoriquement pas être modifié, puisque c'est un passage par valeur. Et ce que je ne comprend pas, c'est pourquoi tu passes Max en parametre, s'il existe déja en global au dessus.
Et de plus quand tu dis que la variable est modifiée au niveau de cette fonction, est ce que cela signifie bien que, si tu fais un

Code :

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

1
2
3
4
 
printf("%f\n",Max);
ResultatFonction(Op,&Liste ,Min,Max,Densite)
printf("%f\n",Max);

Cela n'imprime pas la même valeur??
Et sinon ce que je ferais à ta place, c'est d'imprimer la valeur de la variable qui se modifie toute seule un peu partout dans ta fonction, histoire de voir à quel moment elle change vraiment (enfin peut être que tu l'as déja fait...).

[Sve@r]

Bonjour,
Alors voila j'ai un gros probleme.
Je travail sur un mini-Projet en C. Je dois faire une calculatrice.
Bon j'ai termine tout le code de la calculatrice. Et la je travail sur le dessin d'une courbe de dessin. Pour cela j'utilise 3 variable Max,Min,Densite.
Pour le moment je n'ai ecrit aucune fonction qui modifie ces valeurs.
Pourtant la valeur de Max change. J'ai un peu trace le code et il semblerait que la modification se fait au niveau de cette fonction :

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
 
int ResultatFonction(char * Op,DPoint **  Liste ,double Max,double Min,int n)
	{
	double x=0;
	double r=0;
	double pas = 1;//(Max-Min)/n;
	int err=0; 
	int c=0; //Compte le nombre de points valide
	Vider(Liste);
	x=Min;
	while(x<=Max)
		{
		Val_Constantes[4]=x;
		err=CalculerOperation(Op,&r);
		if (err == ERR_SUCCES)
			{
			Push(Liste,x,r);
			c++;
			}
		x+=pas;
		}
	if (c==0) return ERR_NEPEUTDESSINER; // Si nous n'avons aucun point valide alors on la fonction ne peut etre dessiner avec ses parametres
	return ERR_SUCCES;
	}

En fait Max prend la valeur de r dans la derniere iteration. (Si je prend comme fonction f(x)=0 Max = 0, f(x)=2x Max = 2*Max...
Lors de l'appel de la fonction si je met

Code :

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

1
2
 
ResultatFonction(Op,&Liste ,Min,Max,Densite)

au lieu de

Code :

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

1
2
 
ResultatFonction(Op,&Liste ,Max,Min,Densite)

Le probleme disparait (enfin peut etre qu'une autre variable est touche).

Au debut de mon programme je define les trois variables.

Code :

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

1
2
3
4
 
double Max=10;
double Min=-10;
double Densite=50;

Lorsque j'interverti Densite et Max :

Code :

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

1
2
3
4
 
double Densite=50;
double Max=10;
double Min=-10;

c'est Densite qui change.
Alors juste pour tester j'ai cree un variable qui n'est utilise null part et je l'initialise à 100

Code :

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

1
2
3
4
5
 
double Salut=100;
double Densite=50;
double Max=10;
double Min=-10;

Et maintenant c'est Salut qui change.

Si je declare ces variables comme constante, plus de probleme. Seulement je dois les modifer. Au pire je pense les declarer const et ajouter des variable zoom et incDensite. et faire un truc du style

Code :

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

1
2
 
ResultatFonction(Op,&Liste ,Max*zoom,Min*zoom,Densite*IncDensite)

Mais franchement c'est moche. En plus je devrais l'expliquer a mes profs alors.

Je pense que ca doit etre un probleme de chevauchement de variables en memoire mais je ne sais pas trop comment y remedier.
Ps: Mon projet est en C, si vous pouviez me donner que des solutions C et non C++. (Quoique meme en compilant en tant que c++ le probleme reste le meme)

Je travail sous VisualC++(Avec Visual Studio 2005) et je n'utilise que les bibliotheques AINSI et WIN32(Pour l'interface).
Merci pour votre aide.

Tout ceci est une illustration parfaite d'un comportement indéterminé.
A un moment donné, tu vas écrire dans un pointeur ou un tableau plus que ce qu'il peut recevoir et donc ce qui déborde va écraser des emplacements utilisés par d'autres variables. T'es dans la pire des situations. Faut que tu retrouves un bug qui peut dater d'il y a 6 mois et qui est maintenant noyé dans le reste.

Déjà les remarques de Emmanuel sont utiles. Les globales c'est le mal. Tes variables sont utilisées dans 3 fonctions ben les 3 fonctions reçoivent ces variables en paramètre. Cela rend tes fonctions trop lourdes (trop de paramètres) alors tu regroupes les variables allant ensemble dans une structure et tu passes l'adresse de ta structure à tes fonctions (avec l'adresse les fonctions peuvent retrouver les variables associées).