Ver Mensaje Individual
  #7 (permalink)  
Antiguo 22/09/2011, 11:41
ZanPHP
 
Fecha de Ingreso: septiembre-2011
Mensajes: 9
Antigüedad: 13 años, 3 meses
Puntos: 0
Respuesta: ZanPHP: Ejemplos básicos de cómo utilizar el ActiveRecord

Cita:
Iniciado por pablofmorales Ver Mensaje
Veo varias cosas mal, en como manejas esto


1- Tenes el mismo problema que codeigniter de usar un mapa de modelos ( $this->model('nombre_modelo') ), esto es bastante molesto cuando estas trabajando con un IDE y queres desde el controller acceder a un metodo de ese modelo. Por ejemplo si quiero acceder al metodo $this->model('users')->save(); y le doy F3 a save() en eclipse, o ctrl + b en netbeans, no va a abrir nada. Me va a dejar donde estoy, esto es porque no puede relacionar el metodo model(), con el archivo que contiene el modelo.

2- Por otro lado haces $this->Agenda_Model = $this->model("Agenda_Model");

y despues usas $data = $this->Agenda_Model->getContact($contactID);

No vi como esta hecho en el core, pero tranquilamente podrias hacer

$data = $this->model('Agenda_Model')->getContact($contactID);
Sin la necesidad de generar propiedades que no existen, o no declaraste anteriormente.

3- Trabajas sobre supuestos, por ejemplo, $this->app(); $this->model(); $this->core(), $this->theme(); no tengo idea que hacen, los nombres de los modelos deberian ser mas descriptivos como getModel(), setTheme, no me di cuenta que hace el resto.

Lo mismo con esa constante que tenes _webTheme, donde se configura? Porque no es una constante de clase?

y ni hablar del metodo ____(); ahi me descolocaste totalmente.

4- En el modelo tenes un metodo llamado getContactByQuery, pero nunca le pasaste la query

5- Standard en el codigo, esto es importante mas si queres que otras personas trabajen con estas librerias. Por un lado tenes la propiedad $this->application y por otra tenes propiedades Templates, en una la primer letra es mayuscula y la segunda minuscula.



Saludos y suerte con el FW
La función ____() (4 guiones bajos) es el equivalente a hacer un die(var_dump()), saludos.