Petit problème avec le constructeur par copie

**beegees** · 01/04/2008, 13h32

Bonjour tout le monde,

J'ai un programme sur lequel je suis bloqué depuis hier soir.

J'ai en fait deux erreurs dans la même fonction :

error C2661: 'CVoiture::CVoiture' : no overloaded function takes 2 parameters

et

error C2661: 'CVoiture::CVoiture' : no overloaded function takes 2 parameters

La méthode Ajouter prend un seul paramètre qui est un modele de voiture, modele veut dire un tout qui fait une voiture:

Code :

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

bool CTabVoitures::Ajouter(const CVoiture& Modele)

Je pense que cette fonction reçoit le model via le constructeur par copie que voici :

Code :

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

	CVoiture(const CVoiture& Source);

Je vous remercie d'avance pour votre so precious.

beegees

PS: voici le code des deux fonctions qui pour moi posent problème

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
#include "chaine.h"
 
 
class Vehicule3
{
public:
 
	Vehicule3();
	~Vehicule3();
 
 
 
	short		NombreDePlace;
	bool		Nom(const char* UnNom); //modificateur du nom
	bool		EstValide() const;
 
 
private:
	CChaine		m_Nom;
	CChaine		m_Immatriculation;
 
};
 
class CVoiture : public Vehicule3
{
public:
	CVoiture();
	CVoiture(const CVoiture& Source);
	~CVoiture();
 
	bool				EstValide() const;
 
 
private:
	double		m_VTM; //Volume transportable de marchandise
 
 
};
 
class CTabVoitures
{
public:
	CTabVoitures();
	~CTabVoitures();
	bool				Ajouter(const CVoiture& Modele);
	long				Indice(const char* Nom) const;
private:
	long				m_Nombre;
	CVoiture**			m_Tableau;
 
};
 
class CLeTout
{
public:
	CLeTout();
	~CLeTout();
 
	bool		Charger(const char* NomFichier);
 
private:
 
	private:
	static bool			FctnTraiterLigne(const CChargeurFichierTexte& CC, void* Contexte);
	bool				TraiterLigne(const CChargeurFichierTexte& CC);
 
 
private:
	CTabVoitures	m_TV; //Tableau de voitures
};

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
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
#include <BasicConsole.h>
#include "vehicule3.h"
#include "chaine.h"
 
FctnTraiterLigne(const CChargeurFichierTexte& CC, void* Contexte);
 
 
Vehicule3::Vehicule3()
:NombreDePlace(0)
{
}
 
Vehicule3::~Vehicule3()
{
}
 
CVoiture::CVoiture()
:m_VTM(0.0)
{
}
 
CVoiture::~CVoiture()
{
}
 
CTabVoitures::CTabVoitures()
:m_Nombre(0),m_Tableau(NULL)
{
}
 
CTabVoitures::~CTabVoitures()
{
}
 
CLeTout::CLeTout()
{
 
}
 
CLeTout::~CLeTout()
{
 
}
 
bool	CLeTout::Charger(const char* NomFichier)
{
 
	CChargeurFichierTexte	Chargeur;
 
 
	return Chargeur.Charger(NomFichier,'\t',CLeTout::FctnTraiterLigne,this);
}
 
bool CLeTout::FctnTraiterLigne(const CChargeurFichierTexte& CC, void* Contexte)
{
	return ((CLeTout*)Contexte)->TraiterLigne(CC);
}
 
	bool CLeTout::TraiterLigne(const CChargeurFichierTexte& CC)
	{
		if (strlen(CC.Champs(0)) == 1) switch (CC.Champs(0)[0])
		{
			case 'R':
				if (CC.NombreChamps() == 4) m_TV.Ajouter(CVoiture(CC.Champs(1),CC.Champs(2)),CC.Champs(3),CC.Champs(4));
				break;
		}
		return true;
	}
 
 
bool CTabVoitures::Ajouter(const CVoiture& Modele)
{
	if (!Modele.EstValide()) return false;
	//if (Indice(Modele.Nom()) >= 0) return true;
	CVoiture* AAjouter = new CVoiture(Modele);
	if (AAjouter == NULL) return false;
	void* Nouveau = realloc(m_Tableau,(m_Nombre+1)*sizeof(CVoiture*));
	if (Nouveau == NULL)
	{
		delete AAjouter;
		return false;
	}
	m_Tableau = (CVoiture**)Nouveau;
	m_Tableau[m_Nombre] = AAjouter;
	m_Nombre++;
	return true;
}
 
