Tema: Slave Class!
Ver Mensaje Individual
  #4 (permalink)  
Antiguo 03/04/2012, 11:14
Avatar de JairLizcano
JairLizcano
 
Fecha de Ingreso: junio-2008
Ubicación: Santander, Colombia
Mensajes: 608
Antigüedad: 15 años, 10 meses
Puntos: 53
Exclamación Respuesta: Slave Class!

Cita:
Iniciado por SirDuque Ver Mensaje
JairLizcano muy interesante la clase, en lo personal quitaria todos los get por un getProfile (Profile.php). Algo como esto:

Código PHP:
Ver original
  1. <?php
  2. // codigo.....
  3. $profile = new Profile();
  4. //.............
  5.  
  6. echo "Email: ". $profile-> getEmail(); ."<br>";
  7. echo "ID: ". $profile-> getId(); ."<br>";
  8.  
  9. // Cambiara por:
  10.  
  11. $datos = $profile-> getProfile();
  12.  
  13. echo "Email: ".$datos['email']."<br>";
  14. echo "ID: ".$datoss['id']."<br>";
  15.  
  16. // Ó
  17.  
  18. echo "Email: ". $profile-> getProfile('email'); ."<br>";
  19. echo "ID: ". $profile-> getProfile('id'); ."<br>";

Creo que seria mejor, ya que sacas 11 funciones de tu codigo y las remplazas por una.
Ademas con un solo llamado, tenes todos los datos disponible en todo el codigo. (Opcion $datos = $profile-> getProfile();)
Tambien (si bien el codigo es pequeño) optimizas en Kb,caracteres, lineas a procesar por PHP.
Para el comportamiento que defines se hace evidente el uso del método mágico __toString que permite definir el comportamiento cuando a una clase se le manipula como string... y que sinceramente, la había pasado por alto, no sé el porqué... pero se me iba como olvidando.

Gracias por sus opiniones.
__________________
Programar dejó de ser una profesión acusada en la oscuridad y disfrutada fríamente... para convertirse en un arte.