¿Es correcto esto?

[Blade_Stom]

mmm yo por mi parte lo veo medio bien. Es decir eso que pones ahi podria ser el modelo......... Tienes que enteder que el modelo suele ser UNICO pero no confundas UNICO con que todo este en el mismo archivo. Es decir....... tu puedes tener un contrtolador de usuario (controlador_usuario.php) ....y que cuando este necesite datos de usuario en concreto pues ire al archivo (modelo_usuario.php) a pedirselos. El modelo_usuario.php tendra metodos del tipo Obtener_usuario(nombre), obtener_num_dni_del_usuario(nombre) ........cosas asi...... de esta manera aunque el modelo es unico.......lo tendremos "esparcido para ser mas modular y manejable".
Desde el modelo lo unico que se hara sera ir a la BD, cojer datos, y devolverlos o a lo sumo utilizar los datos de la BD para cargarselos a un atributo suyo , para posteriormente devolver ese atributo (parecido lo que haces tu con usuario en el ejemplo de arriba)

Luego el controlador jugara con todos los datos que necesite ......... y creara la vista que necesite ..... (PERO EL CONTROLADOR NUNCA ACCEDE A LA BD)....cuando quiere datos los pide al modelo

espero haberte aclarado algo....