subir y renombrar archivos con php- move_upload_file

[cristo995]

buenas para todos.
Tengo una aplicacion la cual una de las secciones es subir imagenes.
-uno de los problemas que tenia era que si por ejemplo subia una imagen con el mismo nombre de alguna otra imagen que ya se encontrara en dicho directorio:
-la imagen no se subia y aparte la otra que ya estaba se borraba o algo por el estilo.

investigando y en este mismo foro encontre esta solucion.

function random(){

$str = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijkmnopqrstuvwx yz234567890ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijkmn opqrstuvwxyz234567890";
$cad = "";
for($i=0;$i<10;$i++) {
$cad .= substr($str,rand(0,120),1);
}

return $cad;

}

$nombre_archivo = $_FILES['archivo']['name'];
$nombre_archivo=random().".jpg";

if(move_uploaded_file($_FILES['archivo']['tmp_name'], "imagenes/$nombre_archivo")){

$msg.="El archivo ha sido cargado correctamente.<br>";


}

esto funciona de maravilla, ya que si la imagen se llama carro.jpg me queda guardada con el nombre QWEExx876t y cuando subo otra imagen se guarda con otra combinacion y asi sucesivamente ...


pero entonces mi pregunta es si en un futuro no habra problema de que dicha combinacion se repita y guarde una imagen con el mismo nombre de alguna otra que ya este y se me forme problema ??

que probabilidades hay de que alguna combinacion se repita ??

[gartzia6]

Hola buenas.

Pues supongo que las probabilidades serán muy pero que muy bajas.

Pero porque al nombre del archivo no le añades el día de hoy, así las probabilidades bajaran muchísimo, porque que en el mismo día te haga dos veces la misma combinación es casi imposible.

[Eleazan]

¿Pq no compruebas si existe?

Código PHP:

  while(file_exists($nombre_archivo)) $nombre_archivo=random().".jpg";

if(move....  
   

[cristo995]

pues muchas gracias. No se por que no se me ocurreio

aparte en el siclo for lo puse hasta <20 mas los datos de la fecha una imagen me quedaria de nombre: aYEUktCK24ZQmdLyWRPBe2014-03-06.jpg
seria deverdad algo casi imposible que el mismo dia se repitiera la misma combinacion jejej

gracias por tu ayuda.

[cristo995]

da.. no conocia ese file_exis...
lo tendre en cuenta muchas gracias.

[alexisverano]

Te dejo como resolvi ese problema.

Código PHP:

//RUTA DONDE SE GUARDARAN LOS ARCHIVOS
$destino = 'archivos/' ; 
 
//SE DEFINEN LOS DATOS DEL ARCHIVO
$nombre_archivo = $_FILES['archivo']['name'];
$tipo = $_FILES['archivo']['type'];
$tamano = $_FILES['archivo']['size'];
 
//GUARDAMOS EL ARCHIVOS
move_uploaded_file ($_FILES ['archivo']['tmp_name'], $destino . '/' . $_FILES ['archivo']['name']);
            
//CONSTRUIMOS UN CODIGO UNICO PARA RENOMBRAR
$codigo_fecha = date("YmdHis");         
$no_aleatorio = rand(100, 999); //GENERAMOS TRES DIGITOS PARA INCORPORARLO AL FINAL DEL CODIGO
$codigo = $codigo_fecha.$no_aleatorio; //CODIGO DE 17 DIGITOS
 
//RENOMBRAMOS EL ARCHIVO SUBIDO
list($nombre, $ext) = explode(".", $nombre_archivo);
$nombre_actual = "$codigo"."."."$ext" ;
rename("archivos/$nombre_archivo","archivos/$nombre_actual");
 

Esto es solo un pedazo de codigo, faltan las validaciones y las comprobaciones si las extenciones y tamaños permitidos son validos.

[Eleazan]

Y si el archivo lo sube con un nombre ya existente?

Lo sobreescribiria....

Deberías obtener primero el nombre, y después moverlo ;)

[alexisverano]

Bueno, seria un poco dificil (aunque en la vida no hay nada imposible), que estuviera un archivo con un nombre identico al codigo construido, en el cual esta involucrada la fecha.
De todos modos, se pueden ingeniar variantes, como primero subir la imagen a una carpeta temporal y despues de cambiado el nombre, moverla a la carpeta original.

[alexisverano]

Bueno, seria un poco dificil (aunque en la vida no hay nada imposible), que estuviera un archivo con un nombre identico al codigo construido, en el cual esta involucrada la fecha.
De todos modos, se pueden ingeniar variantes, como primero subir la imagen a una carpeta temporal y despues de cambiado el nombre, moverla a la carpeta original.

[menteccato]

Efectivamente, en mi opinión, la solución aportada por alexisverano es la más adecuada. No solo aporta una cadena única -fecha y hora- sino que además incluye en sí misma una información adicional. Con la fecha y la hora sería suficiente en un entorno monousuario, pero un entorno que simultáneamente suban archivos multitud de personas, es más que probable los conflictos, por eso añadir una cadena adicional de 3 o más caracteres es lo más seguro.

Aún así me sorprende leer las dudas que plantea el método propuesto por el autor de este hilo: el de una cadena alfanumérica de 10 caracteres. Él preguntaba "¿qué probabilidades hay de que alguna combinacion se repita?" y en general todos se han limitado a decir que es muy poco probable. ¿Muy poco probable? ¿Sólo muy poco probable? Las variaciones de 10 elementos tomados de una serie de 62 (26 mayúsculas + 26 minúsculas + 10 números) es, matemáticamente: 62 elevado a 10. Y eso es lo bastante cercano a una posibilidad CERO. Para que os hagáis una idea es casi 1 posibilidad entre UN TRILLÓN. Una posibilidad entre un MILLÓN de BILLONES...

