Especialmente cuando trabajemos con servicios, los cuales se supone que exponen una interfaz a clientes externos, sean cuidadosos con que métodos son expuestos por dicha interfaz.
¿Qué quiero decir con esto?
En su clase Sistema tienen un método crearMail(String, String). Está genial que hayan separado eso como un método aparte pero no está bueno que lo terminen exponiendo como parte de la interfaz del servicio. Quizás deban marcar ese método en particular como privado.
Especialmente cuando trabajemos con servicios, los cuales se supone que exponen una interfaz a clientes externos, sean cuidadosos con que métodos son expuestos por dicha interfaz.
¿Qué quiero decir con esto?
En su clase Sistema tienen un método crearMail(String, String). Está genial que hayan separado eso como un método aparte pero no está bueno que lo terminen exponiendo como parte de la interfaz del servicio. Quizás deban marcar ese método en particular como privado.