comment dupliquer une liste de pointers ? (les objets pointés aussi).

comment dupliquer une liste de pointers ? (les objets pointés aussi). - C++ - Programmation

Marsh Posté le 10-01-2003 à 22:14:26    

comment dupliquer une liste de pointers ? (les objets pointés aussi).
 
pour stocker des elements dans une liste chainée generique avec templates (g fait l'implementation) afin de pouvoir stocker des classes et aussi des classes dérivées du type de base, j'utilise une liste chainée dont les noeuds contienent un champ avec la valeur du pointeur.
 
donc pour rentrer un element je dois faire l.add(new machin(bidule,truc))
Là je peux conserver le type reels de objets contrairement au cas ou j'utilise un attribut du type machin dans mon noeud.
Et donc je vois pas comment faire pour dupliquer la liste. :heink:
J'ai tenté des trucs avec typeof etc, mais le compilo (g++) subit une erreur interne, et il veut que j'envoit un mail [:ddr555]
 
Mon but etait de faire une liste generique qu'il soit possible de dupliquer.


Message édité par karim63 le 15-01-2003 à 20:35:27
Reply

Marsh Posté le 10-01-2003 à 22:14:26   

Reply

Marsh Posté le 10-01-2003 à 22:20:37    

muni tes objets d'une méthode clone()

Code :
  1. Foo* Foo::clone() const
  2. {
  3.   return new Foo(*this);
  4. }

Reply

Marsh Posté le 10-01-2003 à 22:22:20    

Code :
  1. class A {
  2.   virtual A* Clone() const {return new A(*this);};
  3. }
  5. class B : public A {
  6.   virtual A* Clone() const {return new B(*this);}
  7. }


 
C'est ca que tu veux ?
 
LeGreg
edit: grille comme d'hab


Message édité par LeGreg le 10-01-2003 à 22:23:12

---------------
voxel terrain render engine | animation mentor
Reply

Marsh Posté le 10-01-2003 à 22:37:20    

mmm effectivement c ça qu'il me faut. J'avais vu que y avait ça en java mais j'ai pas percuté merci  :) .
Sinon je mettrais bien le code histoire de ce marrer [:ddr555]

Reply

Marsh Posté le 10-01-2003 à 22:43:13    

sinon seul prob, j'aurais voulu pouvoir l'utiliser sur des types standart.
l.add(new int (i) );
 
comment rajouter la methode clone sur ce type ?
Je vois en specialisant le patron, mais y pas d'ot moyens ?

Reply

Marsh Posté le 10-01-2003 à 22:57:42    

non. tiens une interrogation. est ce que c'est possible de spécialiser un template pour les types de bases, spécialisation de tempalte qui ferait appel à une unique classe template servant de wrapper+clone aux types de bases

Reply

Marsh Posté le 10-01-2003 à 23:14:34    

Code :
  1. template<class T>
  2. class Container
  3. {
  4. public:
  6.   static void check_preconditions()
  7.   {
  8.     static T* ptr=NULL;
  10.     if(ptr==NULL)
  11.       {
  12. const T tmp;
  13. ptr=tmp.clone();
  14. delete ptr;
  15.       }
  16.   }
  17.    
  19. public:
  21.   Container()
  22.   {
  23.     check_preconditions();
  24.   }
  25. };
  28. template<class T>
  29. class BaseTypeWrapper
  30. {
  31.   T* item;
  32. public:
  33.  
  34.   T* clone() const
  35.   {
  36.     return new T(*item);
  37.   }
  38. };
  41. template<class BaseType> class Container< BaseTypeWrapper<BaseType> >
  42. {
  43. };
  46. template<> class Container<int> : public Container< BaseTypeWrapper<int> >
  47. {
  48. };
  49.  
  51. // etc et à compléter evidemment


 
ça me parait devoir marcher...
 
le type template doit etre muni d'une méthode T* clone() const. et pour obtenir ce const, on doit instancier un objet const: on fait donc appel a son constructeur par défaut. Donc T doit aussi posséder un constructeur par défaut.
 
edit: il y a sans doute pas mal de modification à apporter, mais c'est une piste interessante. mais en y réflechissant, disposer d'un conteneur pour manipuler des objets polymorphes qui s'adaptent aussi aux types de bases n'a pas vraiment de sens. Il faut chercher dans la spécialisation de template, certes, mais je pense qu'il faut surtout designer un conteneur dédié à ce genre de travail. Mais l'exercice de style est interessant et a retenir. Ici la synthaxe est sans doute mauvaise, les noms des paramètres templates mal choisis, des pointeurs là alors qu'il n'en faudrait pas, bref c'est la jungle. Me jetez pas la pierre, j'ai essayez de répondre à la question: on peut pas faire du tout en un. Le problème m'interesse mais j'ai pas le temps de le traiter, je vais pas non plus faire tout le boulot. et il me reste 23épisodes de 24H chrono a regardé


Message édité par Taz le 11-01-2003 à 01:47:10
Reply

Marsh Posté le 10-01-2003 à 23:50:08    

Je peux pas te répondre du tout.
 
Je met le code pour avoir des critiques constructives vu que y en a qui sont callés.
 

Code :
  1. template <class T> class Pnode
  2. {
  3.  
  4. private:
  5.   T info;
  6.   Pnode<T>* next;
  7.   Pnode<T>* pred;
  8. public:
  9.   Pnode(T,Pnode<T>*,Pnode<T>*);
  10.   Pnode(const Pnode<T>&);
  11.   ~Pnode(); 
  12.   Pnode<T>* getnext();
  13.   Pnode<T>* getpred();
  14.   T getinfo();
  15.   void setnext(Pnode<T>*);
  16.   void setpred(Pnode<T>*);
  17.   void setinfo(T);
  18.  
  19. };
  21. template<class T> class Plist
  22. {
  23. private:
  24.   int size;
  25.   Pnode<T>* first;
  26.   Pnode<T>* last;
  27.   Pnode<T>* getnode(int)const;
  28. public:
  29.   Plist();
  30.   Plist(const Plist&);
  31. ~Plist(); 
  32.   void operator=(const Plist&);
  33.   bool empty();
  34.   int getsize()const;
  35.   Plist<T>& add(T);
  36.   Plist<T>& adde(T);
  37.   T get(int)const ;
  39.   Plist<T>& del(int);
  40.   void affiche(ostream&);
  41. };
  43. template<class T> ostream& operator<<(ostream&,Plist<T>&);


 

Code :
  1. #include <iostream>
  2. #include <string>
  3. #include "Plist2.h"
  5. class Kexception
  6. {
  7. public:
  8.   Kexception(string message)
  9.   {
  10.     cout << message;
  11.   };
  12. }; 
  15. template<class T> Pnode<T>::Pnode(T Pelement,Pnode<T>* p,Pnode<T>* n)
  16. {
  17.   info=Pelement;
  18.   next=n;
  19.   pred=p;
  20. };
  21.  
  22. template<class T> Pnode<T>:: ~Pnode()
  23. {
  24.   delete info;
  25. };
  26.  
  27. template<class T> Pnode<T>::Pnode(const Pnode<T> &n)
  28. {
  29.   info=n.info->clone();
  30.   next=NULL;
  31.   pred=NULL;
  32. };
  35. template<class T> Pnode<T>*  Pnode<T>::getnext()
  36.   {
  37.     return next;
  38.   };
  40. template<class T> Pnode<T>*  Pnode<T>::getpred()
  41.   {
  42.     return pred;
  43.   };
  44.  
  45. template<class T> T  Pnode<T>::getinfo()
  46.   {
  47.     return info;
  48.   };
  50. template<class T> void  Pnode<T>::setinfo(T Pelement)
  51.   {
  52.     delete info;
  53.     info=Pelement;
  54.   };
  55.  
  56. template<class T> void  Pnode<T>::setpred(Pnode<T>* p)
  57.   {
  58.     pred=p;
  59.   };
  60.  
  61. template<class T>  void  Pnode<T>::setnext(Pnode<T>* n)
  62.   {
  63.     next=n;
  64.   };
  66. template<class T> Plist<T>::Plist()
  67. {
  68.   first=NULL;
  69.   last=NULL;
  70.   size=0;
  71. };
  73. template<class T> Plist<T>::Plist(const Plist &l)
  74. {
  75.   first=NULL;
  76.   last=NULL;
  77.   size=0;
  78.   if (!(l.getsize()==0))
  79.     {
  80.       Pnode<T>* n=l.getnode(1);
  81.       for(int i=1;i<=l.getsize();i++)
  82. {
  83.   adde(n->getinfo()->clone());
  84.   n=n->getnext();
  85. };
  86.     };
  87.  
  88. };
  90. template<class T> void Plist<T>::operator=(const Plist &l)
  91. {
  92.   if (!(&l==this))
  93.     {
  94.       while (!(size==0))
  95.       {
  96.       del(1);
  97.       };
  99.       if (!(l.getsize()==0))
  100. {
  101.   Pnode<T>* n=l.getnode(1);
  102.   for(int i=1;i<=l.getsize();i++)
  103.     {
  104.       adde(n->getinfo()->clone());
  105.       n=n->getnext();
  106.     };
  107. };
  109.     };
  110. };
  113. template<class T> Plist<T>::~Plist()
  114.   {
  115.     while (!(size==0))
  116.       {
  117.      del(1);
  118.       };
  119.   };
  121. template<class T>  bool Plist<T>::empty()
  122. {
  123.     return size==0;
  124. };
  126. template<class T>  int Plist<T>::getsize()const
  127. {
  128.     return size;
  129. };
  131. template<class T>  Plist<T>& Plist<T>::add(T Pobj)
  132. {
  133.   Pnode<T>* n=new Pnode<T>(Pobj,NULL,first);
  134.   if (!(first==NULL)) {first->setpred(n);};
  135.   first=n;
  136.   if (size==0) {last=n;};
  137.   ++size;
  138.   return *this;
  139. };
  141. template<class T>  Plist<T>& Plist<T>::adde(T Pobj)
  142. {
  143.   Pnode<T>* n=new Pnode<T>(Pobj,last,NULL);
  144.   if (!(last==NULL)) {last->setnext(n);};
  145.   last=n;
  146.   if (size==0) {first=n;};
  147.   ++size;
  148.   return *this;
  149. };
  151. template<class T>  Pnode<T>* Plist<T>::getnode(int j)const
  152. {     
  153.   if (size==0) throw Kexception("get on empty list" );
  154.   if ((j<=0) || (j>size)) throw Kexception("get out of bound of list" );
  155.   Pnode<T>* n;
  156.   if (j<=((size / 2)+1))
  157.     {
  158.       n=first;
  159.       for(int i=1;i<j;i++)
  160. {
  161.   n=n->getnext();
  162. };
  163.     }
  164.   else
  165.     {
  166.       n=last;
  167.       for(int i=size;i>j;i--)
  168. {
  169.   n=n->getpred();
  170. };
  171.     };
  172.   return n;
  173. };
  176. template<class T>  T Plist<T>::get(int j)const
  177. {
  178.   return this->getnode(j)->getinfo();
  179. };
  181. template<class T>  Plist<T>& Plist<T>::del(int j)
  182. {
  183.   if (size==0) throw Kexception("del on empty list" );
  184.   if ((j<=0) || (j>size)) throw Kexception("del out of bound of list" );
  185.   Pnode<T>* neud;
  186.   Pnode<T>* pred;
  187.   Pnode<T>* next;
  188.   neud=this->getnode(j);
  189.   pred=neud->getpred();
  190.   next=neud->getnext();
  191.    
  192.   if (first==neud)
  193.     {
  194.       first=next;
  195.     }
  196.   else
  197.     {
  198.       pred->setnext(next);
  199.     };
  201.   if (last==neud)
  202.     {
  203.       last=pred;
  204.     }
  205.   else
  206.     {
  207.       next->setpred(pred);
  208.     };
  209.   delete neud;
  210.   --size;
  211.   return *this;
  212. };
  214. template<class T>  void Plist<T>::affiche(ostream& os)
  215. {
  216.   Pnode<T>* f;
  217.   Pnode<T>* l;
  218.   f=first;
  219.   l=last;
  220.   for(int i=1;i<=size;i++)
  221.   {
  222.     os << *f->getinfo() << " ";
  223.     f=f->getnext();
  224.   };
  225. };
  228. template<class T> ostream& operator<<(ostream& os,Plist<T>& l)
  229. {
  230.   l.affiche(os);
  231.   return os;
  232. };
  234. class entier
  235. {
  236. public:
  237.   int val;
  238.   entier(int x=0){val=x;++nbe;/*cout << nbe<<endl;*/};
  239.   entier(const entier &e){val=e.val;++nbe;};
  240.   ~entier(){--nbe;/*cout << nbe<<endl;*/};
  241.   virtual entier* clone()
  242.   {
  243.     return new entier(*this);
  244.   };
  245.   static int nbe;
  246.   virtual void affiche(ostream& os)
  247.   {
  248.     os<<val;
  249.   };
  250. };
  252. int entier::nbe=0;
  254. class entier2:public entier
  255. {
  256. public:
  257.   int val2;
  258.   entier2(int x=0,int y=0):entier(x){val2=y;};
  259.   entier2(const entier2 &e){*this=e;};
  260.   virtual entier2* clone()
  261.   {
  262.     return new entier2(*this);
  263.   };
  264.   virtual void affiche(ostream& os)
  265.   {
  266.     os<<"(" <<val<<","<<val2<<" )";
  267.   };
  268.  
  269. };
  271. ostream& operator<<(ostream& os,entier &e)
  272. {
  273.   e.affiche(os);
  274.   return os;
  275. };
  278. main()
  279. {
  280.   Plist<entier*> l;
  281.   cout <<l<<endl;
  282.   for(int i=1;i<=10;i++)
  283.     {
  284.       l.add(new entier2(3*i,i) );
  285.     };
  286.   l.add(new entier(55));
  287.   cout <<l<<endl;
  288.   Plist<entier*> m;
  289.   m.add(new entier2(5,98));
  290.   m=l;
  291.   cout <<m<<endl;
  292. }


Message édité par karim63 le 10-01-2003 à 23:52:40
Reply

Marsh Posté le 11-01-2003 à 00:04:34    

ca fait un peu beaucoup à lire:
 
petites remarques:  
 
 
- essaye de plus utiliser les références constantes dans tes paramètres (comme pour setinfo)
 
- si unefonction membre de modifie pas l'objet, mets l'attribut const (getinfo, empty)
 
- void operator=(const Plist&). si tu fais ça tu empeches l1=l2=l3. peut etre est-ce volontaire. sinon tu fais  

Code :
  1. const Plist & operator=(const Plist&)
  2. {
  3.   return *this;
  4. }


 
- utilise des types unsigned pour tes longueurs (je suis un chieur)
 
- ta methode affiche, bof, les ostream n'ont rien a faire avec ta classe (c'est une surspécification, dans la vraie vie, personne n'utilisera Plist.affiche(cout), on ne demande pas au structure de données de faire des GUI). la méthode consacrée, c'est string print() const, qui renvoie une chaine, représentation de l'objet
 
- vas y avec les inline
 
- construits tes membres

Code :
  1. class Foo
  2. {
  3.   Bar b;
  4.   std::vector<int> v;
  6. public:
  7.   Foo(const Bar &a_b) : b(a_b), v(10, 69)
  8.   { // des trucs
  9.    }
  10. };


 
- !(a==b)) <=> (a!=b)  !!!!!!!!!!!!!!!!!
 
