Discussion :

[le_voisin]

Salut,

je debute tout juste en c++, j ai commence par créer une classe de polynome. J ai un probleme pour faire une methode me retournnat le degre de mon polynome, quelqu un voit il d ou ca pourrait venir ?? C est dans la methode "degre".
Autre chose : pourquoi je ne peux pas instancier un polynome de degre 2 en tapant polynome(2) ?
Si vous avez d autres remarques a rpopos de ma classe, elles sont les bien venues !! Merci et voici ma classe :

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
 
#include <iostream> 
#include <cmath> 
using namespace std;
 
#ifndef polynome_H
#define polynome_H
 
float *tab; //pointeur de flottant
 
class polynome
{
//définition des attributs
 
int n;
 
//déclaration des méthodes
public :
void saisir(int);
void afficher(int);
float extraire(int);
void unitaire(int);
int degre(void);
};
 
void polynome::saisir(int deg)
{
	tab=new float[deg]; //reserve un nombre d adresse correspondant à deg
	for(int i=0; i<=deg; i=i+1)
	{
	cout<<"coefficient de degre"<<i<<"\n";
	cin>>tab[i];
	}
}
 
void polynome::afficher(int deg)
{
	for(int i=0; i<=deg; i=i+1) 
	{cout<<tab[i]<<" \n";}
}
 
float polynome::extraire(int puiss)
{
	float a=tab[puiss];
	return(a);
}
 
void polynome::unitaire(int deg)
{
	float c=extraire(deg);
	for(int i=0; i<=deg; i=i+1)
	{tab[i]=tab[i]/c;}
}
 
int polynome::degre(void)
{
	return(n);
}
#endif

[loufoque]

J ai un probleme pour faire une methode me retournnat le degre de mon polynome, quelqu un voit il d ou ca pourrait venir ?? C est dans la methode "degre".

Tu n'as jamais défini la valeur de n.

Autre chose : pourquoi je ne peux pas instancier un polynome de degre 2 en tapant polynome(2) ?

Parce que tu n'as pas défini de constructeur qui prenne un entier.

Si vous avez d autres remarques a rpopos de ma classe, elles sont les bien venues !!

tab qui n'a rien à faire en global, allocation mémoire insuffisante (un polynôme de degré 2 a 3 composantes), pas de destructeur qui libère la mémoire, choses faites dans saisir() qui devraient être faites dans le constructeur, inclusion inutile de <cmath> et pollution de l'espace de nom principal sans raison, fonctions membres afficher() et unitaire() qui prennent un argument degré alors que cela est inutile, sémantiques de copie mal définies...
Bref la liste des défauts est longue.

D'ailleurs je te conseille plutôt d'utiliser std::vector<float>, ça simplifiera les choses.

[le_voisin]

A propos de la methode devant me retourner le degre...la valeur n a été defini en attribut de la classe ...?? cela n est il pas suffisant ?

Je me doutais bien que je ne n'avais pas besoin d argument degre dans mes methodes mais je n arrive pas à la programmer autrement !?

Si tu peux mettre sur la voie, je serai tres interesse !

[bladerunners]

Tu n'as pas besoin de float *tab; en global. Tu peux le declarer en attribut.

Declare un constructeur et un destructeur

Code :

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

1
2
3
4
5
6
7
8
9
polynome::polynome(int degre):
tab(NULL)
{
  n = degre;
}
 
polynome::~polynome()
{
}

Et pour tes methodes
void polynome::afficher(int deg)
-> en

Code :

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

1
2
3
4
5
6
7
void polynome::afficher()
{
	for(int i=0; i<=n; i=i+1) //ton polynome est de degre n 
	{
           cout<<tab[i]<<" \n";
        }
}

[loufoque]

Je te soumets une correction possible.

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
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
#ifndef polynome_H
#define polynome_H
 
#include <iostream>
#include <vector>
 
class polynome
{
	std::vector<float> tab;
 
public:
	polynome(int);
	void saisir();
	void afficher();
	float extraire(int);
	void unitaire();
	int degre();
};
 
polynome::polynome(int deg) : tab(deg+1)
{
}
 
void polynome::saisir()
{
	for(int i=0; i<=tab.size(); i++)
	{
		std::cout<<"coefficient de degre"<<i<<"\n";
		std::cin>>tab[i];
	}
}
 
void polynome::afficher()
{
	for(int i=0; i<=tab.size(); i++) 
	{
		std::cout<<tab[i]<<" \n";
	}
}
 
float polynome::extraire(int puiss)
{
	return tab[puiss];
}
 
void polynome::unitaire()
{
	float c=extraire(tab.size());
	for(int i=0; i<=tab.size(); i++)
	{
		tab[i] /= c;
	}
}
 
int polynome::degre()
{
	return tab.size();
}
#endif

[le_voisin]

