[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

[wookieb]

Do manuala
http://pl2.php.net/manual/en/control-structures.switch.php
http://pl2.php.net/manual/en/control-structures.if.php

Bo to co masz wynika z twoich braków z wiedzy opisanej w byle jakim podstawowym kursie.

[Początkujący]

Dopiero co zacząłem uczyć się php.
Przerabiam lekcję dotyczącą switcha i postanowiłem zrobić własny kod na podstawie zdobytej wiedzy. Dotychczas nigdy nie korzystałem z manuala, a teraz nie znalazłem w nim interesującej informacji.
Wiem, że za pomocą funkcji if mógłbym to zrobić, ale chcę po prostu zobaczyć, jak można to zrobić za pomocą funkcji switch.
Bo chyba nie trzeba wpisywać kilkadziesiąt razy case z kolejnymi godzinami, żeby funkcja switch działała prawidłowo?

Edycja: teraz działa, wystarczyło usunąć breaki

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

[wookieb]

Edycja: teraz działa, wystarczyło usunąć breaki

Breaki nie mają z tym nic wspólnego.

[minolone]

Witam. Rozwiązanie troche inne, lecz działa.

[PHP] pobierz, plaintext 
<?php 
    $godzina = 4;
 
    if ( $godzina > 0 && $godzina < 14 )
    {
        echo 'Dzień dobry!';
    }elseif( $godzina >= 14 && $godzina < 18 )
    {
        echo 'Dobrego popołudnia';
    }elseif ( $godzina >= 18 && $godzina <= 24 )
    {
        echo 'Dobry wieczór';
    }else
    {
        echo 'Błąd';
    }
 
?>
[PHP] pobierz, plaintext 

Ten post edytował minolone 27.08.2010, 17:30:12

[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

[Początkujący]

Dziękuję za porady. Jednak Po case dałem dwukropki, usunąłem operator porównywania w zmiennej godzina. I niestety dalej nie działa. Chcę to zrobić na switchu, żeby zapoznać się ze schematem robienia instrukcji w nim. Jednak dalej wyświetla dzień dobry, zamiast "ta godzina to zła godzina".

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

[wookieb]

Kumam jeden może nie umieć ale dwóch pod rzad?
DLACZEGO WASZE CASE-Y SĄ STRINGAMI wyrażeń logicznych?
Nie dość, że nie czytacie manuala to nawet jak czytacie to wymyślone bzdety wstawiacie

[PHP] pobierz, plaintext 
 
switch (true) {
 
case  $godzina>0 && $godzina<14:
 
(.. jedziecie dalej)
[PHP] pobierz, plaintext 

[Początkujący]

Działa. Wookieb, możesz wyjaśnić, w jakich sytuacjach bo case powinien być apostrof/cudzysłów, a w jakich nie?
Bo z tego " The statement list for a case can also be empty, which simply passes control into the statement list for the next case. " nie bardzo rozumiem.
Z gory dzięki

[wookieb]

To co masz jest specyficznym przypadkiem. Poczytaj o wyrażeniach logicznych.
Wynik "case" musi być równy temu co podajesz do switch-a.

[Webmasterr]

Sorki za błąd. Pisałem na szybko, ale trzeba będzie to powtórzyć...