Discussion :

[pacolito]

Bonjour,

j'ai eu un petit souci avec une méthode. Je l'ai résolu mais je ne comprends pas comment...

Pour expliquer, voici mon code: (j'ai enlevé les include et tout ça...)

Dans main.cpp:

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
 
void displayObj(const vector<myObject> &theObjects);
 
int main()
{
    ...
}
 
void displayObj(const vector<myObject> &theObjects)
{
    for(unsigned int i(0); i < theObjects.size(); i++)
    {
        theObjects[i].afficher();
    }
}

Dans myObject.cpp

Code :

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

1
2
3
4
5
 
void myObject::afficher()
{
    cout << "Nom : " << m_nom << ", Age : " << m_age << endl;
}

Dans cette configuration là mon code ne compile pas. L'erreur est la suivante:

Code :

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

1
2
 
error: passing 'const Arme' as 'this' argument of 'void myObject::afficher()' discards qualifiers

Et lorsque j'enlève le 'const' en paramètre de ma fonction displayObj, là je n'ai plus de problèmes! Mais je ne comprends pas pourquoi...

Si certains ont une explication, merci

[zul]

Il faut que tu déclare ta méthode myObject::afficher() const. Par défault, les méthodes sont non-const.

En l'occurence, tu dis à ton compilateur, j'ai une liste d'objet que je n'ai pas le droit de modifier, et je vais appliquer dessus la méthode afficher() qui va potentiellement modifier cet objet (le compilateur ne fait pas d'analyse poussée de ce que fait réellement ta fonction) donc il t'envoie ballader .

[3DArchi]

Entre deux choix possibles, l'incompréhension conduit au mauvais
dsl

const accolé à une variable indique au compilateur que cette variable ne va pas être modifiée dans le scope où elle se trouve : pour un paramètre cela veut dire que le paramètre ne va pas être modifié : tu ne peux lui affecter une nouvelle valeur ou tu ne peux appeler que des fonctions constantes sur cet objet. Gérer la constance est important, cf ici et suivant.

Donc ta modif aurait du être :

Code :

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

void myObject::afficher() const

et laisser :

Code :

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

void displayObj(const vector<myObject> &theObjects);

[pacolito]

Ok merci pour vos informations.

Je ne savais pas qu'il fallait aussi déclarer la fonction 'const' !!

Merci encore

[koala01]

Salut,

Ok merci pour vos informations.

Je ne savais pas qu'il fallait aussi déclarer la fonction 'const' !!

Merci encore

En fait cela rentre dans un contexte assez particulier que l'on appelle "const correctness" (la correction par rapport à la constance).

Comme l'a fait remarquer zul, les fonctions membres de classes ne sont, par défaut, pas constantes.

Cela signifie qu'elles ne s'engagent absolument pas à ne pas modifier l'objet en cours d'utilisation.

Le problème vient du fait que tu déclare explicitement theObjects comme... un objet constant.

Or, si tu modifie l'un des élément de theObjects, tu modifie fatalement theObjects, ce que le compilateur ne veut pas accepter, et pour cause: tu t'es engagé à ne pas modifier theObjects en le déclarant constant.

La règle est, en effet, relativement simple: on peut invoquer n'importe quelle fonction (qu'elle se soit engagée à ne pas modifier l'objet en cours ou non) au départ d'un objet non constant, par contre, on ne peut invoquer de des fonctions qui se seront engagées à ne pas modifier l'objet courent au départ... d'un objet qui ne peut pas être modifié.

Une "saine pratique" à adopter consiste donc:

1. à utiliser des objets constants chaque fois qu'ils ne faut pas qu'ils soient modifiés
2. à déclarer explicitement toute fonction qui ne modifiera pas l'objet comme étant constante

De cette manière, tu sera certain que les seuls endroits où ton objet est modifié sont... ceux où tu accepte que ce soit le cas