Wann ich den Zustand eines Objekts überprüfen soll, allgemeine OOP, Beispiel in C#

Question

Ich bin nicht ganz sicher, wie man diese Frage in einem Satz formuliert, so dass ich Schwierigkeiten hatte, vorherige Beiträge zu suchen. Dies kommt häufig für mich und ich würde gerne einen Konsens darüber, wie man es angehen.

Angenommen, Sie haben zwei Klassen, ExampleClass und ExampleClassManager. ExampleClass hat eine Update(Data data) Methode, die von ExampleClassManager aufgerufen wird. es will ExampleClass kann jedoch in einem von zwei Zuständen, und im Enabled Zustand sein, die data, um es verarbeiten in Update vergangen, und im Behinderten Zustand ist es nicht mit den data überhaupt nichts tun.

Soll ich für den Staat in der ExampleClassManager werden überprüft und nicht die data überhaupt vorbei, wenn es deaktiviert ist, oder sollte ich die data unabhängig und ignorieren es innerhalb ExampleClass passieren?

Hier ist ein Codebeispiel, falls ich es nicht klar artikulieren würde.

public class ExampleClass { 
    public bool Enabled { 
     get; 
     set; 
    } 

    public void Update(Data data) { 
     if(Enabled) { 
      //do stuff with data 
     } 
    } 
} 

public class ExampleClassManager { 
    private List<ExampleClass> exampleClassList=new List<ExampleClass>(); 

    public void UpdateList() { 
     foreach(ExampleClass exampleClass in exampleClassList) { 
      exampleClass.Update(data); 
     } 
    } 
} 

oder

public class ExampleClass { 
    public bool Enabled { 
     get; 
     set; 
    } 

    public void Update(Data data) { 
     //do stuff with data 
    } 
} 

public class ExampleClassManager { 
    private List<ExampleClass> exampleClassList=new List<ExampleClass>(); 

    public void UpdateList() { 
     foreach(ExampleClass exampleClass in exampleClassList) { 
      if(exampleClass.Enabled) { 
       exampleClass.Update(data); 
      } 
     } 
    } 
} 

Answer 1

Da es hängt von einer Eigenschaft von ExampleClass, würde ich die Option 1 und überprüfen innerhalb ExampleClass.Update wählen. Andernfalls könnte jedes Objekt mit Zugriff auf ein ExampleClass Objekt Updateungeachtet des Staates aufrufen. Indem Sie innerhalb der Update Methode überprüfen, stellen Sie sicher, dass es nur weitergeht, wenn das Objekt aktiviert ist. Die Frage ist, wer kann den Status des Objekts ändern?

Siehe Law of Demeter: nur für die aktuelle Einheit Einheiten „eng“ bezogen werden:

Jede Einheit nur begrenzte Kenntnisse über andere Einheiten haben sollte.

Answer 2

"Wie viel Management sollte meine Managerklasse machen?" Ihre Option 2 ist der "Mikromanagement" -Ansatz - dass der Manager der ExampleClass jedes kleine Detail vorschreiben sollte, was und wie es zu tun ist.

In der Regel ist das strukturbasierte Programmierung als objektorientierte Programmierung. Ich würde erwarten, dass Ihr Vorgesetzter Ihrem Objekt sagt "Mach das" und mach dir keine Sorgen um die Details. Die ExampleClass sollte das gesamte Verhalten intern verkapseln und den Manager nicht zwingen, sich darum zu kümmern.

Meine Stimme ist für die Option 1, mit dem Vorbehalt, dass diese allgemeinen Prinzipien, und Sie werden immer in der Lage sein, etwas Fall zu finden, wo Sie Option 2.

eine Frage stellen wollen - gibt es einen Grund dass ExampleClass.Enabled.get öffentlich ist, abgesehen davon, dass der Manager dies überprüft? Wenn das der einzige Grund ist, warum es öffentlich ist, dann ist es eine interne Implementierung, und der gettor sollte privat sein. Wenn Sie viele Dinge in diesem Bereich suchen, könnte dies bedeuten, dass Ihr Option 2-Workflow für diese Klasse natürlicher ist.

Answer 3

Ich würde Option 2 wählen, classManager sollte die Objekte verwalten, sollte er wissen, ob das Update feuern oder nicht. Die Update-Methode des einzelnen Objekts sollte tun, was es sagt (das Objekt aktualisieren und nichts tun).

@caerolus: Ich kann noch nichts sagen ... sowieso weder von dem gegebenen Beispiel des Gesetz des demeter brach

Auf jeden Fall denke ich, das eine etwas persönliche Wahl Frage ist, vielleicht für Stapelaustausch mehr geeignet, es ist?

Answer 4

Nehmen wir an, Sie haben zwei Objekte, Firma und Mitarbeiter. Haben Sie die Firmenkontrolle, wenn der Mitarbeiter hungrig ist, oder haben Sie die Mitarbeiterkontrolle, wenn er hungrig ist? Klingt so, als ob der Angestellte überprüfen soll, ob er hungrig ist, oder? Aber wenn das Unternehmen für alle Mitarbeiter Essen bestellen möchte, muss das Unternehmen prüfen, ob der Mitarbeiter hungrig ist.

Also mein Punkt ist, es hängt vom Design und dem Kontext ab, wann zu überprüfen ist und wer überprüft und warum. In Ihrem Beispiel würde ich sagen, dass es nicht wichtig ist, weil der Kontext nicht festgelegt ist.