[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

[vermis]

A co konkretnie masz na myśli?

[crazy_programmer]

Wcześniej miałem pewien problem. Otóż gdy zmienne utworzyłem w dalszej części skryptu a na początku użyłem instrukcji warunkowej:

[PHP] pobierz, plaintext 
if($ilosc>0 && $wartosc>0)
echo 'Zamowienie zostało przyjęte o '.date('H:i, jS F');
[PHP] pobierz, plaintext 

Instrukcja ta była umieszczona na początku skryptu. Cały czas wywałało mi błąd. Gdy jednak przeniosłem zmienne na początek skryptu instrukcja warunkowa zaczęła działać. O co chodzi?

[vermis]

Chodzi o to, że zmienne przed użyciem muszą zostać zainicjowane. Jeśli ta instrukcja był na początku to porównywałeś zmienne, które nie istnieją stąd błąd (pewnie że zmienna nie istnieje).

[Crozin]

[PHP] pobierz, plaintext 
$podatekvat = 0.22;
[PHP] pobierz, plaintext 

Chciałbyś 22% co nie? 23% mamy.

Instrukcja ta była umieszczona na początku skryptu. Cały czas wywałało mi błąd. Gdy jednak przeniosłem zmienne na początek skryptu instrukcja warunkowa zaczęła działać. O co chodzi?

Pokaż dokładny kod jaki miałeś.

Hint:
1. Cena* nie powinna być stałą, a normalną zmienną.
2. Przestrzegaj konwencji nazewnictwa, czyli: $nazwaZmiennej, NAZWA_STAŁEJ

[crazy_programmer]

[PHP] pobierz, plaintext 
 
	$iloscopon = (integer)$_POST['iloscopon'];
	$iloscoleju = (integer)$_POST['iloscoleju'];
	$iloscswiec = (integer)$_POST['iloscswiec'];
 
	if($ilosc > 0 && $wartosc > 0)
		echo 'Zamówienie zostało przyjęte o '.data('H:i, jS F');
 
		echo $iloscopon.'	opon<br />';
		echo $iloscoleju.'	oleju<br />';
		echo $iloscswiec.'	swiec<br />';
 
 
	$ilosc = 0;						
	$wartosc = 0.00;				
 
 
	define("CENAOPON", 150);
	define("CENAOLEJU", 25);
	define("CENASWIEC", 50);
 
	$ilosc = $iloscopon + $iloscoleju + $iloscswiec;
 
	echo 'Zamówionych produktów:	'.$ilosc.'<br />';
 
	$wartosc = $iloscopon * CENAOPON
			 + $iloscoleju * CENAOLEJU
			 + $iloscswiec * CENASWIEC;
 
	echo 'Cena Netto:	'.number_format($wartosc, 2).' PLN<br />';
 
	$podatekvat = 0.22;
	$wartosc = $wartosc * (1 + $podatekvat);
 
	echo 'Cena Brutto:	'.number_format($wartosc, 2).' PLN<br />';
 
[PHP] pobierz, plaintext 

Wykonująć instrukcję warunkową if i używając funkcji data(), wyświetlany jest blą nieznana zmienna.

[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 ( { ... } ).

[potreb]

You are crazy_programmer. A gdzie masz wcześniej zdefiniowaną zmienną wartosc? Sprawdzasz zmienna wartosc, która nie istnieje.

[crazy_programmer]

Dzięki za wytłumaczenie Przpraszam że zadaje takie proste pytania - dopiero się uczę