Ver Mensaje Individual
  #4 (permalink)  
Antiguo 29/04/2010, 12:39
Avatar de HiToGoRoShi
HiToGoRoShi
 
Fecha de Ingreso: abril-2008
Mensajes: 849
Antigüedad: 16 años
Puntos: 31
Respuesta: Es una buena practica esto en poo ?

Ahora, esta mejor ?

Código PHP:
Ver original
  1. <?php
  2. //Importa la clase ConDB
  3. include_once("../config.php");
  4. class Cpanel{
  5.    
  6.     private $estado;
  7.     private $msg;
  8.    
  9.     function loadConfig(){      
  10.         //Conectarse a Mysql
  11.         $cn = new Conexion;
  12.         $cn = $cn->conDB();
  13.         //Conectarse a Mysql
  14.         $query = mysql_query("select * from mees_config",$cn) or die (mysql_error());
  15.         $row = mysql_fetch_array($query);
  16.         //Inicializar la variable estado
  17.         $this->estado = $row['estado'];
  18.         return $row;
  19.     }
  20.    
  21.     function estado(){
  22.     //Verificar estado del Mees
  23.     if($this->estado == 1)
  24.     $estado = '<b><font color="green">ACTIVO</font></b>';
  25.     else
  26.     $estado = '<b><font color="red">INACTIVO</font></b>';
  27.    
  28.     return $estado;
  29.     }
  30.    
  31.     function actualizarConfig($n,$d,$cp,$e,$m,$es){
  32.         if($cp=="")
  33.         {
  34.         $this->msg='<div class="advertenciamees">MeES necesita tener un correo padre, no puedo dejarlo sin valor !!</div>';
  35.         }
  36.         else{
  37.         //Conectarse a Mysql
  38.         $cn = new Conexion;
  39.         $cn = $cn->conDB();
  40.         //Conectarse a Mysql
  41.         $query = mysql_query("update mees_config set nombre = '$n', descripccion = '$d', correo_padre = '$cp', estado = $e, motivo = '$m', estilo = '$es';",$cn) or die (mysql_error());
  42.         $this->msg = '<div class="satisfactoriomees">Los datos de configuracion han sido actualizados correctamente !!</div>';
  43.         }
  44.     }
  45.  
  46. //Getters and Setters
  47.  
  48.     public function getEstado(){
  49.     return $this->estado;
  50.     }
  51.    
  52.     public function setEstado($estado){
  53.     $this->estado = $estado;
  54.     }
  55.  
  56.     public function getMsg(){
  57.     return $this->msg;
  58.     }
  59.    
  60.     public function setMsg($msg){
  61.     $this->msg = $msg;
  62.     }
  63. }
  64. ?>

Respecto a a los query, creo que si estaria mejor separarlo en otra clase, pero no en la clase de conexion ya que este es generado automaticamente al momento de instalar el SW, lo tendre en mente y pondre todos mis querys dentro de una clase, asi lo instancio y solo llamo a las clase query para usar sus funciones.