- je crois pas avoir vu de using namespace std;
 
- initialise(sens large) tes varaibles au plus pres de leur déclaration

Code :
  1. // appel constructeur, puis operator=
  2. Pnode<T>* f;
  3. Pnode<T>* l;
  4. f=first;
  5. l=last;
  7. // il vaut mieux, seulement constructeur
  8. Pnode<T>* f(first);
  9. Pnode<T>* l(last);


 
- et le type de retour de main doit etre int (pas la peine de spécifier un return 0;)
 
- dans tes déclarations de classe: saute des lignes et evite de mixer déclaration de variables et déclaration de fonctions membres
 
- etc


Message édité par Taz le 11-01-2003 à 00:20:15
Reply

Marsh Posté le 11-01-2003 à 00:17:15    

-references constante:
hem vu que ce que je passe en param est un pointeur c pas tres utile de metre un const. Ok pour le passage par ref, mais vu que c'est un pointeur c même pas sur que le compilateur accepte.
 
-si unefonction membre de modifie pas l'objet, mets l'attribut const (getinfo, empty)
 
En fin de fonction c ça ?
 
-void operator=(const Plist&). si tu fais ça tu empeches l1=l2=l3. peut etre est-ce volontaire.
 
vaguement volontaire, je voyais pas l'interet, mais pkoi pas, je vais le rajouter.
 
