Méthode surchargée ou quelque chose de plus élégant?

Question

Je suis en train d'écrire du code en C# et j'ai remarqué un scénario comme ci-dessous. Je me demande comment je peux rendre cela plus élégant et facile à entretenir.

Si je suivant le scénario avec

surcharge

public void DoSmtg(string a) { DoSmtg(a, 0, 0f); } 
public void DoSmtg(string a, int x) { DoSmtg(a, x, 0f); } 
public void DoSmtg(string a, int x, double d) { // method logic } 

dire maintenant, je dois ajouter un autre paramètre bool. Je devrais modifier cela à

Ceci est un exemple très simple. Il est possible d'avoir 10 autres versions de la méthode DoSmtg(). Clairement, ce code sent. Alors que la saturation est tout à fait valable, il est évidemment difficile de maintenir ce code depuis:

1. il existe de nombreuses méthodes à écrire

2. il est évident que la méthode est appelée jusqu'à une enquête minutieuse (surtout si méthode a plus de paramètres)

3. méthode

   se poluted par la croissance liste des paramètres

4. le nouveau paramètre ajouté nécessite des changements à de nombreux endroits (pensez à la métho ds ci-dessus appelé à partir de nombreux endroits différents)

Ce qui serait un moyen élégant, simple et bon d'éviter quelque chose comme ça?

Answer 1

Vous pouvez essayer d'ajouter tous les paramètres sur une fonction, en utilisant les valeurs par défaut et nommez les paramètres lorsque vous appelez la fonction:

public void DoSmtg(string a, int x=0, double d=0f, bool doIt=false) { 
    // method logic 
} 

Lorsque vous appelez la fonction, alors vous feriez:

DoSmtg("yo!") 
DoSmtg("yo!", d:0.59, doIt:true) 

Answer 2

Vous pouvez envelopper tous les paramètres dans un POCO:

public class SomethingParameters 
{ 
    public string A { get; set; } 
    public int X { get; set; } 
    public double D { get; set; } 
    public bool DoIt { get; set; } 
} 

Ensuite, la méthode la signature devient:

public void DoSmtg(SomethingParameters parameters) { // method logic } 

J'aime ce motif car il est facile de le prolonger dans le futur. Si vous avez besoin d'ajouter cinq paramètres supplémentaires, ou des paramètres optionnels, pas de problème!

Vous pouvez l'appeler comme:

var parameters = new SomethingParameters() 
{ 
    A = "foobar", 
    X = 123, 
    D = 0.123, 
    DoIt = false 
} 

DoSmtg(parameters); 

Si vous avez beaucoup de code appelant l'ancienne signature de la méthode que vous ne voulez pas casser, vous pouvez garder vos surcharges existantes, mais ne les appeler la nouvelle un:

public void DoSmtg(string a, int x, double d, bool doIt) 
    => DoSmtg(new SomethingParameters() 
       { 
        A = a, 
        X = x, 
        D = d, 
        DoIt = doIt 
       }); 

Answer 3

Je préfère la possibilité d'utiliser une catégorie distincte pour les paramètres. Mais peut-être, comme vous l'avez dit, vous appelez déjà la méthode à plusieurs endroits et vous ne voulez pas la modifier.
Dans ce cas, vous pouvez ajouter un paramètre optionnel:

public void DoSmtg(string a, int x, double d, bool doIt = false) 

Rien ne doit changer nulle part ailleurs, sauf que vous pouvez fournir le paramètre si vous le désirez.

Si vous vous trouvez en train de faire cela, j'écrirais toujours la surcharge supplémentaire en utilisant une classe de toute façon et commencer à l'utiliser. Les paramètres facultatifs peuvent également devenir désordonnés s'ils sont trop nombreux.