Skip to main content
Commonmark migration
Origen Enlace

#Otras observaciones

Otras observaciones

#Otras observaciones

Otras observaciones

Origen Enlace
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Tu código sería menos confuso si hicieras lo siguiente:

  1. En la conversión del JSON evitas forzar una conversión a array. Esto se hace dejando de pasar true como segundo parámetro de json_encode. Al hacer eso sigues trabajando en un contexto de JSON, que permite una sintaxis más clara y más elegante. Es mucho más claro escribir esto: $row->propiedad que escribir esto: $row['propiedad']. En el primer caso no se usan comillas, y eso aporta muchísimo en cadenas como la sentencia INSERT que vas a construir, que lleva por otros lados comillas simples para rodear aquellos valores que son cadenas.
  2. Escribes toda la sentencia dentro de un bloque rodeado por comillas dobles en la apertura y en el cierre. Haciendo eso, puedes escribir toda la sentencia dentro de un mismo bloque, sin tener que usar concatenaciones con el punto (.). También, como te sugiere @DiegoMesa, conviene que escribas código identado, que sea fácil de analizar y de leer.

El código quedaría entonces así:

$servername = "localhost";
$username = "root";
$password = "";
$dbname = "bd_usuarios";

//conexión db
$conexion = mysqli_connect($servername, $username, $password,$dbname) 
or die("Ha sucedido un error inexperado en la conexion de la base de datos");

#CAMBIO AQUÍ: omitimos el parámetro true, para que no fuerce $datos a array como dijimos en (1)
$datos = json_decode($json);

foreach ($datos as $row) {
    #CAMBIO AQUÍ: usamos una variable $sql para más claridad y aplicamos lo dicho en (2)
    $sql="INSERT INTO usuarios 
            (
                document, 
                firstName, 
                lastName, 
                gender, 
                email, 
                phone, 
                productPurchasedTag
            ) 
            VALUES 
            (
                '$row->document',
                '$row->firstName',
                '$row->lastName',
                '$row->gender',
                '$row->email',
                 $row->phone,
                '$row->productPurchasedTag'
            )";
    mysqli_query($conexion,$sql);
}

mysqli_close($conexion);

El código es fácilmente analizable escrito de ese modo. Observa la elegancia que tiene la variable $sql, gana muchísimo en claridad al usar la notación $row->propiedad en cada caso. Aquí, siguiendo el tipo de dato que viene en el JSON, he asumido que phone es de tipo numérico en la base de datos, por eso no he usado comillas simples para encerrar ese dato. Si en la base de datos fuera del tipo VARCHAR entonces tendrías que encerrarlo entre comillas simples, como están los otros datos.

#Otras observaciones

  1. Como te sugería ayer en otra pregunta sobre este código, considera aplicar consultas preparadas para dar seguridad y más eficacia a tu programa. Un usuario mal intencionado podría manipular los datos e inyectar código malicioso en el programa. Aparte de eso, las consultas preparadas son más rápidas en sentencias que deben repetirse, como es el caso de estas, que están en un bucle.

  2. Si no recuerdo mal, se trataba de insertar varias filas. En esos casos conviene usar transacciones.

  3. En el código de ayer usabas el estilo orientado a objetos de mysqli. Aquí veo que has pasado al estilo procedural. Te recomendaría que mantengas el estilo orientado a objetos. Es más claro, más moderno y menos verboso que el estilo procedural.