Prosty stronka ASP.NET MSSQL prośba o recenzje

0

Witam serdecznie,

stworzyłem mały projekt C# ASP.NET + MSSQL. Jest to bardzo prosta stronka (sklep z książkami). Zależy mi na uwagach dotyczących kodu, nie funkcjonalności :)

https://github.com/kawar92/WebApplication1/tree/master/WebApplication1

Za wszelką pomoc będę bardzo wdzięczny.

Serdecznie pozdrawiam!

1
  1. "WebApplication1" to nie jest najlepsza nazwa dla projektu ;-),
  2. https://github.com/kawar92/WebApplication1/blob/master/WebApplication1/Pages/Books.aspx.cs#L34 -> Generowanie tabelki w ten sposób... A nie dało by się np. w GridView? W paru innych miejscach też generujesz kod widoku w Code Behind - ja oczywiście już mam spaczone podejście przez MVC i bym próbował tego nie robić wcale, tylko jak już wydzielić do osobnej klasy czy coś,
  3. https://github.com/kawar92/WebApplication1/blob/master/WebApplication1/App_Code/ConnectionClass.cs#L211 -> Gotta catch'em all! A potem nie róbmy z nimi nic ;-) Gdzie indziej z kolei tam są jakieś messageboksy, a gdzieniegdzie nie ma,
  4. https://github.com/kawar92/WebApplication1/blob/master/WebApplication1/App_Code/ConnectionClass.cs#L324 -> Nie rób COUNT(*), używaj COUNT(id) (gdzie id to kolumna z kluczem głównym),
  5. Może podzielić ConnectionClass na coś bardziej odpowiadające zastosowaniu - jedne to operacje do książek, inne do kont użytkowników. W dodatku czego to jest wszystko statyczne? I czego nazywa się ConnectionClass, a nie np. Connection?. I wcale nie odpowiada za połączenie, a za wiele rzeczy, m.in. połączenie i wybranie odpowiednich rzeczy z bazy,
  6. Nie używaj ArrayList, używaj list generycznych,
  7. Zamiast BookItem może to nazwać po prostu Book i wrzucić do przestrzeni nazw np. Models, razem z np. Orders?,
  8. Nie musisz tworzyć konstruktorów, możesz używać takiej składni stworzenia obiektu i nadania mu jego właściwości 2w1:
var u = new User { Name = "Jan", Password = "12345", Email = "[email protected]", Type = "admin" };
  1. Masz katalog IMG i Images/Book, w obydwu jest 1984.jpg. Po pierwsze dane aplikacji nie powinny być w repozytorium kodu, po drugie coś ci się chyba dubluje.

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