Скажите, не спрашивайте и не делитесь состоянием между задачами

Question

В этом простом примере (конечно, моя проблема с реальным миром сложнее, хотя основы одинаковы), как я могу принудить вас не спрашивать max? Id нравится максимизировать, не спрашивайте в методе процесса, в его текущем состоянии его сложнее насмехаться и тестировать.

public class Processor 
{ 
    public void Process() 
    { 
     var data = new Task1().DoStuff(); 
     new Task2().DoStuffToData(data); 
    } 
} 

public class Task1 
{ 
    public Data DoStuff(){return new Data();} 
} 

public class Data {} 

public class Task2 
{ 
    public void DoStuffToData(Data data){} 
} 

EDIT: Обновленный образец более DIish

public class Processor 
    { 
public Processor(ITask1 task1, ITask2 task) {...} 
     public void Process() 
     { 
      var data = task1.DoStuff(); 
      task2.DoStuffToData(data); 
     } 
    } 

Answer 1

Этот код не кажется слишком плохим с точки зрения tell-don't-ask.

Tell-don't-ask в основном означает, что вы не должны запрашивать объект об этом состоянии, принимать решение на основе его состояния и , а затем сообщить тому же объекту, что делать до. Если объект имеет всю необходимую ему информацию, он должен сам решить.

Вы получаете данные от task1 и потребляете эту информацию с помощью task2, не сообщая task1 что делать. Поэтому я бы сказал, что это нормально с точки зрения не-спросите.

Что касается Processor.Process: там ничего плохого. Клиент будет звонить myProcessor.Process и тем самым сообщает ему, что делать, не спрашивая.

Скажите-don't-ask выглядит нормально, может быть, есть что-то еще о коде, который вам не нравится? Например, вы можете рассмотреть вопрос о помещении DoStuffToData в класс Data, тем самым сочетая состояние и поведение. Но зависит ли это от модели и ее контекста.

Answer 2

Одним из вариантов является использование Depenancy Injection (DI). Однако убедитесь, что это не слишком усложняет ваш код. DI может быть полезен для unit testing and mocking, но это также может привести к слишком маленьким классам.