[SOLUCIONADO] Bucle foreach dentro de una sentencia sql

[mensajeescrito]

Hola amigos/as, miren ya ni entiendo por que esta consulta sql no me funciona (lleva php).

Por mas que la reviso me da error de sintaxis y no se por que.

la consulta sql es un insert into, pero con un bucle foreach dentro y este bucle me da error de sintaxis.

Es extraño por que lo veo bien escrito.

Si pudieran echarme una mano lo agradeceria mucho.

Código PHP:

$sql="INSERT INTO " . $nombre_tabla . " (nombre, caracteristicas, antiguedad) VALUES (" 
    
. foreach($array_datos as $datos) {$datos; } .  
                     
")";
 

[gnzsoloyo]

Te da un error de sintaxis a que nivel? ¿MySQL o PHP?
Son cosas y soluciones diferentes.
Por lo pronto, todo eso construye muy mal la query. Recuerda que MySQL no entiende PHP.
Tienes que crear la query para que MySQL la entienda.

[mensajeescrito]

de php, ya que si quito todo el php y dejo solo la variable , no da error de sintaxis.

dicho de otro modo si pongo esto no da errror.

Código PHP:

$sql="INSERT INTO " . $nombre_tabla . " (nombre, caracteristicas, antiguedad) VALUES ("
       
    . $datos . 
                         
    ")";
 

[gnzsoloyo]

No te dará error, pero a menos que $datos contenga la cadena completa de valores para los campos, te dará error de sintaxis al momento de ejecutar la query.

Mira, esto es lo que en tu anterior código no te dará error:

Código PHP:

  $sql="INSERT INTO " . $nombre_tabla . " (nombre, caracteristicas, antiguedad) VALUES (";
foreach($array_datos as $datos) {$sql.= $datos; }
$sql.= ")";  
   

pero eso tampoco te funcionará. Está mal construido.

No te funcionará porque la consulta debe quedar construida mas o menos así:

Código SQL:

INSERT INTO tabla (nombre, caracteristicas, antiguedad) 
VALUES ('edificio', 'kjhkkhjhjh kjhjh', 50);
 

y eso no se realiza con el codigo que propones. Con el tuyo, estos datos quedarían asi:

Código SQL:

INSERT INTO tabla (nombre, caracteristicas, antiguedad) 
VALUES (edificiokjhkkhjhjh kjhjh50);
 

[mensajeescrito]

Te entiendo perfectamente pero eso se soluciona asi creo:

Código PHP:

echo $sql="INSERT INTO " . $nombre_tabla . " (nombre, caracteristicas, antiguedad) VALUES (";
foreach($array_datos as $datos) {$sql.= $datos . ", "; }
$sql.= ")";
 

El asunto es que sigue sin funcionar aun indicando esto ", "

[gnzsoloyo]

No, no se soluciona así exactamente, porque dependiendo del tipo de dato, debe llevar o no apóstrofos...
No te olvides, por favor, que MySQL no entiende variables de PHP, por lo que no puede saber que cuando pones esto:

Código SQL:

INSERT INTO tabla (nombre, caracteristicas, antiguedad)
    VALUES (edificio, kjhkkhjhjh kjhjh, 50)
 

;
los dos primeros quieres que se tomen como cadenas de texto. Para MySL serían nombres de columnas, lo que te daría un error de ejecución.

Y si le pones a todo apostrofos, puedes afectar la performance de los datos numéricos, ya que obligas a MySQL a hacer conversiones implícitas, y se generarán errores de datos si la variable llega vacía sin que te des cuenta.
En inserciones simples no lo notarás, pero en las masivas puede costarte performance.

No es difícil hacer eso dinámico, pero tiene que ser hecho con cuidado.

[mensajeescrito]

gnzsoloyo, de veras se me estan agotando las ideas, entiendo perfectamente lo que me explicas.

Dare un paso atras para tener una mejor perspectiva, vamos a ver:

Tengo este array:

Código PHP:

$array_datos = array('$nombre', '$caracteristicas', '$antiguedad');
 

El cual necesito meter estos valores dentro de la consulta array:

y tengo esto, que a primera impresion lo veo bien en todos los sentidos, pero que no funciona:

Código PHP:

function InsertarDatos($nombre_tabla, $array_datos)
 
$sql="INSERT INTO " . $nombre_tabla . " (nombre, caracteristicas, antiguedad) VALUES (";
      foreach($array_datos as $datos) {
      $sql.=  $datos . ", "; }
      $sql.= ")"; 
 
}
 

Bien dicho esto hago un "echo" para ver donde podria estar el error y la posible solucion:

y me dice esto:

