C++

données membres et retour par référence de ces dernières

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : données membres et retour par référence de ces dernières

Blakstaf

J'ai un petit peu de mal a comprendre quelle est la meilleure facon de coder les accesseurs aux attributs privés d'une classe et vous demande un peu d'éclaircissement à ce propos.

Illustrons mon problème avec le code suivant que j'ai l'occasion de voir régulièrement.

Code :

class CMaClasse
{
CMachin m_machin;
public:
CMachin& getMachin()
{
return m_machin;
}
};

Comment se fait-il qu'on renvoie l'objet membre de CMaClasse par référence alors que ça signifie qu'on va donner la possibilité à l'utilisateur de modifier l'attribut sans aucun controle ?

Publicité

xterminhate

Si vis pacem, para bellum.

j'ajouterais un const au début de l'accesseur justement.

---------------
Cordialement, Xterm-in'Hate...

Blakstaf

C'est ce qui me semble le plus sain mais dans ce cas pourquoi est ce que l'on trouve l'écriture précédente dans une grande majorité des cas ?

xterminhate

Si vis pacem, para bellum.

j'ajouterais egallement un const à la fin de l'accesseur ;-)

---------------
Cordialement, Xterm-in'Hate...

masklinn

í dag viðrar vel til loftárása

Soit parce que les gens ne savent pas/sont stupides
soit parce qu'ils veulent effectivement laisser l'attribut modifiable sans contrôle

---------------
Stick a parrot in a Call of Duty lobby, and you're gonna get a racist parrot. — Cody

Taz

bisounours-codeur

moi j'aime bien la façon CORBA

Code :

class Foo
{
T member;
public:
T Member() const { return this->member; }
void Member(const T &m) { this->member = m; }
};

la copie défensive est peut être de trop, si oui, mettre

Code :

const T& Member() const { return this->member; }

Lam's

Profil: bas.

Masklinn a écrit :

Soit parce que les gens ne savent pas/sont stupides
soit parce qu'ils veulent effectivement laisser l'attribut modifiable sans contrôle

Je suis d'accord avec toi. D'ailleurs dans le deuxième cas, la signature reste:

Machin & CMaClasse::getMachin () const;

Et ça, c'est pas automatique pour tout le monde de laisser en const tout ce qui doit l'être par défaut.

Taz

bisounours-codeur

manque un const là, c'est pas correct

Blakstaf

Je lis les choses suivantes sur le net :

Citation :

A public member function must never return a non-const reference or pointer to member data ([5], Rule 29).
Returning a non-const reference to member data violates the encapsulation of the class.

source : http://geosoft.no/development/cppp [...] troduction

Citation :

Accessor Styles

Accessor methods provide access to the attibutes of an object. Accessing an object's attributes directly, as is commonly done in C structures, is greatly discouraged in C++. It exposes implementation details of the object and degrades encapsulation.

There are two accepted ways to implement accessors, the preferable way is the following :
Attributes as objects

Example 4-3. Attributes as object accessor style

class X
{
public:
int age() const { return mAge; }
int& rAge() { return mAge; }

const String& name() const { return mName; }
String& rName() { return mName; }
private:
int mAge;
String mName;
}

source : http://www.uwyn.com/resources/uwyn [...] /x496.html

Ainsi, ok l'accesseur à name est bien retourné en const . Mais pour le mutateur, il renvoie name en reference simple, contredisant la phrase plus haut comme quoi il ne faut jamais renvoyer une reference non constante d'un membre d'une classe .

Il faut donc savoir ce que l'on veut. :??:

Message édité par Blakstaf le 12-12-2004 à 19:18:42

Lam's

Profil: bas.

Taz a écrit :

manque un const là, c'est pas correct

Oui, tiens, je me suis emmelé les pinceaux en cours de route. Je pensais à la double-signature (une avec const des deux côtés, et une sans rien), et du coup, j'ai oublié d'enlever la référence en retour.

Mais en relisant le commentaire de Masklinn, ça colle pas. Je dois être fatigué...

Publicité

xterminhate

Si vis pacem, para bellum.

