¿Es correcto esto?

[pragone]

Hola Merodeador / Blade_Storm

Muy interesante lo expuesto (e ilustrativo como intro al vocabulario MVC), quisiera sólo añadir dos comentarios (que agregan a lo escrito)
1.- La idea de que la parte que crea los objetos esté separada de la que accede a la BD, no es sólo para mayor limpieza y mantenimiento, sino que te permitiría en un futuro cambiar de manejador de base de datos con un impacto aún menor.

2.- Con respecto a las validaciones. Lo lógico es que en cada capa hayan las validaciones propias de la capa. Por ejemplo, para validar un campo DNI (Documento Nacional de Identidad en España)está bien, en la capa de presentación deberías verificar todo lo que puedas, pero adicionalmente en la capa de lógica debes verificar que tenga el formato correcto (quizás un regular expression) y en la de acceso a BD, validar que no sea un string de longitud mayor a la que acepta el campo en la BD. Es decir, a cada capa lo suyo.
La razón para esto es que si en algún punto reemplazaras la capa presentación por un API (por ejemplo), ya sabrás que parte de la validación está en las otras capas.

Esto es la teoría. En ocasiones es un fastidio validar algo cuando sabes que sólo se invoca desde un sitio y en el que ya tienes una validación al respecto, pero para ser puristas, la separación de capas también implica una separación en la dependencia y por tanto en las validaciones.