Constructeur pour classe vecteur

**zopet** · 19/01/2011, 11h47

Bonjour tout le monde !

Voila je commence le C++ en cours et j'ai un exercice ou je dois faire:

-un contructeur par défaut
-un autre constructeur qui initialise le vecteur avec des zeros
-un constructeur par copie
-un destructeur

j'ai fait mes lignes de codes mais j'ai une erreur et je vois pas comment la résoudre.

voici le fichier header

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
 
using namespace std;
 
class vecteur
{
 private :
  int size; 
  double* p;
 public :
  vecteur() {size=0;}//contructeur par défaut
  vecteur(int);
  vecteur(const  vecteur& v);
  ~vecteur();
};
 
vecteur :: vecteur(int taille)
{p=new double[taille=size];
  for (int i=0 ;i<taille ;i++)
    p[i]=0;}
 
 
vecteur :: vecteur(const  vecteur& v)
{ p=new double[size=v.size];
  for (int i=0 ;i<size; i++)
    p[i]=v[i];}
 
    vecteur :: ~vecteur()
    {delete[] p;}
 
void vecteur :: affiche()
{ for (int i=0; i<size; i++)
    cout << p[i] <<"";
    cout << "\n";}

Et ici mon fichier .cpp (juste pour tester la compilation)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include <iostream>
#include "vecteur.h"
 
 
using namespace std;
 
int main()
{ 
    vecteur a(3);
    a.affiche();
    return(0);
 
}

J'ai un probleme qui m'indique un no match pour [] et pas de void pour le dernier .

pouvez vous m'indiquer l'erreur dans mon code et m'expliquer le pourquoi du comment?

P.S:Je suis un noob en C++(en programmation en général) et je viens de commencer il y a deux jours.

**dragonjoker59** · 19/01/2011, 12h32

Salut!
Tu as oublié de déclarer ta fonction "affiche" dans ta classe "vecteur".
Puisque tu commences le C++, je te conseille de prendre tout de suiste de bonnes habitudes concernant la lisibilité de ton code.
Voilà comment je l'aurais écrit :

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
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
 
using namespace std;
 
class vecteur
{
private :
	int m_size; 
	double * m_p;
 
public :
	vecteur();
	vecteur( int);
	vecteur( const vecteur & v);
	~vecteur();
 
	void affiche();
};
 
vecteur :: vecteur()
{
	m_size = 0;
	m_p = NULL;// Important car si ce n'est pas fait, ça plantera dans ton destructeur
}
 
vecteur :: vecteur( int taille)
{
	m_size = taille;
 
	if (m_size > 0)
	{
		m_p = new double[m_size];// Tu avais fait une erreur ici : taille=m_size
 
		for (int i = 0 ; i < m_size ; i++)
		{
			m_p[i] = 0;
		}
	}
	else
	{
		m_size = 0;
		m_p = NULL;
	}
}
 
 
vecteur :: vecteur(const  vecteur& v)
{
	m_size = v.m_size;
 
	if (m_size > 0)
	{
		m_p = new double[m_size];
 
		for (int i = 0 ; i < m_size ; i++)
		{
			m_p[i] = v[i];
		}
	}
	else
	{
		m_size = 0;
		m_p = NULL;
	}
}
 
vecteur :: ~vecteur()
{
	delete [] m_p;
}
 
void vecteur :: affiche()
{
	for (int i = 0 ; i < m_size ; i++)
	{
		cout << p[i] << ";";
	}
 
	cout << "\n";
}

Pour ton constructeur initialisant à 0, tu as aussi une autre méthode, je te conseille de voir la documentation de la fonction 'memset' (google est ton ami pour ça ^^).
Enfin, il est conseillé de mettre l'implémentation de la classe dans un .cpp plutôt que dans un .h (à part certains cas particuliers, tu verras plus tard quand on te parlera de 'template'). Ca permet de ne pas avoir à tout recompiler si tu modifies l'implémentation d'une fonction.