Discussion :

[beegees]

Bonjour tout le monde,

Je dois charger en mémoire deux fichiers txt.

Pour cela, j'ai besoin de deux constructeurs par copie qui ne sont différent que par leur liste d'initialisation.

ça ne fonctionne pas, j'ai ce message d'erreur :

function '__thiscall CFauxFrere::CFauxFrere(const class CFauxFrere &)' already has a body

Avec ce code :

Code :

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

1
2
3
4
5
6
7
8
9
CFauxFrere::CFauxFrere(const CFauxFrere& Source) //constructeur par copie
:m_Ligne(Source.m_Ligne)
{
}
 
CFauxFrere::CFauxFrere(const CFauxFrere& Source) //constructeur par copie
:m_Code(Source.m_Code),m_Nom(Source.m_Nom),m_Type(Source.m_Type),m_Feminin(Source.m_Feminin),m_Pluriel(Source.m_Pluriel),m_ComplementInformation(Source.m_ComplementInformation),m_Definition(Source.m_Definition);
{
}

Merci d'avance pour votre aide.

beegees

[loicounet]

Ben là, typiquement je dirais que c'est normal que le compilo râle!

dans ton .HPP, tu dois avoir un seul constructeur par recopie:

Code :

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

CFauxFrere(const CFauxFrere& Source);

Tu ne peux donc pas lui donner plusieurs définitions dans le .CPP

A la rigueur tu peux bidouiller en rajoutant un parametre bidon afin que le compilo prenne le bon constructeur:

Code :

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

1
2
3
4
5
6
7
8
9
CFauxFrere::CFauxFrere(const CFauxFrere& Source, int bidon) //constructeur par copie
:m_Ligne(Source.m_Ligne)
{
}
 
CFauxFrere::CFauxFrere(const CFauxFrere& Source) //constructeur par copie
:m_Code(Source.m_Code),m_Nom(Source.m_Nom),m_Type(Source.m_Type),m_Feminin(Source.m_Feminin),m_Pluriel(Source.m_Pluriel),m_ComplementInformation(Source.m_ComplementInformation),m_Definition(Source.m_Definition);
{
}

est ce que ca répond a ta question?

[beegees]

Ben là, typiquement je dirais que c'est normal que le compilo râle!

dans ton .HPP, tu dois avoir un seul constructeur par recopie:

Code :

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

CFauxFrere(const CFauxFrere& Source);

Tu ne peux donc pas lui donner plusieurs définitions dans le .CPP

A la rigueur tu peux bidouiller en rajoutant un parametre bidon afin que le compilo prenne le bon constructeur:

Code :

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

1
2
3
4
5
6
7
8
9
CFauxFrere::CFauxFrere(const CFauxFrere& Source, int bidon) //constructeur par copie
:m_Ligne(Source.m_Ligne)
{
}
 
CFauxFrere::CFauxFrere(const CFauxFrere& Source) //constructeur par copie
:m_Code(Source.m_Code),m_Nom(Source.m_Nom),m_Type(Source.m_Type),m_Feminin(Source.m_Feminin),m_Pluriel(Source.m_Pluriel),m_ComplementInformation(Source.m_ComplementInformation),m_Definition(Source.m_Definition);
{
}

est ce que ca répond a ta question?

Salut,

Un tout grand merci pour ton aide.

J'ai fais ce que tu as dis :

Dans le point h :

Code :

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

1
2
3
CFauxFrere(const CFauxFrere& Source,int bidon); //constructeur par copie
 
CFauxFrere(const CFauxFrere& Source); //constructeur par copie

Code :

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

1
2
3
4
5
6
7
8
9
CFauxFrere::CFauxFrere(const CFauxFrere& Source, int bidon) //constructeur par copie
:m_Ligne(Source.m_Ligne)
{
}
 
CFauxFrere::CFauxFrere(const CFauxFrere& Source) //constructeur par copie
:m_Code(Source.m_Code),m_Nom(Source.m_Nom),m_Type(Source.m_Type),m_Feminin(Source.m_Feminin),m_Pluriel(Source.m_Pluriel),m_ComplementInformation(Source.m_ComplementInformation),m_Definition(Source.m_Definition);
{
}

J'ai ce message d'erreur sur le deuxième constructeur :

syntax error : '{'

et

';' found in what should be a comma-separated list

Qu'en penses-tu ?

Encore merci
beegees

[aurelman]

Salut,

Tu ne peux pas avoir deux constructeurs avec la même signature. (Sinon comment savoir lequel appeler ?)

[beegees]

Salut,

Tu ne peux pas avoir deux constructeurs avec la même signature. (Sinon comment savoir lequel appeler ?)

Salut,

Tu as raison, comme ça s'est vrai que s'est plus claire.

Merci

beegees

[loicounet]

regarde au bout de ta liste d'initialisation:

Code :

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

m_Definition(Source.m_Definition);

t'as mis un ";" au bout ;-)
enlève le déjà, je pense que le

Citation:
';' found in what should be a comma-separated list

vient de là!

[beegees]

regarde au bout de ta liste d'initialisation:

Code :

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

m_Definition(Source.m_Definition);

t'as mis un ";" au bout ;-)
enlève le déjà, je pense que le

vient de là!

Salut,

Merci pour ta réponse.

En effet, il ne fallait pas mettre de ; à la fin, je ne suis pas encore très bien habitué avec les constructeurs.

Maintenant, mon texte ne s'affiche plus.

Tantôt, avant que j'ajoute bidon, il s'affichait.

Merci encore

beegees

[Alp]

Tu peux aussi écrire un seul constructeur avec un booléen en argument supplémentaire.
Si il vaut 'true', tu initialises de telle manière, sinon de l'autre manière.

[loicounet]

ca marche now donc?
si oui marque ton sujet en "résolu"