Skip to main content
se añadieron 2 caracteres en el cuerpo
Origen Enlace
Pipe
  • 3.3k
  • 1
  • 10
  • 10

Tal como te expliqué en el comentario lo que yo haría sería declarar todos los campos y los que sean opcionales si no tienen valor enviarlos null.

Veo que los unicos campos opcionales son nombre2 y apellido2, entonces sería algo asi:

//A los opcionales les asignas NULL si no tienen valor o tienen un valor vacio
$_REQUEST["nombre2"] = !empty($_REQUEST["nombre2"]) ? "'{$_REQUEST["nombre2"]}'" : "NULL";
$_REQUEST["apellido2"] = !empty($_REQUEST["apellido2"]) ? "'{$_REQUEST["nombre2"]$_REQUEST["apellido2"]}'" : "NULL";
mysqli_query($conexion, "INSERT INTO usuarios (usu_nombre1, usu_nombre2, usu_apellido1, usu_apellido2, usu_email, cel) 
            VALUES ('$_REQUEST[nombre1]', $_REQUEST[nombre2], '$_REQUEST[apellido1]', $_REQUEST[apellido2], '$_REQUEST[email]', '$_REQUEST[cel]')")
            or die ("Problemas en el query: " . mysqli_error($conexion));

Y como también te dijeron anteriormente, no es buena practica concatenar parametros que envia el usuario para crear las sentencias SQL, usa prepared statments y siempre sanitiza los datos de entrada.

Tal como te expliqué en el comentario lo que yo haría sería declarar todos los campos y los que sean opcionales si no tienen valor enviarlos null.

Veo que los unicos campos opcionales son nombre2 y apellido2, entonces sería algo asi:

//A los opcionales les asignas NULL si no tienen valor o tienen un valor vacio
$_REQUEST["nombre2"] = !empty($_REQUEST["nombre2"]) ? "'{$_REQUEST["nombre2"]}'" : "NULL";
$_REQUEST["apellido2"] = !empty($_REQUEST["apellido2"]) ? "'{$_REQUEST["nombre2"]}'" : "NULL";
mysqli_query($conexion, "INSERT INTO usuarios (usu_nombre1, usu_nombre2, usu_apellido1, usu_apellido2, usu_email, cel) 
            VALUES ('$_REQUEST[nombre1]', $_REQUEST[nombre2], '$_REQUEST[apellido1]', $_REQUEST[apellido2], '$_REQUEST[email]', '$_REQUEST[cel]')")
            or die ("Problemas en el query: " . mysqli_error($conexion));

Y como también te dijeron anteriormente, no es buena practica concatenar parametros que envia el usuario para crear las sentencias SQL, usa prepared statments y siempre sanitiza los datos de entrada.

Tal como te expliqué en el comentario lo que yo haría sería declarar todos los campos y los que sean opcionales si no tienen valor enviarlos null.

Veo que los unicos campos opcionales son nombre2 y apellido2, entonces sería algo asi:

//A los opcionales les asignas NULL si no tienen valor o tienen un valor vacio
$_REQUEST["nombre2"] = !empty($_REQUEST["nombre2"]) ? "'{$_REQUEST["nombre2"]}'" : "NULL";
$_REQUEST["apellido2"] = !empty($_REQUEST["apellido2"]) ? "'{$_REQUEST["apellido2"]}'" : "NULL";
mysqli_query($conexion, "INSERT INTO usuarios (usu_nombre1, usu_nombre2, usu_apellido1, usu_apellido2, usu_email, cel) 
            VALUES ('$_REQUEST[nombre1]', $_REQUEST[nombre2], '$_REQUEST[apellido1]', $_REQUEST[apellido2], '$_REQUEST[email]', '$_REQUEST[cel]')")
            or die ("Problemas en el query: " . mysqli_error($conexion));

Y como también te dijeron anteriormente, no es buena practica concatenar parametros que envia el usuario para crear las sentencias SQL, usa prepared statments y siempre sanitiza los datos de entrada.

Origen Enlace
Pipe
  • 3.3k
  • 1
  • 10
  • 10

Tal como te expliqué en el comentario lo que yo haría sería declarar todos los campos y los que sean opcionales si no tienen valor enviarlos null.

Veo que los unicos campos opcionales son nombre2 y apellido2, entonces sería algo asi:

//A los opcionales les asignas NULL si no tienen valor o tienen un valor vacio
$_REQUEST["nombre2"] = !empty($_REQUEST["nombre2"]) ? "'{$_REQUEST["nombre2"]}'" : "NULL";
$_REQUEST["apellido2"] = !empty($_REQUEST["apellido2"]) ? "'{$_REQUEST["nombre2"]}'" : "NULL";
mysqli_query($conexion, "INSERT INTO usuarios (usu_nombre1, usu_nombre2, usu_apellido1, usu_apellido2, usu_email, cel) 
            VALUES ('$_REQUEST[nombre1]', $_REQUEST[nombre2], '$_REQUEST[apellido1]', $_REQUEST[apellido2], '$_REQUEST[email]', '$_REQUEST[cel]')")
            or die ("Problemas en el query: " . mysqli_error($conexion));

Y como también te dijeron anteriormente, no es buena practica concatenar parametros que envia el usuario para crear las sentencias SQL, usa prepared statments y siempre sanitiza los datos de entrada.