Co sądzicie o tym kodzie?

1

wygląda ok

@some_ONE

Widzisz? nawet ten gość wali db init w startupie, a nie jakieś tam HasData :D :D

0

IsDeleted pole w bazie, najbardziej debilny pomysł z jakim się spotkałem!

1

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.
title

0

Wygląda na typowy przykład przeinżynierowanego CRUDa. Tam chyba nie ma żadnej logiki...

0

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.

8
.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.

1

Jeśli dowolny diagram zależności w projekcie przypomina:

  • nieuporządkowany chaos
  • kłębek wełny
  • spaghetti

wiedz że się coś dzieje ;)

0

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

0

https://github.com/dotnet-arc[...]ities/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.

title

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0