[SOLUCIONADO] Novato con POO: me salta error al hacer consultas

[PSPforever]

Hola

Estoy empezando con POO y tengo creado estas 2 clases:

conn.inc.php

Código PHP:

<?php
 
define("BD", "artwork");
define("HOST", "localhost");
define("USER", "root");
define("PASS", "");
 
//global $db;
 
class Conexion
{
    public $db;
 
    public function conectar($dbase)
    {
        try{
            $this -> db = $dbase;
            $this -> db = new PDO('mysql:host=' . HOST . ';dbname=' . $dbase, USER, PASS);
            echo "Ok Connect";
        } catch(PDOException $e) {
            echo "Error: " . $e->getMessage();
        }
    }
 
}
 
$con = new Conexion();
$con -> conectar(BD);
 
//error_reporting(0);
 
?>
 

paginasModelo.php

Código PHP:

<?php
 
require("../cgi-bin/conn.inc.php");
 
class paginasModelo
{
 
    public function consultaPaginas()
    {
        try{
            $consulta = $db -> prepare('SELECT id, titulo, orden FROM paginas');
            $consulta -> execute();
            return $consulta -> fetchAll();
            echo "Ok";
        } catch(PDOException $e) {
            echo "Error: ".$e -> getMessage();
        }
    }
}
 
$pag = new PaginasModelo();
$pag -> consultaPaginas();
 
?>
 

Y al entrar en paginasModelo.php me salta el error:

Código PHP:

Ok Connect
Notice: Undefined variable: db in C:\xampp\htdocs\photoartwork\modelos\paginas.modelo.php on line 11
 
Fatal error: Call to a member function prepare() on a non-object in C:\xampp\htdocs\photoartwork\modelos\paginas.modelo.php on line 11
 

Seguro que es una bobada, pero no logro dar con la solución.

Espero vuestra ayuda, gracias por anticipado.

[h2swider]

Justamente $db no existe para el metodo, te lo dice claramente el error,

$db es un atributo publico de la clase Conexion y tu clase paginasModelo no tiene alcance sobre ese atributo.

Podrias pasarle db como un parametro.

$pag -> consultaPaginas( $con->db);

y luego

public function consultaPaginas( $db)

[rbczgz]

Hola PSPforever,

Yo no estoy muy "puesto" en PHPOO, pero en principio esto solo es un aviso, no un error, y es porque no has declarado la variable antes de asignarle valor:

Iniciado por PSPforever

Código PHP:

Ok Connect
Notice: Undefined variable: db in C:\xampp\htdocs\photoartwork\modelos\paginas.modelo.php on line 11
 

Aquí lo que te está diciendo es que no existe la función prepare() que estás llamando en esa línea, y por lo que se ve en la clase, ahí no está declarada la función.

Iniciado por PSPforever

Código PHP:

Fatal error: Call to a member function prepare() on a non-object in C:\xampp\htdocs\photoartwork\modelos\paginas.modelo.php on line 11
 

Seguro que es una bobada, pero no logro dar con la solución.

Espero vuestra ayuda, gracias por anticipado.

No sé si te servirá de ayuda pero es lo que veo yo en lo que muestras, espero que te sirva...

.

[h2swider]

rbczgz En mi respuesta explico claramente el error, esta tratando de acceder a un atributo de otra clase..

[rbczgz]

Iniciado por h2swider

rbczgz En mi respuesta explico claramente el error, esta tratando de acceder a un atributo de otra clase..

Tienes razón, h2swider, es que nos hemos pisado en la respuesta, yo estaba respondiendo y cuando he terminado, he visto que ya habías respondido tú mejor que yo

.

[PSPforever]

Muchas gracias por vuestra ayuda, ya se ha solucionado.

Pero ahora tengo otro problema, creo un bucle para mostrar los datos fuera de la clase paginaModelo, pero no me muestra nada en pantalla:

Código PHP:

require("../cgi-bin/conn.inc.php");
 
class paginasModelo
{
 
    public function consultaPaginas($db)
    {
        try{
            
            $db -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $consulta = $db -> prepare('SELECT id_pagina, titulo, orden FROM paginas');
            $consulta -> execute();
            $paginas = $consulta -> fetchAll(); 
 
        } catch(PDOException $e) {
 
            echo "Error: ".$e -> getMessage();
 
        }
    }
 
}
 
$pag = new PaginasModelo();
$pag -> consultaPaginas( $con -> db );
 
foreach($paginas as $pagina){
    echo $pagina["id_pagina"].$pagina["titulo"].$pagina["orden"];       
}
 

La idea es mostrar los datos del bucle foreach fuera de la clase pero no me pinta nada. Una vez funcione lo añadiré en una vista HTML.

Espero vuestra ayuda, gracias.

[hhs]

Iniciado por PSPforever

Muchas gracias por vuestra ayuda, ya se ha solucionado.

Pero ahora tengo otro problema, creo un bucle para mostrar los datos fuera de la clase paginaModelo, pero no me muestra nada en pantalla:

Código PHP:

require("../cgi-bin/conn.inc.php");
 
class paginasModelo
{
 
    public function consultaPaginas($db)
    {
        try{
            
            $db -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $consulta = $db -> prepare('SELECT id_pagina, titulo, orden FROM paginas');
            $consulta -> execute();
            $paginas = $consulta -> fetchAll(); 
 
        } catch(PDOException $e) {
 
            echo "Error: ".$e -> getMessage();
 
        }
    }
 
}
 
$pag = new PaginasModelo();
$pag -> consultaPaginas( $con -> db );
 
foreach($paginas as $pagina){
    echo $pagina["id_pagina"].$pagina["titulo"].$pagina["orden"];       
}
 

La idea es mostrar los datos del bucle foreach fuera de la clase pero no me pinta nada. Una vez funcione lo añadiré en una vista HTML.

Espero vuestra ayuda, gracias.

Código PHP:

// El metodo consultaPaginas no esta devolviendo ningún valor
$paginas = $consulta -> fetchAll();
return $paginas;
 

[PSPforever]

Gracias hhs, pero el problema sigue igual, no me muestra nada en pantalla. Pongo el código actualizado:

Código PHP:

<?php
 
require("../cgi-bin/conn.inc.php");
 
class paginasModelo
{
    public function consultaPaginas($db)
    {
        try{
            
            $db -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $consulta = $db -> prepare('SELECT id_pagina, titulo, orden FROM paginas');
            $consulta -> execute();
            $paginas = $consulta -> fetchAll();
            return $paginas;    
 
        } catch(PDOException $e) {
 
            echo "Error: ".$e -> getMessage();
 
        }
    }
 
}
 
$pag = new PaginasModelo();
$pag -> consultaPaginas( $con -> db );
 
foreach($paginas as $pagina){
    echo $pagina["id_pagina"].$pagina["titulo"].$pagina["orden"];       
}
 
?>
 

Está claro que el problema es mi inexperiencia. ¿Qué hago mal?

[alex1084]

lo que pasa que la variable que estas ocupando en el foreach $paginas no veo donde le asignes algun valor....
Por lo tanto en el foreach no encuentra nada...

[PSPforever]

Nuevamente gracias, he solucionado lo del bucle. Y por último, pregunto sobre otro fallo:

Código PHP:

<?php
 
require("../cgi-bin/conn.inc.php");
 
class paginasModelo
{
 
    public $paginas;
    public $nameTable;
 
    public function queryTable($qTable)
    {
        $this -> nameTable = $qTable;
        //return $this -> nameTable;
    }
 
    public function consultaPaginas($db)
    {
        try{
 
            $db -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            $consulta = $db -> prepare(" SELECT id_pagina,titulo,orden FROM ". $this -> queryTable()."" );
            $consulta -> execute();
            $this -> paginas = $consulta -> fetchAll();
            //return $this -> paginas;  
 
        } catch(PDOException $e) {
 
            echo "Error: ".$e -> getMessage();
 
        }
    }
 
}
 
$pag = new PaginasModelo();
$pag -> consultaPaginas( $con -> db );
$pag -> queryTable("paginas");
 
foreach($pag -> paginas as $pagi){
    extract($pagi);
    echo $id_pagina.$titulo.$orden;     
}
 
?>
 

En pantalla me salta el mensaje de error:

Error: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Pero no entiendo porqué. La sintaxis creo que es correcta y está bien concatenado.

El problema está en la línea 22 que creo el SELECT y escribo $this -> queryTable()

[Italico76]

El problema es que queryTable() no te has decidido si es un getter o un setter

Código PHP:

<?php
 
    private $paginas;
    private $nameTable;
 
    public function setTable($qTable)
    {
        $this -> nameTable = $qTable;
        return $this;
    }
 
    public function getTable()
    {
        return $this -> nameTable;        
    }
 
// ..
 

Código PHP:

$consulta = $db -> prepare(" SELECT id_pagina,titulo,orden FROM ". $this -> getTable()."" );
 

Casi no deberias usar atributos publicos y los getters depende si los necesitas pero asi como esta solucionas el problema

[PSPforever]

gracias italico, he probado los cambios que dices pero me sigue saltando el error de sintaxis.

[Italico76]

Intenta debuguear:

Código PHP:

$sql = "SELECT id_pagina,titulo,orden FROM {$this-> queryTable()}";
echo $sql;
$consulta = $db -> prepare($sql);
var_dump($consulta);
 

[gnzsoloyo]

Error: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

