Losowanie lotto. Opcje

[Macios25]

Witajcie.

Próbuje nauczyć się na własną rękę pisać skrypty w php. Zapoznałem się z podstawami jednak nie zawsze wiem czy wykorzystuje zdobytą w najprostszy sposób. Dlatego chciałbym abyście rzucili okiem na mój skrypt losowania lotto
i powiedzieli co mogłem zrobić lepiej i w jaki sposób.

[PHP] pobierz, plaintext 
<?php
$zakres = array(1=>1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30,31,32,33,34,35,36,37,38,39,40,41,42,43,44,45,46,47,48,49);
// zakres liczb z jakich ma być losowane 6 cyfr. 
 
$wprowadzone="3,9,7,2,22,12"; 
// Liczby wprowadzone przez użytkownika. ( Muszą one być odzielone odcinkami. ) 
 
if(!preg_match('#^[0-9,}]+$#ui', $wprowadzone)) {
// tą linijkę kodu ukradłem z innego skryput. Nie mogę zrozumieć wyrażeń warunkowych, nie wiem czy użyłem tej funkcji prawidłowo. 
echo("<br>Blad! Musisz podac 6 liczb z zakresu od 1 do 49, odzielonych za pomoca przecinka!<br>");
// W wypadku gdy użytkownik poda niedozwolony ciąg znaków zostanie wyśwetlony komunikat o błędzie. 
die();
// Zostanie również wtedy zakończone działanie skryptu. 
}
 
 
$wprowadzone =explode(",",$wprowadzone);
// Każda liczba odzielona przecinkiem zostaje osobno przypisana do tablicy wporowadzone.
 
 
 
for ($i = 0; $i<= 5; $i++) {
if ($wprowadzone[$i]>49 or $wprowadzone[$i]<1  ){
echo("Możesz podawać tylko liczby zakresu od 1 do 49!<br>");
die();
}
// Pętla sprawdza czy podane liczby pochodza z zakresu od 1 do 49; 
 
 
}
 
 
echo("Wybrane przez Ciebie liczby to: <br>");
for ($i = 0; $i <= 5; $i++) {
     echo($wprowadzone[$i]." ");
              }
// Wypisuje 6 wybranych przez użytkownika liczb. 
 
 
 
 
echo("<br>Wylosowano:<br>");
 
for ($i = 1; $i<= 6; $i++) {
// pętla wykona się 6 razy, za każdym razem wylosuje jedną liczbę i umieści ją w tablicy losuj
$losuj[$i] = array_rand($zakres, 1);
// losuje liczbę z tablicy zakres. 
unset($zakres[$losuj[$i]]);
// usuwa wylosowaną liczbę z tablicy zakres - każada liczba ma być unikalna. 
echo($losuj[$i]." "); 
// Zwaraca wylosowaną liczbę. 
} 
 
 
 
 
 
$licznik =0;
 
 
for ($i = 0; $i <= 5; $i++) {
      if (in_array($wprowadzone[$i], $losuj)) {
          $licznik++;
                                  }
              }
//Porównuje liczby wylosowane z liczbami podanymi przez użytkownika. 
 
 
 
echo ("<br>Trafione: ".$licznik);
// Wyświelta liczbę trafień. 
 
 
 
 
 
 
 ?>
 
 
 
[PHP] pobierz, plaintext 

Chyba trochę przesadziłem z komentarzami...

Proszę o sugestie i pozdrawiam
Maciek.

[Macios25]

Głosiłem jednak herezję z in_array(). Funkcja ta sprawdza czy element znajduje się w tablicy.
Skoro element jest pobierany z tablicy $wpisane, to również w niej musi się znajdować.

Zgodnie z radą kamil4u przejrzałem dokładnie listę funkcji w manualu i znalazłem dosyć ciekawą perełkę.
array_unique(). Usuwa ona duplikaty z tablicy.

Jako, że potem każdy z elementów tablicy jest sprawdzany czy mieści się w przedziale od 1 do 49 zostanie zwrócony fałsz, a działanie skryptu przerwane.

Jako, że dodałem tylko jedno linijkę kodu i zmodyfikowałem echo(), nie wrzucam całego kodu, który jest dostępny wyżej
ponieważ mija się to z celem.

[PHP] pobierz, plaintext 
$wprowadzone =explode(",",$wprowadzone);
// Każda liczba odzielona przecinkiem zostaje osobno przypisana do tablicy wporowadzone.
 
 
 
$wprowadzone = array_unique($wprowadzone);
// Funkcja array_unique usuwa z tablicy powtórzenia. 
 
 
 
 
for ($i = 0; $i<= 5; $i++) {
if ($wprowadzone[$i]>49 || $wprowadzone[$i]<1  ){
die("Możesz podawać tylko liczby zakresu od 1 do 49!<br>Liczby nie mogą się powtarzać.");
}
 // Jeśli liczba jest mniejsza 1 lub większa od 49 lub została usunięta ze względu na powtórzenia zostaje wyświetlony stosowny komunikat.   
[PHP] pobierz, plaintext 

Przy użyciu generowania tablic, z której ma być losowana cyfra przy użyciu funkcji range() dzieje się u mnie na serwerze
dosyć dziwna rzecz. Pomimo, że zakres jest ustawiony od 1,49 to czasami w losowaniu potem z tej tablicy jest wyciągnięte 0. Ciekawe. (IMG:style_emoticons/default/smile.gif)

Udało mi się z Waszą pomocą ulepszyć ten skrypt. Jednak nie myliłem się co do tego, że potrzebuje jakichś mentorów,
którzy będą mi pokazywali błędy. Teraz spróbuje napisać coś większego, może jakiś system logowania. (IMG:style_emoticons/default/wink.gif)
Tylko muszę się dokształcić z wyrażeń regularnych i obsługi sesji.
Jeśli uda mi się stworzyć coś sensownego to wrzucę abyście wskazali mi błędy.

Jeszcze raz dziękuje za pomoc i zainteresowanie. (IMG:style_emoticons/default/smile.gif)