Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP]Można to jakoś usprawnić ? :)
ReporteR
post
Post #1





Grupa: Zarejestrowani
Postów: 7
Pomógł: 0
Dołączył: 26.10.2010

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

Cześć wszystkim, przeczytałem w godzinkę trochę poradnika, i postanowiłem o własnych siłach napisać jakiś najprostszy skrypt. Aktualnie działa, ale czy możecie powiedzieć mi co jest w nim źle, co można zmienić / usprawnić, jak można go zabezpieczyć przed wartościami minusowymi lub wpisywanym tekstem itd. Z góry wielkie dzięki, i nie wkurzajcie się, ale na prawdę siedzę w tym dopiero od kilku godzinek... (IMG:style_emoticons/default/arrowheadsmiley.png) 

[PHP] pobierz, plaintext 
  1. <html>
  2.  <head>
  3.   <title>
  4. 	System pozyczkowy
  5.   </title>
  6.  </head>
  7.  <body>
  8.  <form action="index.php" method="GET">
  9.   Ile potrzebujesz: <input type="text" name="pozyczka"/><br/>
  10.   <input type="submit" value="Wyslij"/>
  11.   </form>
  12.  </body>
  13.  
  14. </html>
  15.  
  16. <?php
  17.  
  18. $max_pozyczka = $_GET['pozyczka'];
  19.  
  20. 	if ($_GET['pozyczka']==null)
  21. 	{
  22. 	echo "";
  23. 	}
  24. 	elseif ($_GET['pozyczka']<=1000)
  25. 	{
  26. 		echo "Nie doceniasz sie! Możesz wziac wiecej niz $max_pozyczka zlotych pozyczki :)";
  27. 	}
  28. 	elseif ($_GET['pozyczka']>=1001 && $_GET['pozyczka']<=5000)
  29. 	{
  30. 		echo "Gratulacje! Wziales pozyczke o wysokosci $max_pozyczka zlotych (IMG:style_emoticons/default/biggrin.gif) ";
  31. 	}
  32. 	elseif ($_GET['pozyczka']>=5001)
  33. 	{
  34. 		echo "Pozyczka nie moze wyniesc $max_pozyczka zlotych - maksimum to 5000 zlotych!";
  35. 	}
  36. 	else
  37. 	{
  38. 		echo "Bledna wartosc!";
  39. 	}
  40. ?>
[PHP] pobierz, plaintext
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
lobopol
post
Post #2





Grupa: Zarejestrowani
Postów: 1 729
Pomógł: 346
Dołączył: 4.04.2009

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

1.Przed wszystkim powinno być sprawdzenie czy ten get jest ustawiony
[PHP] pobierz, plaintext 
  1. if(isset($_GET['pozyczka'])){
  2.  
  3. }
[PHP] pobierz, plaintext

bądź przy tej formie użyć krótkiego ifa:
[PHP] pobierz, plaintext 
  1. $max_pozyczka = isset($_GET['pozyczka']) ? $_GET['pozyczka'] : null;
[PHP] pobierz, plaintext


2. Lepszą opcją byłoby wykonanie tego przed wyświetleniem html, a nie za nim i przypisanie wyniku do zmiennej no i oczywiście używanie $max_pozyczka zamiast tego geta
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. $max_pozyczka = isset($_GET['pozyczka']) ? $_GET['pozyczka'] : null;
  4.  
  5. if ($max_pozyczka == null){
  6. 	   $wynik = '';
  7. } elseif ($max_pozyczka<=1000){
  8. 	    $wynik = "Nie doceniasz sie! Możesz wziac wiecej niz $max_pozyczka zlotych pozyczki :)";
  9. } elseif $max_pozyczka>=1001 && $max_pozyczka<=5000){
  10. 	    $wynik = "Gratulacje! Wziales pozyczke o wysokosci $max_pozyczka zlotych (IMG:style_emoticons/default/biggrin.gif) ";
  11. } elseif ($max_pozyczka>=5001){
  12. 	    $wynik = "Pozyczka nie moze wyniesc $max_pozyczka zlotych - maksimum to 5000 zlotych!";
  13. } else{
  14. 	    $wynik = "Bledna wartosc!";
  15. }
  16. ?>
  17. <html>
  18.  <head>
  19.   <title>
  20. 	System pozyczkowy
  21.   </title>
  22.  </head>
  23.  <body>
  24.  <form action="index.php" method="GET">
  25.   Ile potrzebujesz: <input type="text" name="pozyczka"/><br/>
  26.   <input type="submit" value="Wyslij"/>
  27.   </form>
  28.   <?php echo $wynik ?>
  29.  </body>
  30.  
  31. </html>
  32.  
  33.  
  34. [/php[
  35.  
  36. 3. A jak spodziewasz się tylko wartości intowych to warto zrobić rzutowanie na inta
  37. [php]
  38. $zmienna = (int)$zmienna;
[PHP] pobierz, plaintext

Go to the top of the page
+Quote Post

Posty w temacie
- ReporteR   [PHP]Można to jakoś usprawnić ? :)   3.09.2011, 23:12:39
- - Ailvenge   Dodałbym funkcję filtrującą dane liczbowe: functio...   4.09.2011, 02:13:50
- - lobopol   1.Przed wszystkim powinno być sprawdzenie czy ten ...   4.09.2011, 07:46:21
- - ReporteR   Dzięki wielkie, jak wrócę to będę sobie z tym komb...   4.09.2011, 12:49:51
- - CuteOne   poczytaj o switch   4.09.2011, 12:51:21
- - wookieb   Temat: Jak poprawnie zatytulowac watek   4.09.2011, 12:52:26

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

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