¿Debería una clase C# generar instancias de sí mismo?

Question

Tengo una clase que define un tipo de CallRate. Necesito agregar la capacidad de crear varias instancias de mi clase leyendo los datos de un archivo.

Agregué un método estático a mi clase CallRate que devuelve List<CallRate>. ¿Está bien para una clase generar nuevas instancias de sí mismo llamando a uno de sus propios constructores? Funciona, solo me pregunto si es lo correcto.

List<CallRates> cr = CallRates.ProcessCallsFile(file); 

Answer 1

Está perfectamente bien obtener objetos propios del método estático.

p. Ej.

Una de las bibliotecas de la red del punto hace lo mismo como lo hizo,

XmlReadrer reader = XmlReader.Create(filepathString); 

Answer 2

Claro que está bien, incluso fomentado en algunos casos. Existen varios design patterns that deal with object creation, y algunos de ellos hacen exactamente lo que usted describe.

Answer 3

Me parece bien. En otros idiomas probablemente escribirías una función, pero en un lenguaje como C#, los métodos estáticos toman ese rol.

Answer 4

Está bien. Lo que acaba de crear es algo así como un simple método de fábrica. Tiene un método estático que crea una instancia válida de un tipo. En realidad, su método ni siquiera tiene que ser estático y aún tiene un código válido. Hay un patrón de diseño (prototipo) que crea un nuevo objeto válido a partir de un objeto existente. Ver detalles en http://www.dofactory.com/Patterns/PatternPrototype.aspx.

Answer 5

Claro, para escenarios simples de análisis (o similares) - De hecho, prefiero el factory method sé parte de la clase. Sí, rompe SRP, pero cumple KISS, así que lo llamo ganancia neta. Para aplicaciones más grandes, o rutinas de análisis más complicadas, tiene más sentido que sea una clase de fábrica externa.

para su caso particular, probablemente preferiría un método que tomó en un IEnumerable <cadena> en lugar de un nombre de archivo - que todavía le daría la lógica de análisis, pero permitir pruebas unitarias fáciles y "reutilización". La persona que llama puede envolver el archivo en un IEnumerable con bastante facilidad.

Answer 6

Los métodos de fábrica a menudo son un buen diseño. Cuando las escribo en C#, yo los llamo 'Nuevo', de modo que:

new MyClass() 

convierte

MyClass.New() 

Trivialmente se implementa como esto:

class MyClass 
{ 
    public static MyClass New() 
    { 
     return new MyClass(); 
    } 
} 

Parcialmente hago esto cuando hay condiciones adicionales sobre si realmente crear la clase o simplemente devolver null, o si devolver MyClass o algo derivado de él.

Answer 7

A veces uso métodos públicos estáticos como alternativa a la sobrecarga del constructor.

Especialmente en situaciones en las que no es bueno depender únicamente de los tipos de parámetros para indicar qué tipo de construcción de objeto se pretende.

Answer 8

Soy un fanático de tener métodos estáticos para devolver instancias, como se sugirió muchas veces, más arriba.

@Paul: no olvide marcar el comentario anterior, que encuentra que es la mejor respuesta.

Answer 9

al igual que señalar "generar nuevas instancias de sí mismo llamando a uno de sus propios constructores"

No es por el constructor, es a partir del método estático.

Answer 10

Generalmente uso esto cuando necesito implementaciones instantáneas de una clase. Por ejemplo

public class Car 
    { 
     public static Car RedExpensiveCar = new Car("Red", 250000); 

     public Car() 
     { 

     } 

     public Car(string color, int price) 
     { 
      Color = color; 
      Price = price; 
     } 

     public string Color { get; set; } 
     public int Price { get; set; } 
    } 

Y con esto, no necesito recordar ni escribir parámetros de constructor en mi código.

Car car = Car.RedExpensiveCar; 

Answer 11

A menudo uso este patrón cuando necesito verificar la validez de los parámetros. Se desaconseja encarecidamente lanzar una excepción desde un constructor. No es tan malo desde un método de fábrica, o puede elegir devolver nulo.

Answer 12

Es perfectamente aceptable hacerlo. Cuando lo hago, normalmente hago que los constructores reales para la clase sean privados, por lo que está claro que la forma solo de construir instancias es a través del método estático.

Esto es muy útil en los casos en que "construcción" no siempre devuelve una nueva instancia. Por ejemplo, es posible que desee devolver un objeto previamente almacenado en caché.