- utilise des types unsigned pour tes longueurs (je suis un chieur)
 
[:ddr555]
 
- vas y avec les inline
 
J'ai commencé tout en inline [:ddr555] et j'ai mis dans un point h apres pour me forcer a la faire.
 
- construits tes membres
 
 :??: que veux tu dire par là ?
 
- (!(a==b)) <=> (a!=b)
 
exact je trouvais ça tout moche aussi .
 
- je crois pas avoir vu de using namespace std;
 
C'est quoi ?
 
-// il vaut mieux

Code :
  1. Pnode<T>* f(first);


tu veux dire:
 
Pnode<T>* f=first;
 
?

Reply

Marsh Posté le 11-01-2003 à 00:17:15   

Reply

Marsh Posté le 11-01-2003 à 00:24:52    

je crois que tu as lu un peu vite ma réposne précédente
 
mais remarques vise pas un bout de code particulier, mais l'ensemble. considères en une et regarde l'ensemble de ton code...
 
 
les namespace... il est ard: rajoute ça "using namespace std;" juste apres tes #include
 

Citation :


Code :
  1. Pnode<T>* f(first);


tu veux dire:
 
Pnode<T>* f=first;

c'est strictement équivalent, personnellement je préfère la premiere ecriture (bien entendu ici, le = n'a donc rien avoir avec operator=)

