Question

Estoy empezando a utilizar Clases, Namespaces y demas usando Php 5.3

Quisiera saber si es una buena practica utilizar consultas dentro de los metodos de una clase.

Por ejemplo algo asi (En el ejemplo esta puesto con mysql_ pero voy a utilizar la clase mysqli),

class Categoria
{
private $nombre, $id;

function __construct()
{
    $this->id=0;
    $this->nombre="";
}

function Initialize($cod=0)
{
    $this->id=intval($cod);
    if($this->id!=0) 
    {
        //Obtengo todos los datos de la categoria
        $query="SELECT * FROM Categorias WHERE cat_id=".$this->id;
        $result=mysql_query($query);
        if(mysql_num_rows($result)>0)
        {
            $cat=mysql_fetch_assoc($result);
            $this->nombre=$cat['cat_name'];
        }else
        {
            $this->id=0;
        }
        mysql_free_result($result);         
    }
}

function Get_Id()
{
    return $this->id;
}

function Get_Name()
{
    return $this->nombre;
}

function Set_Name($name="")
{
    if(trim($name)!="")
    {
        $this->nombre=htmlentities(trim($name),ENT_QUOTES);;
    }
}

function Get_Item_List()
{
    //Obtengo todos los articulos correspondientes a la categoria
    $query="SELECT art_id AS id FROM Articulos WHERE art_cat_id=".$this->id;
    $result=mysql_query($query);
    if(mysql_num_rows($result)>0)
    {
        $listado=array();
        $i=0;
        while($i < mysql_num_rows($result)) 
        {
            $aux=mysql_fetch_assoc($result);
            $art= new Articulo;
            $art->Initialize($aux['id']);
            $listado[]=$art;
            $i++;
        }
        return $listado;
    }
}

function Get_Total()
{
    if($this->id!=0) 
    {
        $query="SELECT COUNT(*) AS Total FROM Articulos WHERE art_cat_id=".$this->id;
        $result=mysql_query($query);
        $tot=mysql_fetch_assoc($result);
        return $tot['Total'];
    }else 
    {
        return 0;
    }
}   

function Get_Total_Ofertas()
{
    if($this->id!=0) 
    {
        $query="SELECT COUNT(*) AS Total FROM Articulos WHERE art_cat_id=".$this->id." AND art_oft_id IS NOT NULL";
        $result=mysql_query($query);
        $tot=mysql_fetch_assoc($result);
        return $tot['Total'];
    }else 
    {
        return 0;
    }
}

function Get_Total_Disabled() 
{
    if($this->id!=0) 
    {
        $query="SELECT COUNT(*) AS Total FROM Articulos WHERE art_cat_id=".$this->id." AND art_est_id=0";
        $result=mysql_query($query);
        $tot=mysql_fetch_assoc($result);
        return $tot['Total'];
    }else 
    {
        return 0;
    }
}

private function Exist()
{
    if($this->nombre!="")
    {
        $query="SELECT * FROM Categorias WHERE cat_name LIKE '".$this->nombre."'";
        $result=mysql_query($query);
        return (mysql_num_rows($result)>0)? true : false;
    }else 
    {
        //Aca va un objeto Error            
        return false;
    }
}

private function Insert()
{
    if(!$this->Exist())
    {
        $query="INSERT INTO Categorias(cat_name) VALUES ('".$this->nombre."')";
        $result=mysql_query($query);
        return (mysql_errno()==0)?true:false;
    }else 
    {
        return false;
    }
}

function Save()
{
    if($this->id==0) 
    {
        return $this->Insert();
    }
}

function Delete()
{
    if($this->Exist())
    {
        $query="DELETE FROM Categorias WHERE cat_id=".$this->id;
        $result=mysql_query($query);
        return (mysql_errno()==0)?true:false;
    }else 
    {
        return false;
    }
}
}

Answer 1

Hola Ángela,

en respuesta a tu pregunta, no es una buena práctica tener llamadas a la base de datos en un diseño en Objetos.

Ojeé los comentarios a las otras respusetas y si no entiendo mal, lo que estás queriendo hacer es un software simple y aplicar buenas prácticas de Ingenieria del Software usando el paradigma de Programación Orientada a Objetos.

Siendo este el caso, tus clases deberían hacer lo que el Modelo requiere según un buen diseño basado en responsabilidades, y no es un enfoque saludable pensar que la Categoría deba encargarse de las conexiones a la DB ni de las consultas, ambas cosas que tienen poco que ver con el modelo.

Sin conocer más que esta clase, puedo afirmarte además que la clase Categoría tiene una crisis de identidad: responde tanto a cuestiones sobre sí misma como a cuestiones sobre Artículos, siendo que no hay una agregación o composición con una Colección de Artículos. Para terminar sobre esta disgresión, aplicando el principio KISS, es equivalente y más simple etner un Artículo que tenga una agregación con una Categoría, que una Categoría que tenga una agregación o composición de una Colección de Artículos.

Volviendo al tema, lo ideal para resolver la cuestión de la persistencia en PHP (vamos, que las cosas se guarden en una DB) es implementar o usar un Mapeador Objeto/Relacional y delegar en el la parte de las querys, para poderte concentrar en el Modelo y la solución al software que estás implementando.

Si te interesa ese camino, te pongo el enlace a un ORM que construí bajo licencia GPL que hace Persistencia No Invasiva y por Alcance: https://github.com/PegasusTech/Persistent

Nosotros lo usamos acá para proyectos comerciales que están en producción hace ya un par de años. Permite trabajar y testear Objetos puros que implementan la solución al problema.

Aceptaré gustoso críticas, consultas y sobre todo ejemplos de uso para publicar ;-)

Answer 2

Es la mejor forma de evitar repetir el mismo código constantemente

Te dejo una clase bastante completa
PHP MySQL Class

Answer 3

Te recomiendo desde ya ir viendo y aprendiendo algun Framework sobre PHP como codeigniter.

Saludos

Answer 4

no es que sea inapropiado hacer consultas SQL dentro de una clase, mas que nada, porque ¡en algun sitio habra que hacerlas!. el problema es que 'teoricamente' orientas tu codigo a objetos para 'independizar' trozos de codigo, y ahora mismo tu objeto es dependiente de que exista una conexion abierta a la BBDD (o sea, has tenido que hacer un mysql_connect y un mysql_select_db) ANTES de llamar a ninguna funcion.

Te recomendaria que usaras el patron SINGLETON para encapsular las consultas a las BBDD, usaras esa clase singleton para realizar las consultas y que dicha clase devolviera un array asociativo con las filas devueltas por el mysql_query.

Tambien puedes fabricarte un ORM para cada una de las tablas, pero asi pierdes los JOINS.

en definitiva, no esta mal lo que has hecho (como te dije ¡en algun lado habra que hacer las consultas SQL!), pero no controlas varias cosas (conexion a la BBDD, resultados incorrectos de la BBDD).