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>';
        }
    }
}
?>