1

Buenas tardes,

Tengo este código que se supone que me tiene que dar el número de libros que hay en la base de datos, pero no funciona.

public function getTotalBooks() {

            $sql = $this->db->prepare("SELECT * FROM libros");
            $result = $sql->execute();
            $rows = $result->num_rows();

            echo $rows;

        }

Me da este error:

Fatal error: Uncaught Error: Call to a member function num_rows() on boolean in C:\xampp\htdocs\CBSLibrary\assets\private\classes\class.statics.php:17 Stack trace: #0 C:\xampp\htdocs\CBSLibrary\administration\settings\library.php(265): Statics->getTotalBooks() #1 {main} thrown in C:\xampp\htdocs\CBSLibrary\assets\private\classes\class.statics.php on line 17

He probado a hacerlo de mil maneras después de 1 hora buscando en internet el error, pero no he encontrado ninguna que funcione.

2
  • ¿Estas usando MySQLi o PDO?
    – Marcos
    el 13 jun. 2017 a las 18:14
  • @MarcosGallardo Estoy usando PDO que yo sepa, como soy nuevo capaz que se me ha colado algo de mysqli.
    – Charlie
    el 13 jun. 2017 a las 18:19

1 respuesta 1

2

Tenemos que:

  • PDO::prepare devuelve un PDOStatement, por lo que la variable $sql es un objeto de este tipo.
  • PDOStatement::execute, devuelve bool, por lo que la variable $result es un booleano, es decir no es un objeto, es por esta razón que recibes el error.

Solución:

Una forma de hacerlo sería así:

  • Si la DB posee la función COUNT()

    public function getTotalBooks() {
    
        $sql = $this->db->prepare("SELECT COUNT(*) FROM libros");
        $sql->execute();
        $rows = $sql->fetchColumn();
        echo $rows;
    }
    
  • Si la DB no posee la función COUNT()

    public function getTotalBooks() {
    
        $sql = $this->db->prepare("SELECT * FROM libros");
        $sql->execute();
        $results = $sql->fetchAll(PDO::FETCH_ASSOC);
        echo count($results);
    }
    
5
  • He probado las dos opciones, pero la primera no devuelve nada y la segunda devuelve 0. (Hay 2 libros en la bd) Y el tema de la conexión a la db funciona ya que otras funciones no tienen ningún problema. La verdad es que no entiendo nada, he probado muchas formas y ninguna funciona. @MarcosGallardo (ninguna de las dos opciones me lanza ningun error)
    – Charlie
    el 13 jun. 2017 a las 18:47
  • @CharlieClewer, he actualizado mi respuesta. Olvide llamar a execute después del prepare. Vale la pena mencionar que la opción usando COUNT es la óptima.
    – Marcos
    el 13 jun. 2017 a las 19:01
  • funciona perfectamente ahora, muchas gracias :)
    – Charlie
    el 13 jun. 2017 a las 19:05
  • buena respuesta, como tips : estas en una funcion y no devuelves nada deberias de devolver el resultado el 13 jun. 2017 a las 19:07
  • Buena respuesta, ya que rowCount no es la mejor opción para contar registros. Falta dotar a la función de un return y optimizar la segunda opción, ya que seleccionar todos * para contar ... mejor: SELECT id_libro FROM libros; algunas usan también SELECT 1 FROM tabla aunque como se ha dicho COUNT(*) podría ser la opción más óptima,
    – A. Cedano
    el 14 jun. 2017 a las 2:32

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.