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