Zadanie rekrutacyjne i logika w API

Zadanie rekrutacyjne i logika w API
Krzysztof Pe
Krzysztof Pe
Krzysztof Pe
Krzysztof Pe
  • Rejestracja:prawie 7 lat
  • Ostatnio:około 3 lata
  • Postów:78
0

Witam szanowne grono forumowiczów,

Ktoś odkopał wczoraj temat z przed 2 lat (Zadanie rekrutacyjne) i akurat jeden post rzucił mi się w oczy. Chciałbym poruszyć kilka kwestii odnośnie proponowanego rozwiązania, dla czytelności przekleje cały kod tutaj.

Kopiuj
public class User
{
    public Guid Id { get; protected set; }
    private decimal _balance;
    public decimal getBalance => _balance;
    private static readonly object _lock = new object();

    public void Withdraw(decimal value)
    {
        lock (_lock)
        {
            if (value >= balance)
            {
                _balance -= value;
            }
        }
    }
}

public static class UserService
{
    public static ActionResult Perform_Withdraw_Logic(User user, decimal value)
    {
        if (value <= 0)
        {
            return new BadRequestResult("Amount of money to withdraw must be greater than 0");
        }
        if (value > user.getBalance)
        {
            return new BadRequestResult("You've not enough money");
        }

        user.Withdraw(value);

        return new OkRequestResult("Money has been withdrawn successfully");
    }
}

public class Controller
{
    public HomeController : Controller
    {
        [Authorize]
        public ActionResult WithdrawMoney([FromBody] WithdrawCommand)
        {
            var user = db.Single(c => c.Id == WithdrawCommand.Id);

            if (user == null)
            {
                return new BadRequestResult("User not found");
            }

            return UserService.Perform_Withdraw_Logic(user, WithdrawCommand.Amount);
        }
    }
}

Otóż zastanawia mnie kilka kwestii.

  1. Czy klasa User jest w tym kontekście modelem? Co tam robi metoda Withdraw? W podejściu ktore sam praktykuję modele zawierają propertisy, a logika operacji jakie mogą zostać wykonane na modelu znajduje się w serwisie.
  2. Jak zaimplementowany jest UserService? Czy nie powinien on implementować serwisu IUserService kóry byłby wstrzykiwany do Controllera i stamtąd wywoływane odpowiednie metody?
  3. Czy przez okres 2 lat coś się zmieniło w temacie architektury? Od momentu kiedy został utworzony temat Zadania rekrutacyjnego do dzisiaj.
    4. Jak w podejściu które sam praktykuje zabezpieczyć się przed dostępem wielu wątków do jednego zasobu? Czy proponowany poniżej sposób jest poprawny?
Kopiuj
public class UserService : IUserService
{
   private readonly DbContext _db;
   private object _lock = new object();


   public UserService(DbContext db)
   {
      _db = db;
   }
   public void Update(User user)
   {
      lock(_lock)
      {
         _db.Users.Update(user);
      }
   }
}
edytowany 8x, ostatnio: Krzysztof Pe
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Saalin
SA
Saalin
SA
  • Rejestracja:około 12 lat
  • Ostatnio:około 8 godzin
  • Postów:1430
2
  1. Jeśli klasa domenowa zawiera w sobie komplet informacji potrzebnych do przeprowadzenia sensownej operacji to po co serwis?
  2. UserService nie powinien zwracać ActionResult, wydzielenie interfejsu ma sens jak chcesz go zmockować w testach, kontener IoC może wstrzykiwać tez typy konkretne.
  3. Dlaczego w ogóle chcesz współdzielić DbContext między wątkami?
Zobacz pozostałe 7 komentarzy
WeiXiao
o ch*j, to ja pisałem? xD
WeiXiao
@Saalin: hej, po co mnie tu pingowałes? bo nie jestem pewien
SA
@WeiXiao: Twój kod komentowany, więc pingnąłem. Możesz popatrzeć z nostalgią.
WeiXiao
straszny ten kod, ale chociaż lock jest :D
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Terrored
TE
Terrored
TE
  • Rejestracja:ponad 7 lat
  • Ostatnio:4 dni
  • Postów:267
0
  1. Tutaj mimo, że jestem nowy w tematach DDD to przychodzi mi na myśl DDD właśnie. Według tego podejścia model nie ma być anemiczny tylko ma pokazywać zachowania danej klasy, jego domenę bazując na modelu a nie serwisach w Twoim rozumowaniu. Na pewno nie ma być zestawem publicznych geterów i seterów.

Przykładowo, chcąc zmienić hasło dla użytkownika nie będę odnosił się do propertiesa

Kopiuj
user.Password("123")

tylko do metody modyfikującej ten obiekt, która zawiera w sobie walidacje i jakieś inne dodatkowe logiki itp. To obiekt ma dbać o takie rzeczy jak swój stan.

