Laster tregt

[barentshavet]

Siden min vil ikke fungere pga. denne funksjonen (laster til time-limit.)

 

function randomcode($len="8") {
$code = NULL;
for($i=0;$i<$len;$i++) {
$char = chr(rand(48,122));
while(!preg_match("[a-zA-Z0-9]", $char)) {
if($char == $lchar) { continue; }
$char = chr(rand(48,90));
}
$pass .= $char;
$lchar = $char;
}
return $pass;
}

 

Forklaring på det? Det er forresten gammel kode fra 2005.

Endret 30. november 2012 av winter2012

[TheClown]

Jeg har ingen god forklaring på akkurat hvorfor scriptet ditt bruker så lang tid på å gjøre seg ferdig. Mitt guess er at randomkoden du har laget (rand og char) på en eller annen måte lager en unødvendig stor loop fordi den velger karakterer som ikke er gyldig.

 

Jeg har skrevet om koden din til følgende:

 

 

<?php
function randomcode($len="8") {
   $code = '';
   $chars = 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',
    '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',
    '0','1','2','3','4','5','6','7','8','9');

   while (strlen($code) <= $len) {
    $code .= $chars[rand(0,(count($chars)-1))];
   }

   return $code;
}
echo randomcode(20);
?>

 

Den gjør seg ferdig på noen millisekunder og jeg tror den har akkurat den samme funksjonaliteten som den originale koden du postet.

[etse]

Øverst har du brukt variablen "code", mens sebere bruker du "pass". Jeg antar disse er ment til å være samme variabel. Det er en feil i koden i hvertfall.

 

Det neste er at pragmatchen din alltid er false, selv for verdier som er riktige. Så du kommer aldri ut av for-loopen. Du har rett og slett glemt en skrå-strek før og etter utrykket.

 

<?php
function randomcode($len="8") {
   $pass = "";
   for($i=0;$i<$len;$i++) {
    $char = chr(rand(48,122));
    while(!preg_match("/[a-zA-Z0-9]/", $char)) {
	    if($char == $lchar)
		    continue;
	    $char = chr(rand(48,122));
    }
    $pass .= $char;
    $lchar = $char;
   }
   return $pass;
}
echo randomcode();
?>