[SWITCH] operator || Opcje

[TheArt]

Czy prawidłowym jest zapis

[PHP] pobierz, plaintext 
case 'oferta' || 'produkty' || 'sklepy' :
[PHP] pobierz, plaintext 

[wookieb]

nie

[PHP] pobierz, plaintext 
case 'oferta':
case 'produkty':
case 'sklepy':
 
[PHP] pobierz, plaintext 

Ale oczywiście mogłeś to sprawdzić...

Ten post edytował wookieb 5.01.2010, 00:08:54

[TheArt]

właśnie dziwiłem się że operator || nie działa, jak inaczej to zrobić żeby nie pisać setek linii kodu

[Meares]

Zabijecie mnie za to - użyj ifów.

[darko]

Możesz trzymać wartości w tablicy $gdzie i sprawdzać :

[PHP] pobierz, plaintext 
if(in_array($co, $gdzie)) {
  // i tutaj np. wyciągasz klucz odpowiadający znalezionej wartości za pomocą array_search
}
 
// albo
 
if (array_key_exists($co, $gdzie)) {
  // i tu znowu jakaś operacja powiązana ze znalezionym $co
}
[PHP] pobierz, plaintext 

Możliwości jest sporo.

[thek]

TheArt.. Wookieb Ci już napisał :/
użyj

[PHP] pobierz, plaintext 
case "jedna":
case "druga":
case "trzecia:"
    //tu instrukcje;
    break;
[PHP] pobierz, plaintext 

Switch działa tak, że wykonuje instrukcje do czasu aż napotka break. Jeśli go nie napotka to wykonuje wszystko dalej, nawet jeśli to następny case. Tak więc jeśli nie trafi na żaden z trzech to pójdzie dalej szukać. Jeśli trafi to wykona instrukcję spod "trzecia", bo są one instrukcjami widocznymi także dla "pierwsza" i "druga". Dlatego właśnie zamykasz zazwyczaj poprzez break ciąg instrukcji dla każdego case. Jeśli jednak wiesz, że coś jest wspólne, to walisz je bez break obok i ewentualnie przy drobnych różnicach w głównym bloku instrukcji wyszczególniasz gdzie mają być drobne modyfikacje.

[PHP] pobierz, plaintext 
switch (uri::segment(3)) {
    case 'nowi':
        //ustawianie podstawowych parametrów
        $this->podstrona = new View('domyślny widok');
        js::add('media/js/jquery.dataTables.min.js');
        css::add('media/css/datatables.css');
        //ciąg instrukcji
        $this->podstrona->zmienna = $this->model->pobieranie_danych();
        break;
    case 'aktywni':
    case 'wylaczeni':
    case 'wygasa':
    case 'logowania':
    case 'leniwi':
    case 'callcenter':
        //sprawdzanie, czy użytkownik ma prawo odwiedzić podstronę:
        if(!valid::level(0,1)) {url::redirect_home();}
        //ustawianie podstawowych parametrów
        $this->podstrona = new View('inny_panel');
        $this->podstrona->typ = uri::segment(3);
        js::add('media/js/jquery.dataTables.min.js');
        css::add('media/css/datatables.css');
        break;
    case 'edit':
        if(!uri::segment(4) || !is_numeric(uri::segment(4))) {
            //tu jeden redirect 
            url::redirect('domyślne przekierowanie');
        }
        //ustawianie podstawowych parametrów
        break;
    default:
        //przekieruj wszystkie wywołania nie odpowiadające warunkom
        url::redirect('domyślne przekierowanie');
}
[PHP] pobierz, plaintext 

Zwróć w przykładzie jednego z moich kodów jak mam rozwiązane to LUB. Posługuję się potem w bloku identycznego kodu dla wszystkich tylko uri::segment(3) decydując co wyświetlę (to akurat odpowiada za ustawienie klasy "active" dla wybranej zakładki). Oczywiście przy sprawdzaniu rozszerzonym helperem (valid::level) wcześniej, czy ma do tego prawo

Ten post edytował thek 5.01.2010, 09:36:00

[wookieb]

[PHP] pobierz, plaintext 
$keys1 = array('f1', 'f2', 'f3');
$keys2 = array('t1', 't2', 't3');
 
switch(true)
{
   case in_array($find, $keys1):
 
 
  break;
 
   case in_array($find, $keys2):
 
   break;
 
}
[PHP] pobierz, plaintext