Skip to main content
1 de 2
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

fetch no funciona en INSERT o UPDATE, porque se usa para consultas que devuelven conjuntos de resultados, en combinación con bind_result, para obtener resultados de columnas específicas. En pocas palabras, fetch es para consultas del tipo SELECT, en las cuales puedes seleccionar columnas, vincularlas con bind_result y entonces leerlas usando fetch.

Es preciso cambiar:

if ($stmt->fetch()) {

Por esto:

if ($stmt->affected_rows>0){ 
    //Segundo INSERT

De ese modo debería funcionar.

Aunque yo establecería más controles dentro del bucle (puede que el código falle, por ejemplo al crear claves duplicadas...).

También optimizaría el código de la siguiente manera:

  1. Fuera del bucle, prepararía la inserción:

     $stmt = $con->prepare("INSERT INTO order_details (`id_product`, `quantity`, `price`, `id_order`) VALUES (?, ?, ?, ?) ");
    
  2. Dentro del bucle construiría todas las inserciones a realizar. Esto se podría hacer de diferentes formas. Si tus tablas están en InnoDB lo mejor sería usar transacciones. Si usas PDO esto proceso sería muy simple. Ya he expuesto en otras respuestas que todo lo que tenga que ver con consultas preparadas y varios registros se hace más complicado con mysqli. No obstante, es una tecnología que está ahí y hay que buscar formas de hacer las cosas para quienes persisten en usar esta API.

  3. Una vez preparada nuestra inserción en masa, por la vía que sea... Fuera del bucle ejecutamos la inserción. ¿Por qué hacerlo así? Porque se hace una sola llamada para ejecutar la inserción, no varias llamadas. No sabemos cuántas filas habrá que insertar dentro del bucle. Pero si son cientos, miles, millones... estaríamos haciendo cientos, miles, millones de execute, cuando podemos usar uno solo.

  4. En el punto 2 hablé algo de las Transacciones. Por lo general, es recomendable usarlas cuando tratamos de insertar varias filas. Ya que si por ejemplo necesitamos hacer unas inserciones determinadas y sólo se insertan unas filas y otras no, porque el código falló en alguna de las inserciones... vamos a tener una cierta cantidad de filas insertadas y otras no. ¿Cómo vamos a saber las filas que se insertaron y las que no lo hicieron? La consecuencia es una tabla inconsistente en el sentido de que será una tarea harto difícil determinar exactamente qué inserciones se hicieron. Las transacciones aplican el principio de todo o nada. Es decir, si alguna inserción falla, se deshacen las inserciones previas a la misma... y te retorna un mensaje de error por ejemplo indicando que ninguna fila fue insertada por el motivo X.

En fin, son sólo observaciones y quizá sea interesante tratar este tema en una pregunta aparte, para poder tener la opinión de varios expertos de esta excelente comunidad.

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