Partage d'une variable entre une classe et son membre

Question

Si vous avez une classe qui contient une variable d'état et deux classes de membres qui doivent y accéder et fonctionner de manière asynchrone. Quelle est la meilleure façon de mettre en œuvre cela?

Un exemple

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 
} 

Ces classes fonctionnent de manière asynchrone. Les deux classes DeliveryMan et Supplier doivent pouvoir lire/écrire l'état. DeliveryMan pousse l'état du restaurant et le fournisseur est à l'écoute du statut de son fournisseur.

Y a-t-il une meilleure façon de concevoir ceci ou une façon de l'implémenter avec un couplage minimal sans donner à DeliveryMan ou à un fournisseur une référence à son propriétaire Restaurant.

Answer 1

Vous pouvez peut-être créer des événements sur les classes DeliveryMan et Supplier qui sont déclenchées lorsque l'état doit être mis à jour. Le restaurant peut s'abonner à ces événements et mettre à jour son propre état en conséquence lorsque le (s) gestionnaire (s) d'événement est appelé (s).

Answer 2

Eh bien, je passerais l'état en tant que paramètre constructeur à vos deux classes internes et en considérant qu'il s'agit d'un type de référence, il peut également être modifié.

Answer 3

Je voudrais sortir l'état de la classe de restaurant et faire une classe StateManager qui était un singleton ou une usine pour le reste des autres classes. Il est difficile de donner une réponse plus complète puisque votre conception OO ne donne pas beaucoup à continuer.

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

Answer 4

Je voudrais créer une classe Order qui contient les informations dont vous avez besoin dans une autre classe. Utilisez également une file d'attente que vous vérifiez sur un événement Timer. Lorsque vous supprimez une commande, regardez Order.State (par exemple). Placez la file d'attente dans une classe statique publique avec les méthodes Enqueue et Dequeue.

Lorsque l'événement du temporisateur DeliveryMan se déclenche, réinitialisez l'ordre.

Vous mentionnez que tout est asynchrone, donc vous pouvez vérifier ConcurrentQueue. Étant donné que le fournisseur attend une notification, vous pouvez utiliser IObserver/IObservable pour envoyer un message de flux au fournisseur avec un objet de commande sérialisé ...

Juste quelques réflexions qui pourraient aider.

Answer 5

Si RestaurantState est ou peut être transformé en un objet qui contient l'état plutôt que l'état lui-même, alors vous pouvez faire comme réponse @ Davide et le transmettre au constructeur.

Toutefois, si c'est un type de valeur comme un enum alors je pense que event s sont la voie à suivre.

DeliveryMan déclenche un événement avec le nouvel état, que Restaurant écoute et met à jour son état interne.

Restaurant peut ensuite appeler une méthode StateChanged ou quelque chose de similaire sur Supplier lorsque l'état change. Ou Supplier peut lever un event avec un RestaurantStateEventArgs spécial ou quelque chose que le Restaurant peut écouter et remplir les arguments d'événement avec l'état.Cependant, selon le cas d'utilisation, il n'est peut-être pas terrible d'avoir simplement une référence à Restaurant même si elle est fortement couplée.

Modifier: En fait, si DeliveryMan et Supplier besoin d'accéder à RestaurantState alors ils sont déjà un peu liés à des restaurants, sauf si vous avez un type plus générique de « état » que RestaurantState ils sont déjà couplés.

Parfois, il est bon de faire un pas en arrière et voir si

a) le découplage est en fait utile dans un scénario particulier et
b) si ce que vous faites est en fait assez découplé pour être utile.

Dans ce scénario, vous ne pouviez toujours pas réutiliser DeliveryMan et Supplier pour, par exemple, un magasin de meubles.

En sidenote:

OPEN, 
CLOSED, 
LOW_ON_SUPPLIES 

Ce ne sont pas vraiment les meilleurs choix pour un ENUM, car ils ne sont pas mutuellement exclusifs. il peut être préférable si elle était une classe si:

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

Dans ce cas, la réponse de @ Davide pour passer RestaurantState dans le constructeur de DeliveryMan et Supplier fonctionne bien.