bool CVoiture::EstValide() const
{
	return (Vehicule3::EstValide()) && (m_VTM > 0);
}
 
bool Vehicule3::EstValide() const
{
	return !m_Nom.EstVide();
}

**coyotte507** · 01/04/2008, 13h38

Salut,

dans le cpp, je ne vois la définition de la fonction CVoiture(const CVoiture& Source)

Sinon à quelle ligne est ton erreur? Est-ce quand tu appelles une certaine fonction?
etc..

En tout cas c'est hallucinant la vitesse à laquelle tu avances

**beegees** · 01/04/2008, 13h53

Salut Coyotte,

Merci pour ta réponse.

En effet, j'ai oublié le principal, je m'en excuse

Les deux erreurs de produisent sur cette ligne :

Code :

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

if (CC.NombreChamps() == 4) m_TV.Ajouter(CVoiture(CC.Champs(1),CC.Champs(2)),CC.Champs(3),CC.Champs(4));

J'ai créé le constructeur par copie dans Vehicule3.cpp car je l'avais pas fait, bien vu

Par contre, je vais t'avouer que je ne comprends pas bien ce constructeur par copie.

Dois-je initialiser les membres de la classe (m_....) ?

Voici ce que j'ai fais :

Code :

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

1
2
3
4
CVoiture::CVoiture(const CVoiture& Source)
:Vehicule3(Source)
{
}

Voici une partie de l'énoncé (celui se rapportant à une voiture) :

- un véhicule peut être une voiture, un camion ou un bus
- un véhicule est défini par :
- une immatriculation
- un nombre de places (places assises pour des personnes)
- une voiture est caractérisée en plus par :
- un volume transportable de marchandises

et voici le fichier txt que je lis :

B ABBA54 50 25
C CPP40M 2 3000
V HUB345 5 200

Merci 100000000000 fois pour ton aide vraiment très précieuse.

beegees

**Mat007** · 01/04/2008, 14h01

Salut,

Le message d'erreur est pourtant clair (pour une fois, autant en profiter) puisque tu fais :

Code :

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

CVoiture(CC.Champs(1),CC.Champs(2))

Ce qui ne correspond à aucun constructeur puisque le seul qui existe est le constructeur par défaut qui ne prend pas d'argument.

Ton problème n'a rien à voir avec le constructeur par copie.
Tu peux regarder dans la FAQ, il y a toute une partie sur les constructeurs.

MAT.

**beegees** · 01/04/2008, 14h05

Envoyé par Mat007

Salut,

Le message d'erreur est pourtant clair (pour une fois, autant en profiter) puisque tu fais :

Code :

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

CVoiture(CC.Champs(1),CC.Champs(2))

Ce qui ne correspond à aucun constructeur puisque le seul qui existe est le constructeur par défaut qui ne prend pas d'argument.

Ton problème n'a rien à voir avec le constructeur par copie.
Tu peux regarder dans la FAQ, il y a toute une partie sur les constructeurs.

MAT.

Salut Mat,

Merci pour ta réponse.

J'ai ajouté un constructeur par copie que voici :

Code :

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

1
2
3
4
CVoiture::CVoiture(const CVoiture& Source)
:Vehicule3(Source)
{
}

mais ici il y'a les objets qui se trouve dans la classe Vehicule3 que voici :

Code :

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

1
2
3
CChaine		m_Nom;
	CChaine		m_Immatriculation;
	short		m_NombreDePlaces;

Enfin je pense...

Et ensuite je dois ajouter les objets qui sont propres à une voiture comme le volume transportable ?

Merci d'avance pour ta réponse.

beegees

**Médinoc** · 01/04/2008, 14h15

Tu ne peux pas régler directement les champs si tu ne fais pas un constructeur qui les prend en paramètre.
Typiquement, tu vas devoir ajouter un ou plusieurs constructeurs et Vehicule3 et CVoiture...

Petit problème avec le constructeur par copie

C++

Vue hybride

Discussions similaires

Partager

Partager