Skip to main content
Mejorar la consulta
Origen Enlace
Ferran Muñoz
  • 1.3k
  • 6
  • 14

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 devuelvahaga el insert con los 2 campos y luego utilizarlos más adelanteregistros 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. Por lo tanto, deberías parsear el objeto antes de hacer el INSERT INTO mediante el fetch_fields().

Esto debería funcionar, ya tePrueba este código a ver que tal. He eliminado las 2 consultas select y he optimizado yojuntado el códigoinsert into con loel select para que comentaba antes:inserte los registros que encuentre en el select.

<?php
include 'cn.php';

$codigo_beep$sql = $_POST["codigo_beep"];

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

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

$empleados = $resultado->fetch_fields();

foreach ($empleados as $empleado) {
    $sqlInsert = "INSERT INTO registros (empleado,codigo,fecha,hora) VALUES ($empleado->id, $empleado->codigo, CURDATE(),CURTIME())";

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

header("Location:index.html");
?>

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 devuelva los 2 campos y luego utilizarlos más adelante.

mysqli_query devuelve un objeto, no devuelve un string. Por lo tanto, deberías parsear el objeto antes de hacer el INSERT INTO mediante el fetch_fields().

Esto debería funcionar, ya te he optimizado yo el código con lo que comentaba antes:

<?php
include 'cn.php';

$codigo_beep = $_POST["codigo_beep"];

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

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

$empleados = $resultado->fetch_fields();

foreach ($empleados as $empleado) {
    $sqlInsert = "INSERT INTO registros (empleado,codigo,fecha,hora) VALUES ($empleado->id, $empleado->codigo, CURDATE(),CURTIME())";

    $sqlInsert = mysqli_query($conn, $sqlInsert);
}

header("Location:index.html");
?>

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");
?>

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 devuelva los 2 campos y luego utilizarlos más adelante.

mysqli_querymysqli_query devuelve un objetoobjeto, no devuelve un stringstring. Por lo tanto, deberías parsearparsear el objeto antes de hacer el INSERT INTOINSERT INTO mediante el fetch_fields().

Esto debería funcionar, ya te he optimizado yo el código con lo que comentaba antes:

<?php
include 'cn.php';

$codigo_beep = $_POST["codigo_beep"];

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

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

$empleados = $resultado->fetch_fields();

foreach ($empleados as $empleado) {
    $sqlInsert = "INSERT INTO registros (empleado,codigo,fecha,hora) VALUES ($empleado->id, $empleado->codigo, CURDATE(),CURTIME())";

    $sqlInsert = mysqli_query($conn, $sqlInsert);
}

header("Location:index.html");
?>

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 devuelva los 2 campos y luego utilizarlos más adelante.

mysqli_query devuelve un objeto, no devuelve un string. Por lo tanto, deberías parsear el objeto antes de hacer el INSERT INTO mediante el fetch_fields().

Esto debería funcionar, ya te he optimizado yo el código con lo que comentaba antes:

<?php
include 'cn.php';

$codigo_beep = $_POST["codigo_beep"];

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

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

$empleados = $resultado->fetch_fields();

foreach ($empleados as $empleado) {
    $sqlInsert = "INSERT INTO registros (empleado,codigo,fecha,hora) VALUES ($empleado->id, $empleado->codigo, CURDATE(),CURTIME())";

    $sqlInsert = mysqli_query($conn, $sqlInsert);
}

header("Location:index.html");
?>

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 devuelva los 2 campos y luego utilizarlos más adelante.

mysqli_query devuelve un objeto, no devuelve un string. Por lo tanto, deberías parsear el objeto antes de hacer el INSERT INTO mediante el fetch_fields().

Esto debería funcionar, ya te he optimizado yo el código con lo que comentaba antes:

<?php
include 'cn.php';

$codigo_beep = $_POST["codigo_beep"];

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

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

$empleados = $resultado->fetch_fields();

foreach ($empleados as $empleado) {
    $sqlInsert = "INSERT INTO registros (empleado,codigo,fecha,hora) VALUES ($empleado->id, $empleado->codigo, CURDATE(),CURTIME())";

    $sqlInsert = mysqli_query($conn, $sqlInsert);
}

header("Location:index.html");
?>
Origen Enlace
Ferran Muñoz
  • 1.3k
  • 6
  • 14

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 devuelva los 2 campos y luego utilizarlos más adelante.

mysqli_query devuelve un objeto, no devuelve un string. Por lo tanto, deberías parsear el objeto antes de hacer el INSERT INTO mediante el fetch_fields().

Esto debería funcionar, ya te he optimizado yo el código con lo que comentaba antes:

<?php
include 'cn.php';

$codigo_beep = $_POST["codigo_beep"];

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

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

$empleados = $resultado->fetch_fields();

foreach ($empleados as $empleado) {
    $sqlInsert = "INSERT INTO registros (empleado,codigo,fecha,hora) VALUES ($empleado->id, $empleado->codigo, CURDATE(),CURTIME())";

    $sqlInsert = mysqli_query($conn, $sqlInsert);
}

header("Location:index.html");
?>