[PHP]Zbyt wolny skrypt losowania Opcje

[-emzetka-]

Witam. Mam problem ze skryptem losowania. Zrobiłem można powiedzieć "pseudo kod", bo na pewno nie jest to tak jak ma być. Chodzi mi mianowicie o wylosowanie trzech liczb z danego przedziału przy czym żadna z nich nie może być taka sama jak pozostałe. Dane powinny być zamieszczone w tablicy. Mi osobiście udało się to, bo kod działa, jednak nie wiem czemu, ale zależy na jakim serwerze. Na jednym jest wszystko ok, a na innym strona wczytuję się strasznie długo (biały ekran) aż wyświetla się błąd, że za długo musiał oczekiwać w jednej z linii kodu losowania.

Bardzo proszę o podpowiedź jak to powinno wyglądać aby działało szybko i nie sprawiało żadnych problemów ?
Oto kod:

[PHP] pobierz, plaintext 
function losuj()
{
   $randnum[0] = mt_rand(0, $this->numberofr);
 
   do{   
   $randnum[1] = mt_rand(0, $this->numberofr);
   }while($randnum[0] == $randnum[1]);
 
   do{
   $randnum[2] = mt_rand(0, $this->numberofr);
   }while($randnum[2] == $randnum[1] || $randnum[0] == $randnum[2]); 
 
return $randnum;
}
[PHP] pobierz, plaintext 

Z góry dziękuję za odpowiedzi, pozdrawiam.

[gino]

Co przyjmuje zmienna $this->numberofr ?
Podstawiłem do tej funkcji liczby 10 cyfrowe i działa ok, wykonanie poniżej sekundy. Jak duże liczby przyjmuje ta zmienna ?

gino

@toaspzoo rand jest czterokrotnie wolniejsza od mt_rand

Ten post edytował gino 31.05.2011, 16:53:09

[Zyggmunt]

Przypomniałem sobie, że mam jednak tutaj użytkownika.

Co przyjmuje zmienna $this->numberofr ?
Podstawiłem do tej funkcji liczby 10 cyfrowe i działa ok, wykonanie poniżej sekundy. Jak duże liczby przyjmuje ta zmienna ?

Przyjmuje liczbę wierszy pobraną z bazy mysql. Bardzo duża na pewno nie jest, pewnie trochę ponad 100.

Pozdrawiam.

P.s. Może dodam cały kod pliku (moduł ma za zadnie pobieranie danych z bazy mysql, które zapisał tam komponent DJ Catalog 2)

[PHP] pobierz, plaintext 
<?php defined('_JEXEC') or die('Restricted access'); 
class get_records
{
var $numberofr, $result;
 
function get_records()
{
$query="SELECT * FROM jos_djc2_items"; 
$this->result = mysql_query($query);
$this->numberofr = mysql_numrows($this->result);  //number of rows 
$this->numberofr--;
}
 
function get_datas()
{  
   $randomnumber = $this->losuj();
   for($i = 0 ; $i < 3 ; $i++)
   {   
   $data[$i]["image"] = mysql_result($this->result, $randomnumber[$i], "image_url");
   $data[$i]["name"] = mysql_result($this->result, $randomnumber[$i], "name");   
   $cat_id = mysql_result($this->result, $randomnumber[$i], "cat_id");   
   $id = $randomnumber[$i] + 3;
   //$cat_id++;
   $data[$i]["link"] = 'index.php?option=com_djcatalog2&view=item&id='.$id.'&cid='.$cat_id.'&Itemid=568';   
   $cat_id = $cat_id - 3;
   $query = "SELECT * FROM jos_djc2_categories";
   $result2 = mysql_query($query);  
   $data[$i]["cat_name"] = mysql_result($result2, $cat_id, "name");
   }
 
return $data;
 
}
 
function losuj()
{
   $randnum[0] = mt_rand(0, $this->numberofr);
 
   do{   
   $randnum[1] = mt_rand(0, $this->numberofr);
   }while($randnum[0] == $randnum[1]);
 
   do{
   $randnum[2] = mt_rand(0, $this->numberofr);
   }while($randnum[2] == $randnum[1] || $randnum[0] == $randnum[2]); 
 
return $randnum;
}
 
}
?>
[PHP] pobierz, plaintext 

EDIT.

Już wszystko działa, przynajmniej u mnie. Nie wiem jednak dlaczego, ale osoba, której robię ten skrypt mówi, że na Firefoxie dalej mu nie działa (mimo, że u mnie wszystko jest ok na każdej przeglądarce).

Co może być tego przyczyną ?

Ma ktoś jakiś pomysł ?

Ten post edytował Zyggmunt 31.05.2011, 18:02:04