Reply

Marsh Posté le 11-01-2003 à 00:37:50    

Citation :

les namespace... il est ard: rajoute ça "using namespace std;" juste apres tes #include


 
je ne sais pas ce que c'est, ça sert a quoi ?

Reply

Marsh Posté le 11-01-2003 à 05:47:33    

++Taz a écrit :

muni tes objets d'une méthode clone()

Code :
  1. Foo* Foo::clone() const
  2. {
  3.   return new Foo(*this);
  4. }




 
J'ai pas lu tout le topic, mais je me pose une question :
si je veux que ma liste soit sur des références, ou des pointeurs, je fait comment pour ajouter clone() au type (MaClasse &) ou (MaClasse *) ??

Reply

Marsh Posté le 11-01-2003 à 06:01:31    

karim63 a écrit :


Code :
  1. #include <iostream>
  2. #include <string>
  3. #include "Plist2.h"
  5. template<class T> Pnode<T>::Pnode(const Pnode<T> &n)
  6. {
  7.   info=n.info->clone();
  8.   next=NULL;
  9.   pred=NULL;
  10. };




 
Rahaaaaa putaint de ta race, je passe ma vie à t'expliquer la différence entre initialisation et instanciation, et tu écris des merdes pareilles ????
 
C'est un putain de constructeur par copie, tu n'affectes pas les variables d'instance dans un constructeur tu les initialises (avec des putains de ":", deux point en français, colon in english, jamais avec "=", égal en français, equal in english)  :

