Question

I have the following code which redirect pages depends on $path.

...
$path = $this->uri->segment(3);
$pathid = $this->uri->segment(4);
if($path=='forsiden'){
            redirect('','refresh');
        }elseif($path =='contact'){
            redirect('welcome/kontakt','refresh');
        }elseif($path =='illustration'){
            $this->_gallery($path,$pathid);
        }elseif($path =='webdesign'){
            redirect('welcome/webdesign','refresh');    
        }elseif($path==('web_tjenester' || 'webdesigndetails' || 
                        'vismahjemmeside' || 'joomla' || 'vismanettbutikk' || 
                        'vpasp' || 'artportfolio')){
        ...
     CODE A   
     ...    

}else{
        ...
        CODE B
        ...
                }

I am not getting right results with

$path==('web_tjenester' || 'webdesigndetails' || 
'vismahjemmeside' || 'joomla' || 'vismanettbutikk' || 
'vpasp' || 'artportfolio')

contact, illustration, gallery and webdesign are redirected and working alright.
However all other pages are added CODE A.

I am expecting CODE A only when $path is web_tjenester’, ‘webdesigndetails’, ‘vismahjemmeside’, ‘joomla’, ‘vismanettbutikk’, ‘vpasp’ or ‘artportfolio’.

Could anyone point out my mistake and correct me please?

Thanks in advance.

–UPDATE–

The following works, but is there any ways shorten the code?

I am repeating ($path==..).

elseif(($path=='web_tjenester') || ($path=='webdesigndetails') || 
    ($path=='vismahjemmeside') || ($path=='joomla') || ($path=='vismanettbutikk') || 
    ($path=='vpasp') || ($path=='artportfolio')){

Answer 1

The || operator is a logical operator. The semantics of the expression a || b is that it evaluates to true if either a or b evaluates to true, and false otherwise.

So it cannot be used in that way you intended it as the expression ('web_tjenester' || … || 'artportfolio') will be evaluated to a boolean value (to be specific: true as 'web_tjenester' evaluates to true using boolean conversion) that then will be compared to the value of $path.

If you want to test if a value is in a set of values, you can use an array and test the value with in_array:

in_array($path, array('web_tjenester', 'webdesigndetails', 'vismahjemmeside', 'joomla', 'vismanettbutikk', 'vpasp', 'artportfolio'))