Discussion :

[angeless]

Bonjour
Un polynôme est représenté par un tableau de ses coefficients. Le coefficient du monôme de degré i est rangé dans l’élément de rang i du tableau des coefficients.

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
class Polynome{
           friend Ostream& operator<<(ostream&, const Polynome&);
           friend istream& operator>>(istream&, Polynome&);
private:
	int taille;
	double* coefficients;
public:
	Polynome(); 
	Polynome(int);
	Polynome(Polynome&);
	~Polynome() ;
	void set_taille(int) ;
	void set_coefficients(double* ) ;
	int get_taille( ) ;
	double* get_coefficients( );
	int degre();
	double coefAt(int) ;
           Polynome operator+( const Polynome& );
           Polynome operator*(const Polynome& p);
           Polynome operator*(double);
	bool operator==( const Polynome& );
	const Polynome& operator=(const Polynome&);
           Polynome operator+=(const Polynome& );
           Polynome operator*=(const Polynome& );
           double calculValPolynome(double);
};

on est arrivé a la surdefinition de l'operateur +, voici le code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Polynome Polynome:: operator+( const Polynome& p){
	Polynome pp;
	if(this->taille>=p.taille)
		pp.taille=this->taille;
	else 
		pp.taille=p.taille;
	int i=0;
	while(i<pp.taille){
		pp.cf[i]=coefAt(i)+p.coefAt(i);
		i++;
 
		} return pp;
}

mais il donne une erreur:
error C2662: 'coefAt' : cannot convert 'this' pointer from 'const class Polynome' to 'class Polynome &'Conversion loses qualifiers


NB:la méthode coefAt(int i) retourne la valeur du coefficient de rang i.

[JolyLoic]

Tu n'as pas déclarées constantes les fonctions qui ne modifient pas tes instances...

[angeless]

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
bool Polynome::operator==( const Polynome& p)
{
	Polynome pp;
 
	if(pp.taille=p.taille)
	{
		int i;
		while(i<p.taille)
		{
			if(pp.degre()==p.degre()&&(pp.cf[i]==p.cf[i])) 
				i++;
			else 
				return false;
 
		}
	}
	else 
		return false;
 
}

il me donne cette erreur
warning C4715: 'Polynome::operator==' : not all control paths return a value
Linking...
Devoir.obj : error LNK2001: unresolved external symbol "public: int __thiscall Polynome::degre(void)" (?degre@Polynome@@QAEHXZ)

[bacelar]

Pour le warning.

Code :

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

1
2
if(pp.taille=p.taille)
	{

C'est un = ou == ?

alors si pp.taille>0

Code :

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

int i;

oui, mais i il sera pas forcement égale à 0, sauf en debug ou en CLI.

Code :

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

while(i<p.taille)

Si cela est vrai dés le départ, la méthode retourne quoi ?

Code :

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

if(pp.degre()==p.degre()&&(pp.cf[i]==p.cf[i]))

si c'est faux, on n'est pas prêt de sortie de la boucle

En résumé, le code est une horreur !

Pour l'erreur de link, dans un .h, il y a

Code :

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

1
2
3
4
5
6
class Polynome ...
{
...
  int degre();
...
}

mais le linker ne trouve pas l'implémentation de cette méthode.

[koala01]

Salut,

warning C4715: 'Polynome::operator==' : not all control paths return a value

signifie qu'il y a des chemins qui peuvent être suivis qui ne renvoient pas de valeur (pas de return associé)...

Cela t'aurait presque sauté aux yeux si tu avais pris la peine de mettre les accolades pour l'ensemble des blocs d'instructions, car ton code aurait alors eu la forme de

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
bool Polynome::operator==( const Polynome& p)
{
	Polynome pp;
 
	if(pp.taille=p.taille) /* ca devrait etre un "==" ici, car c'est la
                               * comparaison et non l'assignation
                               */
	{
		int i;
		while(i<p.taille)
		{
			if(pp.degre()==p.degre()&&(pp.cf[i]==p.cf[i])) 
                        {
				i++;
			}
                        else 
                       {
				return false;
		       }
		}
	}
	else 
       {
		return false;
       }
}

Comme tu peux le constater, dans le bloc if, il est possible que l'on ne renvoie jamais rien... car il est possible de sortir de la boucle parce que i==p.taille sans forcément etre passé par le bloc "else" du test if(pp.degre()==p.degre()&&(pp.cf[i]==p.cf[i]))

En outre, jusqu'à présent, le seul retour possible est faux... Or, si tu prévois un opérateur ==, c'est que, quelque part, tu envisage le fait que ca puisse aussi être vrai

[3DArchi]

Bonjour,
Une petite remarque en aparté, tu utilises un double* pour tes coefficients. A moins que ce ne soit un exercice sur les pointeurs, utilises plutôt un conteneur. Tu éviteras beaucoup de problème.
Si tu gardes tes pointeurs, fais donc un petit tour vers cette FAQ.

[Médinoc]

Voici un code corrigé, const-correct (pour peu que la fonction membre Polynome::degre() soit const) et qui ne vérifie pas l'égalité du degré plus de fois que nécessaire.

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
bool Polynome::operator==( const Polynome& p) const
{
	if(this->taille==p.taille && this->degre()==p.degre())
	{
		for(size_t i=0 ; i<p.taille ; i++)
		{
			if(this->cf[i] != p.cf[i]) 
				return false;		
		}
		return true;
	}
	else 
		return false;
}

[Luc Hermitte]

Vu les conversions implicites supportées, en libre, ça serait encore mieux.

[loufoque]

Pourquoi tu mets des this-> partout dans la correction ?

[Médinoc]

Pour bien montrer qu'on n'a pas besoin de créer un objet temporaire pp (qui n'était pas initialisé, par ailleurs).