Discussion :

[User Name]

Bonjour a tous,

D'abord je voudrais m'excuser pour le "flood" de topics en ce moment, vu que j'en suis a mon quatrieme pour un meme projet. J'essaie autant que possible de limiter le nombre de questions, et c'est seulement quand je seche au bout de plusieurs heures et que je n'ai aucune autre alternative que j'en poste un, dans l'espoir qu'il soit aussi constructif aux autres.

Pour en revenir a mon probleme, le voici :

Polynome.cxx (uniquement les fonctions qui nous interessent ici)

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
string Polynome::AffichePolynome ()
{
	if (Coefficients!=NULL)
	{
		cout<<Coefficients[0];
		for (int i=1; i<Degre; i++)
		{
			if (Coefficients[i]<0)
			{
				cout<<"-";
			}		
			else
			{
				if (Coefficients[i]!=0)
				{
					cout<<"+"<<Coefficients[i]<<"*x";
					if (i>1)
					{
						cout<<"^"<<i;
					}
				}
			}
		}
	}
	else
	{
		cout<<"POLYNOME_NUL";
	}
}
 
Polynome::~Polynome()
{
	if (Coefficients!=NULL)
	{
		delete[] Coefficients;
		Coefficients=NULL;
	}
}

Polynome.hxx (tout le fichier)

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
#include "RacineFonctions.hxx"
#include <string>
 
using namespace std;
 
#ifndef POLYNOME
#define POLYNOME
 
class Polynome : public RacineFonctions
{
	private:
	int Degre;
	double* Coefficients;
 
	public:
	Polynome(int=0, double* = NULL);
	Polynome(const Polynome&);
	~Polynome(void);
	double Evalue(int, double) ;
	Intervalle Evalue(int, const Intervalle&);
	string AffichePolynome(void);
 
	Polynome operator = (const Polynome&);
};
#endif

Main.cxx (tout le fichier)

Code :

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

1
2
3
4
5
6
7
8
9
#include "Polynome.hxx"
 
int main ()
{
	double a[3] = {2.,-3.,1.};
	Polynome P1(2,a);
	P1.AffichePolynome();
	return 0;
}

Resultat :

Le compilateur me signale un Segmentation Fault.
Il n'a pas lieu si j'enleve P1.AffichePolynome(); du main.
Si je laisse le runtime utiliser le destructeur par defaut en mettant le mien en commentaire avec son prototype, j'ai un glibc detected suivi d'un tas de lignes incomprehensibles (qui correspondrait apparemment a un free() sur une variable non "malloquee").

Je sais que mon code presente quelques autres imperfections, mais je ne leur donne pas la priorite pour le moment. Merci de votre aide.

Edit : Le debogueur "DDD" arrive bien, dans la fonction AffichePolynome(), au "else" si je cree un Polynome sans parametres (donc il prend en fait les parametres par defaut), et au "if" si je cree un Polynome avec 2 et a en parametre. Dans les deux cas, il recoit un SIGVSEGFAULT juste apres l'execution de AffichePolynome(). C'est pour ca que je bloque, bien que cet outil m'ait pas mal depanne dans le passe.

[r0d]

Bonjour,

je ne sais pas où est l'erreur. A mon avis, elle est ailleurs. Peut-être lors de la construction. Il faudrait que tu nous montre le code des constructeurs.
En attendant, quelques remarques:

Code :

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

1
2
 
	Polynome operator = (const Polynome&);

Cet operateur n'est pas bon. Ce n'est pas impossible que le problème vienne de là d'ailleurs.
Tel qu'il est définit ici, ton opérateur d'affectation renvoie une copie de Polynome, or il devrait renvoyer une référence sur lui-même:

Code :

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

1
2
3
4
5
Polynome & operator = (const Polynome& source)
{
   // code
   return *this;
}

Si je laisse le runtime utiliser le destructeur par defaut en mettant le mien en commentaire avec son prototype, j'ai un glibc detected suivi d'un tas de lignes incomprehensibles

C'est normal: tu as un pointeur parmi tes variables membres (Coefficients), ce qui signifie de la mémoire dynamique, ce qui signifie que c'est à toi de gérer cette mémoire. C'est donc à toi de gérer la construction et la destruction. Ta classe est bien définie (il y a tout ce qui faut: default ctor, copy ctor, assign ctor, et dtor), à l'erreur sur l'op. d'assignation près; reste à voir son implémentation.

D'où, pour une nième fois (n tendant vers l'infini), le conseil d'utiliser un std::vector plutôt qu'un tableau c-style (dynamique de surcroît).

[Arzar]

Hmmm, rassure-moi User Name, ton constructeur fait bien une copie du tableau passé en paramètre ?

Code :

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

1
2
3
4
5
6
Polynome(int i, double* d)
{
   Degre = i;
   Coefficients = new double[i];
   std::copy(d, d+ i,  Coefficients );
};

Parce que si tu copies directement le pointeur comme ceci :

Code :

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

1
2
3
4
5
Polynome(int i, double* d)
{
   Degre i;
   Coefficients = d;
};

Alors clairement le code suivant est faux :

Code :

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

1
2
3
4
5
6
7
#include "Polynome.hxx"
 
int main ()
{
   double a[3] = {2.,-3.,1.};
   Polynome P1(2,a);
}

Car alors le destructeur de Polynome tente de faire un delete[] sur "a", tableau qui n'a pas été alloué par new[] et qui se trouve probablement sur la pile !!

(Et comme rod je préconise std::vector pour éviter ce genre de problème, le code du constructeur deviendrait alors :

Code :

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

1
2
3
4
5
Polynome(int i, double* d):
Coefficients(d, d+ i)
{
   // pas besoin de Degre... car == à Coefficients.size()
};

)

[3DArchi]

Salut,

Code :

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

string Polynome::AffichePolynome ()

Cette fonction ne retourne rien, ce qui revient à dire côté appelant que le destructeur de std::string est appelé sur une chaîne non initialisée => boum.

Sinon, +1000 avec Arzar et r0d pour troquer tes pointeurs avec un joli std::vector

[Arzar]

Salut,

Code :

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

string Polynome::AffichePolynome ()

Cette fonction ne retourne rien

Mais c'est vrai ça ! Comment est-ce que ce code peut compiler en premier lieu ? C'est louche cette histoire.

[3DArchi]

Mais c'est vrai ça ! Comment est-ce que ce code peut compiler en premier lieu ? C'est louche cette histoire.

Un vieux reste de (très) vieux C probablement. Ca compile en mingw (même avec une ribambelle de -W et -std=c++98 et -ansi) avec quand même un petit warning. En revanche, Visual refuse de le compiler avec des options par défaut.

[User Name]

Merci pour vos conseils. Je precise que je ne peux pas utiliser de std::Vector, car c'est mon professeur qui m'impose l'utilisation d'un double*. Ne me demandez pas pourquoi, je n'en sais rien (peut-etre parce qu'on n'a jamais entendu parler en cours de Vector en C++, meme si notre curiosite suffit a nous l'apprendre).

Pour le reste, je m'y atele et je vous tiens au courant.

[User Name]

Effectivement, c'etait a cause du bete string a la place du void en retour dans ma fonction PolynomeAffiche() ! J'aurais pu chercher encore longtemps !

Par contre, et ca me scandalise profondement, je ne peux pas faire de return (*this) dans mon operator = : le compilateur m'afficherait cette erreur.

[dragonjoker59]

Et pourtant tu DOIS faire return *this dans ton operateur d'affectation. L'erreur que tu pointes n'a à priori aucun rapport.

[User Name]

Tu as raison pour le return (*this). Les deux problèmes sont résolus. Merci à tous, aussi efficaces que d'habitude !