Aucun des constructeur n'envoie les bonnes informations à ma méthode

Version imprimable

Bonjour tout le monde,

Je dois charger le contenu d'un fichier txt mais j'ai déjà un problème :(

Mon problème se situe dans cette méthode :
Code:

1 2 3 4 5 bool CTabFauxFreres::TraiterLigne(const CChargeurFichierTexte& Chargeur) { if (Chargeur.NombreChamps() == 7) Ajouter(CFauxFrere(Chargeur.Champs(0),Chargeur.Champs(1),Chargeur.Champs(2),Chargeur.Champs(3),Chargeur.Champs(4),Chargeur.Champs(5),Chargeur.Champs(6))); return true; }
et le message d'erreur est le suivant :

Citation:

error C2665: 'CFauxFrere::CFauxFrere' : none of the 3 overloads can convert parameter 1 from type 'const char *'

La méthode TraiterLigne est appelée par cette méthode :
Code:

1 2 3 4 5 6 bool CTabFauxFreres::Charger(const char* NomFichier) { CChargeurFichierTexte Chargeur; return Chargeur.Charger(NomFichier,'\t',FctnTraiterLigne,this); }
et je pense qu'elle reçoit les informations du constructeur par copie (je n'en suis pas sûr) :
Code:

1 2 3 4 CFauxFrere::CFauxFrere(const CFauxFrere& Source) :m_Code(Source.m_Code),m_Nom(Source.m_Nom),m_Nature(Source.m_Nature),m_Feminin(Source.m_Feminin),m_Pluriel(Source.m_Pluriel),m_Descriptif(Source.m_Descriptif),m_Definition(Source.m_Definition) { }
Voici la déclaration de la fonction TraiterLigne dans le header file :

Code:

bool TraiterLigne(const CChargeurFichierTexte& Chargeur);

Edit : Je viens avec du nouveau :

Si le paramètre 1 est bien m_Code de type short, j'ai alors mis :

