[PHP]Problem z programem losującym Opcje

[michat34]

napisałem program, który losuje 8 liczb z przedziału 1-49.

[PHP] pobierz, plaintext 
 <?php
 function sprawdzenie ()
 {
  $powtorzenie = false;
  while ($i !== 0)
  {
   if ($tablica[$i] = $losowa) // jesli nowa wylosowana liczba bedzie juz w tablicy
   {
    $powtorzenie = true; // to powtorzenie przyjmuje postac prawda
   }
   --$i;
  }
 }
 
 for ($i=0;$i!==8;++$i)
 {
   do
   {
   $losowa = rand(1,49); // losowanie nowej liczby i sprawdzanie jej do skutku
   sprawdzenie();
   } while ($powtorzenie == false);
  $tablica[$i] = $losowa;
 }
?>
[PHP] pobierz, plaintext 

niestety jak sie domyslacie nie działa. maximum time execution 12 sec. gdzie w skrypcie popelnilem blad? sprawdzalem juz kilka opcji ale ciagle nie działa wiec pisze tutaj. moze nalezy zreferowac zmienna $powtorzenie?

[qrooel]

W 7. linijce masz przypisanie, a w komentarzu piszesz, że coś tam porównujesz.

[michat34]

fakt, dzięki. ale nadal nie działa

[klocu]

Generalnie to średnio będzie działać bo:
zasięg zmiennych globalny i wewnątrz funkcji to nie to samo; w związku z tym

Kod

while ($powtorzenie == false)

nie będzie widzieć zmiennej modyfikowanej na poziomie funkcji.

Zaproponowane rozwiązanie jest mocno nieoptymalne - dlaczego?
Za każdym razem, w zamyśle, skanujesz tablicę w poszukiwaniu powtarzalnika.
Aż zrobiłem sobie takie zadanko w ramach małego konkursu - w ile linii uda mi się upchnąć takiego małego lottomaniaka.

[PHP] pobierz, plaintext 
$kaseta = array();
 
for($i=0; $i<8; ++$i)
{
	$los = rand(1,49);
	if(!in_array($los, $kaseta)) array_push($kaseta, $los); else --$i;
}
 
sort($kaseta);
var_dump($kaseta);
[PHP] pobierz, plaintext 

Na upartego 4 linie kodu, nie licząc nawiasów, blanków itd.

[michat34]

@up dzieki, przeanalizowalem Twoj skrypt i wszystko chodzi, mozna zamknac

Ten post edytował michat34 14.08.2012, 06:34:44

[pamil]

Wiesz może dlaczego takie błędy trudno rozwiązać? Nikt nie pokazuje błędów typu E_NOTICE (dostałbyś "undefined variable"), a potem płacz i zgrzytanie zębów.