Usar el mismo código para validar varios campos

[kenproxd]

Hola,

Mi problema es el siguiente: tengo 8 campos de imágenes que se activan al marcar el input habX. X representa el número de la foto. Ejemplo:

<input type='checkbox' name='habX' OnClick='..'>
<div id='foto_campoX'>
<input type='file' name='fotoX' />
</div>

Uso el siguiente código para validar estos campos:

Código PHP:

if($_POST['hab1'] == true) {
        
        if($_SESSION['foto1'] != '') {
            if(strlen($_FILES['foto1']['name']) > 3) {
                $nombre_foto = strtolower($_FILES['foto1']['name']);
                $nombre_foto = str_replace(' ','_',$nombre_foto);
                
                unlink("../".$_SESSION['foto1']."");
                move_uploaded_file($_FILES['foto1']['tmp_name'], "../images/pisos/$nombre_foto");
                $foto1 = "images/pisos/$nombre_foto";
                
                $_SESSION['foto1'] = "images/pisos/$nombre_foto";
                $cant_fotos++;
            } else {
                $foto1 = $_SESSION['foto1'];
            }
        } else {
            if(is_uploaded_file($_FILES['foto1']['tmp_name'])) {
                $nombre_foto = strtolower($_FILES['foto1']['name']);
                $nombre_foto = str_replace(' ','_',$nombre_foto);
                
                move_uploaded_file($_FILES['foto1']['tmp_name'], "../images/pisos/$nombre_foto");
                $foto1 = "images/pisos/$nombre_foto";
                $_SESSION['foto1'] = "images/pisos/$nombre_foto";
                $cant_fotos++;
            } else {
                $foto1 = '';
            }
        }
        
    }
 

Funciona de maravilla pero el problema es que para validar los 8 campos tendría que escribir el mismo código 8 veces. Parece fácil copiar, pegar y cambiar el número pero esto va a hacer muy pesado mi archivo y es lo que no deseo.

Espero que puedan ayudarme.

[IEKK]

Hay algo que se llaman funciones...

Saludos

[abimaelrc]

Aparte de que existen funciones como te comenta IEKK, también coloca los nombres de los campos algo así

Código PHP:

<input type="text" name="foo[]" />
 

De esa forma es fácil de trabajar luego con PHP.

[duskrow]

simplmente metelo en un ciclo for y el identificador de 1 2 3 4 5 6 7 8 lo metes en la variable de inicio del ciclo

saludos

[kenproxd]

Gracias a todos por sus respuestas.

El problema de usar una function o un ciclo for es que tengo la siguiente variable: $foto1. La cual debe cambiar a $foto2, $foto3, etc.

El escribir $foto$i me va a dar error. Es por eso que buscaba otra solución.

Gracias nuevamente y espero puedan seguir ayudándome

[IEKK]

Iniciado por kenproxd

El escribir $foto$i me va a dar error. Es por eso que buscaba otra solución.

¿y si concatenas? $foto.$i
De todas formas la solución de abimaelrc sigue siendo la más fácil y limpia.

Saludos

[vgonga1986]

Para hacerlo con un for puedes intentar guardar los datos a un array.

En PHP puedes hacer que los índices de un array sean cadenas y no enteros. Es decir, puedes tener el array fotos de la siguiente manera:
$fotos (['foto1'] -> "contenido_foto_1" ['foto2'] -> "contenido_foto_2" ...)

De esta forma, a la hora de rescatar los valores, tendrías que acceder al array como $foto['foto1'], $fotos['foto2'] y así sucesivamente. De esta forma, sí puedes utilizar un bucle for (por ejemplo) para recorrer el array, usando lo siguiente:

Código PHP:

  for($i = 0; $i < count($fotos); $i++) { 
    echo $fotos['foto' . $i]; 
}.  
   

Espero que te sirva, un saludo.

[kenproxd]

Mil gracias por sus respuestas.

La concatenación no me sirvió. Al final logré acomodarlo usando la idea de abimaelrc.

Saludos! :)

[IEKK]

Iniciado por kenproxd

Mil gracias por sus respuestas.

La concatenación no me sirvió. Al final logré acomodarlo usando la idea de abimaelrc.

Saludos! :)

No dije que la solución fuera concatenar, aunque sí que se puede hacer así, sino que te daba error esto :

$foto$i porque eso no se puede hacer así sino que tienes que concatenar, p.e: $var= $foto . $i

Y sí, la idea de abimaelrc era lo mejor. Saludos