Skip to main content
2 de 2
replaced http://es.stackoverflow.com/ with https://es.stackoverflow.com/

Como ya te han mencionado estas mezclando un poco tu código.

El primer error que veo en procesar.php es que no pasas correctamente el valor por la función empty().

//Tu código.
!empty ($db_POST['nombres'])
        ^^^^^^^^

//Actualizado
!empty($_POST['nombres'])
        

Segundo error, tu sentencia nunca va insertar los datos, ya que se te ha olvidado añadir el parámetro de tu conexión a la función mysqli_query(), y en die() utilizas de uno mysql*

//Tu código
mysqli_query("INSERT INTO datos nombres,apellidos,correo_electronico,telefonos,ciudad) VALUES
           ^^^^
'$nomb','$apel','$tele','$corr','$ciud') ") or die (mysql_error());
                                                      ^^^^^^ ^^^^

Otra cosa tener en cuenta es que creas correctamente tu conexión por estilo por procedimientos, que es la que estas usando.

Podrías mejorar y optimizar tu código de la siguiente manera:

conexion.php (Estilo por procedimientos).

<?php
$cone = @mysqli_connect('localhost', 'root', '', 'yerson');

if (!$cone) {
    exit('Error de conexión: ' . mysqli_connect_error());
}
?>

Formulario HTML

<form method="POST" action="procesar.php">

   <input type="text" name="nombres" placeholder="Nombre" />
   <input type="text" name="apellidos" placeholder="Apellidos" />
   <input type="text" name="telefono"  placeholder="Telefono" />
   <input type="text" name="correo_electronico" placeholder="Correo electronico" />
   <input type="text" name="ciudad" placeholder="Cuidad" />

   <input type="submit" name="insertar_datos" value="insertar dactos" /> 
   <input type="reset" value="limpiar" /> 

</form>

procesar.php

<?php
//Reseteamos variables.
$mensaje = $nombres = $apellidos = $telefono = $correo_electronico = $ciudad = NULL;

//Comprobamos si nuestro formulario está definido y no es NULL (mediante el identificador de nuestro botón de envió).
if (isset($_POST['insertar_datos'])) {
    
    //Comprobamos que nuestros inputs no estén vacíos.
    if (empty($_POST['nombres']) && empty($_POST['apellidos']) && empty($_POST['telefono']) && empty($_POST['correo_electronico']) && empty($_POST['ciudad'])) {
        $mensaje = 'Por favor todos los campos son obligatorios.'
    } else {
        //Requeremos conexión.
        require_once'conexion.php';

        //Obtenemos datos, para mayor seguridad utiliza la función mysqli_real_escape_string.
        $nombres = mysqli_real_escape_string($cone,$_POST['nombres']);
        $apellidos = mysqli_real_escape_string($cone,$_POST['apellidos']);
        $telefono = mysqli_real_escape_string($cone,$_POST['telefono']);
        $correo_electronico = mysqli_real_escape_string($cone,$_POST['correo_electronico']);
        $ciudad = mysqli_real_escape_string($cone,$_POST['cuidad']);

        //Si nuestras variables estan definidas, continuamos...
        if ($nombres && $apellidos && $telefono && $correo_electronico && $ciudad) {
            //Insertamos nuestros datos a la Base de datos.
            $sentencia = mysqli_query($cone, "INSERT INTO datos (nombres,apellidos,correo_electronico,telefonos,ciudad) VALUES ('$nombres','$apellidos','$telefono','$correo_electronico','$ciudad')"); 
            
            //Comprobamos que se ejecuto correctamente.
            if(!$sentencia) { #falso.
               printf("Error en ejecutar sentencia: %s\n", mysqli_error($cone));
            } else { #verdadero.
                //Cierra la conexión
                mysqli_close($cone);
                $mensaje = 'Los datos se insertaron correctamente.'
            }

        }
    }
} else { unset($_POST); }

//Alertas.
echo $mensaje;

?>

Aun así, te aconsejo que utilices sentencias preparadas o PDO, para tener mayor seguridad ante ataques de inyección SQL.

¡Un saludo!

D.Bulten
  • 6.6k
  • 2
  • 25
  • 42