Ocena Api napisane w .NET Core 2.2

Odpowiedz Nowy wątek
2020-01-14 21:34
0

Cześć. Piszę taką większą dla mnie aplikację na .net corze i angularze. Chciałbym abyście zobaczyli Api i ocenli mój kod tzn co robię źle co mogę zmodyfikować czy dodać z góy bardzo dziękuje. oraz nie wiem jak to powiedzieć ocenili poziom kodu :)
P.S wiem kurcze nie mam jeszcze testów ale nadrobię to oraz jest jeden commit bo nie trzymam repo na githubie a chciałem kod wam jakoś udostępnić :)

https://github.com/Michal-Warmuz/Work

edytowany 2x, ostatnio: Michał Warmuz, 2020-01-14 21:36

Pozostało 580 znaków

2020-01-14 21:46
0

https://github.com/Michal-War[...]ollers/Base/BaseController.cs

Jest sens robić z BaseControllera coś, co nazywa się ExceptionFilter / Exception Handling Middleware?

edytowany 1x, ostatnio: WeiXiao, 2020-01-14 21:48

Pozostało 580 znaków

2020-01-15 06:49
0

Takie drobne sugestie, wyrywkowo wszedłem w kilka plików:

  1. https://github.com/Michal-War[...]ntation/UserRepository.cs#L16 - pomyliłeś konwencje, prywatna zmienna z podkreślnikiem, natomiast w konstruktorze ma nazwę bez podkreślnika
  2. https://github.com/Michal-War[...]tion/CompanyRepository.cs#L48 - dlaczego Task.CompletedTask, a wyżej wywołujesz synchronicznie SaveChanges?
  3. https://github.com/Michal-War[...]ion/EmailSenderService.cs#L51 - czemu nie użyjesz tutaj asynchronicznej metody i tylko udajesz, żeby ta metoda była asynchroniczna?

O dziękuję już poprawiam :) - Michał Warmuz 2020-01-15 08:30

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