Discussion :

[agv13]

Bonjour,

Je suis sur un projet ou j'ai actuellement besoin de faire une class "A"
Et je doit déclarer la class "B" de la façon suivante :

MaClassA A(nom_de_ma_class)
MaClassB B = A

Petit problème :
Pas de compilation, une erreur :
conversion from ‘A’ to non-scalar type ‘B’ requested

Merci,
En attente de vos réponses .

[Vodsky]

Si tu déclare ta classe comme cela:

Code :

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

1
2
MaClassA a(...);
MaClassB b = a;

Ce n'est pas l’opérateur '=' mais le constructeur par copie qui est appeler il me semble.

[agv13]

Si tu déclare ta classe comme cela:

Code :

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

1
2
MaClassA a(...);
MaClassB b = a;

Ce n'est pas l’opérateur '=' mais le constructeur par copie qui est appeler il me semble.

Je sais c'est une construction par copie que je fait, j'ai ma fonction :

B& B::operator=(const A& other)
{
if(&other != this)
{
this->_name = other._name;
}
return *this;
}

[Vodsky]

Si tu tien réellement a déclarer ta classe comme ça

Code :

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

1
2
MaClassA a(...);
MaClassB b = a;

Il va falloir que tes classes MaClassA et MaClassB aient une relation d’héritage sinon sa sera pas possible (à moins de coller un reinterpret_cast<> bien moche et encore c'est pas sure que sa marche...).

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
 
class B
{
    //...
};
 
class A : public B
{
    //...
};
 
int main()
{
    A a;
    B b = a;
 
    //...
    return 0;
}

[Arzar]

Je sais c'est une construction par copie que je fait, j'ai ma fonction :

Code :

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

1
2
3
4
5
6
7
8
B& operator=(const A& other)
{
if(&other != this)
{
this->_name = other._name;
}
return *this;
}

Non, ce n'est pas un constructeur que tu as fait là, mais un opérateur d'affectation.
Un constructeur ce serait ça :

Code :

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

1
2
3
4
 B(const A& other)
{
 //...
}

Le constructeur est appelé lors de la construction de l'objet, c'est à dire pour créer un objet qui n'existe pas encore, alors que l'opérateur = d'affectation est appelé lors de l'affectation d'une nouvelle valeur à un objet existant.
Donc :

Code :

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

1
2
3
4
 
A a; //constructeur par défaut de A
B b = a; // appel du constructeur B(const A& other)
b = a; // appel de l'opérateur d'affectation B& operator=(const A& other)

Edit : Et pas besoin de relation d'héritage ou de reinterpret_cast zarb, Vodsky

[Vodsky]

Moi et ma folie des grandeurs

Autant pour moi, j'avais oublié.

[gbdivers]

Bonjour,

Je suis sur un projet ou j'ai actuellement besoin de faire une class "A"
Et je doit déclarer la class "B" de la façon suivante :

MaClassA A(nom_de_ma_class)
MaClassB B = A

Petit problème :
Pas de compilation, une erreur :
conversion from ‘A’ to non-scalar type ‘B’ requested

Merci,
En attente de vos réponses .

Bonjour

D'un point de vue stricte, le constructeur B::B(A const&) n'est pas un constructeur par copie et l'opérateur B& B::operator=(A const&) n'est pas l'opérateur d'affectation. Ces noms (à ma connaissance) sont réservés dans le cas d'une classe sur elle même.

Le constructeur B::B(A const&) sans mot clé explicit est appelé constructeur de conversion. Il n'est pas nécessaire de créer aussi l'opérateur B& B::operator=(A const&) puisque sans le mot clé explicit, le compilateur convertira automatiquement B b = a; en B b(a); (donc en appelant le constructeur de conversion).

Pour rappel, il est d'usage de mettre le mot clé explicit devant tous les constructeurs prenant 1 paramètre sauf volonté d'avoir une conversion implicite. Mais il faut pour cela que ça un sens au niveau sémantique (par exemple pour convertir un entier en réel ou un réel en complexe ; utiliser la conversion implicite pour convertir un type dans un autre type juste parce que ce second type utilise des paramètres du premier type n'est pas très clean)

Donc en conclusion, tu n'as besoin que de déclarer le constructeur de conversion.

[3DArchi]

Salut,
Ce qui donne concrètement :

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
#include <iostream>
#include <string>
 
class A
{
public:
   explicit A(std::string const&){}
};
 
class B
{
public:
   B(A const&){}
};
 
int main()
{
   A a("tutu");
   B b = a;
   return 0;
}

En lisant le post de base, j'ai eu un petit doute :

MaClassA A(nom_de_ma_class)

Ce n'est pas ça A a(A); ? Car ça ne définit pas un A mais une fonction qui prend un A et en retourne un (cf F.A.Q.).