como desacoplar los datos de la lógica de negocio

Question

Aquí está el panorama,

Digamos que tengo una clase de usuario, así:

public class User{ 
    private String firstName; 
    private String lastName; 
//... 
// setter, getters 
} 

Entonces tengo una clase al igual que para manejar usuarios Comentarios:

public class Comments{ 
    // some fields 
    public static loadComments(User user, int count){...} 
} 

Hasta ahora, cosas muy básicas. Sin embargo, quiero agregar algunos ayudantes, por supuesto, para que sea más fácil cargar comentarios para el usuario. Así que podría crear algo de la clase Usuario:

final static int defaultCount = 10; 
... 
public Comment comments(){ 
    return Comments.loadComments(this, defaultCount); 
} 

Creo que esta es una manera fácil de no tener que pasar alrededor de las instancias de usuario alrededor. Pero en este momento, no estoy contento porque he acoplado mi objeto bean de usuario con la lógica de negocios que carga el comentario. También guardé el recuento predeterminado en la clase de usuario, que no debería pertenecer allí. Entonces, ¿cuál es la mejor manera de hacer esto? Mi objetivo es pasar este objeto a un jsp para poder llamar a las funciones JSTL. Tuve la idea de crear un UserWrapper que se vería así ...

public class UserWrapper{ 
    private final static defaultCount = 10; 
    private final User user; 
    public UserWrapper(User user){ 
    this.user = user; 
    } 

    // should probably cache this but i am not going to show this for simplicity 
    public Comments getComments(){return Comments.loadComments(user, 10);} 
} 

Espero haber sido claro. No me gusta usar la etiqueta useBean porque simplemente no es necesario para algo como esto. ¡Espero que haya una forma más limpia de abordar algo como esto! ¡Cualquier ayuda sería apreciada!

Editar: Una cosa que olvidé mencionar. Quiero poder usar este código en JSTL. Lo que significa que debe ser un getter. El modelo DAO es bien conocido pero no ayuda demasiado cuando mi desarrollador front-end necesita escribir un scriplet o necesito cargarlo para lugares que puede o no necesitar.

Answer 1

Desde un punto de la tecnología de stand-independiente ...

Sí, tienes toda la razón en que el acoplamiento de ser prohibitivo. Consulte el Layers architectural pattern para ofrecer alguna orientación sobre la estructura, lógica de negocio y datos. Por ejemplo, podría ser una excelente idea diseñar dos subsistemas: uno para su lógica y el otro para las capas. Restrinja la comunicación entre ellos por solo permitiendo que la lógica pase mensajes a la capa de datos. Además, puede usar el Facade pattern para representar cada subsistema, y ​​así disminuir el acoplamiento aún más. Trata cada capa como una caja negra.

Otro patrón que puede ser útil es el Data Access Object pattern. Le ayudará a definir un contrato entre las capas para pasar datos cuando sea necesario.

Answer 2

Me gusta usar objetos de acceso a datos (DAO) para esto. Sus clases User y Comment solo contendrán los campos para esos objetos y luego creará clases separadas para recuperar datos sobre esos objetos. Los DAO son donde incluye la lógica sobre cómo recuperar, actualizar, seleccionar, etc. Por ejemplo,

public class UserDAO { 

    public UserDAO() { 
     //maybe setup a Hibernate connection or JDBC connection here 
    } 

    public User getUser(int userId) { 
     User user; 
     //code to retrieve user from datasource 
     return user; 
    } 

    //other methods for update, delete, select, etc 
    } 

    public class CommentsDAO { 
    private static int DEFAULT_COUNT = 10; 

    public CommentsDAO() { 
     //maybe setup a Hibernate connection or JDBC connection here 
    } 

    public Collection getCommmentsForUserWithCount(User user, int count) { 
     Collection comments = new Set(); 
     //retrieve comments from datasource limit to count 

     return comments; 
    } 

    public Collection getCommentsForUser(User user) { 
     return getCommentsForUserWithCount(user, DEFAULT_COUNT); 
    } 

    //other methods for update, delete, etc 
    }