Co sądzicie o tym kodzie?

Odpowiedz Nowy wątek
2018-10-26 17:35
2018-10-26 18:12
1

wygląda ok

@some_ONE

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

edytowany 1x, ostatnio: WeiXiao, 2018-10-26 18:17
Może jak pisał ten artykuł (nie znalazłem daty), to .HasData jeszcze nie było, bo dodali to w jakiejś późniejszej wersji EF Core. Zresztą ja nigdzie nie pisałem, że robienie inicjalizacji bazy w startupie to złe rozwiązanie. - some_ONE 2018-10-26 19:48

Pozostało 580 znaków

2018-10-26 18:31
0

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

Pokaż pozostałe 80 komentarzy
Ładnie to opisałeś. To nie mówcie nic Adamowi o EventStore, niech myśli, że jest tam tylko baza do odczytu i problem rozwiązany. Ciii... bo usłyszy. :-| - ._. 2018-10-28 12:42
Dziękuję za zainteresowanie, troskę o moje samopoczucie i stan zdrowia :). Niestety sezon grillowa już się skończył, w sklepach brak węgla więc proszę nie grillujcie mnie na drewnie bo drewno daje większą temperaturę (empiryczne sprawdzone). Ofiara szybciej wyzionie ducha. ;) ;). - .Adam 2018-10-28 12:45
A to nie jest problem ja mam własną kopalnię węgla i uranu, z którego produkuje baterie dla telefonów Samsunga. Nie słyszałeś, że baterie mu wybuchają.? - ._. 2018-10-28 12:52
A to najlepszy chyba taki prasowany wegiel drzewny, jeśli masz las i wypalaczy na własność to jak najbardziej reflektuję na takiego grilla, a co do baterii samsunga, każdemu zdarzają się chwile słabości, ;) ;) - .Adam 2018-10-28 12:55
:-| Nie ma się co za dużo wydurniać, w tak poważnym miejscu. - ._. 2018-10-28 12:57

Pozostało 580 znaków

2018-10-26 19:07
._.
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

Pozostało 580 znaków

2018-10-26 19:52
0

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

Pozostało 580 znaków

2018-10-26 20:39
._.
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.

edytowany 1x, ostatnio: ._., 2018-10-26 21:00

Pozostało 580 znaków

2018-10-27 00:37
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.


"HUMAN BEINGS MAKE LIFE SO INTERESTING. DO YOU KNOW, THAT IN A UNIVERSE SO FULL OF WONDERS, THEY HAVE MANAGED TO INVENT BOREDOM."

Pozostało 580 znaków

2018-10-27 07:52
1

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

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

wiedz że się coś dzieje ;)


It's easy to hate code you didn't write, without an understanding of the context in which it was written.
Ten gościu ma 6 lat doświadczenia i jest seniorem. Moje teorie spiskowe się potwierdzają. - ._. 2018-10-27 07:58

Pozostało 580 znaków

2018-10-29 10:14
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. ;(

Logiki biznesowej to ja akurat tam w kontrolerach nie widzę. - some_ONE 2018-10-29 12:25

Pozostało 580 znaków

2018-10-29 13:23
._.
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

A to, że ktoś pisze, że odkrył takie coś jak enkapsulacje a to, co powinno być enkapsulowane w DM jest wepchnięte do serwisu aplikacyjnego. W projekcie panuje ogólny syf. - ._. 2018-10-29 14:30
no ale to przecież nie jest w serwisie tylko w agregacie - john_klamka 2018-10-29 15:25
Jak koś łamie wszystkie możliwe założenia DDD, a potem pisze, że prywatna kolekcja jest zgodna z założeniami DDD, nie jest zabawne? Nie macie za grosz poczucia humoru. - ._. 2018-10-29 15:55

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

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