Tienen un poco desprolija la división de paquetes. El uso de paquetes básicamente tiene que ser algo que les sirva a ustedes para organizar mejor su trabajo.
Actualmente:
image
Yo les sugiero las siguientes convenciones:
ar.edu.unq.epers.model: debería contener clases relativas solo a su modelo de negocio. Está bien que Usuario esté ahí. Pero todo lo relativo al envío de mails (EnviadorDeMails, Mail) quizás pueda separarse en un paquete distinto (por ejemplo: ar.edu.unq.epers.mailing)
Lo mismo para su Sistema (que en este caso es un servicio). Los servicios generalmente no son considerados parte del modelo de negocio,sino mas bien una puerta de entrada que expone una Fachada (pattern Facade). Quizás puedan colocarlo en un paquete distinto, como ser ar.edu.unq.epers.services.
Lo mismo para su _HomeSistema. Debería ir un paquete distinto que ar.edu.unq.epers.model:
Tienen un poco desprolija la división de paquetes. El uso de paquetes básicamente tiene que ser algo que les sirva a ustedes para organizar mejor su trabajo.
Actualmente: image
Yo les sugiero las siguientes convenciones: