Problema con bind_param()

jonyjdv · #1 (**permalink**) 22/12/2015, 13:04

Hola, ya he echo varias consultas en este foro para llegar hasta aquí pero sigue dándome problemas este trozo de codigo que tiene mi web paralizada.

Tengo un pequeño problema con bind_param() y es que necesito que sea capaz de leer un array

Código PHP:

  if ($stmt = $SQL_insert->prepare($consulta)) { 
 
//Contar numero de elementos que contiene el array 
$numero_variables=count($escapar_datos); 
 
$type='s'; 
 
$type=str_repeat($type, $numero_variables); 
 
/* bind parameters for markers */ 
//Asi funciona 
$stmt->bind_param($type, $escapar_datos[0], $escapar_datos[1]); 
 
//PERO NECESITO ALGO ASI Y NO FUNCIONA 
$stmt->bind_param($type, $escapar_datos); 
                    
/* execute query */ 
$stmt->execute(); 
 
/* close statement */ 
$stmt->close(); 
 
}

Saludos

pateketrueke · #2 (**permalink**) 22/12/2015, 14:34

¿Ya has probado con la función call_user_func_array() a propósito?

Código PHP:

Ver originalcall_user_func_array([$stmt, 'bind_param'], $escapar_datos);

jonyjdv · #3 (**permalink**) 23/12/2015, 08:46

Cita:

Iniciado por pateketrueke

¿Ya has probado con la función call_user_func_array() a propósito?

Código PHP:

Ver originalcall_user_func_array([$stmt, 'bind_param'], $escapar_datos);

Hola amigo.

Si y he partido la cabeza intentando hacerlo funcionar, creia que esa era la solucion de hecho pero parece ser que no (el problema creo que esel array). Me pone:

Warning: call_user_func_array() expects parameter 2 to be array, string given in...

Código PHP:

  if ($stmt = $SQL_insert->prepare($consulta)) { 
  //Contar numero de elementos que contiene el array
  $numero_variables=count($escapar_datos);

  $escapar_datos=implode('","',$escapar_datos);
                
  $type='s';

  $type=str_repeat($type, $numero_variables);

  call_user_func_array([$stmt, 'bind_param'], '"'.$escapar_datos.'"');

   /* execute query */
  $stmt->execute();

  /* close statement */
  $stmt->close();
}

[EDITO]
He probado poniendo y quitando las comillas:

Código PHP:

    $escapar_datos=implode('","',$escapar_datos);
  call_user_func_array([$stmt, 'bind_param'], '"'.$escapar_datos.'"');

  //Sin comillas
  $escapar_datos=implode(',',$escapar_datos);
  call_user_func_array([$stmt, 'bind_param'], $escapar_datos);

pateketrueke · #4 (**permalink**) 23/12/2015, 09:32

Caray, dicha función espera un array y tu lo arruinas todo.

Código PHP:

Ver original// MAL
// esta linea no es necesaria, no necesitas convertir dicho array a un string: es es tu error
  $escapar_datos=implode(',',$escapar_datos);
  call_user_func_array([$stmt, 'bind_param'], $escapar_datos);
 
// BIEN
//  $escapar_datos=implode(',',$escapar_datos);
call_user_func_array([$stmt, 'bind_param'], $escapar_datos);

¿Por qué no lees el manual de dicho método para que entiendas cómo usarlo?

jonyjdv · #5 (**permalink**) 23/12/2015, 12:07

Cita:

Iniciado por pateketrueke

Caray, dicha función espera un array y tu lo arruinas todo.

Código PHP:

Ver original// MAL
// esta linea no es necesaria, no necesitas convertir dicho array a un string: es es tu error
  $escapar_datos=implode(',',$escapar_datos);
  call_user_func_array([$stmt, 'bind_param'], $escapar_datos);
 
// BIEN
//  $escapar_datos=implode(',',$escapar_datos);
call_user_func_array([$stmt, 'bind_param'], $escapar_datos);

¿Por qué no lees el manual de dicho método para que entiendas cómo usarlo?

Dios que paciencia tienes conmigo

Te juro que los leo pero debido a mi bajo conocimiento cuando no me sale algo empiezo a googlear, y hay cosas que las busco y consigo solucionarlo pero otras las complico mas.

He introducido el codigo que me has pasado, el implode lo tenia cuando usaba

Código PHP:

  $stmt->bind_param('s', $escapar_datos);

y despues me dio por usarlo con el call_user_func_array porque poniendo el codigo tal cual me lo has pasado me da el siguiente error:

Warning: Parameter 2 to mysqli_stmt::bind_param() expected to be a reference, value given in

Código PHP:

              if ($stmt = $SQL_insert->prepare($consulta)) {  
                 
                $array_final = array('ss',$escapar_datos[0],$escapar_datos[1]); 
 
                call_user_func_array([$stmt, 'bind_param'], $array_final); 
 
                                //Tampoco funciona con $escapar_datos que a diferencia del anterior $array_final no contiene las 'ss' 
 
                                call_user_func_array([$stmt, 'bind_param'], $escapar_datos); 
 
                $stmt->execute(); 
 
                $stmt->close(); 
            }

Muchisimas gracias por todo

pateketrueke · #6 (**permalink**) 23/12/2015, 12:13

Una forma más dinámica de generar las secuencias "ss" es usando array_unshift() y sería mucho mejor considerar otros tipos más allá de strings únicamente:

Código PHP:

Ver original$data = ['a', 1, 'b'];
$params = '';
 
foreach ($data as $value) {
  $params .= is_numeric($value) ? 'i' : 's';
}
 
array_unshift($data, $params);
 
var_dump($data); // ["sis", "a", 1, "b"]

jonyjdv · #7 (**permalink**) 23/12/2015, 12:21

Cita:

Iniciado por pateketrueke

Una forma más dinámica de generar las secuencias "ss" es usando array_unshift() y sería mucho mejor considerar otros tipos más allá de strings únicamente:

Código PHP:

Ver original$data = ['a', 1, 'b'];
$params = '';
 
foreach ($data as $value) {
  $params .= is_numeric($value) ? 'i' : 's';
}
 
array_unshift($data, $params);
 
var_dump($data); // ["sis", "a", 1, "b"]

no seria peor para el rendimiento del servidor? me sigue apareciendo el mismo fallo despues de pasar el valor $data:
Warning: Parameter 2 to mysqli_stmt::bind_param() expected to be a reference, value given in

pateketrueke · #8 (**permalink**) 23/12/2015, 12:26

¿Empeorar el rendimiento en qué sentido?

Igual sería bueno que muestres como has dejado el código para ver si algo anda mal.

jonyjdv · #9 (**permalink**) 23/12/2015, 12:29

Cita:

Iniciado por pateketrueke

¿Empeorar el rendimiento en qué sentido?

Igual sería bueno que muestres como has dejado el código para ver si algo anda mal.

El codigo queda asi:

Código PHP:

              if ($stmt = $SQL_insert->prepare($consulta)) { 
                 
                $params = ''; 
                  
                foreach ($escapar_datos as $value) { 
                  $params .= is_numeric($value) ? 'i' : 's'; 
                } 
                  
                array_unshift($escapar_datos, $params); 
                  
                print_r($escapar_datos); 
 
//Hasta aqui todo bien 
 
                call_user_func_array([$stmt, 'bind_param'], $escapar_datos); 
 
                $stmt->execute(); 
 
                $stmt->close(); 
                 
            }

Pues al tener que hacer foreach no tendria que emplear mas recursos de servidor, no se si me explico correctamente

pateketrueke · #10 (**permalink**) 23/12/2015, 12:54

¿Podrías mostrar el resultado de "Hasta aqui todo bien" para entender lo que resulta?

Código PHP:

Ver originalPues al tener que hacer foreach no tendria que emplear mas recursos de servidor, no se si me explico correctamente

¿Y? Un foreach() no es nada costoso, no debería impactar en nada al rendimiento del servidor.

jonyjdv · #11 (**permalink**) 23/12/2015, 13:27

Cita:

Iniciado por pateketrueke

¿Podrías mostrar el resultado de "Hasta aqui todo bien" para entender lo que resulta?

Código PHP:

Ver originalPues al tener que hacer foreach no tendria que emplear mas recursos de servidor, no se si me explico correctamente

¿Y? Un foreach() no es nada costoso, no debería impactar en nada al rendimiento del servidor.

Ah vale pense que al ser un bucle pense que consumia recursos extra en el servidor.
pues al hacer print_r muestra: Array ( [0] => is [1] => 4 [2] => Titular de la nueva noticia )

pateketrueke · #12 (**permalink**) 23/12/2015, 13:32

Cita:

pues al hacer print_r muestra: Array ( [0] => is [1] => 4 [2] => Titular de la nueva noticia )

Pues yo veo perfecto dicho resultado, ¿podrías entonces mostrar el mensaje de error completo?

jonyjdv · #13 (**permalink**) 23/12/2015, 14:02

Cita:

Iniciado por pateketrueke

Pues yo veo perfecto dicho resultado, ¿podrías entonces mostrar el mensaje de error completo?

Array ( [0] => is [1] => 4 [2] => Titular de la nueva noticia )
Warning: Parameter 2 to mysqli_stmt::bind_param() expected to be a reference, value given in /opt/lampp/... on line 102
La linea 102 contiene:

Código PHP:

  call_user_func_array([$stmt, 'bind_param'], $escapar_datos);

Te pongo a continuacion el cdigo completo

Código PHP:

               if ($stmt = $SQL_insert->prepare($consulta)) {  
                 
                $params = ''; 
                  
                foreach ($escapar_datos as $value) { 
                  $params .= is_numeric($value) ? 'i' : 's'; 
                } 
                  
                array_unshift($escapar_datos, $params); 
                  
                print_r($escapar_datos); 
 
                call_user_func_array([$stmt, 'bind_param'], $escapar_datos); 
 
                $stmt->execute(); 
 
                $stmt->close(); 
                 
            }

pateketrueke · #14 (**permalink**) 23/12/2015, 14:07

¿Estás combinando estilos de OOP y funciones al mismo tiempo?

¿Qué es y qué hace el método $SQL_insert->prepare() eh?

jonyjdv · #15 (**permalink**) 23/12/2015, 14:10

Cita:

Iniciado por pateketrueke

¿Estás combinando estilos de OOP y funciones al mismo tiempo?

¿Qué es y qué hace el método $SQL_insert->prepare() eh?

Pues si :/ de hecho el codigo completo es:

Código PHP:

  function eliminar_nuevos_datos($conexion, $consulta, $escapar_datos){

        // Create connection
        require($_SERVER['DOCUMENT_ROOT'].$conexion);
        
        $SQL_insert = new mysqli($db_host, $db_user, $db_pass, $db_name);

        //Si se quiere escapar los datos
        if($escapar_datos!=false){


            if ($stmt = $SQL_insert->prepare($consulta)) {
                
                $params = '';
                 
                foreach ($escapar_datos as $value) {
                  $params .= is_numeric($value) ? 'i' : 's';
                }
                 
                array_unshift($escapar_datos, $params);
                 
                print_r($escapar_datos);

                call_user_func_array([$stmt, 'bind_param'], $escapar_datos);

                $stmt->execute();

                $stmt->close();
                
            }
}

[EDITO]

Cita:

Iniciado por pateketrueke

¿Qué es y qué hace el método $SQL_insert->prepare() eh?

Pues insertar datos en la bbdd, con $stmt->bind_param('ss', $escapar_datos[0], $escapar_datos[1]); funcionaba :/

pateketrueke · #16 (**permalink**) 23/12/2015, 14:26

Estuve leyendo el manual y creo que falta pasar los datos por referencia, ¿podrías probar lo siguiente?

Código PHP:

Ver originalcall_user_func_array([$stmt, 'bind_param'], &$escapar_datos);

jonyjdv · #17 (**permalink**) 24/12/2015, 08:11

Cita:

Iniciado por pateketrueke

Estuve leyendo el manual y creo que falta pasar los datos por referencia, ¿podrías probar lo siguiente?

Código PHP:

Ver originalcall_user_func_array([$stmt, 'bind_param'], &$escapar_datos);

Nada amigo no hay manera Fatal error: Call-time pass-by-reference has been removed in...

Creo que voy hacer varios if contando el numero de elementos que tiene el array y colocarle algo como:
si tiene 2
$stmt->bind_param('ss', $escapar_datos[0], $escapar_datos[1]);
si tiene 3
$stmt->bind_param('ss', $escapar_datos[0], $escapar_datos[1], $escapar_datos[2]);

pateketrueke · #18 (**permalink**) 24/12/2015, 08:18

Ya veo, ¿podrías intentar una última cosa?

Código PHP:

Ver originalfunction eliminar_nuevos_datos($conexion, $consulta, &$escapar_datos)

Agregar el & sólo en la declaración de la función, nada más.

jonyjdv · #19 (**permalink**) 24/12/2015, 08:34

Cita:

Iniciado por pateketrueke

Ya veo, ¿podrías intentar una última cosa?

Código PHP:

Ver originalfunction eliminar_nuevos_datos($conexion, $consulta, &$escapar_datos)

Agregar el & sólo en la declaración de la función, nada más.

Warning: Parameter 2 to mysqli_stmt::bind_param() expected to be a reference, value given in

Pues nada amigo, no hay manera es un tanto frustrante