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?
0
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.
0
Już jest 404, więc nawet nie mam jak przejrzeć ;)