[Consulta] Clase para conexion a Base de Datos en PHP

[netcirion]

Hola !
Hace poco empecé a programar con objetos y estoy haciendo un pequeño blog. Me gustaría saber si lo que estoy haciendo esta bien, saber que tengo que correjir y demas.

Este es lo que hice:

config.php

Código PHP:

  <?php
define("HTTP_HOST", $_SERVER['HTTP_HOST']);
 
if (HTTP_HOST == "www[dot]misitio[dot]com" || HTTP_HOST == "misitio[dot]com") {
    $dbHost = 'misitio[dot]com';
    $dbUser = 'user';
    $dbPass = 'pass';
    $dbName = 'dbname';
} else {
    $dbHost = 'localhost';
    $dbUser = 'local_user';
    $dbPass = 'local_pass';
    $dbName = 'local_dbname';
}
?>   

connection.class.php

Código PHP:

  <?php
 
class ConfigDataBase {
    private $_dbHost;
    private $_dbUser;
    private $_dbPass;
    private $_dbName;
    
    public function __construct() {
        require 'includes/config.php';
        $this->_dbHost = $dbHost;
        $this->_dbUser = $dbUser;
        $this->_dbPass = $dbPass;
        $this->_dbName = $dbName;
    }
    
    private function __clone() {}
    
    public function getHostDB() {
        $var = $this->_dbHost;
        return $var;
    }
    
    public function getUserDB() {
        $var = $this->_dbUser;
        return $var;
    }
    
    public function getPassDB() {
        $var = $this->_dbPass;
        return $var;
    }
    
    public function getNameDB() {
        $var = $this->_dbName;
        return $var;
    }
}
 
class ConnectionDataBase {
    private $dbHost;
    private $dbUser;
    private $dbPass;
    private $dbName;
    private $link;
    private $result;
    
    function __construct() {
        $this->configConnection();
        $this->setConnection();
    }
    
    protected function configConnection() {
        $config = new ConfigDataBase();
        $this->dbHost = $config->getHostDB();
        $this->dbUser = $config->getUserDB();
        $this->dbPass = $config->getPassDB();
        $this->dbName = $config->getNameDB();
    }
    
    private function __clone() {}
    
    private function setConnection() {
        $this->link = mysql_connect($this->dbHost, $this->dbUser, $this->dbPass);
        if ($this->link)
            mysql_select_db($this->dbName);
        else
            throw new Exception('Database Error No. ' . mysql_errno() . ': ' . mysql_error());
    }
    
    function __destruct() {
        mysql_close($this->link);
    }
}
 
?>   

post.class.php
¿Conviene hacer la conexion aca adentro o es mejor hacerla en el index.php (mas abajo) y despues instanciar cla clase Post?

Código PHP:

  <?php
 
class Post {
    private $link;
    private $query;
    private $result;
    
    function __construct() {
        $this->setConnection();
    }
    
    protected function setConnection() {
        require_once 'app/connection.class.php';
        $this->link = new ConnectionDataBase();
    }
    
    public function getList($totalRow, $orderBy) {
        $this->query = "SELECT * FROM post WHERE status = 'A' ORDER BY " . $orderBy . " DESC LIMIT 0," . $totalRow;
        $this->result = mysql_query($this->query);
        return $this->result;
    }
    
    public function getPostById($id) {
        $this->query = "SELECT * FROM post WHERE status = 'A' AND id = " . $id;
        $this->result = mysql_query($this->query);
        return $this->result;
    }
}
 
?>   

index.php

Código PHP:

  <?php
 
require_once 'app/post.class.php';
$posts = new Post();
$post = $posts->getList(10, 'date');
            
while ($p = mysql_fetch_object($post)) {
        echo '<div style="margin-bottom: 5px;"><a href="view-post.php?      id_post='.$p->id.'" title="">'.$p->title.'</a></div>';
}
mysql_free_result($post);
 
?>   

Espero sus consejos !!

Saludos !

[pateketrueke]

no crees que combinar las funciones de mysql_*() junto con lo que devuelve tu clase es algo incoherente??

osea..

mira este ejemplo (imaginando que sabrás como implementarlo)

Código PHP:

  require_once 'app/post.class.php';
$posts = new Post();
$post = $posts->getList(10, 'date');
            
foreach ($post as $p) {
        echo '<div style="margin-bottom: 5px;"><a href="view-post.php?      id_post='.$p->id.'" title="">'.$p->title.'</a></div>';
} 
   

osea, las funciones de MySQL deben ser internas de tus clases, así.. si algún día cambias tu clase de MySQL a MySQLi pues... ¡¡que ventaja!!!

[bng5]

Iniciado por pateketrueke

no crees que combinar las funciones de mysql_*() junto con lo que devuelve tu clase es algo incoherente??

Estoy totalmente de acuerdo con esa observación.

Y si de verdad querés un consejo: Te recomiendo que, para empezar, hagas una clase que extienda de PDO.