Ocena Projektu

0

Cześć. Napisałem Forum w ASP.NET MVC. Chciałbym was się spytać co o nim sądzicię co mogę dodać usunać albo zmienić. Z góry dziękuje za pomoc :)
https://github.com/Michal-Warmuz/Froum-v.1.0

Nie xD już zmieniam :D

0

A gdzie domena?

0

A co to jest domena ?

1

A gdzie testy?

2

Tak na szybko:

  1. Fajnie, że jest podział na projekty, ale trzymanie ViewModeli, modeli bazodanowych, DbContextu i migracji w jednym projekcie pokonuje cel robienia takiego podziału. ;) Tu masz przykład od @somekind, jak taki podział może wyglądać: https://4programmers.net/Forum/1296917
  2. Zamiast tworzyć klasę Service do wszystkiego, lepiej utworzyć kilka osobnych, znów @somekind: https://4programmers.net/Forum/1496160
  3. Walidację lepiej mieć w filtrach, możesz wygooglować, jak to zrobić
  4. Masz wycieki logiki w kontrolerach, patrz link z punktu 2

Domeną się na razie nie przejmuj ;)

0

Dziękuje za ten komentarz wszystko poprawię Dzięki :)

2

Oprócz tego co wyżej

Z opisu wynika, że jest panel admina. Ale dostęp mają wszyscy.
Brak sprawdzenia czy edytujemy posta zalogowanego usera czy kogoś innego.
Brak obsługi błędów,
Brak logów,
Usuwasz z bazy wywołując get'a.
Zamiast faktycznie usuwać, może rób soft delete
Edycja, usuwanie postów dostępne dla nie zalogowanego usera.

Jak robisz commity to może zamiast Adding paging napisz Add pagination

W Foldrze Forum.Api masz klasyczną aplikacje z widokami, to czemu nazywasz to Api?

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