Code :
  1. template<class T> Pnode<T>::Pnode(const Pnode<T> &n) : info(n.info), pred(NULL), next(NULL=){/* ce putain de code est vide*/}


Fait-moi plaisir, imprime un truc dans ta vie : la différence entre initialisation (la variable ne contenait aucune valeur significative avant) et affectation (la variable change de valeur).


Message édité par nraynaud le 11-01-2003 à 06:03:28
Reply

Marsh Posté le 11-01-2003 à 07:40:11    

nraynaud a écrit :


 
J'ai pas lu tout le topic, mais je me pose une question :
si je veux que ma liste soit sur des références, ou des pointeurs, je fait comment pour ajouter clone() au type (MaClasse &) ou (MaClasse *) ??
 

ben comme je le propose, il faut introduire une double spécialisation de template par l'intermédiaire d'une classe wrapper&clone. le problème c'est surtout la sémantique: je ne vois pas trop l'interet d'avoir une fonction membre clone a part pour le polymorphisme qui se fait grace aux pointeurs en C++

Reply

Marsh Posté le 11-01-2003 à 14:02:54    

Citation :

Fait-moi plaisir, imprime un truc dans ta vie : la différence entre initialisation (la variable ne contenait aucune valeur significative avant) et affectation (la variable change de valeur).


 
hé connard tu vas parler sur un autre ton ?? :fou:  
Ce que je comprends et pas avec tes explications(y en a pas), c'est que si j'affecte a un attribut non instanticé un objet avec = , si il est pas initiliasié ça peut merder lors du vidage de l'objet censé ecrasé.
Soit plus clair dans tes explications. Il suffit pas de dire ce qu'il faut faire, il faut aussi dire pourquoi sinon ça sert a rien.

Reply

Marsh Posté le 11-01-2003 à 14:43:34    

karim63 a écrit :

Citation :

Fait-moi plaisir, imprime un truc dans ta vie : la différence entre initialisation (la variable ne contenait aucune valeur significative avant) et affectation (la variable change de valeur).


 
hé connard tu vas parler sur un autre ton ?? :fou:  
Ce que je comprends et pas avec tes explications(y en a pas), c'est que si j'affecte a un attribut non instanticé un objet avec = , si il est pas initiliasié ça peut merder lors du vidage de l'objet censé ecrasé.
Soit plus clair dans tes explications. Il suffit pas de dire ce qu'il faut faire, il faut aussi dire pourquoi sinon ça sert a rien.


http://forum.hardware.fr/forum2.ph [...] t=#t272543
ho bah t'as raison, c'est une choucroute volante, j'ai cru que c'était une explication. A la page précédente, j'avais déjà fait la remarque (rapidement).

Reply

Marsh Posté le 11-01-2003 à 14:52:43    

++Taz a écrit :


 
- void operator=(const Plist&). si tu fais ça tu empeches l1=l2=l3. peut etre est-ce volontaire. sinon tu fais  

Code :
  1. const Plist & operator=(const Plist&)
  2. {
  3.   return *this;
  4. }




 
C'est pas plutôt  

Code :
  1. const Plist & operator=(const Plist& other)
  2. {
  3.   return other;
  4. }


 

Code :
  1. int i = 10, j = 11, k;
  3. k = (i = j) //vaut 11 dans ma tête

Reply

Marsh Posté le 11-01-2003 à 15:15:17    

ben c'est à peu pres la meme chose, a moins que ta sémantique de operator= soit differente de celle communément utilisée

Reply

Marsh Posté le 11-01-2003 à 15:26:50    

++Taz a écrit :

