wygląda ok
Widzisz? nawet ten gość wali db init w startupie, a nie jakieś tam HasData
:D :D
IsDeleted pole w bazie, najbardziej debilny pomysł z jakim się spotkałem!
Prosta strzałka do bazy przez trzy warstwy to jest jakaś architektura?. A jak taka architektura się nazywa.? Interfejsy, które niczemu nie służą. Testy, które nie mają sensu jak zamokowanie bazy i sprawdzenie, czy można coś do niej dodać.
Widać, że autor kierował się jakimiś wyższymi pryncypiami oraz metrykami projektując paczki. Jest w tym coś pięknego.
Wygląda na typowy przykład przeinżynierowanego CRUDa. Tam chyba nie ma żadnej logiki...
Miałoby to sens, bo to ma tylko zapisywać i odczytywać (jedną encje), ale to wygląda jakby, autor pomyślał, wyszła tylko jedna klasa do zapisu i odczytu, cholera muszę dodać trochę więcej mięsa.
.Adam napisał(a):
IsDeleted pole w bazie, najbardziej debilny pomysł z jakim się spotkałem!
No to chyba konkurs na najbardziej debilny post roku w działach technicznych mamy rozstrzygnięty.
Jeśli dowolny diagram zależności w projekcie przypomina:
- nieuporządkowany chaos
- kłębek wełny
- spaghetti
wiedz że się coś dzieje ;)
Nawet ta przykładowa aplikacja: https://github.com/dotnet-architecture/eShopOnWeb, która jest promowana przez Microsoft jako punkt odniesienia przy projektowaniu architektury, zdaje się pokazywać złe praktyki (logika biznesowa w kontrolerach, sterowanie przepływem za pomocą wyjątków i takie tam). Już w nic nie można wierzyć w internecie. ;(
public Address ShipToAddress { get; private set; }
// DDD Patterns comment
// Using a private collection field, better for DDD Aggregate's encapsulation
// so OrderItems cannot be added from "outside the AggregateRoot" directly to the collection,
// but only through the method Order.AddOrderItem() which includes behavior.
Zarejestruj się i dołącz do największej społeczności programistów w Polsce.
Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.