INSERT INTO coches (nombre, caracteristicas, antiguedad) VALUES (
Warning: Invalid argument supplied for foreach() in on line 46

Acto seguido veo el foreach y lo veo bien pero no logro ver el error.

(

[gnzsoloyo]

Te estás complicando la vida.

Código PHP:

  function InsertarDatos($nombre_tabla, $array_datos)
 {
$sql="INSERT INTO " . $nombre_tabla . " (nombre, caracteristicas, antiguedad) VALUES ('";
$sql.=  implode("', '", $array_datos);
$sql.= ")"; 
}  
   

foreach() no es la función adecuada porque necesitas saber anticipadamente cuantos ciclos recorrerá a fin de excluir la coma después del último valor.

[pateketrueke]

El error te dice que la variable que usas en el foreach() no es un array, así que $array_datos seguramente no es lo que tu crees que es.

Eso depende de cómo ejecutas el método InsertarDatos() precisamente, ¿podrías mostrar el código completo de cómo lo haces o hay que adivinar?

[mensajeescrito]

gnzsoloyo, el codigo que has puesto con el implode , lo he probado y no funciona no inserta.

Dice:

Warning: implode(): Invalid arguments passed in on line 43


pateketrueke, este es el codigo que tengo:

archivo 1:

Código HTML:

<form method="POST" action="2---insertar_parte_2.php">
        
            <input type="text" name="nombre" placeholder="Nombre"/>
            
            </br></br>
        
            <input type="text" name="caracteristicas" placeholder="Caracteristicas"/>
            
            </br></br>
        
            <input type="text" name="antiguedad" placeholder="antiguedad"/>
            
            </br></br>
            
            <input type="submit" value="Insertar">
                                  
        </form>
 

archivo 2:

Código PHP:

include 'OperacionesMysql.php'; 
 
    $op = new OperacionesMysql();
        
 
    
    $nombre = $_POST['nombre']; 
    $caracteristicas = $_POST['caracteristicas'];   
    $antiguedad = $_POST['antiguedad']; 
  
  
    
    //CAMPOS OBLIGATORIOS QUE NO PUEDEN ESTAR VACIOS..................................................
    
    if ($nombre == '' or $caracteristicas == '' or $antiguedad == '')
    
        {
            header('Location:2---insertar_parte_1.php');            
        }
        
        else 
        
        {   
            //Lineas para solo poner el nombre de la tabla en el archivo "OperacionesMysql.php" 
            $nombre_tabla = $op->nombre_tabla;  
            
            $op->InsertarDatos($_POST['nombre'], $_POST['caracteristicas'], $_POST['antiguedad'], $nombre_tabla);                               
                        
            header('Location:2---insertar_parte_1.php');
 

archivo 3:

Código PHP:

class OperacionesMysql
        
        {
    
            private $servidor = 'localhost';
            private $usuario = 'root';
            private $pass = '';
            private $basedatos = 'vehiculos';   
    
            public $nombre_tabla = 'coches';
    
            //public $nombre, $caracteristicas, $antiguedad;
            
            public $array_datos = array('$nombre', '$caracteristicas', '$antiguedad');
 
 
 
 
 
            public function ConectarBbdd()
            
            {
                $con = mysqli_connect($this->servidor, $this->usuario, $this->pass, $this->basedatos) or 
                die ('Lo siento pero no conecta a la BBDD');
                
                return $con;        
            }
 
 
 
 
 
            function InsertarDatos($nombre_tabla, $array_datos)
            
                {           
                                    
                echo $sql="INSERT INTO " . $nombre_tabla . " (nombre, caracteristicas, antiguedad) VALUES (";
                      foreach($array_datos as $datos) {
                      $sql.=  $datos . ", "; }
                      $sql.= ")";   
                    
                
                    
                    $con = $this->ConectarBbdd();
                    
                    mysqli_query($con, $sql) or die ('Lo siento pero no se inserta en BBDD' . mysqli_error($con));                                                          
                }   
 
}
 

[pateketrueke]

Mira bien lo que programas:

Código PHP:

function InsertarDatos($nombre_tabla, $array_datos)
 

Ahí especificas sólo dos argumentos, así que el chiste se explica solo:

Código PHP:

$op->InsertarDatos($_POST['nombre'], $_POST['caracteristicas'], $_POST['antiguedad'], $nombre_tabla);
 

Claramente no respetas la firma que tu mismo declaraste en dicho método.

[gnzsoloyo]

Iniciado por mensajeescrito

gnzsoloyo, el codigo que has puesto con el implode , lo he probado y no funciona no inserta.

Dice:

Warning: implode(): Invalid arguments passed in on line 43

¿En serio?

Código PHP:

  //Enter your code here, enjoy! 
$nombre_tabla = "tabla"; 
$array_datos = array('nombre', 'caracteristicas', 'antiguedad'); 
               
$sql="INSERT INTO " . $nombre_tabla . " (nombre, caracteristicas, antiguedad) VALUES ('"; 
$sql.=  implode("', '", $array_datos); 
$sql.= "')";  
echo $sql;  
   

Esto da por resultado (probado):

Código SQL:

INSERT INTO tabla (nombre, caracteristicas, antiguedad) VALUES ('nombre', 'caracteristicas', 'antiguedad')
 

Ásí que NO, mi código no está mal.

[mensajeescrito]

gnzsoloyo, eureka funciona ya.

En el fondo los dos teniamos razon , el codigo que me dijiste con el implode funciona y a la vez no me funcionaba.

La razon es que el codigo funciona pero en el otro archivo como decia

tenia por un lado:

Código PHP:

function InsertarDatos($nombre_tabla, $array_datos)
 

y por otro:

Código PHP:

$op->InsertarDatos($_POST['nombre'], $_POST['caracteristicas'], $_POST['antiguedad'], $nombre_tabla);
 

y esto ultimo debia estar asi:

Código PHP:

$array_datos = array($_POST['nombre'], $_POST['caracteristicas'], $_POST['antiguedad']);            
    
            $op->InsertarDatos($nombre_tabla, $array_datos);
 

Dicho de otra manara para cuando me dijiste lo del implode, todavia no habia cambiado este ultimo codigo.

Ahora ya funciona todo como un reloj suizo.

Mil gracias a los dos por la paciencia. y la ciencia adquirida