ben c'est à peu pres la meme chose, a moins que ta sémantique de operator= soit differente de celle communément utilisée


 
C'est pas la mienne mais la tienne ... (si c'est vrai, nananère)

Reply

Marsh Posté le 11-01-2003 à 21:07:58    

*s'imagine une choucroute qui vole*...beuurkkk  :lol:  
*se demande si la choucroute garde sa cohésion lorsqu'elle dépasse la vitesse du son*... :pt1cable:


---------------
mon NAS/HTPC Q11 H55N-USB3, Core i3 http://forum.hardware.fr/hfr/Hardw [...] m#t7599404
Reply

Marsh Posté le 11-01-2003 à 21:24:38    

Galett a écrit :

*s'imagine une choucroute qui vole*...beuurkkk  :lol:  
*se demande si la choucroute garde sa cohésion lorsqu'elle dépasse la vitesse du son*... :pt1cable:  


http://kadreg.free.fr/perso/moules [...] olante.jpg

Reply

Marsh Posté le 12-01-2003 à 04:45:47    

nraynaud a écrit :

C'est pas plutôt

Code :
  1. int i = 10, j = 11, k;
  3. k = (i = j) //vaut 11 dans ma tête


Pour un type intégré, k reçoit la valeur de i après affectation (plus précisément, la valeur de j convertie dans le type de i).
Donc, si on veut être cohérent, on doit renvoyer le type affecté si celui-ci est différent du type lu.
 
Mais surtout, l'objet lu peut être un objet temporaire généré par le compilateur (issu d'une conversion implicite par exemple).
Renvoyer une référence de ce temporaire peut être catastrophique s'il est supprimé par le compilateur dès la première affectation accomplie.

Code :
  1. const A & operator=(const A& other){
  2. return other;
  3. }
  5. A i,j;
  6. i=j=0; //i= peut-être rien !


C'est le fait que l'argument soit const qui autorise à transmettre un temporaire à une référence. Un truc tordu !
 
Donc, on n'a en fait pas le choix si on renvoie des références:

Code :
  1. return *this;


---------------
Bricocheap: Montage de ventilo sur paté de mastic silicone
Reply

Marsh Posté le 12-01-2003 à 16:05:22    

ok, j'ai ma réponse  :)


---------------
mon NAS/HTPC Q11 H55N-USB3, Core i3 http://forum.hardware.fr/hfr/Hardw [...] m#t7599404
Reply

Marsh Posté le 12-01-2003 à 16:23:00    

Musaran a écrit :


Mais surtout, l'objet lu peut être un objet temporaire généré par le compilateur (issu d'une conversion implicite par exemple).
Renvoyer une référence de ce temporaire peut être catastrophique s'il est supprimé par le compilateur dès la première affectation accomplie.
 
 
Donc, on n'a en fait pas le choix si on renvoie des références:

Code :
  1. return *this;




Tu es très loin de m'avoir convaincu, si le compilo te file à affecter une valeur temporaire (qu'il a créé de façon implicite en plus), il se démerde, c'est à lui de te filer des trucs qui vont durer au moins jusqu'à la fin du scope de la variable.
Par contre, le feit que = renvoie l'ancienne valeur contenue dans la variable et pas la nouvelle, ça me parraît nettement plus grave, c'est une rupture de contrat.

Reply

Marsh Posté le 12-01-2003 à 22:10:37    

Citation :

Tu es très loin de m'avoir convaincu, si le compilo te file à affecter une valeur temporaire (qu'il a créé de façon implicite en plus), il se démerde, c'est à lui de te filer des trucs qui vont durer au moins jusqu'à la fin du scope de la variable.


 
Oui mais dans ce cas la ce n'est plus la responsabilite du compilateur. L'operateur = est de la responsabilite du programmeur qui l'a surcharge. Et malheureusement on ne peut pas l'empecher d'ecrire n'importe quoi.
 

Citation :

Par contre, le feit que = renvoie l'ancienne valeur contenue dans la variable et pas la nouvelle, ça me parraît nettement plus grave, c'est une rupture de contrat.


 
Ce qui me paraitrait bizarre plutot c'est qu'a la fin de l'operation d'affectation on aurait *this qui vaudrait toujours son ancienne valeur alors que normalement on voudrait qu'elle soit egale a celle de la reference passee en parametre.
 
Bref si l'operateur = est bien ecrit et qu'il renvoie
une reference constante vers *this alors le contrat
est bien respecte imho.
 
LeGreg

Reply

Marsh Posté le 12-01-2003 à 22:25:09    

legreg a écrit :


Bref si l'operateur = est bien ecrit et qu'il renvoie
une reference constante vers *this alors le contrat
est bien respecte imho.
 


