Ma méthode ajouter ne fonctionne pas

Bonjour tout le monde,

J'ai un problème avec une méthode que voici :

Code:

---

1 2 3 4 5 6 7 8 9 10

bool CTraitementProduits::TraiterLigne_Prix(const CChargeurFichierTexte& Chargeur) { long n = 0; if(Chargeur.NombreChamps() == 5)   for (n =0; n<Chargeur.NombreChamps();n++) { if(!m_PrixDeVente.Ajouter(CPrixDeVente(atoi(Chargeur.Champs(n))))) return false; } return true;

---

J'obtiens ce message d'erreur :

Citation:

---

error C2440: 'type cast' : cannot convert from 'int' to 'class CPrixDeVente'

---

Il travaille avec ce constructeur :

Code:

---

1 2 3 4 5 6

CPrixDeVente::CPrixDeVente(int UnPrix, int UneQuantite) : m_PV(0),m_Quantite(0) { setPV(UnPrix); setQuantite(UneQuantite); }

---

Bon je vois bien que mon constructeur à deux paramètres et que je n'en donne qu'un mais le message d'erreur s'applique au premier objet que j'ai passé.

En vous remerciant d'avance pour votre aide.

beegees

hello,

vu ton message d'erreur et les infos que tu donne,
je dirais que tu n'as qu'un seul constructeur possédant un seul paramètre(celui de recopie donc)

le compilo cherche d'abord s'il y a un constructeur de type:

Code:

---

CPrixDeVente( int val)

---

comme il ne trouve pas, il cherche a convertir ton int en CPrixDeVente...

a mon avis, si tu tente:

Code:

---

1 2 3 4

int bidon = 0; ... if(!m_PrixDeVente.Ajouter(CPrixDeVente(bidon,atoi(Chargeur.Champs(n))))) return false;

---

ca compilera

Citation:

---

Envoyé par loicounet

hello,

vu ton message d'erreur et les infos que tu donne,
je dirais que tu n'as qu'un seul constructeur possédant un seul paramètre(celui de recopie donc)

le compilo cherche d'abord s'il y a un constructeur de type:

Code:

---

CPrixDeVente( int val)

---

comme il ne trouve pas, il cherche a convertir ton int en CPrixDeVente...

a mon avis, si tu tente:

Code:

---

1 2 3 4

int bidon = 0; ... if(!m_PrixDeVente.Ajouter(CPrixDeVente(bidon,atoi(Chargeur.Champs(n))))) return false;

---

ca compilera

---

Hello Loic,

Un tout grand meric pour ton aide très efficace.

ça fonctionne mais je t'avoue que je ne comprends pas bien pourquoi.

S'est parce que mon constructeur prend un entier et un autre entier ?

Encore un tout grand et sincère merci ;)

beegees

Dans ton cas le mieux serait peut être de définir un constructeur prenant un entier en paramètre et un second optionnel:

Code:

---

1 2 3 4 5 6 7

CPrixDeVente::CPrixDeVente(int UneQuantite,int UnPrix=0) //ou l'inverse : m_PV(0),m_Quantite(0) { setPV(UnPrix); setQuantite(UneQuantite); }

---

D'ailleurs après relecture ton code gagnerait en simplicité et en efficacité en te contentant d'écrire:

Code:

---

1 2 3 4

CPrixDeVente::CPrixDeVente(int UneQuantite,int UnPrix=0) //ou l'inverse : m_PV(UneQuantite),m_Quantite(UnPrix) {}

---

Sauf si tes setters font d'autres traitements que de positionner les valeurs en questions.

re,

lorsque tu crée un objet, il faut lui donner 0,1,2,etc arguments pour qu'il puisse savoir quel constructeur utiliser.

ce qui est important c'est le nombre et le type des arguments passés en paramètres:

exemple:

Code:

---

1 2 3 4 5 6 7 8

Mon_objet(); //appelle le constructeur par defaut car 0 arguments Mon_objet(int a); Mon_objet(char b); //ce sont 2 constructeurs differents meme s'il n'y a qu'un seul argument Mon_objet( Mon_objet const & obj); //constructeur de clonage Mon_objet(int a, int b) Mon_objet(char a, int b) Mon_objet(Mon object const & val, int a)

---

tous ces constructeurs sont différents...tu vois le truc?

Citation:

---

Envoyé par loicounet

re,

lorsque tu crée un objet, il faut lui donner 0,1,2,etc arguments pour qu'il puisse savoir quel constructeur utiliser.

ce qui est important c'est le nombre et le type des arguments passés en paramètres:

exemple:

Code:

---

1 2 3 4 5 6 7 8

Mon_objet(); //appelle le constructeur par defaut car 0 arguments Mon_objet(int a); Mon_objet(char b); //ce sont 2 constructeurs differents meme s'il n'y a qu'un seul argument Mon_objet( Mon_objet const & obj); //constructeur de clonage Mon_objet(int a, int b) Mon_objet(char a, int b) Mon_objet(Mon object const & val, int a)

---

tous ces constructeurs sont différents...tu vois le truc?

---

Citation:

---

Envoyé par befalimpertinent

D'ailleurs après relecture ton code gagnerait en simplicité et en efficacité en te contentant d'écrire:

Code:

---

1 2 3 4

CPrixDeVente::CPrixDeVente(int UneQuantite,int UnPrix=0) //ou l'inverse : m_PV(UneQuantite),m_Quantite(UnPrix) {}

---

Sauf si tes setters font d'autres traitements que de positionner les valeurs en questions.

---

Citation:

---

Envoyé par befalimpertinent

Dans ton cas le mieux serait peut être de définir un constructeur prenant un entier en paramètre et un second optionnel:

Code:

---

1 2 3 4 5 6 7

CPrixDeVente::CPrixDeVente(int UneQuantite,int UnPrix=0) //ou l'inverse : m_PV(0),m_Quantite(0) { setPV(UnPrix); setQuantite(UneQuantite); }

---

---

Re,

Un tout grand merci à vous deux, vous êtres vraiment sympa et très efficace (en plus vos explications sont simples).

Franchement, qu'est-ce que je ferais sans ce forum :lahola:

beegees