Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP]Zgadywanie generowaniej liczby, Analiza kodu
xamrex
post
Post #1





Grupa: Zarejestrowani
Postów: 267
Pomógł: 1
Dołączył: 10.08.2007

Ostrzeżenie: (0%)
-----

Witam. Napisałem skrypt, który generuje losową liczbę, a zadaniem użytkownika jest zgadnięcie właśnie tej liczby.
Pisałem to w PHP, jestem pewny że używając java script'u było by o wiele prościej, jednak chcę przetestować własne umiejętności.
Proszę o analizę tego kodu i napisanie co można by tutaj poprawić, gdzie zrobiłem jakieś błędy.
[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();//Problemy z headers already sent
  3. echo '<html>';
  4. echo '<head>';
  5. echo '<title>Zgadywanie wygenerowaniej cyfry</title>';
  6. echo '<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-2" >';
  7. echo '</head>';
  8. echo '<body>';
  9. //Tworzeniec cookie z wygenerowana losowo cyfra
  10. if (empty($_COOKIE['losowa']))
  11. {
  12.    setcookie("losowa", rand(1,10),  time()+3600);
  13.    echo'<meta http-equiv="Refresh" content="0">';   
  14. }
  15. //tworzenie zmiennych
  16. $cyfra_wygenerowana=$_COOKIE['losowa'];
  17. $cyfra_usera=$_POST['cyfra'];
  18. //podpowiedzi
  19. if (empty($cyfra_usera))
  20. {
  21.    echo 'Wpisz cyfrę (od 1 do 10)';   
  22.    echo '<form action="index.php" method="POST">CYFERKA: <input type=text name="cyfra"><input type=submit value="Wyślij"></form>';
  23. }elseif ($cyfra_wygenerowana>$cyfra_usera){
  24.     echo 'Za mało, co ty taki skromny??';
  25.     $proba++;
  26.     setcookie ('proba', "$proba", time()+3600);
  27.     echo '<form action="index.php" method="POST">CYFERKA: <input type=text name="cyfra"><input type=submit value="Wyślij"></form>';
  28. }elseif ($cyfra_wygenerowana<$cyfra_usera){
  29.     echo 'No nie przesadzaj aż tak.';
  30.     $proba++;
  31.     setcookie ('proba', "$proba", time()+3600);
  32.     echo '<form action="index.php" method="POST">CYFERKA: <input type=text name="cyfra"><input type=submit value="Wyślij"></form>';
  33. }elseif ($cyfra_wygenerowana==$cyfra_usera){
  34.     $proba++;
  35.     setcookie ('proba', "$proba", time()+3600);
  36.     echo 'Zgadłeś za '.$proba.' razem BRAWO!<br><a href="index.php">Kliknij</a> by zagrać jeszcze raz.';
  37.     setcookie("losowa", "",  time());
  38.     setcookie("proba", "", time());
  39. }else{
  40.     echo 'To sie nie powinno zdarzyć';
  41. }
  42. ob_end_flush();//Problemy z headers already sent
  43. echo '</body>';
  44. echo '</html>';
  45. ?>
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
xamrex
post
Post #2





Grupa: Zarejestrowani
Postów: 267
Pomógł: 1
Dołączył: 10.08.2007

Ostrzeżenie: (0%)
----- 

[PHP] pobierz, plaintext 
  1. <?php
  2. ob_start();//Problemy z headers already sent
  3. ini_set('display_errors' , 'On'); error_reporting(E_ALL | E_STRICT); 
  4. echo '<html>';
  5. echo '<head>';
  6. echo '<title>Zgadywanie wygenerowaniej cyfry</title>';
  7. echo '<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-2" >';
  8. echo '</head>';
  9. echo '<body>';
  10. //Tworzeniec cookie z wygenerowana losowo cyfra
  11. if (empty($_COOKIE['losowa']))
  12. {
  13.    setcookie("losowa", rand(1,10),  time()+3600);
  14.    echo'<meta http-equiv="Refresh" content="0">';   
  15. }
  16. //tworzenie zmiennych
  17. if(isset($_COOKIE['losowa'])){  
  18.     $cyfra_wygenerowana=(int)$_COOKIE['losowa'];
  19. }
  20. if(isset($_POST['cyfra'])){
  21.     $cyfra_usera=(int)$_POST['cyfra'];
  22. }
  23. //ustawianie proby
  24. if(isset($_COOKIE['proba'])){    
  25.     $proba = $_COOKIE['proba'] ;
  26. }else{
  27.     $proba=0;
  28. }
  29. //podpowiedzi
  30. if (empty($cyfra_usera))
  31. {
  32.    echo 'Wpisz cyfrę (od 1 do 10)';   
  33.    echo '<form action="index.php" method="POST">CYFERKA: <input type=text name="cyfra"><input type=submit value="Wyślij"></form>';
  34. }elseif ($cyfra_wygenerowana>$cyfra_usera){
  35.     echo 'Za mało, co ty taki skromny??';
  36.     $proba++;
  37.     setcookie ('proba', $proba, time()+3600);
  38.     echo '<form action="index.php" method="POST">CYFERKA: <input type=text name="cyfra"><input type=submit value="Wyślij"></form>';
  39. }elseif ($cyfra_wygenerowana<$cyfra_usera){
  40.     echo 'No nie przesadzaj aż tak.';
  41.     $proba++;
  42.     setcookie ('proba', $proba, time()+3600);
  43.     echo '<form action="index.php" method="POST">CYFERKA: <input type=text name="cyfra"><input type=submit value="Wyślij"></form>';
  44. }elseif ($cyfra_wygenerowana==$cyfra_usera){
  45.     $proba++;
  46.     setcookie ('proba', $proba, time()+3600);
  47.     echo 'Zgadłeś za '.$proba.' razem BRAWO!<br><a href="index.php">Kliknij</a> by zagrać jeszcze raz.';
  48.     setcookie("losowa", "",  time());
  49.     setcookie("proba", "", time());
  50. }else{
  51.     echo 'To sie nie powinno zdarzyć';
  52. }
  53. ob_end_flush();//Problemy z headers already sent
  54. echo '</body>';
  55. echo '</html>';
  56. ?>
[PHP] pobierz, plaintext

Proszę zobaczyć wiersz od 23-28 czy to jest poprawnie? Czy cały skrypt jest poprawny? (oprócz tego nagłówka bo z tym nie umiem sobie inaczej poradzić)
Go to the top of the page
+Quote Post

Posty w temacie
- xamrex   [PHP]Zgadywanie generowaniej liczby   16.06.2010, 09:55:21
- - zend   1) $cyfra_wygenerowana<$cyfra_usera -...   16.06.2010, 10:33:49
- - xamrex   1)Poprawiłem $cyfra_wygenerowana=(int)$_...   16.06.2010, 11:34:16
- - nospor   Co do undefined to najpierw sprawdzaj czy cos jest...   16.06.2010, 11:43:00
- - xamrex   myślałem ze dry to jakaś funkcja z php ;( mój kod ...   16.06.2010, 11:54:44
- - nospor   CytatPrzecież ja ustawiłem na $proba=0 jeśli ...   16.06.2010, 11:57:50
- - xamrex   no tak, lecz zend napisał 4) $proba++; a gdzi...   16.06.2010, 12:02:08
- - nospor   Nie, cookie z proba przeciez ustawiasz przy kazdym...   16.06.2010, 12:04:58
- - xamrex   [PHP] pobierz, plaintext <?phpob_start(...   16.06.2010, 12:12:27
- - nospor   Powinno byc ok. A co jesli ktoś będzie chciał na n...   16.06.2010, 12:32:19
- - xamrex   CytatA co jesli ktoś będzie chciał na nowo zacząc ...   16.06.2010, 12:59:58
- - zend   DRY - pl. Nie Powtarzaj Się. Wwal cały kod z if...   16.06.2010, 13:08:41
- - xamrex   zaden kos sie nie powtarza. <input type=text na...   16.06.2010, 13:15:50
- - zend   [PHP] pobierz, plaintext if(!$wygrał)...   16.06.2010, 13:20:08
- - xamrex   [PHP] pobierz, plaintext if (empty($cyfra_...   16.06.2010, 14:00:59
- - zend   Dalej powtarzasz ten sam kod   16.06.2010, 14:04:33

« Następny starszy · Przedszkole · Następny nowszy »
 

Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Aktualny czas: 9.10.2025 - 00:59
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn