Discussion :

[r0d]

Bonjour à tous,

voilà, je me rend compte que j'ai de grosses lacunes concernant le mot clé const. Il peut être utilisé de différentes façons, et je m'embrouille un peu. Aussi, je me disais que peut-être vous pourrez éclairer ma lanterne.

Alors voilà, pour commencer, j'ai quelques questions. Pour appuyer mes exemples, j'utiliserai la classe suivante:

Code :

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

1
2
3
4
5
6
7
8
9
Class TestConst
{
public:
 TestConst():m_iConst (1),m_iNonConst(2){}
 
protected:
 const int m_iConst;
 int m_iNonConst;
};

Et voici mes questions:
1/ Dans cette classe, j'ai implémenté la méthode suivante:

Code :

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

void SetConstInt(const int i)	const	{m_iConst = i;}

Et j'ai une erreur de compilation:

error C2166: l-value specifies const object

Pourquoi?

2/ A quoi sert, concrètement, ce mot clé const? Quelles en sont ses différentes utilisations?

3/ Avez-vous des liens ou de la doc qui explique clairement et précisément l'utilisation de const? Car je n'ai rien trouvé de clair et précis dans tous mes cours de c++.

Merci

[Swoög]

Salut !

Je pense que ceci devrait t'éclairer : http://www.developpez.net/forums/sho...ighlight=const

J'ajouterais une précision, pour les méthodes membres spécifiées constantes (ne modifiant pas l'instance), la constance est automatiquement transférée au données membre de l'instance en cours pour détecter les infractions à la constance.

[Pragmateek]

Code :

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

const int m_iConst;

Code :

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

m_iConst = i;

Le compilateur n'accepte pas l'affectation d'une valeur à une constante hors initialisation:

Code :

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

const int n=0;

est bon mais pas:

Code :

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

1
2
const int n;
n=0;

De plus:

Code :

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

void SetConstInt(const int i)	const

Donc la fonction n'est pas sensée modifier l'instance de la classe.

Donc il faut initialiser la constante avec un constructeur qui ne peut pas être "const".

[r0d]

Après quelques tests et surtout grâce à vos conseils et vos liens, je crois que je suis maintenant au point. C'était nettement plus simple que ce que je pensais

Encore

[r0d]

Ha, une dernière question:
Imaginons que j'ai une classe qui a besoin de constantes. Comme par exemple, une classe qui effectue des calculs de mécanique et qui a besoin d'une constante g=9.81.

1/ Quelle est la meilleure solution parmi les suivantes:

Code :

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

1
2
//MaClasse.cpp
#define g 9.81 //define et variable globale

ou

Code :

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

1
2
3
4
5
//MaClasse.h
[...]
private:
  const float g = 9.81;//variable membre contante
[...]

Y-a-til mieux?

2/ Quels sont les "chartes de nommage" communément admises pour les variables const?

[Swoög]

personnellement, j'aurais plutôt tendance à faire une variable static const dans la classe... en C++, toujours éviter les instruction préproc au maximum...

pour les chartes de nommages je ne pourrais pas te renseigner par contre, désolé...

[Pragmateek]

Pour définir une variable static constante:

Code :

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

1
2
3
4
5
6
class A{
private:
      static const int g;
 
      };
const int A::g=9.81;

[r0d]

Argh, je viens de tomber sur un os:

Code :

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

1
2
3
4
5
6
7
8
9
10
typedef std::vector<CMonObjet*> MonObjetList;
class Toto
{
public:
[...]
    MonObjetList* GetMonObjetList() const {return &m_MonObjetList;}
 
protected:
    MonObjetList m_MonObjetList;
};

Ce code génère l'erreur de compilation suivante:

error C2440: 'return' : cannot convert from 'const class std::vector<class CMonObjet *,class std::allocator<class CMonObjet *> > *' to 'class std::vector<class CMonObjet *,class std::allocator<class CMonObjet *> > *'

Alors que:
1/ si j'enlève le const, ça marche
2/ le const_cast est implicite, donc normalement, bien que m_MonObjetList ne soit pas déclaré comme const, il doit être implicitement casté.
3/ je ne comprends pas cette erreur

Que se passe-t-il?

[Swoög]

du fait que ta méthode soit déclaré comme const, m_MonObjetList est un const MonObjetList donc &m_MonObjetList est un const MonObjetList*

or si la conversion A* => const A* est implicite, la conversion const A* => A* est interdite.

il faut déclarer ta fonction comme renvoyant un const MonObjetList

[r0d]

Haaaa d'accord.
Mais là, ça ne me conviens pas! (je sais, je suis c****t ). Ce que je voudrais, c'est de m'assurer que cette méthode ne modifie pas la classe (donc une méthode const) mais qui renvoie un objet que je puisse modifier de l'extérieur de cette classe. Dois-je utiliser le mot clé mutable?

Je ne sais pas pourquoi, j'ai l'impression qu'il y a quelque chose qui cloche dans mon histoire

[Eusebe]

Je pense qu'il ne faut pas que tu déclares cette fonction comme constante, parce que tu ne peux pas assurer que ta classe ne sera pas modifiée suite à son utilisation.

[Swoög]

la fonction en elle même ne fait que retourner un pointeur, elle ne pourra jamais modifier les données membres de la classe en elle-même...

Cependant, donner un pointeur sur une donnée membre peut être dangeureux pour plusieurs raisons (pointeurs fantômes, modifications "ex-nihilo" de l'instance etc...)

donc soit tu dis que ta fonction n'est pas const, soit tu reçoit un pointeur const

ensuite, si la sémantique le justifie VRAIMENT tu peux aussi utiliser un const_cast, mais la plupart du temps, ça implique une erreur de modélisation

[r0d]

Je pense, en effet, qu'il y a une erreur de modélisation dans mon appli. Rien qu'à voir le diagramme UML... il commence à ressembler à un plat de spaghettis, cela ne présage rien de bon.
Je vais revoir un peu ça, ça tombe bien, j'ai du temps en ce moment