0

Lo que estoy intentando hacer es, 2 select para recoger 2 valores y comprobarlos en la base de datos antes del insert y luego hacer el insert, pero cuando lo hago me salta este error:

Recoverable fatal error: Object of class mysqli_result could not be converted to string in C:\xampp\htdocs\xxx\beep\registrar.php on line 13

<?php
include 'cn.php';

$codigo_beep = $_POST["codigo_beep"];


    $sql1 = "SELECT empleados.id FROM empleados, codigos WHERE empleados.codigo LIKE codigos.id AND codigos.codigo = $codigo_beep;";
    $sql2 = "SELECT empleados.codigo FROM empleados, codigos WHERE empleados.codigo LIKE codigos.id AND codigos.codigo = $codigo_beep;";

    $resultado1 = mysqli_query($conn, $sql1);
    $resultado2 = mysqli_query($conn, $sql2);

    $sql3 = "INSERT INTO registros (empleado,codigo,fecha,hora) VALUES ('$resultado1', '$resultado2', CURDATE(),CURTIME())";

    $resultado3 = mysqli_query($conn, $sql3);
    //$resultado3 = $mysqli->query($sql3);

header("Location:index.html");

 ?>

1 respuesta 1

2

Antes que nada podrías optimizar el código, ya que estás haciendo 2 consultas para devolver 2 campos de la misma tabla y la misma condición. Podrías hacer una única consulta que haga el insert con los registros del select que intentas hacer.

mysqli_query devuelve un objeto o devuelve FALSE en caso que la sentencia no haya ido bien. Sea como fuere, no devuelve un string.

Prueba este código a ver que tal. He eliminado las 2 consultas select y he juntado el insert into con el select para que inserte los registros que encuentre en el select.

<?php
include 'cn.php';

$sql = "INSERT INTO registros (empleado,codigo,fecha,hora) SELECT empleados.id, empleados.codigo, CURDATE(), CURTIME() FROM empleados, codigos WHERE empleados.codigo LIKE codigos.id AND codigos.codigo = ".$_POST["codigo_beep"].";";

$sql = mysqli_query($conn, $sql);

header("Location:index.html");
?>
12
  • ¿Qué sentido tiene el uso de fetch_fields() aquí? Por otra parte, el INSERT se puede hacer en base al SELECT sin necesidad de lanzar el SELECT por una parte y el INSERT por otra.
    – A. Cedano
    Commented el 11 mar. 2020 a las 14:29
  • Correcto, sé que se puede hacer todo en una misma sentencia, pero si lo he hecho así es para mantener la estructura que daba. Ahora, si lo quieres todo en una misma sentencia, lo edito y lo pongo, no me supone ningún problema... Commented el 11 mar. 2020 a las 15:09
  • @A.Cedano lo que me has puesto es correcto, en el sentido de que no me da el error de string y tu explicación me ha quedado clara de lo que estaba fallando. Ahora es, no da error pero no me inserta nada en la bd
    – amr07
    Commented el 11 mar. 2020 a las 15:39
  • @amr07 por eso preguntaba cuál es el sentido de usar fetch_fields aquí. Si lo que quieres es insertar los datos obtenidos en el SELECT se debe cambiar fetch_fields por un método fetch que traiga los datos, no los campos. Y, si se necesita insertar todas las filas que traiga el SELECT hay que cambiar la forma de lectura (el foreach así como está no servirá). Ya que Ferran ha escrito la respuesta, sugiero que la modifique en ese sentido para poder insertar los datos que se extraen del SELECT. Si hay algún problema y puedo ayudar, lo haré con gusto.
    – A. Cedano
    Commented el 11 mar. 2020 a las 15:43
  • Entendido, pues voy a ver que puedo hacer Commented el 11 mar. 2020 a las 15:45

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.