Forum d'entraide PHPFrance

Bonjour à tous,

Je travail sur un nouveau projet et j'ai de la peine à me lancer.

Je dois générer 10'000 licences de 8 caractères en chiffre et lettre type "15HZ2A97".
Ces licences vont être envoyée pour impression et fournie avec un achat spécifique.

Le client va acheter le produit et trouver une carte avec le code pour activer son produit.

J'ai créé ce script qui me génère les codes et les transmets dans une BDD

<?php 

// Fichier de connexion à la base de donnée

// require_once('../fr/admin/Connections/xxx.php');

 

function MakeRandomLicence($length=7) {



// Déclaration des variables

$newlicence = "";

$num = array ('1', '2', '3', '4', '5', '6', '7', '8', '9');   

$alpha = array ('A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I','J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z');

$code = array ();   





foreach ($num as $x) {

    foreach ($alpha as $y) {   

        array_push($code,"$y");   

        array_push($code,"$x");

		}

	}



for ( $i=0;$i<=$length;$i++)  

$newlicence= $newlicence.$code[array_rand($code) ];



return $newlicence; 

 

}

 



// On déclare le nombre total d'enregistrement voulu et on créé un boucle

// Inscription dans la base de donnée

for($i=1; $i<1001; $i++){

	$value = array(MakeRandomLicence());  

    	 

}$test = array_unique($value);

	foreach($test as $val){

 /* $insertSQL = "INSERT INTO `bwdb`.`keys` (`id`, `key`, `status`, `date`) VALUES (NULL, '$val','0', '')";																										 	mysql_select_db($database_xxx, $xxx);

	$Result1 = mysql_query($insertSQL, $xxx) or die(mysql_error());*/

	echo $val.'<br/>';

		}



?> 

Ce code fonctionne très bien, mais j'y rencontre un problème de taille et que je n'ai malheureusement pas trouvé de solution. Il y a pleins de DOUBLON!

J'ai essayé avec un array_unique() mais je ne dois probablement pas l'utiliser correctement.

tu pourrais le faire direct via MYSQL grace a un HASH

$insertSQL = "INSERT INTO `keys` (`key`) VALUES (HEX(CRC32('$val' + NOW())))";

EDIT : dans $val , tu peu mettre le login de la personne par exemple

Bonjour.

Tu as vérifié le contenu de tes variables ?

for($i=1; $i<1001; $i++){
	$value = array(MakeRandomLicence());  
}
print_r($value); // Affiche le contenu du tableau $value
$test = array_unique($value);

Ca devrait te donner une bonne indication du problème

Salut, merci pour vos réponses...

oui j'avais déjà testé le contenu vu que j'y trouve des doublons:
Array
(
[0] => U9VH6D3S
)

Array
(
[0] => ZLA4U58P
)

Array
(
[0] => K25U6E42
)

Array
(
[0] => K2O1UC49
)

Array
(
[0] => C1G8RA1L
)

Array
(
[0] => GPN893TV
)

Array
(
[0] => S7LH27M4
)


c'est supprimer les doublons que j'aimerais...

L'affichage du contenu, tu le fais à quel moment ?
Parce que dans ce que tu mets, on voit qu'il y a 7 Array de 1 élément

ahhh ouai!!! je me suis planté dans la conception de mon array ! Merci je pense que c'est une bonne piste.

hello

//tableau de licence
$arrayLicence = array();
//var tmp
$tmpLicence = '';
for($i=1; $i<1001; $i++){

do{
//créer une licence
$tmpLicence =  MakeRandomLicence();
}
//temps qu'elle appartient au licence déjà faite on recommance
while(!in_array($tmpLicence,$arrayLicence)); 

$arrayLicence[] = $tmpLicence; // on la stock
         

}

print_r($arrayLicence);

nan ? j'ai pas testé attention au boucle infini

++

hum... merci beaucoup... effectivement cette apporche me semble beaucoup plus juste.
ligne 38 =

 array_push($code,"$y");  

qu'es tu en penses?

regarde du coté de set_time_limit()

ca changera rien la boucle est infinie...

ca moulline (10sec) un peut mais ca passe )

<?php
set_time_limit(0);
$maxLoop = 10000;

function MakeRandomLicence($length=7){
	
	$car = array('a','z','e','r','t','y','u','i','o','p','q','s','d','f','g','h','j','k','l','m','w','x','c','v','b','n','1','2','3','4','5','6','7','8','9','0');
	$key = '';
	for($i=0;$i<$length;$i++){
		$key .= $car[rand(0,35)];
	}
	return $key;
}

//tableau de licence
$arrayLicence = array();
//var tmp
$tmpLicence = '';
for($i=0; $i<10000; $i++){
	$count = 0;
	do{
		//créer une licence
		$tmpLicence =  MakeRandomLicence();
		$count++;
		if($count>$maxLoop) exit('WARNING : plus de '.$maxLoop);
	}
	//temps qu'elle appartient au licence déjà faite on recommance
	while(in_array($tmpLicence,$arrayLicence));
	
	$arrayLicence[] = $tmpLicence; // on la stock
}

print_r($arrayLicence);

?>

Ce n'est pas trop étonnant que ça mouline, vu que tu fais (au moins) 10 000 fois un test pour vérifier que la licence n'existe pas déjà - donc en comparant la licence créée aux X licences déjà créées.
En gros (je ne connais pas le fonctionnement exact de in_array() ), tu fais donc 1 + 2 + ... + 9999 comparaisons.
=> 49 995 000 comparaisons
Même si in_array() est optimisé, le temps de calcul est conséquent...

Il y a moyen d'optimiser un peu ton code.
Exemples :
Au lieu de comparer chaque licence créée à celles déjà dans le tableau, on fait l'opération à la fin du remplissage. Ou en considérant que statistiquement parlant on n'aura jamais plus de 10% de doublons (par exemple) on peut créer un tableau plus gros. Avec ces systèmes, tu devrais passer sous la barre de la seconde =D>

en SQL c'est plus simple, chaque membre a son propre code

Hum... merci pour vos tuyaux... je crois que je touche au but.