[PHP]Czy ten kod może być zapisany w taki sposób? Opcje

[crazy_programmer]

Cześć wszystkim. Właśnie kończę się uczyć pierwszego rozdziału z książki "PHP I MySQL Vademecum Profesionalisty Wydanie III" i mam do was pewne pytnako. Czy ten kod może być zapisany w taki sposób?

[PHP] pobierz, plaintext 
$iloscopon = (integer)$_POST['iloscopon'];
$iloscoleju = (integer)$_POST['iloscoleju'];
$iloscswiec = (integer)$_POST['iloscswiec'];
 
$ilosc = 0;
$wartosc = 0;
$podatekvat = 0.22;
 
define('CENAOPON', 250);
define('CENAOLEJU', 100);
define('CENASWIEC', 50);
 
$ilosc = $iloscopon + $iloscoleju + $iloscswiec;
$wartosc = CENAOPON * $iloscopon
              + CENAOLEJU * $iloscoleju
  	+ CENASWIEC * $iloscswiec;
 
if( $ilosc > 0 && $wartosc > 0)
	{
	echo 'Zamówienie zostało przjęte o '.date('H:i, jS F').'<br />';
	echo 'Zostały zamówione następujące produkty: '.'<br />'
				           $iloscopon.' opon(y)<br />'
                                                                   $iloscoleju.' butelki oleju<br />'
				           $iloscswiec.' sztuk swiec<br />';
	echo 'Cena netto: '.number_format($wartosc, 2).'<br />';
	echo 'Cena brutto: '.number_format($wartosc * (1 + $podatekvat), 2);
	}
	else
	{
	echo 'Nie pwrowadziłeś żadnych danych';
	}
[PHP] pobierz, plaintext 

Ten post edytował crazy_programmer 13.01.2011, 22:36:44

[Crozin]

1. Używałeś zmiennych przed ich zadeklarowaniem ($ilosc i $wartosc nie istnieją w momencie ich użycia w linii #6). PHP nie traktuje tego jako błąd (a powinno) i w tle utworzy takie zmienne z wartością 0, co sprawi, że warunek nigdy nie zostanie spełniony.
2. Wszystkie instrukcje kontroler (if, else, for, while, foreach itd.) odnoszą się do dokładnie jednej instrukcji po nich lub jednego bloku kodu ( { ... } ).