Forum PHP.pl > [PHP]Można to jakoś usprawnić ? :)

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP]Można to jakoś usprawnić ? :)

ReporteR

3.09.2011, 23:12:39

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

[PHP] pobierz, plaintext 
<html>
 <head>
  <title>
	System pozyczkowy
  </title>
 </head>
 <body>
 <form action="index.php" method="GET">
  Ile potrzebujesz: <input type="text" name="pozyczka"/><br/>
  <input type="submit" value="Wyslij"/>
  </form>
 </body>
 
</html>
 
<?php
 
$max_pozyczka = $_GET['pozyczka'];
 
	if ($_GET['pozyczka']==null)
	{
	echo "";
	}
	elseif ($_GET['pozyczka']<=1000)
	{
		echo "Nie doceniasz sie! Możesz wziac wiecej niz $max_pozyczka zlotych pozyczki :)";
	}
	elseif ($_GET['pozyczka']>=1001 && $_GET['pozyczka']<=5000)
	{
		echo "Gratulacje! Wziales pozyczke o wysokosci $max_pozyczka zlotych ";
	}
	elseif ($_GET['pozyczka']>=5001)
	{
		echo "Pozyczka nie moze wyniesc $max_pozyczka zlotych - maksimum to 5000 zlotych!";
	}
	else
	{
		echo "Bledna wartosc!";
	}
?>
[PHP] pobierz, plaintext

Ailvenge

4.09.2011, 02:13:50

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.

lobopol

4.09.2011, 07:46:21

1.Przed wszystkim powinno być sprawdzenie czy ten get jest ustawiony

[PHP] pobierz, plaintext 
if(isset($_GET['pozyczka'])){
 
}
[PHP] pobierz, plaintext

bądź przy tej formie użyć krótkiego ifa:

[PHP] pobierz, plaintext 
$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 
<?php
 
$max_pozyczka = isset($_GET['pozyczka']) ? $_GET['pozyczka'] : null;
 
if ($max_pozyczka == null){
	   $wynik = '';
} elseif ($max_pozyczka<=1000){
	    $wynik = "Nie doceniasz sie! Możesz wziac wiecej niz $max_pozyczka zlotych pozyczki :)";
} elseif $max_pozyczka>=1001 && $max_pozyczka<=5000){
	    $wynik = "Gratulacje! Wziales pozyczke o wysokosci $max_pozyczka zlotych ";
} elseif ($max_pozyczka>=5001){
	    $wynik = "Pozyczka nie moze wyniesc $max_pozyczka zlotych - maksimum to 5000 zlotych!";
} else{
	    $wynik = "Bledna wartosc!";
}
?>
<html>
 <head>
  <title>
	System pozyczkowy
  </title>
 </head>
 <body>
 <form action="index.php" method="GET">
  Ile potrzebujesz: <input type="text" name="pozyczka"/><br/>
  <input type="submit" value="Wyslij"/>
  </form>
  <?php echo $wynik ?>
 </body>
 
</html>
 
 
[/php[
 
3. A jak spodziewasz się tylko wartości intowych to warto zrobić rzutowanie na inta
[php]
$zmienna = (int)$zmienna;
[PHP] pobierz, plaintext

ReporteR

4.09.2011, 12:49:51

Dzięki wielkie, jak wrócę to będę sobie z tym kombinował...

Ktoś jeszcze ma coś do dodania ?

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

Tego nie rozumiem, możesz mi to objaśnić ?

CuteOne

4.09.2011, 12:51:21

poczytaj o switch

wookieb

4.09.2011, 12:52:26

Temat: Jak poprawnie zatytulowac watek

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.