Quando controllare lo stato di un oggetto, OOP generale, esempio in C#

Question

Non sono abbastanza sicuro di come pronunciare questa domanda in una frase, quindi ho avuto difficoltà a cercare i post precedenti. Questo succede spesso per me e mi piacerebbe avere un consenso su come affrontarlo.

Supponiamo di avere due classi, ExampleClass e ExampleClassManager. ExampleClass ha un metodo Update(Data data) che viene chiamato da ExampleClassManager. Tuttavia, ExampleClass può essere in uno dei due stati, e nello stato Enabled si vuole elaborare il data passata ad essa in Update, e nello stato disabilitato non fa nulla con il data a tutti.

Dovrei controllare per lo stato nel ExampleClassManager e non superato il data affatto se è disabilitato, o forse dovrei passare il data indipendentemente e ignorarlo all'interno ExampleClass?

Ecco un esempio di codice nel caso in cui non lo articolassi chiaramente.

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

o

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

Dato che dipende da una proprietà di ExampleClass, sceglierei l'opzione 1 e assegno entro ExampleClass.Update. In caso contrario, qualsiasi oggetto con accesso a un oggetto ExampleClass potrebbe chiamare Updateindipendentemente dallo dello stato. Controllando all'interno del metodo Update, ci si assicura che procederà solo se l'oggetto è abilitato. La domanda qui è chi può cambiare lo stato dell'oggetto?

Vedere la Law of Demeter:

Ogni unità deve essere a conoscenza solo limitata su altre unità: solo le unità "da vicino" correlate a l'unità corrente.

Answer 2

"Quanto deve essere gestita la classe del mio manager?" La tua opzione 2 è l'approccio "micromanagement" - che il Manager dovrebbe dettare ogni piccolo dettaglio alla ExampleClass su cosa dovrebbe fare e come.

In genere, questa è una programmazione più basata sulla struttura rispetto alla programmazione orientata agli oggetti. Mi aspetto che il tuo manager dica al tuo oggetto "vai a fare questo" e non preoccuparti dei dettagli. ExampleClass dovrebbe incapsulare tutto il suo comportamento internamente e non costringere il gestore a preoccuparsi di ciò.

mio voto è per l'opzione 1, con l'avvertenza che si tratta di principi generali, e sarai sempre in grado di trovare qualche caso in cui si desidera l'opzione 2.

Una domanda per chiedere - c'è un motivo che ExampleClass.Enabled.get è pubblico, a parte lasciare che il gestore controlli? Se questo è l'unico motivo per cui è pubblico, allora è un'implementazione interna e il gettor dovrebbe essere privato. Se hai un sacco di cose da guardare in questo campo, ciò potrebbe implicare che il tuo flusso di lavoro opzione 2 è più naturale per questa classe.

Answer 3

Sceglierei l'opzione 2, classManager dovrebbe gestire gli oggetti, dovrebbe sapere se attivare o meno l'aggiornamento. Il metodo di aggiornamento del singolo oggetto dovrebbe fare ciò che dice (aggiornare l'oggetto e non fare nulla).

@caerolus: Non posso commentare ancora ... in ogni caso nessuno dei due l'esempio dato ha rotto la legge di Demetra

Comunque penso che questa è una domanda un po 'personale scelta, forse è più adatto per lo scambio di stack?

Answer 4

Supponiamo di avere due oggetti, Azienda e Dipendente. Hai l'assegno dell'azienda quando il dipendente ha fame o hai il dipendente che controlla se ha fame? Sembra che tu voglia che il dipendente controlli se ha fame, giusto? Ma cosa succede se l'azienda vuole ordinare cibo per tutti i dipendenti, quindi l'azienda deve verificare se il dipendente ha fame.

Quindi il mio punto è, dipende dal design e dal contesto di quando controllare e chi sta controllando e perché. Nel tuo esempio, direi che non ha importanza perché il contesto non è impostato.