Discussion :

[remi5]

Bonjour,

j'utilise une fonction récursive (Permet de générer aléatoirement un nom de fichier, puis vérifie si il existe déjà en BDD):

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
function random_name($fileName) {
 
    // Récupère l'extention du fichier
    $pos = strrpos($fileName, '.');
    $ext = strtolower(substr($fileName, $pos));;
 
    // Génère un fileName aléatoire
    $fileName_random = "";
    $len = rand(1,11);
    $str= "abcdefghijklmnopqrstuvwxyz";
    for($i=0;$i<$len;$i++)
    {
        switch(rand(1,2)){
            case 1: $fileName_random.= $str[rand(0,25)];
            case 2: $fileName_random.= rand(0,9);
		}
    }
 
    // Replace l'extention du fichier
    $fichier = $fileName_random.$ext;
 
    // Vérifie si le nom de fichier n'existe pas déjà
    $query = $GLOBALS['db']->prepare("select fileName from image where fileName = :fichier limit 1");
    $query->bindParam(':fichier', $fichier, PDO::PARAM_STR);
    $query->execute();
    $row = $query->fetchAll();
    $query= null;
 
    // return
    if (count($row) > 0){
        return random_name($fileName);
    } else {
        return $fichier;
   }
}

Lorsque je l'utilise, mon navigateur plante :

• La connexion a été réinitialisée (pour Firefox)
• Erreur 101 (net::ERR_CONNECTION_RESET) : Erreur inconnue (pour Google Chrome)

J'ai 3800 entrées en BDD, donc la fonction s'arrête bien à un moment.
Si j'enlève la récursivité, le code fonctionne.
Si j'enlève la requête SQL, le code fonctionne.
Voici ma connexion dans mon fichier de config appelé plus haut :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
$dbhost = 'localhost';
$dbuser = 'xxx';
$dbpass = 'xxx';
$dbname = 'xxx';
try{
	$sqlPDO = new PDO('mysql:host='.$dbhost.';dbname='.$dbname, $dbuser, $dbpass);
	$sqlPDO->setAttribute(PDO::ATTR_CASE, PDO::CASE_NATURAL);
	$sqlPDO->setAttribute(PDO::MYSQL_ATTR_USE_BUFFERED_QUERY, true);
} catch(PDOException $e){
	//echo 'Erreur : '.$e->getMessage().'<br /><br />';
	die("Une erreur est survenue, le site est temporairement indisponible");
}
$GLOBALS['db'] = $sqlPDO;

J'ai également tenté de mettre $sqlPDO en paramètre de la fonction récursive, même souci.

Merci de votre aide, j'espère que l'erreur n'est pas trop idiote

Merci

Edit : je souhaiterai ajouter que si vous n'avez pas la réponse, n'hésitez pas à me proposer des tests pour mieux cibler le problème.
Je travail sous WAMP en Local.

[Bebel]

Bonjour,

quelques petites remarques comme ca :

pourquoi passer par une fonction récursive, alors que tu pourrais t'en sortir avec un do while, avec quelque chose dans ce genre la

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
 
function random_name($fileName) {
 
    // Récupère l'extention du fichier
    $pos = strrpos($fileName, '.');
    $ext = strtolower(substr($fileName, $pos));
 
    $isUnique = false;
    do
    {
       // Génère un fileName aléatoire
	    $fileName_random = "";
	    $len = rand(1,11);
	    $str= "abcdefghijklmnopqrstuvwxyz";
	    for($i=0;$i<$len;$i++)
	    {
	        switch(rand(1,2)){
	            case 1: $fileName_random.= $str[rand(0,25)];
	            case 2: $fileName_random.= rand(0,9);
					}
	    }
 
       // Replace l'extention du fichier
	    $fichier = $fileName_random.$ext;
 
	    // Vérifie si le nom de fichier n'existe pas déjà
	    $query = $GLOBALS['db']->prepare("select fileName from image where fileName = :fichier limit 1");
	    $query->bindParam(':fichier', $fichier, PDO::PARAM_STR);
	    $query->execute();
	    $row = $query->fetchAll();
	    $query= null;
 
	    // return
	    if (count($row) == 0)
	    	$isUnique = true;
    }
    while(!$isUnique);
 
		return $fichier;
}

Sinon, tu as testé en forcant avec un nom de fichier non existant ?

Affiche également la requete sql et teste la sous phpmyadmin pour voir si elle fonctionne.

[remi5]

Tout d'abord merci pour ta réponse Bebel.

J'aime ton idée, c'est beaucoup plus basique qu'une fonction récursive. et sa fait le même travail.

Mais cela ne fonctionne toujours pas.

j'ai ajouté ça à ton code :

Code :

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

1
2
3
4
5
6
7
8
9
// return
if (count($row) == 0){
	$isUnique = true;
}else{
	$ttt = 'existe deja';
	$query3 = $GLOBALS['db']->prepare("INSERT INTO image (id_image, fileName) VALUES ('', :fileName_random)");
	$query3->bindParam(':fileName_random', $ttt, PDO::PARAM_STR);
	$query3->execute();
}

Et le résultat est : quelques images passe, puis, dès qu'il trouve un nom qui existe déjà, il ne sort plus du 'do', il m'insère 10K de lignes, si se n'est plus (si je n'avais pas stoppé le script) de 'existe deja' en BDD.
La requête 'select' (qui vérifie le nom de l'image) doit forcement planter, car je n'ai pas assez d'image en BDD pour avoir utilisé tout les noms possibles généré aléatoirement.
J'ai testé la requête dans phpmyadmin, elle fonctionne bien.

Je remarque également que la requête 'insert' (pour le 'existe deja'), elle, fonctionne très bien.

Nième Edit : A partir de 4000 noms d'image (environ) en BDD, il n'en trouve plus de nouveau.
Je ne comprend pas, j'avais compté bien plus de possibilité.. Combien en comptez-vous, vous?

Code :

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

1
2
3
4
5
6
7
8
9
10
$fileName_random = "";
	    $len = rand(1,11);
	    $str= "abcdefghijklmnopqrstuvwxyz";
	    for($i=0;$i<$len;$i++)
	    {
	        switch(rand(1,2)){
	            case 1: $fileName_random.= $str[rand(0,25)];
	            case 2: $fileName_random.= rand(0,9);
					}
	    }

[Bebel]

Le nom du fichier est il important ?

Si non pourquoi ne pas utiliser une valeur incrémentale (l'id de l'enregistrement par exemple) ou un guid ?

Si oui, tu peux toujours sauver le nom du fichier d'origine en bd, renommer le fichier avec l'id et au moment d'afficher le nom du fichier utilisé celui stocké en BD.

Edit : le rand (1,11) pour la longueur commence peut etre trop bas. Peut etre un rand(5,11) (par exemple) augmenterais les possibilités. Sinon un strlen = 11 donnerait également une plus grand panel de possibilité.

[RunCodePhp]

Salut

Il y a une fonction qui me semblerais plus adapté dans ton cas :
fonction uniqid()
Elles est basée sur la date et heure courante en microsecondes, ce qui fait qu'il est (théoriquement) impossible que ça génère 2 IDs identiques (d'où son nom).
A quoi bon réinventer la roue

De plus, elle permet d'ajouter un préfixe, fort pratique.

[remi5]

Merci pour vos réponses, tout fonctionne maintenant, j'ai utilisé les id.

Sa m'apprendra à vouloir réutiliser des morceaux de code, alors qu'il était bien plus facile de tout refaire.

Merci encore, à la prochaine