Kopiuj
user.SetPassword("123")

Proszę o poprawę jeśli się mylę sam próbuję zrozumieć te tematy.

Zobacz pozostałe 8 komentarzy
Krzysztof Pe
Rozumiem, że rich model jest elementem (żeby nie pisać domeną :)) DDD, więc jeśli brak anemicznego modelu nie oznacza DDD to po co w ogóle stosujesz taką klasę? Możesz pokazać konkretną implementacją razem z zaletami takiego rozwiązania?
SA
Zalety są takie: nie pozwalasz na utworzenie niepoprawnego z punktu logiki biznesowej obiektu, a obiekt nareszcie stanowi wartość biznesową i zawiera logikę, którą można testować niezależnie od wszystkiego wokół. Nie wiem gdzie widzisz trudniejsze testowanie, bo jest zupełnie odwrotnie.
Krzysztof Pe
Okej, a w podejściu gdzie model jest anemiczny w jaki sposób ktoś może stworzyć niepoprawny model z punktu widzenia logiki biznesowej skoro po stronie serwisu jak i GUI zaimplementowana jest walidacja? Jesli masz jakiś fajny tutorial to będę wdzięczny za podesłanie, bo to co znajduje porusza temat, ale jest imo słabo opisane.
Krzysztof Pe
@Saalin: zakładając, że chcesz dodać obiekt do bazy jak to zrobisz? Jak ten obiekt jest zmapowany na obiekt np. EF?
somekind
A w jaki niby sposób DDD utrudnia testowanie?
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Saalin
SA
Saalin
SA
  • Rejestracja:około 12 lat
  • Ostatnio:około 8 godzin
  • Postów:1430
2

Okej, a w podejściu gdzie model jest anemiczny w jaki sposób ktoś może stworzyć niepoprawny model z punktu widzenia logiki biznesowej skoro po stronie serwisu jak i GUI zaimplementowana jest walidacja?

No tak, bo programiści nie popełniają błędów, nie zapominają o uzupełnieniu pól, w ogóle jak jest walidacja to nic złego się stać nie może. Ograniczenie możliwości i hermetyzacja już na poziomie obiektu domenowego zamyka możliwości tego co można w nim popsuć. A psują zwykle programiści, a nie użytkownicy.

@Saalin: zakładając, że chcesz dodać obiekt do bazy jak to zrobisz? Jak ten obiekt jest zmapowany na obiekt np. EF?

@Krzysztof Pe: nie współdzieliłbym modelu bazodanowego z modelem domeny i mapowałbym jedne na drugie.

Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Kokoniłaj
Kokoniłaj
Kokoniłaj
Kokoniłaj
  • Rejestracja:ponad 7 lat
  • Ostatnio:około 4 godziny
  • Postów:182
0
Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
somekind
somekind
somekind
somekind
Moderator
  • Rejestracja:około 17 lat
  • Ostatnio:3 dni
  • Lokalizacja:Wrocław
3
Krzysztof Pe napisał(a):

W podejściu ktore sam praktykuję modele zawierają propertisy, a logika operacji jakie mogą zostać wykonane na modelu znajduje się w serwisie.

Ale serwis też jest częścią modelu. Model to warstwa aplikacji modelująca to, czym aplikacja się zajmuje, a nie klasa z właściwościami.

To, co Ty stosujesz to wzorzec Transaction Script. Autor tamtego kodu starał się mieć Rich Domain Model, co jest o tyle inne, że obiekty są obiektami w sensie OOP, a nie rekordami.

  1. Jak zaimplementowany jest UserService? Czy nie powinien on implementować serwisu IUserService kóry byłby wstrzykiwany do Controllera i stamtąd wywoływane odpowiednie metody?

Lepiej wstrzyknąć, lepiej nie robić statycznie, lepiej nie zwracać obiektów infrastruktury webowej i lepiej nie implementować interfejsu.

Udostępnij
Kopiuj link do postu Kopiuj link do postu jako Markdown
Komentuj
Treść
Podgląd
Kliknij, aby dodać treść...
Kliknij, aby dodać załącznik lub wklej ze schowka.
Instrukcja obsługi Markdown
Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj <u>underline</u>.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, <b>, <i>, <kbd>, <del>, <strong>, <dfn>, <pre>, <blockquote>, <hr/>, <sub>, <sup> oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H<sub>2</sub>O i m<sup>2</sup> otrzymasz: H2O i m2.

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript
document.write('Hello World');
```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy  by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

  • Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
  • Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
  • Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
  • Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

  • Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

  • Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

  • Alt+L - zaznaczenie całej linii
  • Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
  • Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
  • Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

  • Ctrl+Enter - dodaj post
  • ⌘+Enter - dodaj post (MacOS)