Witam,

Muszę losować kilkukrotnie cyfrę 1 z 10, z tym że jedna cyfra po drugiej nie może być taka sama. A liczby muszą być oczywiście losowane z zakresu 1-10.
Znalazłem gotowe rozwiązania typu lotto: losuje się jedna liczbę i ona się już więcej w całym losowaniu nie powtórzy. Nie o to mi chodziło.

Poniżej proponowane rozwiązanie.
Czy (a jeżeli tak, to gdzie) widzicie tu błąd?

[PHP] pobierz, plaintext 
function losujcyfre() {
 
 	$liczby = range(1,10);
	$wylosowane = array_rand($liczby,1);
	$wylosowane += 1;
		if(($wylosowane) == $_SESSION ['wylosowana']) {
			losujcyfre();
			} else {
				$_SESSION ['wylosowana'] = $wylosowane;
				}
		}
[PHP] pobierz, plaintext 

Zapomniałem dodać, że liczba (liczby są losowane) ale zdarza się ze po kilka razy jest losowana ta sama liczba

Z braku laku twoje rozwiązanie może być ale posiada pewien minus - jeśli wylosuje się ta sama liczba to jest kolejne losowanie - może się to okazać mało wydajne dla losowania z małego zakresu liczb.

Proponuję użyć:

[PHP] pobierz, plaintext 
    $numbers = range(1,10);
    if(isset($_SESSION ['wylosowana']))
        unset($numbers[$_SESSION['wylosowana']-1]);
    $_SESSION['wylosowana'] = $numbers[array_rand($numbers,1)];
[PHP] pobierz, plaintext 

A nie lepiej za pomocą funkcji shuffle

[PHP] pobierz, plaintext 
 
$liczby = range(1, 10);
shuffle($liczby);
echo $liczby[0];
echo $liczby[1]; 
 
[PHP] pobierz, plaintext 

W taki sposób Mozesz wylądować 2 liczby z zakresu od 1 do 10 bez powtórzeń

Dzięki.

Jest tylko mały problem. Korzystając z drugiego rozwiązania jest prawdopodobieństwo ze będą się powtarzały liczby po shuffle np.:
- shuffle mam dwie liczby które nie będa takie same
- wykonuje się shuffle
- i w tym miejscu może się zdarzyć, że druga liczba z pierwszego shuffle moze być taka sama jak pierwsza z drugiego.

Jak to obejść?

[PHP] pobierz, plaintext 
function losujcyfre() {
 
		if(!isset($_SESSION['los1'])) {
			$liczby = range(1,10);
			shuffle($liczby);
			$_SESSION ['wylosowana'] = $liczby[0];
			} else {
				$_SESSION ['wylosowana'] = $liczby[2];
				unset($_SESSION['los1']);
				}
		}	
[PHP] pobierz, plaintext 

jest sens tworzyć następne if(!isset($_SESSION['los1'])), if(!isset($_SESSION['los2'])), ...

Ale kombinujesz ;P

Mój przykład działa na pewno tak jak Ci to potrzbne - czyli losuje jedną liczbę, inną niż ta, która była ostatnio w sesji.

Z użyciem shuffle musisz dać coś takiego:

[PHP] pobierz, plaintext 
if(isset($_SESSION['wylosowana'])) {
	$liczby = range(1, 10);
	shuffle($liczby);
	foreach($liczby AS $l) {
		if($l != $_SESSION['wylosowana']) {
			$_SESSION['wylosowana'] = $l;
			break;
		}
	}
} else $_SESSION['wylosowana'] = rand(1,10);
[PHP] pobierz, plaintext