Discussion :

[toss.net]

Bonjour,

Je pensais que ça existait par défaut le clonage d'un objet en c# : .clone()
Faut écrire soit même la procédure ?

L'ajout de ICloneable ne suffit pas

il faut mettre ça ?

Code :

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

1
2
3
4
public object Clone()
   {
      return this.MemberwiseClone();      // call clone method
   }

rien d'autre ..!?

[StormimOn]

L'ajout de ICloneable ne suffit pas

Si mais il faut implémenter la mise en oeuvre de l'interface, c'est le principe des interfaces. On exprime un contrat (l'interface) et la classe qui souhaite utiliser ce contrat doit ensuite l'implémenter.
L'interface n'est néanmoins pas forcément nécessaire. Elle peut être utile si tu as besoin de cloner des objets sans forcément les connaître. Auquel cas tu les manipuleras via l'interface

Concernant

Code :

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

1
2
3
4
public object Clone()
{
      return this.MemberwiseClone();
}

cela peut suffire et cela va surtout dépendre du type de copie que tu veux faire : superficielle ou en profondeur. MemberwiseClone effectuant une copie superficielle.

Si le clonage est superficiel et que l'objet cloné possède des références sur d'autres objets, alors ces derniers ne seront pas clonés.
Le clonage en profondeur consiste à cloner le graphe d'objets entier, ou bien jusqu'à une certaine profondeur suivant les cas.

Il est évident que si ton objet ne possède pas de référence vers d'autres objets alors MemberwiseClone est suffisant.

Donc la question est, de quoi as-tu besoin ?

[tomlev]

A noter que le principe de l'interface ICloneable est largement critiqué dans la communauté .NET, parce que ça n'indique pas du tout le type de copie qui est effectuée (profonde ? si oui, jusqu'à quel niveau ? superficielle ?)

Au final, si c'est juste pour un objet, autant créer directement une méthode Clone qui fait ce que tu veux, sans forcément implémenter l'interface ICloneable (qui n'apporte finalement rien d'utile)

[toss.net]

En gros je comprends que je vais reprendre mes vielles méthodes VB6 , du genre pour un classe simple sans référence...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
Public Function Clone() As cMonObjet
 
    Set Clone = New cMonObjet
    Clone.Forme = Forme
    Clone.CoteA = CoteA
    Clone.CoteB = CoteB
    Clone.CoteC = CoteC
    Clone.Distance = Distance
    Clone.Angle = Angle
    Clone.Multiple = Multiple
    Clone.Repere = Repere
    Clone.Couleur = Couleur
    Clone.FormuleCoteA = FormuleCoteA
    Clone.FormuleCoteB = FormuleCoteB
    Clone.FormuleCoteC = FormuleCoteC
End Function

[tomlev]

En gros je comprends que je vais reprendre mes vielles méthodes VB6

Ben si fais juste une copie superficielle de tous les membres, autant utiliser MemberWiseClone. Si tu veux choisir les membres qui sont copiés, là il faut le faire à la main... Je ne vois pas comment on pourrait s'y prendre autrement

[kheironn]

Bonjour

Juste en passant, si tu veux cloner des objets en contenant d'autres, pourquoi ne pas les faire hériter d'un ancêtre qui offre une methode clone() et effectue un clone memberwise.

Ensuite les macro-objets redéfinissent la methode en faisant un base + appel de methode clone des objet inclus...

(et faire ce qui va bien avec tout ça )