Discussion :

[3DArchi]

Salut,

Quelle pourrait être une solution? Un renvoie par référence ou par pointeur nefonctionne pas non plus, puisque l'objet qui est pointé (ou dont la référence est le nom) est détruit...

Gérer la copie dans la classe String et l'utiliser :

Code :

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

1
2
3
4
int main(){
String const str=f2();
 
cout<<static_cast<char const*>(str);

[deubelte]

Ok, il nous faut un constructeur par copie.
j'ai d'ailleurs lu à ce propos qu'il faut un constructeur par copie dès que dans le constructeur il y avait l'opérateur new et la présence d'allocation dynamique
Est-ce une bonne règle générale à suivre?

[bruno_pages]

Bonjour,

vous avez déjà un problème au niveau de String::setter(char* donnes) car *data=*donnes ne fait que modifier le premier caractère de data avec le premier caractère de donnes. A priori je pense que vous vouliez intégralement copier la chaine de caractères, non ?

de toute façon il ne fallait pas définir une opération setter mais l'opérateur= car actuellement toute affectation d'un String sera catastrophique car les deux String partageront ensuite le même data

pour la même raison il faut impérativement que vous définissiez le constructeur par copie.

[koala01]

Ok, il nous faut un constructeur par copie.
j'ai d'ailleurs lu à ce propos qu'il faut un constructeur par copie dès que dans le constructeur il y avait l'opérateur new et la présence d'allocation dynamique
Est-ce une bonne règle générale à suivre?

Cela va même encore plus loin...

Il existe ce que l'on appelle la forme canonique de Coplien:

Dans le cas où tu as une allocation dynamique effectuée dans le constructeur, tu dois prévoir un total de quatre fonctions particulières:

1. Le constructeur lui-même
2. Le constructeur par copie
3. L'opérateur d'affectation "="
4. Le destructeur

Mais il faut cependant prendre en compte la sémantique de ton objet.

Il faudra définir ces quatre fonctions si ton objet a sémantique de valeur (que tu veux pouvoir l'assigner, le comparer, le copier, ...), par contre, il faudra souvent "désactiver" l'opérateur d'affectation et le constructeur par copie (en les déclarant privés sans les définir, ou avec le nouveau spécificateur delete (C++11) ) pour les types ayant sémantique d'entité (les différents noeuds de structures dynamique, par exemple)

[Joel F]

Avec C++11, faut aussi rajouter les constructeurs et operateurs d'affection via rvalue.

J'ai aussi tendance à inclure swap (et move) dans le package de base de toute classe proprement implantée.

[bruno_pages]

swap (et move)

je pense comprendre swap (mais je ne partage la nécessite de toujours le définir, on ne fait pas que des tris )

move quésaco ?

[Joel F]

je pense comprendre swap (mais je ne partage la nécessite de toujours le définir, on ne fait pas que des tris )

swap ne sert pas que au tri, il sert aussi à implanter le copy-by-swap.

move quésaco ?

std::move effectue l'echange de ressource au travers de reference sur rvalue.

[koala01]

swap ne sert pas que au tri, il sert aussi à implanter le copy-by-swap.

S'il est vrai que le copy and swap est un idiome régulièrement utilisé, je ne suis pas vraiment sur que le comportement lui-même vaille la peine d'être généralisé en tant que fonction membre "indispensable" (enfin, aussi indispensable que les quatre autre).

Le copy and swap peut parfaitement être implémenté sous une forme proche de

Code :

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

1
2
3
4
5
6
7
8
 
Type & operator=(Type rhs) // copy inside
{
     std::swap(*this.membre1,rhs.membre1);
     std::swap(*this.membre2,rhs.membre2);
     /*...*/
     return *this;
}

[Joel F]

Que se passe-t-il si un de tes membres necessite un swap différent de celui fournit par std (genre pour faire du vol de ressources) ?

Comme on a pas droit à surcharger std::swap, le mieux reste quand meme de se donner un membre swap() et la fonction associée dans le namespace pour que en plus l'ADL la trouve tout seul.

[koala01]

J'ai utilisé std::swap juste pour l'exemple, parce que c'est la solution "la plus simple" à appliquer, par exemple, sur des pointeurs, pour éviter les fuites mémoire

Mais rien ne t'empêche d'utiliser quelque chose d'autre

... De là à généraliser une fonction membre swap, publique ou non, il reste de la marge, non (même si sa présence peut présenter des avantages dans certains cas à évaluer correctement )

[yan]

Comme on a pas droit à surcharger std::swap, le mieux reste quand meme de se donner un membre swap() et la fonction associée dans le namespace pour que en plus l'ADL la trouve tout seul.

std::swap ça se spécialise.( j'ai peut être pas compris ta phrase)

http://www.developpez.net/forums/d44...rer-algorithm/

[Joel F]

Normalement non. Ouvrir std, c'est jamais une idée génial:

http://www.boost.org/doc/libs/1_42_0...lity/swap.html

[yan]

Normalement non. tu n'as pas le droit (dixit le standard) d'ouvrir std pour surcharger des éléments de std.

std::swap est une Template. Tu peut donc la spécialiser non?
pourquoi se serait interdit?

[Joel F]

std::swap est une Template. Tu peut donc la spécialiser non?
pourquoi se serait interdit?

j'ai le vague souvenir d'un article de Sutter sur le fait que le standard interdit d'ouvrir std pour y coller des spécialisations.

[metagoto]

j'ai le vague souvenir d'un article de Sutter sur le fait que le standard interdit d'ouvrir std pour y coller des spécialisations.

Il me semble que ce sont les overloads qui sont prohibés, pas les spécialisations.

[yan]

http://www.gotw.ca/gotw/068.htm

ça me rassure

[Joel F]

Il me semble que ce sont les overloads qui sont prohibés, pas les spécialisations.

OK

[Jean-Marc.Bourguet]

Il me semble que ce sont les overloads qui sont prohibés, pas les spécialisations.

Attention, pour une fonction comme swap, il n'y a pas moyen de fournir des spécialisations partielles, juste des surcharges qui sont interdites -- peut-être le point dont Joel se souvenait.

[metagoto]

yan a le bon lien je pense.

http://www.gotw.ca/gotw/068.htm

ça me rassure

it's usually a good idea to specialize std::swap() (or provide your own overloaded swap() in another namespace) that's specific to your new type

H. Sutter

[yan]

Attention, pour une fonction comme swap, il n'y a pas moyen de fournir des spécialisations partielles, juste des surcharges qui sont interdites -- peut-être le point dont Joel se souvenait.

que veut tu dire?