Discussion :

[Pilloutou]

Bonjour,

Je cherche à définir une fonction convertissant une instance de la classe Produit en TreeNode pour être ensuite affiché dans un TreeView.
J'ai besoin de 2 fonctions, l'une convertissant un produit en un treenode et l'autre convertissant plusieurs produit en plusieurs treenode.
d'après vous parmi les 3 solutions suivantes, laquelle est réellement une surcharge ?

Solution 1:
La méthode ProduitToTreeNode est surchargée, la première convertit un produit en un treenode et l'autre utilise la première pour convertir plusieurs produits en plusieurs treenode. Les 2 méthodes ne font pas rigoureusement la même chose, je me demande donc si elle doit être surchargée.

Code c# :

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
17
18
19
 
public TreeNode ProduitToTreeNode(Produit pdt)
{
    TreeNode tree = new TreeNode(pdt.Reference);
    tree.Name = pdt.Identifiant.ToString();
    return tree;
}
 
public TreeNode[] ProduitToTreeNode(Produit[] pdtArray)
{
    TreeNode[] trees = new TreeNode[pdtArray.Length];
 
     for (int i = 0; i<pdtArray.Length; i++)
     {
          trees[i] = ProduitToTreeNode(pdtArray[i]);
     }
 
     return trees;
}

Solution 2:
Pour cette solution, la méthode ProduitToTreeNode n'est pas surchargée.
Les 2 méthodes n'ont plus le même nom car je me dis que ne faisant pas rigoureusement la même chose elles ne peuvent avoir le même nom.

Code c# :

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
17
18
public TreeNode ProduitToTreeNode(Produit pdt)
{
    TreeNode tree = new TreeNode(pdt.Reference);
    tree.Name = pdt.Identifiant.ToString();
    return tree;
}
 
public TreeNode[] ProduitToTreeNodeArray(Produit[] pdtArray)
{
    TreeNode[] trees = new TreeNode[pdtArray.Length];
 
     for (int i = 0; i<pdtArray.Length; i++)
     {
          trees[i] = ProduitToTreeNode(pdtArray[i]);
     }
 
     return trees;
}

Solution 3:
Et enfin la dernière solution qui me semble rigoureusement plus être une sucharge.

Code c# :

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
17
18
19
 
public TreeNode ProduitToTreeNode(Produit pdt)
{
    return ProduitToTreeNode(new Produit[] { pdt})[0];
}
 
public TreeNode[] ProduitToTreeNode(Produit[] pdtArray)
{
    TreeNode[] trees = new TreeNode[pdtArray.Length];
 
     for (int i = 0; i<pdtArray.Length; i++)
     {
          Produit pdt = pdtArray[i];
          trees[i] = new TreeNode(pdt.Reference);
          trees[i].Name = pdt.Identifiant.ToString();          
     }
 
     return trees;
}

Merci d'avance pour vos conseils.
Bye

[Piotrek]

Salut

Pour qu'il y ait surcharge il faut que les methodes aient le meme nom, donc c'est la solution 1.

[ced600]

Dan la solution 3 elles possèdent aussi le même nom

[Pilloutou]

Bonjour,

Pour qu'il y ait surcharge il faut que les methodes aient le meme nom, donc c'est la solution 1.

Oui oui je suis d'accord, mais en fait ma question portait plus sur:
Quelle solution adopter pour réaliser une méthode qui transforme un produit en TreeNode et une méthode qui transforme plusieurs produits en plusieurs TreeNode ?
Soit je réalise une méthode surchargée (Solution 1 et 3), soit je ne fais pas de méthode surchargée si l'on part du principe que conceptuellement les méthodes ne font pas rigoureusement la même chose.
J'avais lu quelque part que si deux méthodes ne réalise pas rigoureusement la même chose, elles ne doivent pas être surchargées.
Ma question portait plus sur ce type de raisonnement. Je sais que ça fait un peu question tordu mais c'est parce que je m'interroge sur l'utilisation judicieuse de la sucharge.

Merci d'avance.
Bye

[SirJulio]

Salut,

du point de vue langage, il n'est pas incorrect dans des surcharges de retourner des valeurs differentes, mais AMHA, c'est pas une bonne solution. Les surcharges devraient retourner un 'truc' soit sous des conditions differentes (argument variable en nature), soit avec une precision differentes (variable en nombre). De fait, je trouve la deuxieme solution la plus clair et intuitive, c'est ce que j'aurai fait pour ma part. =)

[Contrec]

Idem, du moment qu'elles ne retournent pas la meme chose, autant faire 2 methodes distinctes sinon d'un point de vue technique ca passe mais d'un point de vue conception ca peut preter a confusion quand on voit que les signature de methodes ont le meme nom alors qu'elle ne retournent pas la meme chose...

[Franckintosh]

Oui oui je suis d'accord, mais en fait ma question portait plus sur:
Quelle solution adopter pour réaliser une méthode qui transforme un produit en TreeNode et une méthode qui transforme plusieurs produits en plusieurs TreeNode ?

Ben alors pose la question "Quelle solution adopter pour réaliser une méthode qui transforme un produit en TreeNode et une méthode qui transforme plusieurs produits en plusieurs TreeNode ?" !
...
Si tu demandes quelle solution est une surcharge alors tu auras des réponses à propos de surcharge.

[ced600]

+1
Tout à fait d'accord, on n'a pas de boule de cristal pour deviner ce que tu veux vraiment, alors si tu nous met des pièges ....