L'ancienne valeur reçoit le message = avec comme argument la nouvelle valeur et, dans la fonction, il n'y a rien de magique qui te permette de faire toi-même l'affectation, par contre on te prévient qu'il y a affectation (de qui sur qui), et que si tu as quelquechose à faire, c'est maintenant ou jamais. On te file pas un pointeur sur la varible et la nouvelle valeur en disant "tiens vas-y affecte", mais on te donne la possibilité de t'accrocher à l'événement affectation.

Reply

Marsh Posté le 12-01-2003 à 22:53:22    

heink??
 
:crazy:
 
J'ai beau retourner ta phrase dans tous les sens??
 
LeGreg

Reply

Marsh Posté le 12-01-2003 à 22:59:09    

legreg a écrit :

heink??
 
:crazy:
 
J'ai beau retourner ta phrase dans tous les sens??
 
LeGreg
 


Je réessaye : L'opérateur = n'est pas destiné à faire une affectation. Il sert de handler qui sera appelé au moment de l'affectation, mais on te demande pas physiquement de faire la copie dedans. C'est juste que si tu as un truc à faire au moment de l'affectation, tu le fais dedans, c'est tout.
 
mon vieux pointeurs-intelligents pour un exemple :
http://forum.hardware.fr/forum2.ph [...] t=#t268792

Reply

Marsh Posté le 12-01-2003 à 23:14:38    

nraynaud a écrit :


Je réessaye : L'opérateur = n'est pas destiné à faire une affectation. Il sert de handler qui sera appelé au moment de l'affectation, mais on te demande pas physiquement de faire la copie dedans. C'est juste que si tu as un truc à faire au moment de l'affectation, tu le fais dedans, c'est tout.


 
Oui mais si lors d'une affectation tu decides de ne rien faire lors de cette affectation c'est tout de meme pervers non ?
 

Citation :

     Pointeur operator = (Pointeur r) { // ici est toute la feinte  
          if (r.real != real) {  
              if (real)  
                  real->unbind();  
              if (r.real)  
                  r.real->bind();  
              real = r.real;  
          }  
          return r;  
      }


 
Quoique j'ai du mal a faire le lien entre ton affirmation
quotee et ce bout de code ou tu fais les choses volontairement mal apparemment mais ca devrait tout de meme marcher (en partie grace aux recopies inutiles..).  
 
LeGreg

Reply

Marsh Posté le 12-01-2003 à 23:43:37    

legreg a écrit :


 
Oui mais si lors d'une affectation tu decides de ne rien faire lors de cette affectation c'est tout de meme pervers non ?
 

Citation :

     Pointeur operator = (Pointeur r) { // ici est toute la feinte  
          if (r.real != real) {  
              if (real)  
                  real->unbind();  
              if (r.real)  
                  r.real->bind();  
              real = r.real;  
          }  
          return r;  
      }


 
Quoique j'ai du mal a faire le lien entre ton affirmation
quotee et ce bout de code ou tu fais les choses volontairement mal apparemment mais ca devrait tout de meme marcher (en partie grace aux recopies inutiles..).  
 
LeGreg


 
si tu décides de ne rien faire, tu ne fais rien, tu ne surcharges même pas = et ça va s'affecter tout seul (d'ailleur, même si tu surcharges et que tu mets un corps vide ça va le faire aussi), je vois pas la question.
 
Gni ??? c'est n'importe quoi ce code ! Y'a un mec qui m'a fait chier avec des références à la con mais il est pas capable de voir ça.
 
Bon, correction :
 

Code :
  1. Pointeur operator = (Pointeur r) { // ici est toute la feinte
  2.     if (r.real != real) {
  3.       if (real)
  4.         real->unbind();
  5.       if (r.real)
  6.         r.real->bind();
  7.     }
  8.     return r;
  9.   }


 
donc dans cette version, on ne touche pas aux objets eux-même , on ne fait même rien passer de l'un à l'autre, on fait juste des actions justifiées par l'imminence d'une affectation.

Reply

Marsh Posté le 12-01-2003 à 23:48:29    

ah ben si ca marche apres ta "correction"
ce sera tout de meme un sacre miracle..
 
Bon je retourne bosser. Bonne nuit a vous.
 
LeGreg

Reply

Marsh Posté le 13-01-2003 à 01:18:21    

Après documentation, je me suis planté.
En fait il assigne tous les champs un par un et il gueule s'il trouve un const ("non-static const member `const int A::i', can't use default assignment operator" ). C'est ce const qui m'a fait planter, je pensais qu'il virait l'objet de la case et qu'il en foutait un nouveau à la place.

Reply

Marsh Posté le 13-01-2003 à 12:56:10    

nraynaud a écrit :

