Copie instance d'une class avec changement d'attribut

Version imprimable

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:

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

Quelque chose dans ce style là ?

Code:

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

Plus simple :

Code:

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

12/03/2015, 17h40
Bousk
Code:

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". :calim2:
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.

Oups :aie:

Code:

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

13/03/2015, 08h52
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().

Citation:

Envoyé par MrPchoun

Quelque chose dans ce style là ?

Code:

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

27/03/2015, 15h52
Bousk
Code:

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 --/++
27/03/2015, 16h34
jo_link_noir

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