Ben daha fazla veri odaklı şeyi yapma öneriyoruz. Menü seçenekleri bir çok-boyutlu bir dizi oluşturmak ve sonra doğru HTML çıktısı, dizi yineleme.

İşte ben bunu yapacağını nasıl (bu test edilmemiştir, ama fikri ben aktarmaya çalışıyorum budur):

<?
	$p = $_GET['p'];

  	$navArray = array();
    $navArray['home'] = array('main' => array('id'=>'home', 'url'=>'/', 'title'=>'Home'));
    $navArray['more'] = array('main' => array('id'=>'more', 'url'=>null, 'title'=>'More'), 
							  'pages'=> array(
								array('id'=>'widgets', 'url'=>'/more-widgets.php', 'title'=>'Widgets'),
								array('id'=>'news', 'url'=>'/news.php', 'title'=>'News')));

?>


<div id="bottomrow"> 
      <div class="pad"> 
       	<ul class="menu left">

			<? foreach($navArray as $navHeading) : ?>
				<? $current = $navHeading['main']['id'] == $p ? 'current' : ''; ?>
	          	<li class="first <?=$current?>"><a href="<?=$navHeading['main']['url']?>"><em><?=$navHeading['main']['title']?></em></a></li>

				<? if (!empty($navHeading['pages'])) : ?>
					<ul id="sub-<?=$navHeading['main']['id']?>">
					<? foreach($navHeading['pages'] as $navPage) : ?>
						<? $current = $navPage['id'] == $p ? 'current' : ''; ?>
						<li class="<?=$current?>"><a href="<?=$navPage['url'];?>"><em><?=$navPage['title']?></em></a></li>
					<? endforeach; ?>
					</ul>
				<? endif; ?>
			<? endforeach; ?>
		</ul>
	</div>
</div>

Öncelikle, ben bu menünün oluşturulması idare ve index.php sınıf yöntemi çağırmak için bir menü sınıf oluşturma öneririm. Bu, sunum kodunu ayırmak çağırdığınızda bunu sayfayı geçebileceği vb soyutlama, sağlamak ve sınıf uygulaması ilgilenir ve tam geri menü geçer olur.

İkincisi, ben bir sürü veya koşullarına sahip bir switch deyimini kullanarak yerine else if ... öneririm. Sen mola anahtar dahil değil tarafından davaları basamakla:

switch( $page) {
  case: 'home':
    $current_home = 'current';
    break;
  case: 'users.online':
  case: 'users.location':
  case: 'users.featured':
  .......
    $current_users = 'current';
    break;
}

Bildiğim kadarıyla PHP kullanımı gider gibi, size çok kullanmıyorsunuz demektir. Ben eğer ifadeleri ile de söylediğim gibi, eğer ben her aksine, $ sayfa kez değerlendirilir ve bir kez uyumlu olacaktır, bir anahtar çok daha iyi yapacağını düşünüyorum. Eğer menü için bir döngü kullanmak olsaydı, bu kod gerçek değişmez HTML miktarını azaltmak olabilir. O zaman menü sistemi oluşturmak için dizi içeriği, bazı html, ve değişkenleri dizi üzerinde döngü ve birleştirmek olabilir, her bir ana menü parça (belki diğerleri ne anlama geldiğini) tutmak için dizileri yarattı eğer. PHP daha az edebi HTML ve daha verimli kullanımı.

Sizin HTML çok daha basitleştirilmiş olamaz, ancak PHP muhtemelen daha iyi yapılabilir. Genellikle küresel değişkenleri kullanmak kötü bir fikir, ama muhtemelen onun yerine böyle bir şey yapabilirsiniz:

function current_tab($page) {
    global $p;
    // Check if $page is a prefix for $p
    if ($page == substr($p, 0, strlen($page))) {
        return "current";
    }
}

Sonra HTML, bu kullanın:

<?=current_tab('account');?>

Olmayan uyumdur sayfa isimleri için, özel koşullar kullanmanız gerekir gidiyoruz, ya da adlandırma değiştirebilir.

Orada (sürdürülebilirlik ve genişletilebilirlik oldukça düşük) burada artırabilir bir sürü var, ama ben ilki, örneğin ... Bu Koşullamalar birkaç ortadan kaldırarak ile başlamak istiyorum, sen dize 'ile başlar olup olmadığını kontrol edebilirsiniz Kullanıcı 'bütün özel durumlarda ilgilenir,' Kullanıcılar.

if(strpos($p, 'users') !== false) { ... }

Aynı diğer durumlarda bir kaç için de geçerlidir.

Her neyse, ben size sunucu tarafı komut yapısına bir göz atın öneririz. Genellikle, bu tür şeyler için daha genel bir çözüm bulabilirsiniz.

Ben (PHP uzayda çok olmasına rağmen, CI benim seçimim) CodeIgniter gibi bir MVC çerçevesinde bakmak zorunda öneririm.

Eğer biraz farklı kaygılar ve katı çiftleşmiş temiz ayrılmasını kazanıyor bunu yapısı neredeyse tüm mevcut mantık ve sadece yeniden kullanabilirsiniz CI gibi bir çerçeve ile ... zemin-up kendi yazmaya yolda.