Duda con PDO y mi clase

[neodani]

Iniciado por portalmana

Bueno no tengo del todo claro lo de PDO, puesto que hace poco estoy investigando el tema....
Pero con respecto a tus inquietudes lo que tengo entendido y saque en limpio es lo siguiente:
1> Si es cierto, pero si usas las sentencias preparadas y aparte utilizas en tus consulta o ? o :categoria que son los placeholders o marcadores, que indican que en ese lugar va a ir un valor. En ese caso si la variable es sanada, no en el caso que ya pases la consulta directamente.

2> Generalmente en producción, se trata de no volcar errores por pantalla que puedan dar información de la estructura de tu base, pero si capturas todos los errores y los cambias por otro tipo de mensaje quizás no tengas problemas.

3> Lo que sacaría es el uso del sprintf de:
$query = sprintf("SELECT nombre FROM categorias");
No tiene mucho sentido pues no das formato ni insertas variables a la consulta.
Lo mismo que en $query = sprintf("INSERT INTO categorias (nombre) values ('$this->nombreCategoria')"); pues ya le pasas la variable
Generalmente se usa para consultas prefabricadas y tu solo le pasas los parametros...
En el segundo caso podrías variar por ejemplo

Código PHP:

$sql = "INSERT INTO categorias (nombre) values (%s) ";
$query= sprintf( $sql, $this->nombreCategoria );
 

O puedes usar las consultas preparadas de PDO con los marcadores, que en el enlace que dejas te muestra como se hace.

4> En lo personal me parece mas practico tener una clase separada como la tienes.

5> Si utilizas una instancia, pues estas usando un patrón singleton, es decir tu no accedes directamente al constructor pues este es privado, entonces lo haces con DB::getInstance() que lo que hace es fijarse si ya se instancio la clase si es así te retorna la instancia que ya tiene si no genera la instancia y te la retorna...

Espero te sea de ayuda...
Saludos

Muchas gracias portalmana, no pude responder antes por problemas en la conexión ADSL. Me aclaraste las dudas que tenía. Mil gracias.

Para usar los datos de acceso, en lugar de escribirlos hardcore en la la clase, verdad que puedo tener una clase llamada configBD y cogerlos desde ahí, de la siguiente forma?

Es correcto hacerlo así, cambiaríais algo?

Código PHP:

class ConfigBD{
    public $DBServer;
    public $DBName;
    public $UserName;
    public $Password;
    
    function __construct($DBServer,$UserName,$Password,$DBName){
        $this->DBServer=$DBServer;
        $this->UserName=$UserName;
        $this->Password=$Password;
        $this->DBName=$DBName;
    }
}
 

Código PHP:

class db{
    /*** Declare instance ***/
    private static $instance = NULL;
 
    /**
    *
    * the constructor is set to private so
    * so nobody can create a new instance using new
    *
    */
    private function __construct() {
      /*** maybe set the db name here later ***/
    }
 
    /**
    *
    * Return DB instance or create intitial connection
    *
    * @return object (PDO)
    *
    * @access public
    *
    */
    public static function getInstance() {
 
    if (!self::$instance)
        {
        $conexion = new ConfigBD("localhost","root","jupiter","codelab");
        self::$instance = new PDO("mysql:host=$conexion->DBServer;dbname=$conexion->DBName",$conexion->UserName,$conexion->Password);; //asi funciona y con variables NO.
        self::$instance-> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        }
    return self::$instance;
    }
 
    /**
    *
    * Like the constructor, we make __clone private
    * so nobody can clone the instance
    *
    */
    private function __clone(){
    }
 
} /*** end of class ***/
 
class Categoria{
    private $nombreCategoria;
    
    // Listar todas las categorias de la base de datos
    public function listarCategorias(){
        $query = "SELECT nombre FROM categorias";
        $comando = DB::getInstance()->prepare($query);
        $comando->execute();
 
        while( $datos = $comando->fetch() ) {
        // Imprimimos resultados
        echo $datos[0].'<br />';
        }
    }
 
    // Agrega una nueva categoria
    public function agregarCategoria($nombre){
        $this->nombreCategoria=$nombre;
        $sql = "INSERT INTO categorias (nombre) values (%s) ";
        $query = sprintf( $sql, $this->nombreCategoria );
        $comando = DB::getInstance()->prepare($query);
        $comando->execute();
    }
}
$cat = new Categoria();
$cat->listarCategorias();
?>
 

Muchas gracias de antemano!