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]

Dziękuje za rady. Zastosowałem wszystkie i kod stał się dużo bardziej przejrzysty, a do tego poznałem kilka przydatnych funkcji. (IMG:style_emoticons/default/wink.gif)

Poprawiony kod:

[PHP] pobierz, plaintext 
<?php
 
 
$zakres =  range (1, 49);
// funkcja renge() generuje tablicę $zakres z, której następnie ma być wybrane 6 liczb.  
 
 
$wprowadzone="1,2,3,4,5,6"; 
// 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. 
die("<br>Blad! Musisz podac 6 liczb z zakresu od 1 do 49, odzielonych za pomoca przecinka!<br>");
// zostanie wyświetlony odpowiedni komunikat i działaie skryptu zostanie zakończone. 
}
 
 
$wprowadzone =explode(",",$wprowadzone);
// Każda liczba odzielona przecinkiem zostaje osobno przypisana do tablicy wporowadzone.
 
 
 
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>");
                                                              }
                                         }
// Pętla sprawdza czy podane liczby pochodza z zakresu od 1 do 49, zostaje wyświetlony odpowiedni komunikat, po czym działanie skryptu zostaje zatrzymane.; 
 
 
 
 
 
echo("Wybrane przez Ciebie liczby to: <br>");
for ($i = 0; $i <= 5; $i++) {
     echo($wprowadzone[$i]." ");
              }
// Pętla Drukuje wybrane przez użytkownika liczby. 
 
 
echo("<br>Wylosowano:<br>");
$wylosowane = array_rand($zakres, 6);
$licznik =0;
// Funkcja array_rand() losuje  6 liczb z tablicy $zakres. ( bez zwracania. )  
 
 
 
foreach($wylosowane as $klucz=> $wylosowana ) {
    echo ($wylosowana." ");  
         if (in_array($wylosowana,$wprowadzone)) {
             $licznik++;
                           }             
            }
// Pętla foreach drukuje wzlosowane liczby. Sprawdza czy każda z kolejna znajduje się w tablicy $wprowadzone. Jeśli tak zwiększa licznik o 1.  
 
 
echo ("<br>Trafione: ".$licznik);
 ?>
[PHP] pobierz, plaintext 

Trapi mnie jeszcze jedne rzecz. Użytkownik może podać 6 takich samych liczb, przez co prawdopodobieństwo na
trafienie "szóstki" byłoby bardzo duże ( 1 do 49 ).
Do sprawdzania czy wszystkie liczby są różne mógłbym użyć funkcji in_array. Jednak musiałbym sprawdzać każdy element z osobna i porównywać go do całej tablicy. Czy nie ma jakiegoś bardziej wydajnego i odpowiedniego wyjścia
w tej sytuacji?

Jeszcze raz dziękuje za zainteresowanie i udzieloną pomoc.