[PHP]Problem z rand(); - nie losuje Opcje

[SarnaMC]

Witam. Piszę coś w stylu lotto dla mojej strony ( konkurs). Wszystko mi pięknie działało dopóki nie zacząłem pisać losowania liczb. Znalazłem na necie rand() ale ona sie dziwnie zachowuje. Oto plik class:

[PHP] pobierz, plaintext 
<?php
 
class lotto
{
function generowanie()
{
$liczba1 = rand(1, 40); 
$liczba2 = rand(1, 40); 
$liczba3 = rand(1, 40); 
$liczba4 = rand(1, 40); 
$liczba5 = rand(1, 40);
echo rand(0, 40); 
 
}}
?>
[PHP] pobierz, plaintext 

i teraz plik index:

[PHP] pobierz, plaintext 
$lotto->generowanie();
echo "$liczba1 $liczba2 $liczba3 $liczba4 $liczba5 test ";
[PHP] pobierz, plaintext 

Jedyne co mi się wyświetla to to ostatnie rand które dałem już w echo oraz napis test. Nie wiem czemu tak się dzieje, pomożecie?

#EDIT
hmmm przemyślałem to i widzę że powinienem chyba użyć return w pliku class, tylko że po dopisaniu return $liczba1; dalej to nie działa

Ten post edytował SarnaMC 28.05.2014, 11:59:00

[IProSoft]

Nowe:

[PHP] pobierz, plaintext 
class lotto
{
    public $rands = array();
	function generowanie()
    {
		$this->rands = array_map(
			function ($value) {
			  return rand( 0, 40 );
			},
			range(1, 10)
		);
    }
}
 
$lotto = new lotto();
$lotto->generowanie();
 
echo implode( ' ', $lotto->rands );
[PHP] pobierz, plaintext 

albo w swoim w funkcji musisz zwrócic:

[PHP] pobierz, plaintext 
return "$liczba1 $liczba2 $liczba3 $liczba4 $liczba5";
[PHP] pobierz, plaintext 

[kreatiff]

Jeśli lotto ma być bez powtórzeń, to musisz to inaczej zbudować, bo w obu powyższych przypadkach mogą wystąpić powtórzenia.
Ja to zawsze robię tak:

[PHP] pobierz, plaintext 
$a = range(1,40);
shuffle($a);
// i wybieram sobie pierwsze x elementów tablicy, gdzie x to ilość potrzebnych liczb
[PHP] pobierz, plaintext 

ale to na własny użytek, nie produkcyjny.

Jak zbiór, z którego losujemy jest ogromny*, to lepiej losować i sprawdzać, czy dana wartość już padła i jeśli tak, to ponowić losowanie.
* wszystko sprowadza się do optymalizacji

Ten post edytował kreatiff 28.05.2014, 12:52:46