Prośba o code review małego projektu w JSF

0

Witam. Czy mógłby ktoś rzucić okiem na mój kod i powiedzieć czy nie stosuję przypadkiem jakiś złych praktyk? Albo czy coś robię źle? Aplikacja napisana w JSF ma wyświetlać tabele z bazy danych i umożliwiać usunięcie rekordu.

https://github.com/kukiel91/code-review

Z góry dzieki

0
  1. Zaponaj się z adnotacją @PostConstruct. Nie używaj konstruktora w Managed Beans, ponieważ w konstuktorze nie są dostępne zarządzalne (wstrzykiwalne) obiekty np. EJB/beany CDI. Dlatego tam lepiej przeprowadzać inicjalizację.
@PostConstruct
private void init() {
   // tu robisz to co w konstruktorze, ale masz
   // dostep do wszystkich obiektow
}
  1. SessionScope żyje raczej dłużej niż tego potrzebujesz. Wystarczy Ci @ViewScoped.
  2. Projektem warto zarządzać za pomocą Apache Maven (pom.xml).
  3. DataController (DAO) to jest coś co zwykle wstrzykuje się za pomocą adnotacji @ejb lub @Inject. Poczytaj o SOLID i dependency injection. Dzięki zastosowaniu DI pisanie testów jednostkowych jest łatwiejsze.

Tak sobie pomyślałem, że korzystnie może być jeśli na początku zrefaktoryzujesz projekt do CDI/JDBC (niewiele brakuje). Dopiero później przerób to na JPA2/EJB. :)

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