Discussion :

[it45200]

Bonsoir à tous,

Je voulais savoir par quel moyen peut-on en c++ copier une instance d'une classe en modifiant certains de ces attributs; je m'explique: j'ai une classe Chat et je veux via une méthode Chat reproduction() copier une instance c1 en modifiant certains de ses attributs. Imaginons que mon chat a un attribut string couleur initialisé à "noir" et un attribut int vie initialisé à 7 et qu'en faisant Chat c2 = c1.reproduction(); cela me crée un nouveau chat c2 ayant une couleur grise mais qu'en même temps c1 perde 1 point de vie. J'ai essayé de décrémenter l'attribut force directement via le constructeur ou le constructeur de recopie sans succès car cela m'initialise le chat initial (c1) avec une force déjà rétrogradée avant reproduction (c1.force = 7) ce qui n'est conforme à ce qui est attendu, c'est à dire c1.force = 7.

Code :

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

1
2
3
4
5
6
7
8
9
10
public:
	Chat (string nom = "Miaou", int force = 7, string couleur  = "noir")
	: nom(nom),  force(force), couleur(couleur)
	{
 
	}
 
	Chat (Chat const& c)
	: nom(c.nom), force((c.force)), couleur(c.couleur)
	{

Merci pour votre aide.

Bien cordialement,
IT

[MrPchoun]

Quelque chose dans ce style là ?

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
22
23
24
25
26
27
28
29
struct Chat
{
    std::string         m_nom;
    int                 m_force;
    std::string         m_couleur;
 
    Chat () {};
    Chat (std::string nom, int force, std::string couleur) : m_nom(nom), m_force(force), m_couleur(couleur) {};
 
    Chat reproduction()
    {
        m_force--;
        return Chat(m_nom, m_force+1, m_couleur);
    }
};
 
 
int main()
{
    Chat c1;
    c1.m_force = 7;
 
    Chat c2 = c1.reproduction();
 
    std::cout<<c1.m_force; // affiche 6
    std::cout<<c2.m_force; // affiche 7
 
    return 0;
}

[bacelar]

Plus simple :

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
struct Chat
{
    std::string         nom;
    int                 force;
    std::string         couleur;
 
    Chat (string nom = "Miaou", int force = 7, string couleur  = "noir") : nom(nom),  force(force), couleur(couleur) {};
 
    Chat (Chat const& c): nom(c.nom), force((c.force)), couleur(c.couleur)	{ }
 
    Chat reproduction()
    {
        return Chat(m_nom+"-"+m_nom, m_force, "gris");
        m_force--;
    }
};

[Bousk]

Code :

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

1
2
3
4
5
Chat reproduction()
    {
        return Chat(m_nom+"-"+m_nom, m_force, "gris");
        m_force--;
    }

Le premier compilo venu t'annoncera un warning "code unreachable".
Ton constructeur par copie est inutile, celui par défaut aura le même comportement. En plus, si besoin d'un constructeur par copie, on aura aussi généralement besoin d'un opérateur d'affectation, et souvent du destructeur.

[bacelar]

Oups

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
struct Chat
{
    std::string         nom;
    int                 force;
    std::string         couleur;
 
    Chat (string nom = "Miaou", int force = 7, string couleur  = "noir") : nom(nom),  force(force), couleur(couleur) {};
 
    Chat reproduction()
    {
        return Chat(nom+"-"+nom, force--, "gris");
    }
};

[ternel]

La réflexion à retenir, c'est que puisque tu ne veux pas faire une copie (au sens de duplicatat), il vaut mieux faire une fonction dédiée.

La méthode proposée par mes prédécesseurs reproduit le concept de clonable, ou un objet n'est pas copiable, mais dispose d'une fonction clone().

[it45200]

Quelque chose dans ce style là ?

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
22
23
24
25
26
27
28
29
struct Chat
{
    std::string         m_nom;
    int                 m_force;
    std::string         m_couleur;
 
    Chat () {};
    Chat (std::string nom, int force, std::string couleur) : m_nom(nom), m_force(force), m_couleur(couleur) {};
 
    Chat reproduction()
    {
        m_force--;
        return Chat(m_nom, m_force+1, m_couleur);
    }
};
 
 
int main()
{
    Chat c1;
    c1.m_force = 7;
 
    Chat c2 = c1.reproduction();
 
    std::cout<<c1.m_force; // affiche 6
    std::cout<<c2.m_force; // affiche 7
 
    return 0;
}

Merci je l'ai traduite en classe Chat et cela fonctionne impeccablement.

Cordialement,
IT

[Bousk]

Code :

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

1
2
m_force--;
        return Chat(m_nom, m_force+1, m_couleur);

peut s'écrire tout simplement return Chat(m_nom, m_force--, m_couleur);, en connaissant comment fonctionne le post-opérateur --/++

[jo_link_noir]

@Bousk: petite bévue: return Chat(m_nom, m_force--, m_couleur);