Compartir una variable entre una clase y su miembro

Question

Si tiene una clase que contiene una variable de estado y dos clases miembro que necesitan acceso y funcionan de forma asíncrona. ¿Cuál es la mejor manera de implementar esto?

Un ejemplo

public enum RestaurantState 
{ 
    BREAKFAST, 
    LUNCH, 
    DINNER 
} 

public class Restaurant 
{ 
    //Below need access to state 
    private DeliveryMan pizzaDriver ; 
    private Supplier butcherShop ; 

    internal RestaurantState state ; 
} 

public DeliveryMan 
{ 
    //Uses a System.Timers.Timer 
    //Wakes up and does work every a minute 
    //Needs to inform state of restaurant 
} 

public Supplier 
{ 
    //Waits and listens for requests to accept deliveries 
    //If suppliers run out we need to change the restaurant state based on our own current state 
} 

Estas clases operan de forma asíncrona. Ambas clases, DeliveryMan y Supplier, necesitan poder leer/escribir el estado. DeliveryMan resalta el estado del restaurante y el Proveedor escucha el estado de su proveedor.

Existe una mejor manera de diseñar esto o una forma de implementarlo con un acoplamiento mínimo sin dar a DeliveryMan o Proveedor una referencia a su propietario Restaurant.

Answer 1

Quizás pueda crear eventos en las clases DeliveryMan y Supplier que se activan cuando el estado necesita actualizarse. El Restaurante puede suscribirse a estos eventos y actualizar su propio estado en consecuencia cuando se invocan los controladores de eventos.

Answer 2

Bueno, pasaría el estado como parámetro constructor a sus dos clases internas y, dado que es un tipo de referencia, también se puede modificar.

Answer 3

Quitaría el estado de la clase de restaurante y haría una clase StateManager que fuera un singleton o una fábrica para el resto de las otras clases. Es difícil dar una respuesta más completa ya que su diseño OO no da mucho para seguir.

var restaurant = new Restaurant(); 
var supplier = new Supplier(); 
StateManager.GetState(restaurant); 
StateManager.GetState(supplier); 

Answer 4

me gustaría crear una clase Order que contiene la información que necesita en otra clase. También use una Cola que verifique en un evento Timer. Cuando dequeue un pedido, mire Order.State (por ejemplo). Ponga la cola en una clase pública estática con los métodos Enqueue y Dequeue.

Cuando se dispara el evento del temporizador DeliveryMan, dequeue el pedido.

Menciona que todo es asíncrono, por lo que puede consultar ConcurrentQueue. Dado que el Proveedor espera la notificación, puede usar IObserver/IObservable para enviar un mensaje continuo al Proveedor con un objeto de pedido serializado ...

Solo algunas ideas que pueden ayudar.

Answer 5

Si RestaurantState es o puede convertirse en un objeto que contiene el estado en lugar del estado en sí, entonces puede hacer como respuesta de @Davide y pasarlo al constructor.

Sin embargo, si se trata de un tipo de valor como enum entonces creo que event s son el camino a seguir.

DeliveryMan provoca un evento con el nuevo estado, que Restaurant escucha y actualiza su estado interno.

Restaurant puede llamar a un método StateChanged o algo similar en Supplier cuando el estado cambia. O Supplier puede generar un event con un RestaurantStateEventArgs especial o algo que el Restaurant puede escuchar y rellenar los argumentos del evento con el estado.

Dependiendo del caso de uso, no es terrible tener una referencia a Restaurant a pesar de que se acopla fuertemente.

Editar: En realidad, si DeliveryMan y Supplier necesitan tener acceso a RestaurantState entonces son ya algo atados a restaurantes, por lo menos que tenga un tipo más genérico de "estado" que RestaurantState que ya están acoplados.

A veces es bueno dar un paso atrás y ver si

a) desacoplamiento es realmente útil en un escenario particular y
b) si lo que está haciendo en realidad es desacoplado suficiente como para ser útil.

En este escenario, todavía no podría volver a usar DeliveryMan y Supplier para decir una tienda de muebles.

Como comentario:

OPEN, 
CLOSED, 
LOW_ON_SUPPLIES 

Estos no son realmente las mejores opciones para una enumeración, ya que no todos son mutuamente excluyentes. puede ser mejor si se tratara de una clase para:

public class RestaurantState 
{ 
    public bool IsOpen { get; set; } 
    public bool IsLowOnSupplies { get; set; } 
} 

En ese caso, la respuesta de @ Davide para pasar RestaurantState en el constructor de DeliveryMan y Supplier funciona bien.