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.

skorzystaj z funkcji rand i array_rand

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

A array_rand ?

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



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ł ?

powod to jego kłamstwo

Mówi, że gdy włączy przeglądarkę i pierwsze co zrobi to wejdzie na tą stronę z modułem, to nie działa (biały ekran podczas wczytywania, po 80 sek przestaje i ten biały ekran zostaje - nie wyświetla się żaden błąd), jednak gdy wejdzie najpierw na inną podstronę i później wróci do miejsca gdzie jest moduł to działa. Czy to możliwe ? Jeśli tak to jaka może być tego przyczyna ?

Prosiłbym jeszcze o spróbowanie zrobienia tego samego, czyli wyłączyć przeglądarkę, a po włączeniu wejść od razu tutaj: http://oldtimery.com/index.php?option=com_...&Itemid=323.

Czy jest w ogóle możliwe żeby coś było nie tak gdy nie ma najmniejszego błędu ? Nawet po użyciu:


Wyślę jeszcze kod szablonu:


Co o tym sądzicie ?

http://browsershots.org/http://oldtimery.c...&Itemid=323

Zmieniłem całkowicie skrypt losowania i pojawiają się błędy:


Jednak po odświeżeniu już wszystko działa. Wygląda na to jakby nie łączyło się na początku z bazą i nie pobierało prawidłowo zapytania do zmiennej $result. Moduł oparty jest o Joomla jeśli to coś zmienia.

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

Tak się nie robi!
Co jakbyś miał losować w ten sposób 999'999 różnych liczb spośród 1'000'000? Teoretycznie ta pętla może się wykonywać w nieskończoność (a php ma mało czasu ...). Jasne, że dla podanego przykładu można zastosować inną metodę i wylosować jedną liczbę, którą się odrzuci, ale nie zawsze wiadomo co i z czego będzie losowane ...

Spróbuj tak (jeśli wynik może być posortowany, bo jeśli nie może to pokombinujemy dalej ...):

[PHP] pobierz, plaintext 
<?php
define('MAX',20); //>=1
define('ILE',10); //ILE<=MAX
for($i=0;$i<=ILE-1;$i++)
{
  $t[$i]=rand(1,MAX-$i);
  for($j=0;$j<$i;$j++)
    if($t[$i]>=$t[$j])
      $t[$i]++;
  sort($t);
}
for($i=0;$i<=ILE-1;$i++)
  echo $t[$i].'<br />';
?>
[PHP] pobierz, plaintext 

No właśnie problem jest taki, że ja już zmieniłem skrypt losowania na:


I widać zmianę, ponieważ teraz strona się wczytuje, ale pojawiają się błędy takie jak podałem wyżej. Dziwne, że gdy stronę się odświeży, to już działa normalnie.