Sollte eine C# -Klasse Instanzen von sich selbst generieren?

Question

Ich habe eine Klasse, die einen CallRate-Typ definiert. Ich muss die Möglichkeit hinzufügen, mehrere Instanzen meiner Klasse zu erstellen, indem ich die Daten aus einer Datei lese.

Ich habe meiner Klasse CallRate eine statische Methode hinzugefügt, die eine List<CallRate> zurückgibt. Ist es in Ordnung, wenn eine Klasse neue Instanzen von sich selbst generiert, indem sie einen eigenen Konstruktor aufruft? Es funktioniert, ich frage mich nur, ob es das Richtige ist.

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

Answer 1

Es ist vollkommen in Ordnung, Objekte aus der statischen Methode zu erhalten.

z.B.

Einer der dot net Bibliotheken macht das Gleiche wie du,

XmlReadrer reader = XmlReader.Create(filepathString); 

Answer 2

Sicher ist das in Ordnung, sogar in einigen Fällen gefördert. Es gibt mehrere design patterns that deal with object creation, und ein paar von ihnen tun genau das, was Sie beschreiben.

Answer 3

Scheint mir gut. In anderen Sprachen würden Sie wahrscheinlich eine Funktion schreiben, aber in einer Sprache wie C# übernehmen statische Methoden diese Rolle.

Answer 4

Es ist in Ordnung. Was Sie gerade erstellt haben, ist so etwas wie eine einfache Fabrikmethode. Sie haben eine statische Methode, die eine gültige Instanz eines Typs erstellt. Eigentlich muss Ihre Methode nicht einmal statisch sein und Sie haben immer noch einen gültigen Code. Es gibt ein Entwurfsmuster (Prototyp), das ein neues gültiges Objekt aus einem vorhandenen Objekt erstellt. Siehe Details unter http://www.dofactory.com/Patterns/PatternPrototype.aspx.

Answer 5

Sicher, für einfache Analyse (oder ähnliche) Szenarien - ich eigentlich bevorzugen die factory method Teil der Klasse sein. Ja - es bricht SRP, aber es erfüllt KISS - so nenne ich es einen Nettogewinn. Bei größeren Apps oder komplizierteren Parseroutinen ist es sinnvoller, wenn es sich um eine externe Factory-Klasse handelt. Für Ihren speziellen Fall würde ich wahrscheinlich eine Methode bevorzugen, die eine IEnumerable <Zeichenkette> anstelle eines Dateinamens - das würde Ihnen immer noch die Parsing-Logik, sondern ermöglichen einfache Unit-Tests und "Wiederverwendung". Der Aufrufer kann die Datei problemlos in ein IEnumerable umbrechen.

Answer 6

Fabrik Methoden sind oft ein gutes Design. Als ich sie in C# zu schreiben, ich nenne sie 'Neu', so dass:

new MyClass() 

wird

MyClass.New() 

Trivial es so implementiert ist:

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

Meistens mache ich das, wenn Es gibt zusätzliche Bedingungen darüber, ob die Klasse tatsächlich erstellt werden soll oder ob nur null zurückgegeben wird oder ob MyClass oder etwas davon zurückgegeben wird.

Answer 7

Ich benutze manchmal öffentliche statische Methoden als eine Alternative zum Überladen von Konstruktoren.

Insbesondere in Situationen, in denen es nicht schön ist, sich allein auf Parametertypen zu verlassen, um anzuzeigen, welche Art von Objektkonstruktion beabsichtigt ist.

Answer 8

Ich bin ein Fan von statischen Methoden, die Instanzen zurückgeben, wie oft vorgeschlagen, oben.

@Paul: Vergiss nicht den obigen Kommentar anzukreuzen, was du findest ist die beste Antwort.

Answer 9

mag nur darauf hinweisen „neue Instanzen von sich selbst erzeugen, indem sie einen seiner eigenen Konstrukteuren Aufruf“

Es ist nicht von der Konstruktor, es von der statischen Methode.

Answer 10

Ich verwende das im Allgemeinen, wenn ich sofortige Implementierungen einer Klasse brauche. Zum Beispiel

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; } 
    } 

Und mit diesem, ich brauche nicht zu erinnern oder schreiben Konstruktor Parameter in meinem Code.

Car car = Car.RedExpensiveCar; 

Answer 11

Ich benutze oft dieses Muster, wenn ich die Gültigkeit der Parameter überprüfen muss. Es wird dringend davon abgeraten, eine Ausnahme von einem Konstruktor zu werfen. Es ist nicht so schlecht von einer Factory-Methode, oder Sie können wählen, null zurückzugeben.

Answer 12

Es ist durchaus akzeptabel, dies zu tun. Wenn ich das tue, mache ich normalerweise die echten Konstruktoren für die Klasse private, so dass klar ist, dass die nur Weg, Instanzen zu konstruieren, durch die statische Methode ist.

Dies ist sehr nützlich in Fällen, in denen "Konstruktion" möglicherweise nicht immer eine neue Instanz zurückgibt. Beispielsweise möchten Sie möglicherweise ein zuvor zwischengespeichertes Objekt zurückgeben.