ok je te remercie de tes reponses tres instructives...par contre pourrais tu m expliquer comment est construit ton constructeur, d apres les docs que j ai lu ca n a pas l air d etre une struture classique de constructeur, en general on le defini dans les accolades, non ?

[Eusebe]

ok je te remercie de tes reponses tres instructives...par contre pourrais tu m expliquer comment est construit ton constructeur, d apres les docs que j ai lu ca n a pas l air d etre une struture classique de constructeur, en general on le defini dans les accolades, non ?

Je suppose que tu ne comprends pas la liste d'initialisation qu'utilises loufoque ?
Jette un oeil dans la FAQ à cette réponse, ça devrait pouvoir t'éclairer :
http://c.developpez.com/faq/cpp/?pag...EUR_liste_init

[koala01]

Salut,

ok je te remercie de tes reponses tres instructives...par contre pourrais tu m expliquer comment est construit ton constructeur, d apres les docs que j ai lu ca n a pas l air d etre une struture classique de constructeur, en general on le defini dans les accolades, non ?

En fait, c'est relativement simple:

Un constructeur, par nature, ca initialise les membre de la classe...

Quand, meme sur un type de base, tu écris, par exemple

Code :

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

int i=23;

le compilateur cherche un endroit suffisemment grand en mémoire où il puisse mettre ta variable ("instanciation"), puis "initialise" la valeur de celle-ci à 23.

En sommes, il fait exactement le meme travail, au moment de l'initialisation, que celui du constructeur de ta classe...

Le membre Tab est, en plus, une classe (de la STL )... et il dispose à ce titre, entre autres, d'un constructeur qui demande un entier pour être appelé...

Tout comme, quand tu écrit une classe dérivée par héritage d'une autre, il se peut que tu doive fournir des valeur pour construire la classe parente sous la forme de

Code :

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

1
2
3
4
5
6
 
ClasseDerivee::ClasseDerivee(type arg1, type2 arg2):
                    ClasseParent(arg1)
{
 
}

rien ne t'empeche de faire l'initialisation de tes membres en "appelant leur propre constructeur" (ici, le contstructeur de ton std::vector qui prend un entier)

C'est d'autant plus intéressant que, sinon, l'initialisation de ton membre tab nécessiterait une instruction (un peu) plus complexe...

[le_voisin]

ok merci je comprend mieux.
Je veux aussi faire un destructeur pour ma classe. D apres ce que j ai compris dans la FAQ un destructeur n est pas obligatoire puisqu il y en a toujours un de créé automatiquement.
Quel est donc alors l interet de definir un constructeur ? Ai je interet à en definir un ?

Voilà celui que j ai fait :

Code :

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

1
2
3
4
5
6
 
polynome::~polynome(void)
{
	tab.clear();
    return;
}

Je me suis inspire d'un cours...et j ai pas compris à quoi sert le return ??

Apart ca ,si j ai a peu pres compris, ma classe polynome est derivee de la classe vecteur, mon destructeur detruit mon polynome (il vide le vecteur), et un destructeur appelé automatiquement detruit mon vecteur ??? c est ca !?

Merci

[Eusebe]

Quel est donc alors l interet de definir un constructeur ?

Je suppose que tu voulais dire "Quel est donc alors l'intérêt de définir un destructeur ?
Quand tu dois libérer de la mémoire allouée dynamiquement par exemple (dans ce cas, le destructeur créé par défaut ne la libèrera pas). C'est le cas par exemple quand ta classe contiens des pointeurs.

Ai je interet à en definir un ?

Dans ton cas, d'après ce que j'ai vu, ce n'est pas la peine, puisque ton seul membre est un vector<float> qui est un objet, et donc sera automatiquement détruit par le destructeur par défaut.

Je me suis inspire d'un cours...et j ai pas compris à quoi sert le return ??

A rien dans ce cas... Et tab.clear(); ne sert à rien non plus puisque ce sera fait automatiquement.

Apart ca ,si j ai a peu pres compris, ma classe polynome est derivee de la classe vecteur, mon destructeur detruit mon polynome (il vide le vecteur), et un destructeur appelé automatiquement detruit mon vecteur ??? c est ca !?

Je n'ai pas vu dans ce topic que ta classe dérivait de la classe vector. Elle contiens un membre vector, mais elle n'en dérive pas.
Au risque de radoter, il ne sert à rien de vider le vector, c'est fait dans son destructeur (qui sera appelé automatiquement lors de la destruction de tes objets).

[Luc Hermitte]

pourrais tu m expliquer comment est construit ton constructeur, d'apres les docs que j'ai lu ca n'a pas l'air d'être une struture classique de constructeur, en general on le defini dans les accolades, non ?

Bien au contraire. C'est la forme à préférer. En ce qui me concerne, la partie entre accolades, c'est pour quand je ne peux pas faire autrement, pour le code qui doit être exécuté après les constructions de tous les membres.