Discussion :

[ToxiZz]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

md5(microtime());

J'aimerais savoir si c'est une bonne méthode.
Si non, j'aimerais savoir quelle méthode utiliser.

[trotters213]

utilise un time() tout simplement.

[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.

[trotters213]

alors microtime
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

[juJuv51]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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...

[julp]

Et la fonction uniqid alors ?


Julp.

[ToxiZz]

Pour l extension du fichier je peux faire un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

Je peux ensuite faire un :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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. ???

[ToxiZz]

Ou encore plus sure :
le nom de l image est

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

md5_file($image).$ext;

[sahid]

je serais un peu de l'avis de julp,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
uniqid(mt_rand(), true);

[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. ???

[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

[ToxiZz]

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.

[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

[ToxiZz]

Peux tu me dire comment est-ce que tu obtiens l'identifiant de ta news avant de l'avoir insérer
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.

[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