[PHP]Instrukcja switch - obsługa and? Opcje

[Początkujący]

Mam taki kod:

[PHP] pobierz, plaintext 
<?php 
$godzina==125;
switch ($godzina) {
case "$godzina>0 and $godzina<14";
echo "Dzień dobry!";
break;
case "$godzina>=14 and $godzina<18";
echo "Dobrego popołudnia";
break;
case "$godzina>=18 and $godzina<=24";
echo "dobry wieczór";
break;
default:
echo "Ta godzina to zła godzina";
}
 
?>
[PHP] pobierz, plaintext 

No i niestety, gdy go skompiluję, wyświetla mi się kod: Dzień dobry, pomimo tego, że $godzina=125.
Co zrobiłem źle z tym kodem? A moze instrukcja switch nie obsługuje polecenia and?

Ten post edytował Początkujący 27.08.2010, 15:15:06

[Webmasterr]

[PHP] pobierz, plaintext 
<?php 
$godzina==125;
switch ($godzina) {
case "$godzina>0 and $godzina<14";
echo "Dzień dobry!";
break;
case "$godzina>=14 and $godzina<18";
echo "Dobrego popołudnia";
break;
case "$godzina>=18 and $godzina<=24";
echo "dobry wieczór";
break;
default:
echo "Ta godzina to zła godzina";
}
 
?>
[PHP] pobierz, plaintext 

Na początek 1 linijka. == To operator porównania, tym = przypisuje się wartość.
Więc zamiast '==' ma być '='.

Zamiast 'and' lepiej używać operatora '&&'.

I jeszcze jedno, masz:

[PHP] pobierz, plaintext 
case "wyrażenie";
[PHP] pobierz, plaintext 

a po po sekcji case powinien być dwukropek (:), nie średnik (;).


Mam jeszcze uwagę co do wydajności kodu.
Zamiast cudzysłowów (") należy używać operatora konkatenacji tzn. łączenia tekstu z czymkolwiek innnym(.).
W takim wypadku zamiast:

[PHP] pobierz, plaintext 
case "$godzina>0 and $godzina<14":
[PHP] pobierz, plaintext 

byłoby:

[PHP] pobierz, plaintext 
case $godzina.'>0 and '.$godzina.'<14':
[PHP] pobierz, plaintext 

Po prostu w cudzysłowach parser PHP szuka zmiennych, i innych typów danych, a w apostrofach nie szuka nic.

Ten post edytował Webmasterr 27.08.2010, 16:25:44