Ajouter(CFauxFrere((short)atol(Chargeur.Champs(0))

le message d'erreur est donc maintenant :

Citation:

error C2665: 'CFauxFrere::CFauxFrere' : none of the 3 overloads can convert parameter 3 from type 'const char *'

Je pense que le paramètre est de type booléen, comment puis-je faire la même chose pour un booléen ?

Quand il parle de paramètre 1, s'est bien le champ 0 ?

Voilà, je vous remercie d'avance pour votre aide.

beegees

02/05/2008, 22h14
coyotte507

Salut beegees,

Code:

Ajouter(CFauxFrere((short)atol(Chargeur.Champs(0))

Ne vaut-t-il pas mieux utiliser atoi pour un short?

Pour ton erreur, il faudrait voir le troisième paramètre que tu passe à ton constructeur CFauxFrere, tu ne nous montres que le premier (qui est valide :)).

Le message d'erreur signifie que normalement, dans le cas où tu te trouves, le troisième paramètre ne doit pas être de type const char*.

Peux - tu nous montrer toute la ligne, et les trois constructeurs de CFauxFrere?

Coyotte507
02/05/2008, 22h14
bolhrak

Hello,

Je pense que tu devrais regarder de ce côté :

http://cpp.developpez.com/faq/cpp/?p...NGS_convert_to

Je suppose bien entendu que tu as un constructeur CFauxFrere prenant 7 paramètres ? (Pas trop compris pourquoi tu mentionnes les constructeurs par copie, le pb vient juste du fait qu'il ne trouve pas de constructeur adapté pour CFauxFrere à partir de 7 const char*).

Dans le cas où le lien précédent ne répond pas à ta question, peux-tu poster la déclaration de tes constructeurs de CFauxFrere ?

Citation:

Salut beegees,

Salut Coyotte, merci pour ta réponse.

Citation:

Code:

Ajouter(CFauxFrere((short)atol(Chargeur.Champs(0))

Ne vaut-t-il pas mieux utiliser atoi pour un short?

OK j'ai changé, merci pour l'astuce.

Citation:

Pour ton erreur, il faudrait voir le troisième paramètre que tu passe à ton constructeur CFauxFrere, tu ne nous montres que le premier (qui est valide :)).

Voici tous les paramètres envoyés au constructeur :

Code:

1
2
3
4
5
6
7
8
 
	short			m_Code;       
	CChaine			m_Nom;       
	CChaine			m_Nature;    
	bool			m_Feminin;    
	bool			m_Pluriel;    
	CChaine			m_Descriptif;
	CChaine			m_Definition;

Je ne comprends pas bien comment mon compilo compte les paramètres.

Citation:

Le message d'erreur signifie que normalement, dans le cas où tu te trouves, le troisième paramètre ne doit pas être de type const char*.

Nous sommes d'accord sur ce point. si les paramètres commencent à zéro, alors on parle du bool m_Feminin.

J'avais le problème avec le premier élément et je l'ai changé en short.

Citation:

Peux - tu nous montrer toute la ligne, et les trois constructeurs de CFauxFrere?

bien sûr :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
CFauxFrere::CFauxFrere()
{
}
 
CFauxFrere::CFauxFrere(const CFauxFrere& Source)
:m_Code(Source.m_Code),m_Nom(Source.m_Nom),m_Nature(Source.m_Nature),m_Feminin(Source.m_Feminin),m_Pluriel(Source.m_Pluriel),m_Descriptif(Source.m_Descriptif),m_Definition(Source.m_Definition)
{
}
 
CFauxFrere::CFauxFrere(short CodeIdentification,const char* Nom, char Attribut,bool Feminin,bool Pluriel,const char* type,const char* Definition)
{
}

et voici le code de la fonction TratierLigne qui travaille avec le troisième constructeur :

Code:

1
2
3
4
5
bool CTabFauxFreres::TraiterLigne(const CChargeurFichierTexte& Chargeur)
{
	if (Chargeur.NombreChamps() == 7) Ajouter(CFauxFrere((short)atoi(Chargeur.Champs(0)),Chargeur.Champs(1),Chargeur.Champs(2),bool(Chargeur.Champs(3)),Chargeur.Champs(4),Chargeur.Champs(5),Chargeur.Champs(6)));
	return true;
}

Je te laisse mon fichier.cpp au cas où :

Code:

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
#include <BasicConsole.h>
#include "FFDD1.h"
 
////////////////////
//class CFauxFrere//
////////////////////
 
CFauxFrere::CFauxFrere()
{
}
 
CFauxFrere::CFauxFrere(const CFauxFrere& Source)
:m_Code(Source.m_Code),m_Nom(Source.m_Nom),m_Nature(Source.m_Nature),m_Feminin(Source.m_Feminin),m_Pluriel(Source.m_Pluriel),m_Descriptif(Source.m_Descriptif),m_Definition(Source.m_Definition)
{
}
 
CFauxFrere::CFauxFrere(short CodeIdentification,const char* Nom, char Attribut,bool Feminin,bool Pluriel,const char* type,const char* Definition)
{
}
 
CFauxFrere::~CFauxFrere()
{
}
 
CGenerique*	CFauxFrere::Cloner() const
{
	return new CFauxFrere(*this);
}
 
bool	CFauxFrere::EstValide() const 
{
	return true;
}
 
long	CFauxFrere::NombreClesDeTri() const
{
	return 1;
}
 
 
////////////////////////
//class TabCFauxFrere///
////////////////////////
 
CTabFauxFreres::CTabFauxFreres()
:CTableauGenerique(CFauxFrere())
{
}
 
CTabFauxFreres::CTabFauxFreres(const CTabFauxFreres& Source)
:CTableauGenerique(Source)
{
}
 
CTabFauxFreres::~CTabFauxFreres()
{
}
 
CGenerique* CTabFauxFreres::Cloner() const
{
	return new CTabFauxFreres(*this);
}
 
bool CTabFauxFreres::Ajouter(const CFauxFrere& Modele)
{
	return CTableauGenerique::Ajouter(Modele);
}
 
bool CTabFauxFreres::Charger(const char* NomFichier)
{
	CChargeurFichierTexte	Chargeur;
 
	return Chargeur.Charger(NomFichier,'\t',FctnTraiterLigne,this);
}
 
bool CTabFauxFreres::FctnTraiterLigne(const CChargeurFichierTexte& Chargeur, void* Contexte)
{
	return ((CTabFauxFreres*)Contexte)->TraiterLigne(Chargeur);
}
 
//ceci travaille avec le constructeur par copie de CFauxFrere
bool CTabFauxFreres::TraiterLigne(const CChargeurFichierTexte& Chargeur)
{
	if (Chargeur.NombreChamps() == 7) Ajouter(CFauxFrere((short)atoi(Chargeur.Champs(0)),Chargeur.Champs(1),Chargeur.Champs(2),bool(Chargeur.Champs(3)),Chargeur.Champs(4),Chargeur.Champs(5),Chargeur.Champs(6)));
	return true;
}

Merci pour tout et bon début de Week-End.

beegees

03/05/2008, 09h26
beegees

Citation:

Envoyé par bolhrak

Hello,

Je pense que tu devrais regarder de ce côté :

http://cpp.developpez.com/faq/cpp/?p...NGS_convert_to

Je suppose bien entendu que tu as un constructeur CFauxFrere prenant 7 paramètres ? (Pas trop compris pourquoi tu mentionnes les constructeurs par copie, le pb vient juste du fait qu'il ne trouve pas de constructeur adapté pour CFauxFrere à partir de 7 const char*).

Dans le cas où le lien précédent ne répond pas à ta question, peux-tu poster la déclaration de tes constructeurs de CFauxFrere ?

Hello bolhrak,

Merci pour ta réponse.

Malheureusement ton lien ne m'aide pas beaucoup car notre Prof utilise une console et n'utilise pas std:: ... je sais, s'est pas, bien !!!

J'ai laissé plus d'info dans le message précédent.

Merci encore et bon début de WE.

beegees
03/05/2008, 11h06
coyotte507
Salut,
Code:

1 2 3 CFauxFrere::CFauxFrere(short CodeIdentification,const char* Nom, char Attribut,bool Feminin,bool Pluriel,const char* type,const char* Definition) { }
Comme tu le vois, dans ce constructeur, ton troisième paramètre est de type char et non const char*.

Es-tu sûr que tu veux un char à cet endroit?

Si oui, alors lorsque tu appelles ta fonction, en troisième paramètre tu dois passer un char, et non un const char* (Tu passes Chargeur.Champs(2), qui est un const char*)

Citation:
Envoyé par coyotte507
Salut,
Code:

1 2 3 CFauxFrere::CFauxFrere(short CodeIdentification,const char* Nom, char Attribut,bool Feminin,bool Pluriel,const char* type,const char* Definition) { }
Comme tu le vois, dans ce constructeur, ton troisième paramètre est de type char et non const char*.

Es-tu sûr que tu veux un char à cet endroit?

Si oui, alors lorsque tu appelles ta fonction, en troisième paramètre tu dois passer un char, et non un const char* (Tu passes Chargeur.Champs(2), qui est un const char*)
Salut Coyotte,

Comment va ?

Un tout grand merci pour ton aide.

S'est donc un constructeur spécifique qu'il utilise ?

S'est donc ce constructeur ci :
Code:

1 2 3 CFauxFrere::CFauxFrere(short CodeIdentification,const char* Nom, const char* Attribut,bool Feminin,bool Pluriel,const char* type,const char* Definition) { }
et non celui-ci :
Code:

1 2 3 4 CFauxFrere::CFauxFrere(const CFauxFrere& Source) :m_Code(Source.m_Code),m_Nom(Source.m_Nom),m_Nature(Source.m_Nature),m_Feminin(Source.m_Feminin),m_Pluriel(Source.m_Pluriel),m_Descriptif(Source.m_Descriptif),m_Definition(Source.m_Definition) { }
ça me semble mieux maintenant, merci ;)

J'ai une nouvelle erreur, je l'a met dans ce message au cas où ça te dirais quelque chose :

Citation:

LIBCD.lib(wincrt0.obj) : error LNK2001: unresolved external symbol _WinMain@16
Debug/FauxFreresDD1.exe : fatal error LNK1120: 1 unresolved externals
Error executing link.exe.

Encore un tout grand merci pour ton aide.

beegees

03/05/2008, 14h15
coyotte507

Salut beegees,

ça va bien en ce moment avec les jours fériés :)

Je ne connais rien à visual C++, pour moi ton message veut dire qu'il ne trouve pas la fonction main, mais franchement j'en suis pas sûr.

Pour répondre à ton autre question, le constructeur utilisé dépend des paramètres passés. Le constructeur par copie est appelé dans certains cas spéciaux (enfin souvent quand même, sans que l'on s'en rende compte), ou lorsqu'on lui passe en argument un objet du même type en référence constante.

Dès lors qu'on lui passe un autre type d'argument, ou bien plusieurs arguments, c'est le constructeur qui correspond à ces arguments qui est appelé, jamais le constructeur par copie ;)
03/05/2008, 17h57
beegees

Citation:

Envoyé par coyotte507

Salut beegees,

ça va bien en ce moment avec les jours fériés :)

Je ne connais rien à visual C++, pour moi ton message veut dire qu'il ne trouve pas la fonction main, mais franchement j'en suis pas sûr.

Pour répondre à ton autre question, le constructeur utilisé dépend des paramètres passés. Le constructeur par copie est appelé dans certains cas spéciaux (enfin souvent quand même, sans que l'on s'en rende compte), ou lorsqu'on lui passe en argument un objet du même type en référence constante.

Dès lors qu'on lui passe un autre type d'argument, ou bien plusieurs arguments, c'est le constructeur qui correspond à ces arguments qui est appelé, jamais le constructeur par copie ;)

Salut Coyotte,

Problème résolu :yaisse2:

J'avais juste oublié d'inclure la console magique du Prof :mouarf:

Je suis content d'avoir compris s'était le constructeur spécifique et non celui par copie, j'aurai appris quelque chose d'important grâce à toi.

Merci pour tout et bon samedi.

beegees