Biblioteka maszyn stanu bezstanowego - właściwy sposób na strukturę?

Question

W jaki sposób ludzie budują swój kod podczas korzystania z biblioteki bezpaństwowej C#?

https://github.com/nblumhardt/stateless

Jestem szczególnie zainteresowany w jaki sposób ten wiąże się z wstrzyknięto zależności i prawidłowe podejście do obowiązków i warstw poprawnie.

Moja obecna struktura obejmuje następujące elementy:

public class AccountWf 
{ 
    private readonly AspNetUser aspNetUser; 

    private enum State { Unverified, VerificationRequestSent, Verfied, Registered } 
    private enum Trigger { VerificationRequest, VerificationComplete, RegistrationComplete } 

    private readonly StateMachine<State, Trigger> machine; 

    public AccountWf(AspNetUser aspNetUser, AccountWfService userAccountWfService) 
    { 
     this.aspNetUser = aspNetUser; 

     if (aspNetUser.WorkflowState == null) 
     { 
      aspNetUser.WorkflowState = State.Unverified.ToString(); 
     } 

     machine = new StateMachine<State, Trigger>(
     () => (State)Enum.Parse(typeof(State), aspNetUser.WorkflowState), 
     s => aspNetUser.WorkflowState = s.ToString() 
     ); 

     machine.Configure(State.Unverified) 
     .Permit(Trigger.VerificationRequest, State.VerificationRequestSent); 

     machine.Configure(State.VerificationRequestSent) 
     .OnEntry(() => userAccountWfService.SendVerificationRequest(aspNetUser)) 
     .PermitReentry(Trigger.VerificationRequest) 
     .Permit(Trigger.VerificationComplete, State.Verfied); 

     machine.Configure(State.Verfied) 
     .Permit(Trigger.RegistrationComplete, State.Registered); 

    } 

    public void VerificationRequest() 
    { 
     machine.Fire(Trigger.VerificationRequest); 
    } 

    public void VerificationComplete() 
    { 
     machine.Fire(Trigger.VerificationComplete); 
    } 

    public void RegistrationComplete() 
    { 
     machine.Fire(Trigger.RegistrationComplete); 
    } 

} 

powinniśmy realizować wszystkie procesy (wezwanie do usług) w haku OnEntry lub wdrożyć procesy na zewnątrz po przejściu stan został zweryfikowany, że jest może się odbyć? Zastanawiam się, jak zrobić zarządzanie transakcjami, jeśli tak.

Podejrzewam, że to, czego szukam, to najlepsze wskazówki od tych, którzy już wdrożyli coś za pomocą bezpaństwowców i jak podejść do struktury kodu.

Answer 1

Przed omówieniem struktury samą parę Uwagi:

• OnEntry czynności wykonywane są tylko jeśli spust został z powodzeniem spalać.

• Uruchomione wyzwalacze, które nie są dozwolone w bieżącym stanie, wyrzucą InvalidOperationException. Rozważ zastąpienie OnUnhandledTrigger, jeśli nie spodziewasz się wyjątku (odkryłem, że logowanie nieobsługiwanych wyzwalaczy jest dobrym sposobem na znalezienie błędów w logice).

Moja zasada dla OnEntry/OnExit strukturyzacji jest, że wszelkie stworzenie i logika zostaną umieszczone OnEntry oraz wszelkie wymagane oczyszczania odbywa OnExit.

W twoim przypadku, biorąc pod uwagę, że używasz wstrzykniętych zależności (i zakładając, że nie przejmujesz ich własności, tj. Ktoś inny będzie zarządzał ich cyklem życia), możesz umieścić całą swoją logikę OnEntry.

Mając to na uwadze, sposób, w jaki twoja maszyna stanowa jest obecnie zorganizowana, jest doskonale w porządku.

Jeszcze jedna uwaga, należy pamiętać, że wyzwalanie wyzwalania z tego samego wątku, który przesuwa automat stanowy i wykonuje logikę automatu stanów, może i będzie prowadzić do wyjątków przepływu stackoverflow (patrz here, jak rozwiązać problem z automatycznym wyprzedzeniem).