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
zend
post
Post #2





Grupa: Zarejestrowani
Postów: 580
Pomógł: 85
Dołączył: 25.03.2010
Skąd: Skrzyszów :)

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

1) $cyfra_wygenerowana<$cyfra_usera - tu porównujesz dwa stringi, typuj na (int)$zmienna
2) ini_set('display_errors' , 'On'); error_reporting(E_ALL | E_STRICT); na początku
3) if (empty($cyfra_usera)) => if(isset($_POST['xxx']))
4) $proba++; a gdzie jest deklaracja $proba = 0; ?
5) To można poprawić
[PHP] pobierz, plaintext 
  1. ob_end_flush();//Problemy z headers already sent
  2. echo '</body>';
  3. echo '</html>';
[PHP] pobierz, plaintext

6) setcookie ('proba', "$proba", time()+3600); "$proba" WTF?
7) Poczytaj sobie o DRY
[PHP] pobierz, plaintext 
  1. if (empty($cyfra_usera))
  2. {
  3.    echo 'Wpisz cyfrę (od 1 do 10)';   
  4.    echo '<form action="index.php" method="POST">CYFERKA: <input type=text name="cyfra"><input type=submit value="Wyślij"></form>';
  5. }elseif ($cyfra_wygenerowana>$cyfra_usera){
  6.     echo 'Za mało, co ty taki skromny??';
  7.     $proba++;
  8.     setcookie ('proba', "$proba", time()+3600);
  9.     echo '<form action="index.php" method="POST">CYFERKA: <input type=text name="cyfra"><input type=submit value="Wyślij"></form>';
  10. }elseif ($cyfra_wygenerowana<$cyfra_usera){
[PHP] pobierz, plaintext

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: 5.10.2025 - 18:56
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn