Es una buena practica esto en poo ?

[HiToGoRoShi]

Este es mi codigo de configuracion de un panel de adminstracion, todo el panel de adminstracion llaman a esta clsae, llamada Cpanel en ves de poner el codigo en la parte superior de cada pagina del panel.

Ahora me falto usar el constructor y los metodos set y get, pero es vlaido ya que las variables son del tipo public.

Código PHP:

<?php
//Importa la clase ConDB
include_once("../config.php");
class Cpanel{
    
    public $estado;
    public $msg;
    
    function loadConfig(){      
        //Conectarse a Mysql
        $cn = new Conexion;
        $cn = $cn->conDB();
        //Conectarse a Mysql
        $query = mysql_query("select * from mees_config",$cn) or die (mysql_error());
        $row = mysql_fetch_array($query);
        //Inicializar la variable estado
        $this->estado = $row['estado'];
        return $row;
    }
    
    function estado(){
    //Verificar estado del Mees 
    if($this->estado == 1)
    $estado = '<b><font color="green">ACTIVO</font></b>';
    else
    $estado = '<b><font color="red">INACTIVO</font></b>';
    
    return $estado;
    }
    
    function actualizarConfig($n,$d,$cp,$e,$m,$es){
        if($cp=="")
        {
        $this->msg='<div class="advertenciamees">MeES necesita tener un correo padre, no puedo dejarlo sin valor !!</div>';
        }
        else{
        //Conectarse a Mysql
        $cn = new Conexion;
        $cn = $cn->conDB();
        //Conectarse a Mysql
        $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());
        $this->msg = '<div class="satisfactoriomees">Los datos de configuracion han sido actualizados correctamente !!</div>';
        }
    }
}
?>
 

[GatorV]

No, nunca es buena practica tener propiedades públicas en tus clases, ya que rompes con la encapsulación de tu clase. Para eso existen los setters/getters.

[eulloa]

Creo q sería mejor:

Código PHP:

function estado(){
    //Verificar estado del Mees
    if($this->estado == 1)
        return 1;
    else
        return 0;
    }
 

¿Cual es el sentido de dejar las propiedades públicas? Tampoco veo q las usas
¿No sería mejor que tu clase Conexion se encargara de realizar las consultas a la BD?

[HiToGoRoShi]

Ahora, esta mejor ?

Código PHP:

<?php
//Importa la clase ConDB
include_once("../config.php");
class Cpanel{
    
    private $estado;
    private $msg;
    
    function loadConfig(){      
        //Conectarse a Mysql
        $cn = new Conexion;
        $cn = $cn->conDB();
        //Conectarse a Mysql
        $query = mysql_query("select * from mees_config",$cn) or die (mysql_error());
        $row = mysql_fetch_array($query);
        //Inicializar la variable estado
        $this->estado = $row['estado'];
        return $row;
    }
    
    function estado(){
    //Verificar estado del Mees 
    if($this->estado == 1)
    $estado = '<b><font color="green">ACTIVO</font></b>';
    else
    $estado = '<b><font color="red">INACTIVO</font></b>';
    
    return $estado;
    }
    
    function actualizarConfig($n,$d,$cp,$e,$m,$es){
        if($cp=="")
        {
        $this->msg='<div class="advertenciamees">MeES necesita tener un correo padre, no puedo dejarlo sin valor !!</div>';
        }
        else{
        //Conectarse a Mysql
        $cn = new Conexion;
        $cn = $cn->conDB();
        //Conectarse a Mysql
        $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());
        $this->msg = '<div class="satisfactoriomees">Los datos de configuracion han sido actualizados correctamente !!</div>';
        }
    }
 
//Getters and Setters
 
    public function getEstado(){
    return $this->estado;
    }
    
    public function setEstado($estado){
    $this->estado = $estado;
    }
 
    public function getMsg(){
    return $this->msg;
    }
    
    public function setMsg($msg){
    $this->msg = $msg;
    }
}
?>
 

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.

[eulloa]

Lo de imprimir los errores, si no vas a usar exepciones, sepáralo tambiénm en otras clases. Yo q tu lo haría. El ejemplo de la función estado q te puse arriba ¿lo viste?

[HiToGoRoShi]

Iniciado por eulloa

Lo de imprimir los errores, si no vas a usar exepciones, sepáralo tambiénm en otras clases. Yo q tu lo haría. El ejemplo de la función estado q te puse arriba ¿lo viste?

Si, pero practicamente quiero evitar usar mucho codigo php en las partes visuales.... entonces usare try catch para validar esto, ahi se podria imprimir mensajes normal ?