Entenderéis ahora que la posibilidad de hacer los mismo pero con una longitud de 20 caracateres + fecha, me haya parecido sorprendente. No son necesarias tantas precauciones cristo995, jajaja. Con 20 caracteres más la fecha ya ni lo calculo, no sabría ni nombrar esa cantidad...

Aún así se puede comprobar si el archivo existe... pero vamos, con ese algoritmo... yo no perdería ni quince segundos en ese código, es más fácil que me toque la bono-loto cada día durante una semana.

[Eleazan]

Iniciado por menteccato

Efectivamente, en mi opinión, la solución aportada por alexisverano es la más adecuada. No solo aporta una cadena única -fecha y hora- sino que además incluye en sí misma una información adicional. Con la fecha y la hora sería suficiente en un entorno monousuario, pero un entorno que simultáneamente suban archivos multitud de personas, es más que probable los conflictos, por eso añadir una cadena adicional de 3 o más caracteres es lo más seguro.

Aún así me sorprende leer las dudas que plantea el método propuesto por el autor de este hilo: el de una cadena alfanumérica de 10 caracteres. Él preguntaba "¿qué probabilidades hay de que alguna combinacion se repita?" y en general todos se han limitado a decir que es muy poco probable. ¿Muy poco probable? ¿Sólo muy poco probable? Las variaciones de 10 elementos tomados de una serie de 62 (26 mayúsculas + 26 minúsculas + 10 números) es, matemáticamente: 62 elevado a 10. Y eso es lo bastante cercano a una posibilidad CERO. Para que os hagáis una idea es casi 1 posibilidad entre UN TRILLÓN. Una posibilidad entre un MILLÓN de BILLONES...

Entenderéis ahora que la posibilidad de hacer los mismo pero con una longitud de 20 caracateres + fecha, me haya parecido sorprendente. No son necesarias tantas precauciones cristo995, jajaja. Con 20 caracteres más la fecha ya ni lo calculo, no sabría ni nombrar esa cantidad...

Aún así se puede comprobar si el archivo existe... pero vamos, con ese algoritmo... yo no perdería ni quince segundos en ese código, es más fácil que me toque la bono-loto cada día durante una semana.

Aunq es algo viejo, vuelvo a ver el mismo fallo en ese código. Y no tiene que ver con las cadenas o las probabilidades, sino con que puedo borrar / sobreescribir cualquier fichero.

Imaginate, por un casual, que se que un usuario tiene la foto de perfil siguiente:
"20150504110854654.jpg", que sería una posible combinación que crearía el código de @alexisverano.

Ahora, yo creo una imagen, le pongo ese nombre, y la subo al servidor. ¿Qué pasaría?

Ese era el problema original del autor (podía sobreescribir imágenes) y la solución planteada en ese trozo de código(el que tu dices, de alexis), seguía con el mismo problema (aunq luego comentó la posibilidad correcta de una carpeta temporal).

Nunca está de más comprobar si un archivo existe o no ;). Sea una probabilidad entre 10 o entre 1 trillón.

Por ejemplo, he leído (del 2011) q a Facebook se suben 250 millones de foto al día. Diles que no hace falta más que 3 dígitos por segundo. O sólo 10 carácteres, cuando una foto suya puede ser, por ejemplo, "10632840_10152794513413045_5524611406974723287_n" .

Q si, q probablemente este usuario no tiene un "facebook", ni su capacidad, ni sus usuarios. Y por eso mismo la operación de comprobar si existe el fichero, no será tan costosa como para ignorarla

[BanNsS1]

Bajo mi punto de vista tienes 3 opciones viables:

1- Generar el nombre mediante time() (devuelve el tiempo en segundos desde 31/12/1969 a las 7:00 hasta ahora) y algún carácter aleatorio. De esta forma es muy pero que muy improbable que salgan dos iguales.

2- Un bucle que genere un nombre aleatorio y finalice cuando no exista ningún archivo con ese nombre. (con file_exists como te han dicho)

3- Si guardas las imágenes en una base de datos, estás obligado a que cada imagen tenga una ID. Utiliza esa ID y ya está.

[hhs]

También puedes utilizar sha1_file para el nombre del archivo

[tampon]

Buenos días, disculpen por seguir con el tema estuve probando el ejemplo de alexisverano y funciona muy bien pero al momento de sibir documentos que no sean JPG, me los renombra mal, la exención la cambia por la fecha si subo un .PDF le pone de exención .mar2 ¿Alguien me puede ayudar? Gracias.

veo que el nombre del documento se llama doc.mar2.pdf

[Eleazan]

Iniciado por tampon

Buenos días, disculpen por seguir con el tema estuve probando el ejemplo de alexisverano y funciona muy bien pero al momento de sibir documentos que no sean JPG, me los renombra mal, la exención la cambia por la fecha si subo un .PDF le pone de exención .mar2 ¿Alguien me puede ayudar? Gracias.

veo que el nombre del documento se llama doc.mar2.pdf

Sigo pensando que ese código no es bueno, pero en fin...

Tu problema está en esta línea:

Código PHP:

  list($nombre, $ext) = explode(".", $nombre_archivo);  
   

Guarda en $nombre y en $ext el resultado [0] y [1] del explode... al tener varios puntos, el [1] no es la extensión, y de ahí que falle ;)

Sería mejor usar pathinfo:

Código PHP:

  list($nombre, $ext) = pathinfo($destino . '/' . $nombre_archivo, PATHINFO_BASENAME | PATHINFO_EXTENSION);  
   

De todas formas, recalco que ese código no es seguro.
Allá tu con lo que pongas en tu servidor ;)