Discussion :

[monlou]

Bonjour,

je suis a regarder a rajouter un système de controle de doublons pour les Upload photo de membres pour mon futur site ainsi que les INSERT dans une table MySql

j'ai déja mis en opération un système de gestion des erreurs et dans celui-ci, ya la fonction preg_repalce qui renomme le fichier suite aux caracteres non désirés mais par contre, ca ne règle pas les doublons de fichiers qui pourrait survenir.

alors, j'ai vu qu'on peut utiliser 2 manières d'opérer soit:

1- en renommant chacun des fichiers par des noms aléatoires

2- soit en les renommant et ajoutant au nom avec la fonction time(0)

quel est a votre avis la meilleure manière d'opérer pour un site qui contiendrait beaucoup de membres et en tenant compte bien entendu que les fichiers renommés soit aussi renommés dans la table ?

merci!

[Madfrix]

Le mieux à faire est soit :

-d'exclure tout les doublons en ne prenant pas en compte le nom bien sur mais en testant avec md5_file()

ou

-si tu désires désires par exemple laisser la possibilité à 2 users d'upload le meme fichier, alors de renommer à l'upload ces fichiers (doublons ou non) avec une fonction génératrice de nom de fichier basé sur le timestamp actuel

[monlou]

merci pour ta réponse,

alors si je comprends bien avec md5_file() ca implique d'exécuter un test dans la table ou dans le répertoire avant de uploader et ensuite renommer

tandis que en les renommant à l'upload, ces fichiers (doublons ou non) avec une fonction génératrice de nom de fichier basé sur le timestamp actuel me parait mieux puisque ca économise des ressources.

Puis-je avoir une idée de la syntaxe a utiliser avec rename afin d'ajouter timestamp a rename?

supposons que mon répertoire est

$dossier = '--/images/'; et que la variable a uploader est $fichier


je pourrais utiliser quelque chose du genre:

Code :

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

rename ($_FILES['photo']['tmp_name'], $dossier.$fichier);

comment je pourrais utiliser timestamp ici ?

[patouche]

En suppossant que $dossier = '--/images/'; et que $ext est l'extension du fichier.

Si l'on reprend le raisonnement de Madfrix, tu peux :

- basé sur le hash du fichier :

Code php :

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

1
2
3
4
5
6
7
8
 
$filenameTarget = md5_file($_FILES['photo']['tmp_name']) 
                                . '.' . $ext;
 
if ( !is_file( $dossier . $filenameTarget ) ) {
    // On déplace le fichier temporaire avec move_uploaded_file
    move_uploaded_file($_FILES['photo']['tmp_name'], $filenameTarget);
}

Si l'on veut autoriser les doublons, alors, on peut rajouter un temps ou une variable aléatoire (uniqid par ex).

Code php :

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

1
2
3
4
5
6
7
 
$filenameTarget = md5_file($_FILES['photo']['tmp_name'])
                                . '-' . time()
                                . '.' . $ext;
 
// On déplace le fichier temporaire avec move_uploaded_file
move_uploaded_file($_FILES['photo']['tmp_name'], $filenameTarget);

Voilà en gros les 2 idées... (à tester)


Cordialement,
Patouche

[monlou]

Merci encore une fois pour la reponse

en poursuivant mes recherches j'avais trouver ceci

Code :

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

1
2
3
$dossier = 'images/';
$nouv_nom = time().'.'.pathinfo($_FILES['photo']['name'], PATHINFO_EXTENSION);
move_uploaded_file($_FILES['photo']['tmp_name'], $dossier.$nouv_nom);

je n'ai pas encore fais les test mais j'imagine que le resultat sera le meme qu'avec la 2e option sur le post precedent mis a part : md5_file

Code :

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

1
2
3
4
5
6
$filenameTarget = md5_file($_FILES['photo']['tmp_name'])
                                . '-' . time()
                                . '.' . $ext;
 
// On déplace le fichier temporaire avec move_uploaded_file
move_uploaded_file($_FILES['photo']['tmp_name'], $filenameTarget);

[patouche]

Euh oui, ce que j'ai fait c'est surtout un copier/coller... Mais tu as l'idée

Ne reste plus qu'à la mettre en pratique