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 (1 - 5)
Ailvenge
post
Post #2





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

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

Dodałbym funkcję filtrującą dane liczbowe: function.intval i wtedy wyrzucil ostatniego elsa
No i te wcięcia póki co jakoś dziwnie ułożyłeś, nie chce mi się tłumaczyć wszystkiego, ale tu znalazłem jakiś poradnik.
Go to the top of the page
+Quote Post
lobopol
post
Post #3





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
ReporteR
post
Post #4





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

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

Dzięki wielkie, jak wrócę to będę sobie z tym kombinował... (IMG:style_emoticons/default/biggrin.gif)

Ktoś jeszcze ma coś do dodania ? (IMG:style_emoticons/default/smile.gif) 

[PHP] pobierz, plaintext 
  1. $max_pozyczka = isset($_GET['pozyczka']) ? $_GET['pozyczka'] : null;
[PHP] pobierz, plaintext


Tego nie rozumiem, możesz mi to objaśnić ?
Go to the top of the page
+Quote Post
CuteOne
post
Post #5





Grupa: Zarejestrowani
Postów: 2 958
Pomógł: 574
Dołączył: 23.09.2008
Skąd: wiesz, że tu jestem?

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

poczytaj o switch (IMG:style_emoticons/default/smile.gif)
Go to the top of the page
+Quote Post
wookieb
post
Post #6





Grupa: Moderatorzy
Postów: 8 989
Pomógł: 1550
Dołączył: 8.08.2008
Skąd: Słupsk/Gdańsk
Temat: Jak poprawnie zatytulowac watek
Go to the top of the page
+Quote Post
« 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 · Linearny+ · Przełącz na: Drzewo

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

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