Skip to main content
Cambiado isset por !empty
Origen Enlace
masterguru
  • 16k
  • 5
  • 15
  • 28
$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset!empty($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

El principal problema descrito en la pregunta eran los errores ocasionados cuando no se seleccionaban algunas imágenes, los cuales venian dados porque no se verificaba la existencia desi las variables $_FILES de todas las imágenes tenian datos, dando por supuesto que siempre existirían, pues de hecho era la única forma de conseguir que funcionara.

donde, repito, no se verificaba la existencia de si todas las variables $_FILES['imagenX']['tmp_name'] tenian datos primero, y provocando error cuando no se subian todos los archivos de imagen esperados.

if (isset!empty($_FILES['imagenX']['tmp_name'])) {
  $imagenX = addslashes(file_get_contents($_FILES['imagenX']['tmp_name']));
} else {
  $imagenX = null;
}
  $imagenX = (isset!empty($_FILES['imagenX']['tmp_name'])) ? addslashes(file_get_contents($_FILES['imagenX']['tmp_name'])) : null;
$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset!empty($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");
    ${'imagen'.$i} = isset!empty($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

El principal problema descrito en la pregunta eran los errores ocasionados cuando no se seleccionaban algunas imágenes, los cuales venian dados porque no se verificaba la existencia de las variables $_FILES de todas las imágenes, dando por supuesto que siempre existirían, pues de hecho era la única forma de conseguir que funcionara.

donde, repito, no se verificaba la existencia de $_FILES['imagenX']['tmp_name'] primero, y provocando error cuando no se subian todos los archivos de imagen esperados.

if (isset($_FILES['imagenX']['tmp_name'])) {
  $imagenX = addslashes(file_get_contents($_FILES['imagenX']['tmp_name']));
} else {
  $imagenX = null;
}
  $imagenX = (isset($_FILES['imagenX']['tmp_name'])) ? addslashes(file_get_contents($_FILES['imagenX']['tmp_name'])) : null;
$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");
    ${'imagen'.$i} = isset($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = !empty($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

El principal problema descrito en la pregunta eran los errores ocasionados cuando no se seleccionaban algunas imágenes, los cuales venian dados porque no se verificaba si las variables $_FILES de todas las imágenes tenian datos, dando por supuesto que siempre existirían, pues de hecho era la única forma de conseguir que funcionara.

donde, repito, no se verificaba si todas las variables $_FILES['imagenX']['tmp_name'] tenian datos primero, y provocando error cuando no se subian todos los archivos de imagen esperados.

if (!empty($_FILES['imagenX']['tmp_name'])) {
  $imagenX = addslashes(file_get_contents($_FILES['imagenX']['tmp_name']));
} else {
  $imagenX = null;
}
  $imagenX = (!empty($_FILES['imagenX']['tmp_name'])) ? addslashes(file_get_contents($_FILES['imagenX']['tmp_name'])) : null;
$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = !empty($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");
    ${'imagen'.$i} = !empty($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
se añadieron 3154 caracteres en el cuerpo
Origen Enlace
masterguru
  • 16k
  • 5
  • 15
  • 28

Explicación

El principal problema descrito en la pregunta eran los errores ocasionados cuando no se seleccionaban algunas imágenes, los cuales venian dados porque no se verificaba la existencia de las variables $_FILES de todas las imágenes, dando por supuesto que siempre existirían, pues de hecho era la única forma de conseguir que funcionara.

Esto venia dado por este tipo de instrucciones:

$imagen1 = addslashes(file_get_contents($_FILES['imagen1']['tmp_name']));

donde, repito, no se verificaba la existencia de $_FILES['imagenX']['tmp_name'] primero, y si te funciona paso a explicarloprovocando error cuando no se subian todos los archivos de imagen esperados.

La solución fácil y repetitiva hubiera sido comprobarlo de esta forma en cada caso:

if (isset($_FILES['imagenX']['tmp_name'])) {
  $imagenX = addslashes(file_get_contents($_FILES['imagenX']['tmp_name']));
} else {
  $imagenX = null;
}

o bien en su forma abreviada con el operador ternario ?: :

  $imagenX = (isset($_FILES['imagenX']['tmp_name'])) ? addslashes(file_get_contents($_FILES['imagenX']['tmp_name'])) : null;

pero eso, aunque funcionaría bien, hubiera sido demasiado repetitivo desde mi punto de vista, y he preferido encerrarlo todo en un bucle donde se pudiera escoger el número de imágenes sin tener que cambiar el resto del código de este modo:

$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

donde:

  • definimos una variable que nos indica el total de imagenes que esperamos recibir:
$total_imagenes = 5;
  • inicializamos la cadena de valores a pasarle a la query:
$valores = '';
  • recorremos las imagenes esperadas, sabiendo que empiezan por 1:
for ($i = 1; $i < $total_imagenes+1; $i++) {
  • realizamos la asignación de las variables de cada imagen:
    ${'imagen'.$i} = isset($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;

ojo con esta línea, pues aquí jugamos con que esto:

${'imagen'.$i}

en el fondo es una forma de expresar esta variable:

$imagenX

donde la X va siendo sustituida por la $i en cada ciclo.

  • finalmente, dentro del ciclo, vamos concatenando los valores mediante esta línea:
    $valores .= "'".${'imagen'.$i}."',";
  • ya fuera del bucle, lo que hacemos es eliminar la última coma , de la variable $valores mediante esta instrucción:
$valores = rtrim($valores, ",");
  • y por último, insertamos $valores en la query:
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

dando por supuesto que $valores contiene más o menos esto (aunque ya con sus valores finales, que pueden ser null o bien el resultado del file_get_contents):

'$imagen1', '$imagen2', '$imagen3', '$imagen4', '$imagen5'

y si te funciona paso a explicarlo.

Explicación

El principal problema descrito en la pregunta eran los errores ocasionados cuando no se seleccionaban algunas imágenes, los cuales venian dados porque no se verificaba la existencia de las variables $_FILES de todas las imágenes, dando por supuesto que siempre existirían, pues de hecho era la única forma de conseguir que funcionara.

Esto venia dado por este tipo de instrucciones:

$imagen1 = addslashes(file_get_contents($_FILES['imagen1']['tmp_name']));

donde, repito, no se verificaba la existencia de $_FILES['imagenX']['tmp_name'] primero, y provocando error cuando no se subian todos los archivos de imagen esperados.

La solución fácil y repetitiva hubiera sido comprobarlo de esta forma en cada caso:

if (isset($_FILES['imagenX']['tmp_name'])) {
  $imagenX = addslashes(file_get_contents($_FILES['imagenX']['tmp_name']));
} else {
  $imagenX = null;
}

o bien en su forma abreviada con el operador ternario ?: :

  $imagenX = (isset($_FILES['imagenX']['tmp_name'])) ? addslashes(file_get_contents($_FILES['imagenX']['tmp_name'])) : null;

pero eso, aunque funcionaría bien, hubiera sido demasiado repetitivo desde mi punto de vista, y he preferido encerrarlo todo en un bucle donde se pudiera escoger el número de imágenes sin tener que cambiar el resto del código de este modo:

$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

donde:

  • definimos una variable que nos indica el total de imagenes que esperamos recibir:
$total_imagenes = 5;
  • inicializamos la cadena de valores a pasarle a la query:
$valores = '';
  • recorremos las imagenes esperadas, sabiendo que empiezan por 1:
for ($i = 1; $i < $total_imagenes+1; $i++) {
  • realizamos la asignación de las variables de cada imagen:
    ${'imagen'.$i} = isset($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;

ojo con esta línea, pues aquí jugamos con que esto:

${'imagen'.$i}

en el fondo es una forma de expresar esta variable:

$imagenX

donde la X va siendo sustituida por la $i en cada ciclo.

  • finalmente, dentro del ciclo, vamos concatenando los valores mediante esta línea:
    $valores .= "'".${'imagen'.$i}."',";
  • ya fuera del bucle, lo que hacemos es eliminar la última coma , de la variable $valores mediante esta instrucción:
$valores = rtrim($valores, ",");
  • y por último, insertamos $valores en la query:
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

dando por supuesto que $valores contiene más o menos esto (aunque ya con sus valores finales, que pueden ser null o bien el resultado del file_get_contents):

'$imagen1', '$imagen2', '$imagen3', '$imagen4', '$imagen5'
cuerpo editado
Origen Enlace
masterguru
  • 16k
  • 5
  • 15
  • 28

Prueba con esto:

$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset($_FILES['imagem'$_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagem'$_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

y si te funciona paso a explicarlo.

Prueba con esto:

$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset($_FILES['imagem'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagem'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

y si te funciona paso a explicarlo.

Prueba con esto:

$total_imagenes = 5;
$valores = '';
for ($i = 1; $i < $total_imagenes+1; $i++) {
    ${'imagen'.$i} = isset($_FILES['imagen'.$i]['tmp_name']) ? addslashes(file_get_contents($_FILES['imagen'.$i]['tmp_name'])) : null;
    $valores .= "'".${'imagen'.$i}."',";
}
$valores = rtrim($valores, ",");
$agregarPropiedad = $instancia -> query("CALL SP_PROPIEDADES_INSERTAR(".$valores.");");

y si te funciona paso a explicarlo.

Origen Enlace
masterguru
  • 16k
  • 5
  • 15
  • 28
Loading