Witajcie mam pewien problem

mam kodzik

[PHP] pobierz, plaintext 
<?php
           if($_GET['show'] == '4'){
           $class4 = 'class="active"';
           }
           elseif($_GET['show'] == '3'){
           $class3 = 'class="active"';
           }
           elseif($_GET['show'] == '2'){
           $class2 = 'class="active"';
           }
            elseif($_GET['show'] == '1'){
           $class1 = 'class="active"';
           }
           elseif(empty($_GET['show']) && empty($_GET['type'])  && empty($_GET['act']) ){
           $class = 'class="active"';     
           }
  ?>
            <li><a <?php echo $class; ?> href="./">cos1</a></li>
            <li><a <?php echo $class4; ?> href="?page=info&amp;show=4">cos1</a></li>
            <li><a <?php echo $class2; ?> href="?page=info&amp;show=3">cos2</a></li>
            <li><a <?php echo $class3; ?> href="?page=info&amp;show=2">cos3</a></li>
            <li><a <?php echo $class1; ?> href="?page=info&amp;show=1">cos4</a></li>
        </ul> <!-- Menu end -->
[PHP] pobierz, plaintext 

czy idzie jakoś wydajniej rozwiać ten kod ,ponieważ mam 2 menu które ma jeszcze 10 elseif() więcej ;p ?

while" title="Zobacz w manualu PHP" target="_manual

A nie przypadkiem

[PHP] pobierz, plaintext 
<?php
switch($_GET['show'] {
   case '2' : ... ; break;
 
 ...itd...
 
?>
[PHP] pobierz, plaintext 

Co Ci tutaj da while ?

co mi da while?

[PHP] pobierz, plaintext 
<?php
$x=1;
while($x<=4){
  if($_GET['cos']==$x){
    $class[$i] = 'class="active"';
  }
$x++;
}?>
           <li><a <?php echo $class[1]; ?> href="./">cos1</a></li>
           <li><a <?php echo $class[2]; ?> href="?page=info&amp;show=4">cos1</a></li>
        //itp
[PHP] pobierz, plaintext 

coś w tym stylu

W takiej sytuacji, gdy nazwa użytej zmiennej zależy od wartości zmiennej, mógłbyś spróbować użyć zmiennej zmiennej. Czyli cały kod z ifami zastąpić 1 linią:

[PHP] pobierz, plaintext 
<?php
${'class'.$_GET['show']} = 'class="active"';
?>
[PHP] pobierz, plaintext 

Ale w takiej sytuacji, gdy zmienna show przychodzi z zewnątrz, musisz kontrolować wcześniej jej zawartość, czyli sprawdzać czy jest liczbą z odpowiedniego zakresu.

to mi wyglada na cos takiego ;-)

http://pl.wikibooks.org/wiki/PHP/Instrukcja_switch


Wiec chyba switch ;-)

Pozdrawiam

Poza tym chodzi o klasy CSS dla lina aktywnego/nieaktywnego, tak?
Więc tak naprawdę całą tę rzeźbę można zastąpić:

[PHP] pobierz, plaintext 
$show = (integer)$_GET['show'];
 
...
 
<li><a class="<?php ($show == 3)? echo 'active' : 'inactive'  ; ?> " href="?page=info&show=3">cos3</a></li>
<li><a class="<?php ($show == 4)? echo 'active' : 'inactive'  ; ?> " href="?page=info&show=4">cos4</a></li>
...
[PHP] pobierz, plaintext 

ani switch ani if, teraz jest kwestia 4 pytań,
ale co jeśli by trzeba 100 warunków wykonać ?
ręcznie byś wklepywać 100 if

//edit
blooregard-> w twoim wypadku to warto by dać <li>/html w pętle