Heavy Constructors oder verwenden Sie eine Methode

Question

Beide Methoden funktionieren für mich, aber ich bin mir nicht sicher, was die Empfehlung aus Sicht der "guten Praxis" wäre.

Ich habe eine Klasse, die verschiedene Steuerfunktionen innerhalb meiner Bibliothek durchführt, also alle Arten von Objekten und Eigenschaften initialisieren muss.

Ist es in Ordnung, alle diese Logik in den Konstruktor für die Klasse zu setzen, oder sollte ich es in eine "Initialisieren" -Methode setzen.

public MyClass() 
{ 
    mSubObjectA = new mSubObjectA(); 
    mSubObjectA.DoStuff(); 
    mSubObjectA.DoMoreStuff(); 

    mSubObjectB = new mSubObjectB(); 
    mSubObjectC = new mSubObjectC(); 

    if (something) 
    { 
     DoStuff(); 
    } 
    else 
    { 
     MagicHappens(); 
    } 
} 

Answer 1

Es hängt davon ab, wie viele Konstruktoren Sie denken, dass Sie haben werden. Code nicht wiederholen. Wenn all diese Schritte nur in Ihrem einen Konstruktor passieren, ist das kein Problem. Wenn Sie etwas ausgefalleneres suchen, ordnen Sie Ihren Code intelligent an.

Answer 2

Es gibt eine Alternative zu beiden - verwenden, um eine factory Klasse oder factory method.

Das Konstruieren komplexer Objektdiagramme ist das, woran sich das Fabrikmuster orientiert - klingt wie eine sehr gute Anpassung an Ihre Situation.

Answer 3

Persönlich mag ich, dass ein Objekt vollständig betriebsbereit ist, wenn ich es erhalte. Auf der anderen Seite, eine Menge Arbeit im Konstruktor tun tut Geruch etwas.

Eine Alternative ist eine statische Factory-Methode (oder eine Fabrik Klasse ^*) zu schreiben, die die ganze Arbeit tun können, um es zu muss, bevor den Konstruktor aufrufen, und dann der Konstruktor selbst ziemlich einfach machen. Abhängig von Ihren Anforderungen können Sie den einfachen Konstruktor anzeigen oder nicht.

^* Eine Fabrik Klasse kann für eine bessere Testbarkeit machen, und ermöglicht potenziell verschiedene Fabrik-Implementierungen. Auf der anderen Seite, an diesem Punkt haben Sie eine ziemlich hohe Ebene der Abstraktion, die eine Ablenkung sein kann.