Discussion :

[Astraya]

Bonjour à tous,
J'explique mon problème, lors de la compilation d'une classe nommé swap j'ai une erreur à mon constructeur de copie. Il ne s'agit que d'un code "d'apprentissage".

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
namespace myclass
{
    class swap
    {
    private:
        int i,j;
    public:
        swap();
        swap(int,int);
        swap(const swap &);
        ~swap();
        const int geti();
        const int getj();
        void seti(int);
        void setj(int);
        void echange(int ,int);
    };
 
    swap::swap():i(0),j(0)
    {
    }
    swap::swap(int _i,int _j)
    {
        i = _i;
        j = _j;
    }
    swap::swap(const swap &_swap)
    {
        i = _swap.geti();
        j = _swap.getj();
    }
    swap::~swap()
    {
    }
    /*Accesseur*/
    const int swap::geti()
    {
        return i;
    }
 
    const int swap::getj()
    {
        return j;
    }
 
    /*Affectation*/
    void swap::seti(int _i)
    {
        i = _i;
    }
 
    void swap::setj(int _j)
    {
        j = _j;
    }
 
    /*Echange*/
    void swap::echange(int _i, int _j)
    {
        int tmp;
        _i = tmp;
        _i = _j;
        _j = tmp;
    }
}

j'ai 2 erreurs pour cette fonction:

Code :

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

1
2
3
4
5
swap::swap(const swap &_swap)
    {
        i = _swap.geti();
        j = _swap.getj();
    }

c:\documents and settings\astraya\mes documents\visual studio 2008\projects\entrenementcpp\entrenementcpp\swap.h(29) : error C2662: 'myclass::swap::geti' : impossible de convertir un pointeur 'this' de 'const myclass::swap' en 'myclass::swap &'
1> La conversion perd les qualificateurs
1>c:\documents and settings\astraya\mes documents\visual studio 2008\projects\entrenementcpp\entrenementcpp\swap.h(30) : error C2662: 'myclass::swap::getj' : impossible de convertir un pointeur 'this' de 'const myclass::swap' en 'myclass::swap &'
1> La conversion perd les qualificateurs

je ne comprend pas trop quel est le problème. Est-ce à cause du namespace? Une solution? merci

[r0d]

Bonjour,

l'espace de nommage n'y est pour rien. Le problème c'est que dans ton constructeur par copie, l'objet de type swap est passé par référence constante (ce qui est parfaitement normal). Cela signifie que l'objet passé en paramètre (dans ton exemple c'est _swap ) ne peut être modifier. Le compilateur doit le savoir à l'avance, et donc, il t'oblige à n'appeler que des fonctions membre constantes de cet objet. Il faut donc que tes accesseurs geti() et getj() soient constants (ce qui est logique pour des accesseurs):

Code :

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

1
2
3
4
5
6
7
class swap
{
// ...
        const int geti() const;
        const int getj() const;
// ...
};

edit: a noter que dans ton cas, tu n'as même pas besoin de passer par les accesseurs, puisque dans le constructeur par copie, tu as accès aux variables privées de la classe, donc de toutes les instances de cette classe.

[Astraya]

a ok d'accord merci beaucoup de l'explication

[koala01]

Salut,

Ceci dit, un objet de type swap a intégralement accès au contenu d'un objet de type... swap (Elle n'est pas belle, la lapalissade )

Cela implique que tu peux utiliser directement les valeurs de i et de j dans ton exemple sans passer par l'accesseur adéquat, sous la forme de

Code :

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

1
2
3
4
5
6
/* juste l'implémentation du constructeur par copie */
swap::swap(swap const & rhs):i(rhs.i),j(rhs.j) // autant toujours utiliser les
                                               // listes d'initialisation ;)
{
    /* rien d'autre à faire ;) */
}

Et, juste en passant, nommer un espace de noms... myclass est pour le moins surprenant

Tu devrais veiller à donner des noms "auto commentés" à tes différents identifiants, parce que, en lisant myclass, on risque de s'attendre à avoir affaire à... une classe, avec tout ce que cela implique, et d'être fort surpris de se rendre compte qu'il s'agit d'un espace de noms

[Astraya]

oui oui je comprend bien pour myclass, c'était juste histoire de donné un nom j'aurai pu trouvé mieux je l'accorde, mais j'ai mit ce qui me passai dans la tête

merci beaucoup pour la petite passade