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.

1. Zamiast klepać tablice $zakres z palca użyj funkcji http://www.php.net/manual/en/function.range.php

Wielkie dzięki, rzeczywiście wystarczyło: $zakres = range (1, 49). Tak to jest jak się nie zna do końca języka
i dodaje się samemu sobie pracy.

Na przyszłość z pewnością zapamiętam.

- array_rand - popatrz co robi drugi parametr
- die - echo jest niepotrzebne
- zamiast "or" używaj ||

Dziękuje za rady. Zastosowałem wszystkie i kod stał się dużo bardziej przejrzysty, a do tego poznałem kilka przydatnych funkcji.

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.

Chyba nie ma... przynajmniej nie przypominam sobie... - na wszelki wypadek popatrz jeszcze do manual-a na funkcję tablicy: http://www.php.net/manual/pl/ref.array.php
Ale i tak już raz masz pętle na wszystkie elementy:

Wykorzystaj ją.

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.

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.
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.