[eluser]manilodisan[/eluser]
It's not bad as long as you don't repeat code such as that switch statement in multiple parts of your application. However, I hate seeing so much php code inside the views (but this is me, nothing wrong to it). I would rather make a function and set the menu part slightly different:
Code:
<div id="topo">
<?php
@include "topo/banner.php";
?>
</div><!--fim de topo-->
<div id="corpo">
<div id="painel">
<div id="menu">
<div id="menulista">
<ul>
<li><a href="index.php">Home</a></li>
<li><a href="index.php?menu=link1">Link 1</a></li>
<li><a href="index.php?menu=link2">Link 2</a></li>
<li><a href="index.php?menu=link3">Link 3</a></li>
<li><a href="index.php?menu=link4">Link 4</a></li>
<li><a href="index.php?menu=link5">Link 5</a></li>
<li><a href="index.php?menu=link6">Link 6</a></li>
<li><a href="index.php?menu=link7">Link 7</a></li>
</ul>
</div><!--fim de menulista-->
</div><!--fim de menu-->
</div><!--fim de painel-->
<div id="principal">
<?= set_menu () ?>
</div><!--fim de principal-->
<div id="colunalateral">
</div><!--fim de colunalateral-->
</div><!--fim de corpo-->
<div id="rodape">
<?php
@include "rodape.php";
?>
</div><!--fim de rodape-->
Code:
function set_menu ()
{
if (isset($_GET['menu'])){
switch ($_GET['menu']){
case linkum:
@include ("linkum.php");
break;
case linkdois:
@include ("linkdois.php");
break;
case linktres:
@include ("linktres.php");
break;
case linkquatro:
@include ("linkquatro.php");
break;
case linkcinco:
@include ("linkcinco.php");
break;
case linkseis:
@include ("linkseis.php");
break;
case linksete:
@include ("linksete.php");
break;
}
}
else {
@include ("home.php");
}
}
Also, there's an interesting thread running in this forum, you might wanna check it out:
http://ellislab.com/forums/viewthread/100620/