si le compilo te file à affecter une valeur temporaire (qu'il a créé de façon implicite en plus), il se démerde, c'est à lui de te filer des trucs qui vont durer au moins jusqu'à la fin du scope de la variable.

Quelle variable ?
Si c'est l'argument de la fonction, c'est le cas : elle est détruite avant même la fin de l'expression.
Si c'est 'i' dans 'i=j=0', ça ne colle pas avec un principe du C++: ce qui se passe après dans l'expression n'influence pas ce qui s'y passe avant.
'i=(j=0)': j=0 produira toujours la même chose, qu'il soit précédé de i= ou pas.
 
Je répète que le mélange valeur temporaire et référence const est... spécial.
Ajouté à l'opérateur = qui est des plus compliqués, c'est pas de la tarte !
 

Citation :

Par contre, le feit que = renvoie l'ancienne valeur contenue dans la variable et pas la nouvelle, ça me parraît nettement plus grave, c'est une rupture de contrat.

Je te comprends pas là.
'return' venant logiquement à la fin de la fonction, l'affectation est accomplie, et la nouvelle valeur (adaptée) renvoyée.
 

Citation :

tu ne surcharges même pas = et ça va s'affecter tout seul (d'ailleur, même si tu surcharges et que tu mets un corps vide ça va le faire aussi)

Non, dès qu'on déclares soi-même l'affectation on est responsable de tout copier.
On peut cependant s'appuyer sur l'affectation de la classe héritée.


---------------
Bricocheap: Montage de ventilo sur paté de mastic silicone
Reply

Marsh Posté le 13-01-2003 à 13:05:09    

Musaran a écrit :

Quelle variable ?
Si c'est l'argument de la fonction, c'est le cas : elle est détruite avant même la fin de l'expression.
Si c'est 'i' dans 'i=j=0', ça ne colle pas avec un principe du C++: ce qui se passe après dans l'expression n'influence pas ce qui s'y passe avant.
'i=(j=0)': j=0 produira toujours la même chose, qu'il soit précédé de i= ou pas.
 
Je répète que le mélange valeur temporaire et référence const est... spécial.
Ajouté à l'opérateur = qui est des plus compliqués, c'est pas de la tarte !
 

Citation :

Par contre, le feit que = renvoie l'ancienne valeur contenue dans la variable et pas la nouvelle, ça me parraît nettement plus grave, c'est une rupture de contrat.

Je te comprends pas là.
'return' venant logiquement à la fin de la fonction, l'affectation est accomplie, et la nouvelle valeur (adaptée) renvoyée.
 

Citation :

tu ne surcharges même pas = et ça va s'affecter tout seul (d'ailleur, même si tu surcharges et que tu mets un corps vide ça va le faire aussi)

Non, dès qu'on déclares soi-même l'affectation on est responsable de tout copier.
On peut cependant s'appuyer sur l'affectation de la classe héritée.


 
Laisse tomber, j'ai dit que des conneries, je me suis documenté depuis.


Message édité par nraynaud le 13-01-2003 à 13:06:48
Reply

Marsh Posté le 15-01-2003 à 18:35:01    

je voudrais implemetenter une fonctio de recherche, mais assez generale et qui pourrait porter sur des criteres que les objets de la listes doivent avoir.
 
par exemple une methode de recherche qui prend en parametre la valeure recherchée, et aussi l'attribut qui doit avoir cette valeur, ou bien une methode qui renvoit une valeur qui me permettra de faire la comparaison, et la fonction renverrait un pointeur sur cet objet par exemple.
 
si j'ai un objet avec comme attribut int A,B,C de pouvoir faire une recherche en disant sur quel attribut ce fait la recherche, plutot que de devoir faire 3 fonctions de recherche a chaque fois.
C'est possible ?

Reply

Marsh Posté le 16-01-2003 à 05:14:10    

Il a posé la question dans un topic à part: http://forum.hardware.fr/forum2.php3?post=30891&cat=10


---------------
Bricocheap: Montage de ventilo sur paté de mastic silicone
Reply

Marsh Posté le 16-01-2003 à 07:49:52    

quelle balance ce musa  :D

Reply

Marsh Posté le 17-01-2003 à 05:38:00    

Tu poses ta question à deux endroits.
Comment se sentirait quelqu'un se donnant la peine de te répondre, pour découvrir que tu as déjà eu une réponse ailleurs ?


---------------
Bricocheap: Montage de ventilo sur paté de mastic silicone
Reply

Marsh Posté le 17-01-2003 à 07:23:49    

J'ai cahngé d'avis c tout.
Même si c'est pour continuer ce que j'ai fait dans ce thread, c'est pas forcement le même sujet.
Et pi c'est du second degr&é en plus.  :o

Reply

Marsh Posté le    

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed