Skip to main content
1 de 2
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Las consultas con PDO devuelven un recurso y necesitas que ese recurso esté activo (junto a la conexión) para leerlo. Me parece que al retornar el recurso desde la clase e intentar leerlo fuera de ella, la conexión (y en consecuencia el recurso) serían nulos en ese ámbito. Lo que debes hacer es leer el recurso en el método selectProducto() llenar un array por ejemplo y devolver ese array. Cuando el método hace return los recursos se cierran y no tienes contexto en el receptor para usar métodos como fetch.

Aparte de eso, si usas prepare(), es obligatorio usar execute(), aunque para este caso puedes usar query() porque no hay riesgo de inyección SQL.

Si lo haces así debería funcionar, salvo otros errores:

public function selectProducto()
{
    $conexion = new conexionBBDD();
    $con = $conexion->conexion();
    $select = $con->prepare('SELECT * FROM productos');
    $select ->execute();
    $mData=array();
    if(!$select) {
        /* No conviene mostrar errores internos en producción*/
        $mData["error"]="Error de consulta de ejecución, porque: ". $con->errorInfo()[2];
    }else {
        while ($row = $select->fetch(PDO::FETCH_ASSOC)) {
              $mData[] = $row;
        }
    }
    return $mData;
}

Como ves, los datos se leen en contexto en el que se obtienen. Luego, en el receptor, lees el array.

$allProducts = $productos->selectProducto();
foreach ($allProducts as $product){
    echo "$product[nombre] $product[precio] $product[cantidad]\n";
}

Tengo algunas críticas a tu forma de modelar las clases, pero ese es otro tema. Haciendo esto, debería funcionar, salvo otros errores en otras partes del código.

A. Cedano
  • 95.2k
  • 31
  • 145
  • 257