Ocena projektu ASP.NET MVC

Ocena projektu ASP.NET MVC
V4
  • Rejestracja: dni
  • Ostatnio: dni
0

Cześć,
Napisałem jakiś czas temu projekt w ASP.NET i byłbym bardzo wdzięczny za ocenę i opinie :) Czy z taką wiedzą można już startować na .Net dev?

Link do projektu

Ktos
  • Rejestracja: dni
  • Ostatnio: dni
3

Na pewno wyniósłbym pobieranie topRatedArticles / newArticles z HomeController do oddzielnej warstwy - podobnie zresztą tworzenie ViewModelu. Mówi się, że kontroler w miarę możliwości nie powinien być zbytnio "mądry", tylko zlecać takie zadania oddzielnym warstwom i tylko przesyłać do widoku - przeczytaj np. taki post od somekinda.

Poza tym: niekonsekwencja stosowania String oraz string (konwencja mówi raczej string). Rzuciło mi się też w oczy, że masz w repozytorium plik desktop.ini - spokojnie możesz usunąć, to tylko plik dotyczący Eksploratora Windows.

XardasLord
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Gdańsk
  • Postów: 271
0

Już jest 404, więc nawet nie mam jak przejrzeć ;)

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.