Wątek przeniesiony 2020-07-18 14:54 z C# i .NET przez cerrato.

Ocena kodu

0

Cześć,

znajdzie się ktoś komu chciałoby się przejrzeć fragment mojego kodu i powiedzieć mi jak bardzo źle jest on napisany ;)?

Przyjmę wszelkie słowa krytyki, dobre rady i wszystko inne ;). Chciałbym poprawić mój warsztat.

https://github.com/itmey/webApi

Ps. Nie patrzcie na commity bo repo utworzone tylko żeby pokazać. Nie korzystam z GitHuba na codzień (mam gita na serwerku).

2

Przecież to nie działa, to co tu komentować?

0
kzkzg napisał(a):

Przecież to nie działa, to co tu komentować?

Nie jest to skończony projekt. Zależy mi na tym, żeby ktoś mi powiedział co robię źle i dlaczego.

1

Tylko, że tam prawie niczego nie ma.

1
grzesiek51114 napisał(a):

Tylko, że tam prawie niczego nie ma.

Jest generowanie tokenów JWT. Nie chciałem pisać dalej i powielać błędów, które mogłyby się pojawić w tym co napisałem.
Chodzi mi o sprawdzenie takiego ogólnego podejścia do tematu.

Pamiętam jak kiedyś wrzuciłem na to forum 5 linijek kodu i dostałem listę chyba z 11 uwag do tego.

Jeżeli to co wrzuciłem to jest za mało to po prostu zapomnijcie o temacie ;)

3
  1. jak masz robić message 'commit' to lepiej zosta pusty
  2. komentarze /// Klasa reprezentująca typu wyliczaln sobie darujj
  3. klasy zagniedzone rowniez
  4. Jak juz piszesz to sie zdecyduj na jeden jezyk
  5. Enum namespace - to jakas bardzo wazna koncepcyjnie czesc systemu ze uzywasz enumow, ze dostaly swoj katalog?
  6. db first? Oszczedz sobie nerwow i archeologi i rob code first
3

Wywal tę logikę dostępu do bazy z akcji kontrolerów.

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