Bonjour,

j'ai créé une classe point :

class pt
    {
        /*déclaration des attributs privés*/
        private float x;
        private float y;
     }

J'ai des accesseurs en écriture, en lecture et un constructeur qui prends en paramètres x et y.

Je crèe un liste de points :

list<pt> lis = new list<pt>();
pt p = new pt(2,0);
pt p1 = new pt(1,0);
lis.add(p);
lis.add(p1);

Je voudrais stocker ces anciennes valeurs car je vais modifier les coordonnées des points, je cree un nouvelle liste :

list<pt> lis_old = new list <pt>();
//recopie de lis dans lis_old
lis_old = lis;

//modification des coordonnées dans ma lis
foreach pt p in lis
{
    p.set_x(p.get_x() + 10);
    p.set_y(p.get_y() - 5);
}

le problème avec ce code est que les valeurs de ma lis_old sont également modifié comment je peux faire svp pour garder ces anciennes valeurs.
Merci d'avance.

Laurent

Salut,

Comme il s'agit d'une classe, ton type pt est un type référence, tu as donc en fait les références aux mêmes objets dans les 2 listes.
Lors de la copie il faut donc que tu crées de nouvelles instances avec les mêmes valeurs.
Pour faciliter la tâche tu peux par exemple créer une méthode Clone (implémentation de ICloneable ou méthode Clone fortement typée) et t'en servir durant la phase de copie.



/*
coq
MVP Visual C#
CoqBlog
*/

Ou bien tu utilise une structure au lieu d'une classe (structure est un type valeur)
il est a noter que dans System.Drawing il éxiste tout un tas de structure de ce type :

Point, PointF, Size, SizeF, Rectangle, RectangleF ....
donc c'est par forcement la peine de réinventer la roue...

Salut,




List< PointF > list = new List< PointF >( );
list.AddRange
(
    new PointF[ ]
    {
        new PointF( 0.1f, 0.1f ),
        new PointF( 0.2f, 0.2f )
    }
);

List< PointF > copy = new List< PointF >( list.ToArray( ) );

coq,

as tu une idée pour la méthode clone.

D'avance Merci

ceci devrait fonctionner :
class pt, IClonable
{
   /*déclaration des attributs privés*/
   private float x;
   private float y;

   public float X
   {
      get{return this.x;}
      set{this.x=value;}
   }

   public float Y
   {
      get{return this.y;}
      set{this.y=value;}
   }

   public pt (float x,float y)
   {
      this.x=x;
      this.y=y;
   }

   public Object Clone ()
   {
      return new pt(this.x,this.y);
   }
}

ATTENTION : une classe prend plus de place en mémoire qu'une structure équivalente donc si tu utilise bcp de classe pt et que tu n'as pas besion de referencer le même ojet depuis plusieurs partie du code, essaie de changer ta classe en structure.

Oups y'a comme une redondance!!! (j'ai été moins rapide ...)

mais la si t'a pas trouverton bonheur on pourrat pas faire mieux!

"class pt, IClonable" => class pt : ICloneable
;-)



/*
coq
MVP Visual C#
CoqBlog
*/

Struct ou class de toute façon le problème est là :





list<pt> lis_old = new list <pt>();
//recopie de lis dans lis_old
lis_old = lis;