¿Es esto una mala práctica sobrecargar un método?

Question

que tenían un método como este que sus consumidores están llamando:

static public void DisplayOrderComments(param1, param2, param3, param4) 

Ahora añade una sobrecarga de esta manera:

static public void DisplayOrderComments(param1, param2, param3, param4, param5) 
{ 
    DisplayOrderComments(param1, param2, param3, param4); 
    param5.Foo(); 
} 

¿Es una mala práctica? ¿Hay mejores formas de hacerlo?

Answer 1

Esto es absolutamente correcto: mantiene el código DRY y evita la duplicación innecesaria.

No solo no es una mala práctica, es una buena práctica de .

---

Si está usando C# 4.0 y superior (VS 2010+), se puede utilizar un optional argument para su param5 en lugar de sobrecarga, como menciona Mikey Mouse en this answer.

Answer 2

Buena pregunta.

Yo diría que no, es una sobrecarga normal. pero voy a cambiarlo como tal (ha puesto en práctica en la que tiene la mayoría de los parámetros):

static public void DisplayOrderComments(param1, param2, param3, param4) 
{ 
    DisplayOrderComments(param1, param2, param3, param4, null); 
} 

static public void DisplayOrderComments(param1, param2, param3, param4, param5) 
{ 
    ... // do the work 
    if(param5!=null) 
    param5.Foo(); 
} 

Answer 3

No, en absoluto. Esto es perfectamente aceptable, pero hay dos problemas.

1. param5 se puede pasar como nulo: es posible que desee escribir un código para verificar esa condición y hacer lo correcto.
2. Esto le impedirá usar parámetros opcionales más adelante. Pero es posible que no te importe eso.

Por cierto codereview.stackexchange.com podría ser un lugar mejor para este tipo de pregunta.

Answer 4

Sí, no diría que es malo, pero si está utilizando C# 4.0, le recomendaría hacer el último parámetro opcional.

Usted puede leer todo acerca de ellos aquí http://msdn.microsoft.com/en-us/library/dd264739.aspx

Answer 5

Ésta es también la práctica bienes comunes para proporcionar más fácil de usar API para una interfaz. Mantiene interfaz de pequeña con el método de múltiples parámetros, sino que utiliza múltiples métodos de extensiones (a veces con el mismo nombre) que proporcionan más fácil de usar API de interfaz sin contaminar:

interface ILog 
{ 
    void Log(string message, int someNumber, float anotherParam, object moreParams); 
} 

public static class LogExtensions 
{ 
    public void Log(ILog this log, message) 
    { 
     log.Log(message, 42, 0, null); 
    } 
// more methods using ILog.Log like LogFormat that takes format string... 
} 

Answer 6

diría que depende de lo que DisplayOrderComments y param5.Foo () hace. Si hace param.Foo() para los parámetros 1 a 4, entonces absolutamente, si no lo hace, entonces está haciendo algo extra, que posiblemente valga la pena su propio nombre.

¿Está cambiando lo que DisplayOrerComments hace o los efectos secundarios, o solo las condiciones bajo las cuales se ejecuta? Si está presentando nuevos efectos secundarios, puede valer la pena tenerlos como su función/nombre que llama a DisplayOrderComments.