Discussion :

[beegees]

Bonjour tout le monde,

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

Code :

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

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 :

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

Il travaille avec ce constructeur :

Code :

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

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

[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 :

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

CPrixDeVente( int val)

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

a mon avis, si tu tente:

Code :

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

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

ca compilera

[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 :

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

CPrixDeVente( int val)

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

a mon avis, si tu tente:

Code :

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

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

[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 :

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

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);
}

[befalimpertinent]

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

Code :

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

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.

[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 :

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

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?

[beegees]

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 :

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

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?

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

Code :

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

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.

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

Code :

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

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

beegees