[Système] Générer un nom d'image unique

Version imprimable

21/12/2006, 17h35
ToxiZz

[Système] Générer un nom d'image unique

Bonjour à tous,

J'ai un module de news sur mon site.
Lorsqu'on ajoute une news on doit soumettre une image.
Une fois le formulaire validé, j'upload cette image dans un dossier sur mon serveur et j'enregistre les informations de cette news dans une base de données MySQL.

Mon problème est que chaque news contient une image différente, il faut donc que chaque image porte un nom différent.
Pour générer cette identifiant unique pour l'image, j'avais imaginer ceci :

Code:

md5(microtime());

J'aimerais savoir si c'est une bonne méthode.
Si non, j'aimerais savoir quelle méthode utiliser.
21/12/2006, 18h08
trotters213

:salut:
utilise un time() tout simplement.
21/12/2006, 18h15
ToxiZz

Non parce que ca poserait problème si deux personnes ajouten une news la même seconde, le time() généré sera le même.
21/12/2006, 18h19
trotters213

alors microtime :mrgreen:
Il faudrait que t'ai vraiment pas de chance et énormément de visiteur pour que 2 requêtes arrivent dans la même microseconde au serveur ;)

Je pense qu'il serait plus judicieux de gardé une sorte de référence avec le nom du fichier d'origine?

je pense que tu peut faire un truc comme ça par exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 <?php function gen_name($name) { $gen_name = md5(microtime()).$name; while (file_exists($gen_name)) $gen_name = md5(microtime()).$name; return $gen_name; } echo gen_name($_GET['image']); ?>
l'avantage de ne pas à avoir à changer l'extension du fichier, et de savoir plus ou moins de quoi traite l'image (par rapport aux nom de l'image)
Comme ça en plus, tu êtres sur qu'aucun fichier ne portera jamais le même nom (grâce aux testes).

Je pense ensuite que tu peux en rajouter:
md5_file()
histoire de contrôler si l'image existe déjà et ne pas utiliser de charge inutilement...

21/12/2006, 18h41
julp

Et la fonction uniqid alors ?

Julp.
21/12/2006, 18h51
ToxiZz

Pour l extension du fichier je peux faire un :

Code:

$ext = strrchr(".", $file);

qui me retournera : .jpg, .jpeg, .gif ou .png

Je peux ensuite faire un :

Code:

$image = md5(microtime()).$ext;

Citation:

Je pense ensuite que tu peux en rajouter:
md5_file()
histoire de contrôler si l'image existe déjà et ne pas utiliser de charge inutilement...

Peux tu m'en dire plus à ce sujet.

Est-ce que je peux faire une table image dans ma bd qui contiendrais en clé primaire le md5_file($image) et une colonne avec le nom de l'image.
Comme ca chaque fois que j'ajoute une image, je regarde si son md5_file() existe déjà dans ma bd, si il existe, c'est que l'image est la même que celle sur le serveur. ???
21/12/2006, 19h30
ToxiZz

Ou encore plus sure :
le nom de l image est

Code:

md5_file($image).$ext;
21/12/2006, 20h55
sahid
je serais un peu de l'avis de julp,
Code:

1 2 uniqid(mt_rand(), true);
22/12/2006, 00h36
ToxiZz

Très bonne idée, je ne connaissais pas cette fonction.

En ce qui concerne le md5_file, je peux donc stocker dans ma bd :
image_id = md5_file();
image_name = uniqid();

Comme ca pour être sur de ne pas stocker deux fois la même image, je regarde si le md5_file existe déjà dans ma bd. ???
22/12/2006, 10h28
Rakken

J'vais ptete dire une bêtise mais... Si tu as un module de news sur ton site et qu'a chaque news l'utilisateur rentre une image... J'imagine que les news et les images sont liée non ?
Avec un peu de bol, tu as auto-incrément dans ta table qui gère les news, donc pour le nom de ton image, tu peux mettre "$id_de_ta_news.jpg". T'as un identifiant unique, tu es sur qu'il n'est pas déjà présent dans ta base, et en plus de ca, c'est super facile de retrouver quelle image est liée a quelle news.

Sinon, les autres solutions basé sur uniqid me semblent tout a fait viable.

--
Rakken
22/12/2006, 11h03
ToxiZz

Citation:

tu peux mettre "$id_de_ta_news.jpg"

Si je procède comme cela, je suis obligé d'insérer la news avant de m occuper de l image.
Si une exception se lance lors du traitement de l'image, la news sera quand même déjà dans la bd.
Pour mettre le champs news_image à $id_de_ta_news.jpg, il faudrais que lorsque j'insère, laisser ce champs vide, puis faire un update de ce champs lorsque j'ai obtenu un identifiant de news.
Il faudrait donc chaque fois que je fasse un insert suivi d'un update.

En stockant mes images dans une table avec comme clé primaire le md5_file de l'image, je suis sur de ne pas avoir deux fois la même image sur mon serveur.
22/12/2006, 11h14
cadoudal56

sauf si tu upload l'image avant puis si c'st bon t'inseres la news avec le nom de ton image...

Personnellement je prends toujours l'identifiant unique dans la BDD c'est plus simple et je m'y retrouve mieux....

@+
cadou
22/12/2006, 11h20
ToxiZz

Peux tu me dire comment est-ce que tu obtiens l'identifiant de ta news avant de l'avoir insérer :roll:
Et avec ce système, tu ne sais pas géré, si deux news on la même image, tu aura deux fois la même image sur ton serveur.
22/12/2006, 15h56
Rakken

Ben généralement, quand j'ai ce genre de cas, j'insère la news, je récupère l'id, et ensuite je crée la news avec le bon nom.
Je ne stocke pas le chemin de l'image dans un champs de la news, c'est inutile dans la mesure ou l'image à pour nom l'id de la news. Si j'affiche une news, je teste l'existence de l'image avec le nom id.jpg et si l'image existe, j'affiche.

Ca a aussi l'avantage que tu peux changer facilement le chemin de stockage de des images pour les news, dans la mesure ou il n'apparait pas dans tout tes champs news ca m'a déjà servi une fois ou deux pour des exports de données.

Maintenant, si tu veux gerer des exceptions ou assimilés, le mieux est encore de démarre une transaction avant d'insérer ta news et de faire un rollback en cas de problème. C'est propre et sur. (Ne pas oublier le commit a la fin par contre ^_^). Tu peux également tester l'existence de ton image et sa validité avant d'insérer ta news. Ca va te catcher 95% des soucis.

--
Rakken