[harakiri888]

Mam zaimplemetowanego command patterna i potrzebuję przed egzekucją komendy zrobić jej walidację (stanu) i nie ogarniam jakiego tu wzorca użyć. Czy po prostu zrobić coś na zasadzie:

if(command.CanExecute()) command.Execute(); else return false;

#programowanie #programista15k #dotnet

[Saly]

@harakiri888: walidacja w czasie tworzenia. Programowanie nie powinno być polem minowym, a tworzenie niepoprawnych obiektów w tym nie pomaga

[villager]

@harakiri888: niech komenda sama zweryfikuje swój stan w Execute() i ew. rzuci wyjątkiem. jak potrzebujesz tego boola zwrócić to zrób to w ramach try-catcha

[harakiri888]

@Saly: tak nie mogę zrobić.Bo wykonanie komendy jest zależne od aktualnego stanu, a klient ma tylko jego kopię. Serwer jest source of truth.

@villager: teraz mam podobnie powiedzmy. Stworzenie ogólnej metody SafeExecution() z powyższą implementacją najlepsza opcja jak na razie ᶘᵒᴥᵒᶅ

[Saly]

@harakiri888: to nie brzmi jak walidacja stanu, tylko zwykły błąd/wyjątek wywołania metody. Tak czy owak fajnie jakbyś nakreślił bardziej kontekst, bo zaczyna być ciekawie

[harakiri888]

@Saly: https://pastebin.com/H3PzxknK

PoC z tymi rulami, przeszedłem z bloku ifowego ale do niego wrócę najprawdopodobniej albo po prostu przeitereuję po funkcjach. 24 - 27 linia. Czyli będądwie metody CanExecute() => bool i Execute() => void.

[Saly]

@harakiri888: ok, wygląda dobrze. Użyłeś niefortunnych słów: jako potrzebuję przed egzekucją komendy zrobić jej walidację (stanu) chyba każdy rozumie walidację stanu komendy (jej), a tutaj walidujesz argumenty. Co do CanExecute() to jestem przeciwny, bo utrudnia to używanie kodu. Co jak ktoś zawoła Execute() bez CanExecute()? I tak trzeba obsłużyć taką sytuację. Jak komendy się nie da wykonać to niech zwraca tego boola/rzuca wyjątek/zwraca specialny typ.

[t.....k]

@harakiri888 czasami jeden if-statement lub wyrzucenie wyjątku jest lepszy niż kolejnych kilka klas lub funkcji aby usatysfakcjonować wzorzec

[harakiri888]

@Saly: sorry, ale już się spać kładłem i mnie natchnęło, żeby na mirko napisać :P

[harakiri888]

@tlaziuk: @Saly: https://pastebin.com/3MNv1SrZ zdecydowałem się na takie rozwiązanie jakbyście byli zainteresowani.