Co sądzicie o tym kodzie?

https://www.toptal.com/asp-dot-net/asp-net-web-api-tutorial

wygląda ok

@some_ONE

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. ;(

https://github.com/dotnet-architecture/eShopOnWeb/blob/master/src/ApplicationCore/Entities/OrderAggregate/Order.cs

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.