En el 99% de los casos con consultas SQL embebidas en PHP, el problema es del contenido de las variables, que o no llegan, o lo que llega es incorrecto o valores inválidos. Pero en tu caso eso es (como casi siempre) imposible de saber, porque no estás mirando lo que queda construido de SQL. Tu asumes que está bien, pero no lo verificas.
Tu construcción es así:

Código PHP:

$consulta = $db -> prepare(" SELECT id_pagina,titulo,orden FROM ". $this -> queryTable()."" );
$consulta -> execute();
 

Pero como mínimo deberías hacer esto:

Código PHP:

$qry = " SELECT id_pagina,titulo,orden FROM ". $this -> queryTable();
echo $qry;
$consulta = $db -> prepare($qry);
$consulta -> execute();
 

De ese modo al menos podrás ver qué es lo que se construye en esa variable, y por ende asegurarte por qué está generando un error de sintaxis. Esto desde el momento en que la sentencia en sí (sin considerar el nombre de la tabla), no contiene errores. Por lógica el error lo produce el valor de ese parámetro.

Verifica eso y veamos.

[Italico76]

@gnzsoloyo: de pronto no fui suficientemente claro pero creo que tambien dije eso con el adicional que le digo que re-organice la consulta para aprovechar las comillas dobles que permite que las variables e invocacion del getter queden dentro:

Código PHP:

$sql = "SELECT id_pagina,titulo,orden FROM {$this->getTable()}";
echo $sql;
 

[gnzsoloyo]

Mi intención es hacer hincapié en los datos, no en el proceso, para que tenga en cuenta que la consulta está bien estructurada, por lo que el error debe forzosamente estar en los datos de la variable.
Por eso no hice referencia a tus posts, sino que tomé segmentos del último post del autor del thread.

[Italico76]

Tranquilo...aprovecho para hacer hincapié en otro aspecto tambien importante:

var_dump() logicamente se usa para debug pero ..

echo se usan para debug solo cuando no se pueden hacer pruebas unitarias y cuando NO se esta dentro de una vista. Un echo fuera de una Vista es sinonimo de brutalidad (!)

[PSPforever]

Nuevamente gracias. He probado lo ultimo que decís, con estas líneas:

Código PHP:

$qry = " SELECT id_pagina,titulo,orden FROM ". $this -> queryTable();
            echo $qry;
            $consulta = $db -> prepare($qry);
            //var_dump($consulta);
 

En pantalla imprime la consulta con el error:

SELECT id_pagina,titulo,orden FROM Error: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Y activando el var dump aparece más detalle:

Código PHP:

SELECT id_pagina,titulo,orden FROM 
object(PDOStatement)#4 (1) { ["queryString"]=> string(36) " SELECT id_pagina,titulo,orden FROM " } 
Error: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1
 

Intuyo que $this->queryTable() no muestra valor.

[Italico76]

Y haces el setter antes ? o sea... esas funciones no tienen mucho misterio pero incluso las validé y estan bien

[PSPforever]

Gracias a todos por vuestra ayuda, por fin ha funcionado perfectamente sin que dé ningún error, lo pongo por si resulta de utilidad a alguien:

Código PHP:

<?php
 
require("../cgi-bin/conn.inc.php");
 
class paginasModelo
{
 
    public $paginas;
    private $nameTable;
 
     public function setTable($qTable)
    {
        $this -> nameTable = $qTable;
        return $this;
    }
 
    public function getTable()
    {
        return $this -> nameTable;        
    }
 
    /*public function queryTable($qTable)
    {
        $this -> nameTable = $qTable;
    }*/
 
    public function consultaPaginas($db)
    {
        try{
 
            $db -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            //$qry = " SELECT id_pagina,titulo,orden FROM ". $this -> getTable();
            //echo $qry;
            //$consulta = $db -> prepare($qry);
            //var_dump($consulta);
            $consulta = $db -> prepare(" SELECT id_pagina,titulo,orden FROM ". $this -> getTable() );
            $consulta -> execute();
            $this -> paginas = $consulta -> fetchAll();
            //return $this -> paginas;  
 
        } catch(PDOException $e) {
 
            echo "Error: ".$e -> getMessage();
 
        }
    }
 
}
 
$pag = new PaginasModelo();
$pag -> setTable("paginas");
$pag -> consultaPaginas( $con -> db );
 
foreach($pag -> paginas as $pagi){
    extract($pagi);
    echo $id_pagina.$titulo.$orden;     
}
 
?>
 

Disculpad la lata que os he dado a algunos, pero me interesa mucho POO e ir aprendiendo conceptos y habituarme a pensar en objetos.

Por último os pregunto, ¿conocéis alguna guía de buenas prácticas de POO en PHP ?

De nuevo, os doy las gracias.

[Italico76]

El libro PHP objects, patterns and practice es muy bueno

[PSPforever]

Gracias por el libro.