Dans quel cas on prefere la copie defensive ?

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Il est pas un peu pourri ton code ?

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Si je réécris ton code dans un C++ un peu moins pourri (surtout la ligne9), mon compilateur me repond "error: static_cast from type `const std::string*' to type `std::string*' casts away constness / error: invalid conversion from `const std::basic_string<char, std::char_traits<char>, std::allocator<char> >*' to `std::string*'

Code :

#include<string>
#include<iostream>
using namespace std;
struct CTest
{
CTest( const string& s ) : m_strTest( s ) {}
const string& GetTest() const { return m_strTest; }
private :
string m_strTest;
};
int main()
{
CTest o("test" );
string* p(static_cast<string*>(&(o.GetTest())));
*p = "Test2";
cout << o.GetTest();
}

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Donc tout va bien, on peut faire un passage par référence de manière sécurisé. Mais ca ne repond pas à ma question.

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Non, le complateur ne l'accepte pas visiblement. Je ne pense pas que ton code soit représentatif.

---------------
Cordialement, Xterm-in'Hate...

xterminhate

Si vis pacem, para bellum.

Je crois que ce que tu as démontré, c'est la faiblesse du cast C.

Code :

{
const string s = "test";
string* p( (string*)&s );
*p = "test2";
cout << *p << s; // test2test2
}

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

je dis que ça me donne envie de vomir. Tant que les gens comprendront pas que plus on a de liberté, plus il faut établir des règles strictes, on aura toujours des merdes comme ça. Dans le cas d'une programmation saine, ceci n'arrive jamais. D'ailleurs, je fais gueuler mon compilo pour jeter les cast C

Taz

bisounours-codeur

aucun. À moins d'avoir un veritable impératif de sécurité (genre mot_de_passe, etc), si quelqu'un s'amuse à corrompre ta bibliothèque, c'est son problème si ça plante comme une merde.

Pour les types de bases, une copie suffit amplement cependant.

Taz

bisounours-codeur

ben si tu travailles en équipe, la moindre des choses c'est d'avoir une charte de codage et comme déjà dit, le compilateur peut aider.

Quand je parler de faire une bibliothèque, je voulais dire que si elle est correcte, tu livres, si tes utilisateurs font n'importe quoi, tu les envoies chier, c'est pas ton problème.

On va pas se faire chier pour des mecs qui ont un niveau C++ 0. Tu verra souvent sur le forum des gens faire des

Code :

((char*)s.c_str())[i] = 'x';

c'est leur problème. Y a un moment, faut être adulte : si tu paies un mec dont le raisonnement se résume à "le compilateur veut pas, je cast"... C'est encore pire si tu paies un mec qui fout des cast machinalement.

Faut prendre c'est responsabilité : si tu cast "pour que ça compile", c'est ton problème. Quelqu'un qui fait ça fera également d'autre saloperies. Bref tu peux le réassigner à la documentation.

Taz

bisounours-codeur

ni l'un, ni l'autre. Je vois juste quand dans les projets OpenSources auxquels je participe, le mec qui vient se plaindre pour des trucs comme ça, on lui rit à la gueule.

Et si je travaillais en entreprise, ça serait pareil.

je vois pas de différence entre

((char*)"bla" )[1] = 'x';

et

abort();

Taz

bisounours-codeur

ben ça change rien si t'intègres du C : tu le compiles avec une compilateur C et tu link avec .|

Niveau exception : d'accord. Mais sur le moyen-terme, quand le volume de code augmente ... les exception / GError, ça t'oblige à penser ta politique de gestion d'erreur __avant__

Publicité

FORUM HardWare.fr

Programmation

C++

données membres et retour par référence de ces dernières

Sujets relatifs
Théorie - Comment organiser une base de données	Undefined reference
Utilisation d'une base de données avec VB 6.0	[VBA Excel] [Résolu] Pb d'import de données txt dans excel avec VBA.
[PROB] PHP debutant - Retour a la ligne	acces multiple donnees fichier txt
[Oracle] Créer une base de données	spamming et bases de donnees (legislation)
Erreur "Type de données incompatible dans l'expression du ..."[RESOLU]	Ajouter playlist au site ss retour au debut a chaque chgt de page
Plus de sujets relatifs à : données membres et retour par référence de ces dernières

Page générée en 0.094 secondes