2

私の Web サイトでは、スイッチ ケースを使用して、ナビゲーション バーのアクティブなリンクに色を付けています。

$case最初に、ナビゲーション バーで可能なすべてのリンクを含む配列を宣言しました。次に、それがいずれかのページでクリックされた関数であるかどうか (クエリ文字列が空でない場合)と、それが$case配列に配置されているかどうか (他のページ内の他のリンクがクリックされたときに色が変化しないようにするため) を確認します。

true の場合、次の$currentような値を取得するdocs_zamoかakuut_wgakuut --> FILENAME_QUERY.、以前に宣言したときにのみ色を変更したい ($case 配列で) そうでない場合は、ファイル名だけになります (「.php」なし)

このコードはかなり扱いにくいと思います。仕事をする別の方法はありますか？私はPHPにかなり慣れていません。私は思った。スイッチ内にケースが存在するかどうかをチェックする関数がある場合、$case 配列は必要ありません。これは、スイッチ ケース内にリンクが存在することを既に宣言しているからです。

FILENAME_QUERY リンクかどうかを確認するコード:

$case = array("index","medica","praesidium","akuut","sponsors","docs","docs_zamo","kalender","evenementen","fotos","werking","jaarraden","akuut_wgakuut","onderwijs","sociaal","sport","werkgroepen","eerstejaars","international",);

if (($_SERVER['QUERY_STRING'] != "") && (in_array(basename($_SERVER['PHP_SELF'],'.php') ."_". str_replace('actie=','',$_SERVER['QUERY_STRING']),$case) == true)) {
    $current = basename($_SERVER['PHP_SELF'],'.php') ."_". str_replace('actie=','',$_SERVER['QUERY_STRING']);
}
else {$current = basename($_SERVER['PHP_SELF'],'.php');}

スイッチ:

$id = array("","","","","","","","","","","","","","","","","","","");
switch($current)
{
    case $case[0]:$id[0]="class = 'here'";break;
    case $case[1]:$id[1]="class = 'here'";break;
    case $case[2]:$id[2]="class = 'here'";break;
    case $case[3]:$id[3]="class = 'here'";break;
    case $case[4]:$id[4]="class = 'here'";break;
    case $case[5]:$id[5]="class = 'here'";break;
    case $case[6]:$id[18]="class = 'here'";break;
    case $case[7]:$id[6]="class = 'here'";break;
    case $case[8]:$id[7]="class = 'here'";break;
    case $case[9]:$id[8]="class = 'here'";break;
    case $case[10]:$id[9]="class = 'here'";break;
    case $case[11]:$id[10]="class = 'here'";break;
    case $case[12]:$id[11]="class = 'here'";break;
    case $case[13]:$id[12]="class = 'here'";break;
    case $case[14]:$id[13]="class = 'here'";break;
    case $case[15]:$id[14]="class = 'here'";break;
    case $case[16]:$id[15]="class = 'here'";break;
    case $case[17]:$id[16]="class = 'here'";break;
    case $case[18]:$id[17]="class = 'here'";break;
}
?>

HTML:

<div id='menu'>
        <ul>
            <li><a href='index.php' <? echo $id[0]; ?>>home</a></li>
            <li><a href='medica.php' <? echo $id[1]; ?>>medica</a>
                <ul>
                    <li><a href='praesidium.php' <? echo $id[2]; ?>>&raquo; praesidium</a></li>
                    <li><a href='akuut.php' <? echo $id[3]; ?>>&raquo; akuut</a></li>
                    <li><a href='sponsors.php' <? echo $id[4]; ?>>&raquo; sponsors</a></li>
                </ul>
            </li>
            <li><a href='docs.php' <? echo $id[5]; ?>>doc's bar</a>
                <ul>
                    <li><a href='docs.php' <? echo $id[5]; ?>>&raquo; doc's</a></li>
                    <li><a href='docs.php?actie=zamo' <? echo $id[18]; ?>>&raquo; zamo</a></li>
                </ul>
            </li>
            <li><a href='kalender.php' <? echo $id[6]; ?>>kalender</a>
                <ul>
                    <li><a href='evenementen.php' <? echo $id[7]; ?>>&raquo; evenementen</a></li>
                </ul>
            </li>
            <li><a href='fotos.php' <? echo $id[8]; ?>>foto's</a></li>
            <li><a href='werking.php' <? echo $id[9]; ?>>Werking</a>
                <ul>
                    <li><a href='jaarraden.php' <? echo $id[10]; ?>>&raquo; jaarraden</a></li>
                    <li><a href='akuut.php?actie=wgakuut' <? echo $id[11]; ?>>&raquo; akuut</a></li>
                    <li><a href='onderwijs.php' <? echo $id[12]; ?>>&raquo; onderwijs</a></li>
                    <li><a href='sociaal.php' <? echo $id[13]; ?>>&raquo; sociaal</a></li>
                    <li><a href='sport.php' <? echo $id[14]; ?>>&raquo; sport</a></li>
                    <li><a href='werkgroepen.php' <? echo $id[15]; ?>>&raquo; werkgroepen</a></li>
                </ul>
            </li>
            <li><a href='eerstejaars.php' <? echo $id[16]; ?>>eerstejaars</a></li>
            <li><a href='https://www.medica-csd.be' target='_blank'>cursusdienst</a></li>
            <li style='border:none;'><a href='http://www.medica.be/w' target='_blank'>wikimedica</a></li>
            <li><a href='international.php' <? echo $id[17]; ?>>international <img src='site/images/gb.png'></a></li>
        </ul>
    </div>
4

1 に答える 1

0

このブロック全体...

switch($current)
{
    case $case[0]:$id[0]="class = 'here'";break;
    case $case[1]:$id[1]="class = 'here'";break;
    case $case[2]:$id[2]="class = 'here'";break;
    case $case[3]:$id[3]="class = 'here'";break;
    case $case[4]:$id[4]="class = 'here'";break;
    case $case[5]:$id[5]="class = 'here'";break;
    case $case[6]:$id[18]="class = 'here'";break;
    case $case[7]:$id[6]="class = 'here'";break;
    case $case[8]:$id[7]="class = 'here'";break;
    case $case[9]:$id[8]="class = 'here'";break;
    case $case[10]:$id[9]="class = 'here'";break;
    case $case[11]:$id[10]="class = 'here'";break;
    case $case[12]:$id[11]="class = 'here'";break;
    case $case[13]:$id[12]="class = 'here'";break;
    case $case[14]:$id[13]="class = 'here'";break;
    case $case[15]:$id[14]="class = 'here'";break;
    case $case[16]:$id[15]="class = 'here'";break;
    case $case[17]:$id[16]="class = 'here'";break;
    case $case[18]:$id[17]="class = 'here'";break;
}

ループに置き換えることができます：

for($i=0;$i<19;$i++) {
    if($case[$i] == $current) {
        $id[$i] = "class = 'here'";
        break;
    }
}

一般に、複数の行に対してほぼ同じことを書いていることに気付いた場合は、おそらく何らかの方法でループに置き換えることができます (ほぼ確実にそうすべきです)。

于 2013-07-25T14:44:44.080 に答える