0

Buenas a todos.

Estoy estudiando PHP y se me ha presentado un problema ingresando datos en MySQL. Tengo un código de ingreso de contactos, me gustaría que me ayuden a mejorarlo, que pueda hacer las consultas y como he de hacerlo, si es posible con algún ejemplo practico.

<?php
if(!empty($_POST))
{
    // Ingreso datos de contactos a la tabla MySQL usando PHP.
    //
    // Primer contacto (Este contacto siempre será ingresado, es obligatorio).
    $contact1_name  = $_POST['cont_nombres0'];
    $contact1_last  = $_POST['cont_apellis0'];
    $contact1_email = $_POST['cont_correo0'];
    $contact1_phone = $_POST['cont_tel0'];
    $contact1_cel   = $_POST['cont_cel0'];
    $contacts[0]    = array($contact1_name, $contact1_last, $contact1_email, $contact1_phone, $contact1_cel);

    //
    // Contactos 2, 3 y 4 (Estos contactos pueden o no estar presentes, tal vez todos o solo uno, o dos o ninguno están presentes)
    //
    $contact2_name  = isset($_POST['cont_nombres1']) ? $_POST['cont_nombres1'] : '';
    $contact2_last  = isset($_POST['cont_apellis1']) ? $_POST['cont_apellis1'] : '';
    $contact2_email = isset($_POST['cont_correo1']) ? $_POST['cont_correo1'] : '';
    $contact2_phone = isset($_POST['cont_tel1']) ? $_POST['cont_tel1'] : '';
    $contact2_cel   = isset($_POST['cont_cel1']) ? $_POST['cont_cel1'] : '';
    $contact2       = array($contact2_name, $contact2_last, $contact2_email, $contact2_phone, $contact2_cel);
    $contacts[1]    = "('" . implode("','", array_unique($contact2)) . "')";
    //$contacts[1]  = isset($data1) ? $data1 : '';

    $contact3_name  = isset($_POST['cont_nombres2']) ? $_POST['cont_nombres2'] : '';
    $contact3_last  = isset($_POST['cont_apellis2']) ? $_POST['cont_apellis2'] : '';
    $contact3_email = isset($_POST['cont_correo2']) ? $_POST['cont_correo2'] : '';
    $contact3_phone = isset($_POST['cont_tel2']) ? $_POST['cont_tel2'] : '';
    $contact3_cel   = isset($_POST['cont_cel2']) ? $_POST['cont_cel2'] : '';
    $contact3       = array($contact3_name, $contact3_last, $contact3_email, $contact3_phone, $contact3_cel);
    $contacts[2]    = "('" . implode("','", array_unique($contact3)) . "')";
    //$contacts[2]  = isset($data2) ? $data2 : '';

    $contact4_name  = isset($_POST['cont_nombres3']) ? $_POST['cont_nombres3'] : '';
    $contact4_last  = isset($_POST['cont_apellis3']) ? $_POST['cont_apellis3'] : '';
    $contact4_email = isset($_POST['cont_correo3']) ? $_POST['cont_correo3'] : '';
    $contact4_phone = isset($_POST['cont_tel3']) ? $_POST['cont_tel3'] : '';
    $contact4_cel   = isset($_POST['cont_cel3']) ? $_POST['cont_cel3'] : '';
    $contact4       = array($contact4_name, $contact4_last, $contact4_email, $contact4_phone, $contact4_cel);
    $contacts[3]            = "('" . implode("','", array_unique($contact4)) . "')";
    //$contacts[3]  = is_null($data3) ? $data3 : '';

    // Necesito guardar los datos en MySQL que está establecido, independientemente de si es solo un contacto, 
    // dos, tres o cuatro. Si dar errores y en una sola consulta.
    for($i=0;$i<4;$i++) {
        if($contacts[$i] != null) {
            $sql = "INSERT INTO contacts VALUES (" . $contacts[$i] . ");";
            echo '<pre>'; 
            var_dump($sql);
            echo '</pre>';
        }
    }
}

Gracias de antemano.

2
  • 1
    Sí, puedes mejorarlo. Como lo tienes ahora el código no ejecuta la consultas, eso es lo primero. Y también, sería bueno que uses consultas preparadas para ejecutar las inserciones. Si optas por PDO será muy fácil preparar la consulta y luego pasarle un array con los datos que quieres insertar, para luego hacer una solo ejecución y por tanto una sola llamada a la base de datos. De todos modos tu pregunta es algo amplia y vaga, en el sentido de que cuando preguntas si puedes mejorarlo, te pueden decir miles de cosas que mejorar. Yo te he dicho las que creo esenciales.
    – A. Cedano
    Commented el 18 dic. 2017 a las 15:26
  • Gracias por tu respuesta, edite el mensaje para que la ayuda sea más extensa. Me gustaría algún ejemplo o donde encontrarlo.
    – Gonzalo R.
    Commented el 18 dic. 2017 a las 15:45

1 respuesta 1

0

Voy a asumir en mi ejemplo que no puedes cambiar el formulario en modo alguno, por lo cual siempre recibes un $_POST similar al que has indicado. Si no fuese así, habría aún más optimizaciones que hacer. También voy a asumir que todo contacto tiene todos los campos rellenados. Si no es así, tendrás que añadir varios tests isset como en tu código original; pero por ahora es más sencillo así.

<?php
if(!empty($_POST))
{
    // Ingreso datos de contactos a la tabla MySQL usando PHP.
    $db = new PDO('...', $usuario, $contra); // Conexión a base de datos usando PDO
    //
    // 
    $i = 1; // Primer contacto. Verás más abajo como lo vamos incrementando.
    while(isset($_POST['cont_nombres'+$i]) {
      $contact_name  = $_POST['cont_nombres'+$i];
      $contact_last  = $_POST['cont_apellis'+$i];
      $contact_email = $_POST['cont_correo'+$i];
      $contact_phone = $_POST['cont_tel'+$i];
      $contact_cel   = $_POST['cont_cel'+$i];
      $contacts[]    = array($contact_name, $contact_last, $contact_email, $contact_phone, $contact_cel);
      $i++;
    }
    // Necesito guardar los datos en MySQL que está establecido, independientemente de si es solo un contacto, 
    // dos, tres o cuatro. Si dar errores y en una sola consulta.
    $db->beginTransaction(); // Así esperaremos a haber terminado todo para enviar la transacción con la base de datos
    $stmt = $db->prepare("INSERT INTO contacts VALUES(?, ?, ?, ?, ?)"); // consulta preparada. Posteriormente, reemplazaremos cada '?' con su valor correspondiente en cada 'execute' que hagamos.
    foreach($contacts as $contact) {
      $stmt->execute(...$contact); // "Splat" (...) convierte un array en una lista de argumentos
    }
    $db->commit(); // Y aquí envíamos la transacción completa.
}
?>

Lectura recomendada: http://php.net/manual/es/book.pdo.php

1
  • gracias por la respuesta, te paso a comentar que el formulario es una tabla que posee dichos campos y son agregados dinámicamente si así son requeridos, pero solo la primera linea de campos está establecida y debe ser rellenada, los demás no son obligatorios y no se sabe si estarán presentes o no. Por otro lado si se ingresa otra linea de datos, todos los campos son obligatorios y por lo tanto estarán todos presentes. Probaré tu código, mil gracias.
    – Gonzalo R.
    Commented el 18 dic. 2017 a las 18:06

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.