2

Al momento de querer hacer un fetch_array me manda este error y no logro resolverlo

Fatal error: Call to undefined method PDOStatement::fetch_array()

mi codigo hasta el momento es el siguiente

<?php 

            $query = "SELECT * FROM mensajes WHERE receiver = $us ORDER BY id DESC";
            $run = $conexion->query($query);

            while ($row = $run->fetch_array()) : <--- aqui marca el error

            ?>
            <!--  -->
              <div id="chat_data">
                <span style="color:green"><?php echo $row['nombre']; ?> </span>
                <span style="color:brown"><?php echo $row['message']; ?></span>
                <span style="float:right"><?php echo fecha($mensas['send']); ?></span>
              </div>

            <?php endwhile; ?>
            </div>
            <form action="single-mensajes.php" method="POST">
              <input type="text" name="name_empresa" placeholder="Ingresa el nombre" value="<?php echo($_SESSION['usuario'][3])  ?>">
              <textarea name="message" id="" cols="30" rows="1" placeholder="Ingresa el mensaje"></textarea>
              <input type="hidden" value="<?php echo $row['receiver']; ?>">
              <input type="hidden" value="<?php echo $row['emitter']; ?>">
              <input type="submit" name="submit" value="send">
            </form>

            <?php

            if (isset($_POST['submit'])) {

              $name = $_POST['name'];
              $message = $_POST['message'];
              $receiver = $_POST['receiver'];
              $emitter = $_POST['emitter'];

              $query = "INSERT INTO mensajes (emitter, receiver, message, name) VALUES('$emitter', '$receiver', '$message', '$name')";

              $run = $conexion->query($query);

              if ($run) {
                echo "<embed loop='false' src='chat.way' hidden='true' autoplay='true' />";
              }
            }            

            ?>
            </div>
5
  • 2
    PDO no tiene un método llamado fetch_array sino un método fetch dentro del cual indicas cómo quieres el resultado. Ejemplo: while ($row = $run->fetch(PDO::FETCH_ASSOC)) {... Ver: El punto 4.2 de esta respuesta y el Manual de PHP
    – A. Cedano
    Commented el 26 jun. 2017 a las 23:35
  • Por otra parte, tu código es vulnerable a la Inyección SQL. Es una mala práctica construir directamente las cadenas SQL con los valores. Los estándares actuales de programación exigen al menos el uso de consultas preparadas, en las cuales se pasa por un lado la consulta SQL con marcadores de posición y por otro lado se pasan los datos.
    – A. Cedano
    Commented el 26 jun. 2017 a las 23:43
  • @A.Cedano claro, que no es un abuena practica, solo lo puse todo junto para que fuera una manera mas facil de leer, pero gracias por el comentario Commented el 27 jun. 2017 a las 3:39
  • 1
    De nada. En lo que tienes que facilitar las cosas cuando preguntas es tratando de explicar lo más claro posible el problema. Poner un código vulnerable no facilita nada, más bien lo complica porque hace que alguien se dedique a indicarte que tu código es vulnerable porque se asume que compartes el código tal y como lo tienes escrito. En mi caso, si te respondo a esta pregunta me voy a tomar el tiempo de indicarte que tu código no es seguro y de mostrarte un código seguro, porque lo considero más importante que un simple error de sintaxis o de método.
    – A. Cedano
    Commented el 27 jun. 2017 a las 11:16
  • @A.Cedano Muchas gracias por tu retroalimentacion, lo tomare en cuenta para futuras preguntas :) Commented el 27 jun. 2017 a las 11:40

1 respuesta 1

3

Cambia la siguiente linea:

while($row = $run->fetch_array()){

por la siguiente:

while($row = $run->fetchAll(PDO::FETCH_BOTH)){

Lo que pasa aquí es que si usas fetch como ya dijo @A.Cedano sólo se esta obteniendo un resultado sin importar que la consulta devuelva 2 o más.

En cuanto a: PDO::FETCH__PALABRA puedes usar PDO::FETCH_BOTH,que devuelve un array que con indices asociativos y con indices numericos o PDO::FETCH_ASSOC que solo te permite usar indices asociativos

2
  • 1
    Creo que para este caso sería mejor usar solo fetch(PDO::FETCH_ASSOC) y no fetchAll, puesto que los resultados van a ser recorridos dentro de un bucle. Ver los puntos 4.2 y 4.4 de: es.stackoverflow.com/a/81029/29967
    – A. Cedano
    Commented el 27 jun. 2017 a las 8:20
  • Esta vez tengo un nuevo problema y he leido el post que compartiste en el link, pero no he tenido acierto ingrese un parametro WHERE y ha tirado error es.stackoverflow.com/questions/82352/… Commented el 28 jun. 2017 a las 23:28

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.