Discussion :

[jiojio]

Bonjour,
je voudrais stocker 5 nombres aléatoires uniques dans un tableau.
je rencontre encore quelques problèmes avec mon code car par moment j'ai des nombres en double :

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
$tabnb[0] = rand(1,15);
 
for($i=1;$i<6;$i++)	 
  {
    $choix = rand(1,15);
 
	 for($j=0;$j<$i;$j++)
	   {
	     if($tabnb[$j] == $choix)
		    {
			  $choix = rand(1,15);
			}
			 else
			{
			  $tabnb[$i] = $choix;
			}
	    }
     echo $i.' : ';
	 echo $tabnb[$i-1];	
	 echo '<br><br>';
  }

[Halleck]

Ouch ! Regarde du côté de in_array

[keaton7]

Pourquoi ne pas utiliser uniqid() de php ?

http://fr.php.net/manual/fr/function.uniqid.php

Cette fonction est plus ou moins faite pour ca.

Il faut nécessairement que ton nombre soit compris entre 0 et 15 ?

Mais peut-etre que c'est plutot un truc comme ca qui te conviendrais (j'ai ecris ca vite fait, j'ai pas teste mais ca te donne une idee de la marche a suivre) :

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
$tabNb = array();
 
function genUniqNb($tab) {
 
	$uniqNb = rand(1,15);
 
	if(in_array($uniqNb,$tab)) genUniqNb($tab);
	else return $uniqNb;
 
}
 
for($i=1;$i<6;$i++) {
 
	$tabNb[$i] = genUniqNb($tabNb);
 
	echo "<p>$i : {$tabnb[$i]}</p>";
 
}

[jiojio]

Merci keaton2000

j'ai utilisé ta fonction mais par moment il n'y a pas de valeur dans le tableau.

j'ai également modifié le $tabNb en majuscule car ce ne fonctionnait pas.

Code :

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

echo "<p>$i : {$tabnb[$i]}</p>";

Que faut il faire pour y remédier.

merci d'avance

[mathieu]

j'ai utilisé ta fonction mais par moment il n'y a pas de valeur dans le tableau.

comment as tu testé son code ? son code fonctionne dans 80 % des cas, il y a juste parfois des cases vides parce qu'il y un "return" qui manque mais sinon le reste de son code fonctionne très bien

et sinon pour la génération du tableau je propose ça pour limiter les tests et les appel récursifs :

Code :

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

1
2
3
4
5
$liste = range(1, 15);              // liste de 1 à 15
shuffle($liste);                    // mélange de la liste
$liste = array_slice($liste, 0, 6); // découpage pour garder les 6 premières valeurs
 
print_r($liste);

[keaton7]

Comme je t'ai dit, je n'ai pas teste le code, c'est plus du domaine de l'ebauche, pour comprendre l'interet de la recursivite dans ce cas la.

Cependant, comme l'a dit mathieu, pour une liste de seulement 15 valeurs, un liste desordonnee puis tronquee seras surement plus rapide a l'execution. Je n'y avait pas pense. Garde le bout de code que je t'ai montre pour les tranches plus importantes (generer une valeur comprise entre 1 et 10000 par exemple).

Et si tu as besoin de plus, il te reste uniqid dans la fonction a la place de rand (moi je fais toujours un test de doublons quand meme au cas ou ^^)

-------------

Pour le return qui manque, ca doit etre par la :

Code :

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

if(in_array($uniqNb,$tab)) return genUniqNb($tab);

quel etourdi je fais

[jiojio]

ah oui c'est parfait là !!

merci à tous !

[jiojio]

j'ai parlé trop vite !
il reste des trous dans la fonction.

[keaton7]

Avec ca (j'ai teste cette fois ), je n'ai aucun trous ...

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
 
$tabNb = array();
 
function genUniqNb($tab) {
 
	$uniqNb = rand(1,15);
 
	if(in_array($uniqNb,$tab)) return genUniqNb($tab);
	else return $uniqNb;
 
}
 
for($i=1;$i<6;$i++) {
 
	$tabNb[$i] = genUniqNb($tabNb);
 
	echo "<p>$i : {$tabNb[$i]}</p>";
 
}