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ć :)
0
0
Jest sens robić z BaseControllera coś, co nazywa się ExceptionFilter / Exception Handling Middleware?
0
Takie drobne sugestie, wyrywkowo wszedłem w kilka plików:
- 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
-
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 synchronicznieSaveChanges
? - 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?