[PHP]Jak wydajniej rozwiąć pewniem problem Opcje

[DR.GhostRider]

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 (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif) ?

[Foxx]

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.