polostor / issuetracking Goto Github PK

současnou implementaci tě business rozhraní pak doporučuji dát do samostatného maven modulu, abyste mohl pak ty implementace střídat při vývoji dalších pouze změnou závislostí v mavenu (což lze řešit pak např. profilem, což je cool věc co má maven)

Z nějakého důvodu duplikujete metody create a update v podtřídách GenericDAO, což je zbytečné. Ty duplikace zahoďte, nechte pouze implementaci v generic dao, od toho tam je

V DAO metody find a findAll implementujete vlastním dotazem. Místo toho použijte entity manager a metody find a findAll. Takto to implementujte v GenericDAO jako protected metodu (parametrem je ID a třída). V podtřídách tuto metodu volejte a předávejte tam správnou třídu.

http://docs.oracle.com/javaee/6/api/javax/persistence/EntityManager.html#find(java.lang.Class, java.lang.Object)

Doporučuji vám zavést rozhraní pro všechny služby (třídy v balíčku service), abyste mohl časem přidat další impelmentaci těchto rozhraní při zachování identické persistentní a prezentační vrstvy

Nahrát aplikaci na server: tomanek.czacm.org

Instrukce byly v emailu

moc se mi nezdá ta vaše anotace @ManagedBean nad GenericService. Tam by to být nemělo, managed beany mají být pouze ve view

Třídu mějte User, tabulku můžete změnit pomocí anotace @table

nechápu proč u některých metod překrýváte tu generic service, např. Comment#update, ale je jich víc. TA implementace je stejná nevidím důvod pro překrytí rodiče. Zkuste to minimalizovat, ať nejsou zbytečně duplicity. máte to tak u většiny services