Ocena Api napisane w .NET Core 2.2

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

0

https://github.com/Michal-Warmuz/Work/blob/master/Work.API/Work.API/Controllers/Base/BaseController.cs

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

0

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

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

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