Question

我有一个名为＆＃39; kmenu＆＃39;

的功能

function kmenu()
{
    if (isset($_GET['pid'])) {
        $pid = $_GET['pid'];
    } else {
        $pid = 1;
    }


    $menu = '<ul class="nav navbar-nav">
                        <li class="active"><a href="?pid=1">HOME <span class="sr-only">(current)</span></a></li>
                        <li><a href="?pid=2">ABOUT US</a></li>
                        <li class="dropdown"><a href="#" class="dropdown-toggle" data-toggle="dropdown" role="button"
                                                aria-expanded="false">OUR PRODUCTS<span class="caret"></span></a>
                            <ul class="dropdown-menu" role="menu">
                                <li><a href="#">Arabic Sweets</a></li>
                                <li><a href="#">Cakes</a></li>
                                <li><a href="#">Bakery</a></li>
                                <li><a href="#">Chocolate</a></li>
                                <li><a href="#">Confectionery</a></li>
                                <li><a href="#">Ice Cream</a></li>
                                <li><a href="#">Malaga</a></li>
                            </ul>
                        </li>
                        <li><a href="#">OUR BRANCHES</a></li>
                        <li><a href="#">GALLERY</a></li>
                        <li><a href="#">CONTACT US</a></li>
                    </ul>';
    return $menu;
}

此功能内容如上所示的所有菜单项。我需要将class='active'添加到当前页面。我之前曾经这样做过 if($pid==1){echo 'class="active"'}但是当我尝试使用函数存储菜单时，我现在该怎么做。

Answer 1

行。起初你可以这样做：

$menu = 'MENU';
$menu = $menu.'SOMETHINGELSE';
print_r($menu); // or return $menu;

在这种情况下，它将打印/返回＆＃34; MENUSOMETHINGELSE＆＃34;。这是字符串连接。所以你可以做简单的if（正如你之前说过的那样），并写下：

$menu = '';
$menu .= 'BEGINNING OF THE HTML';
if($pid==1) {
    $menu .= 'class="active"';
}
$menu .= 'REST OF HTML';

更多的话：但这不是一切。在变量中存储菜单看起来非常糟糕。也像这样输出HTML。您应该尝试仅在数组中保存重要值，然后在循环中返回/ echo菜单。例如：

$menu = array(
    [0] => array('name' => 'NAME0', 'title' => 'TITLE0', 'url' = '/url0'), 
    [1] => array('name' => 'NAME1', 'title' => 'TITLE1', 'url' = '/url1')
);

然后生成菜单：

foreach ($menu as $m) {
    echo '<a href="'.$m['url'].'" title="'.$m['title'].'">'.$m['name'].'</a>';
}

在这种情况下，视觉设置在foreach循环中，菜单语义在简单数组中。您也可以在该数组中存储活动。

如果需要，您还可以查看ob_start函数，并使用echo，然后返回ob_get_contents（），如果这对您更容易（http://php.net/manual/en/ref.outcontrol.php）。

请记住，在PHP代码的每个片段上都可以转义php。

<?php echo "ASD"; ?>
ASD
<?php echo "ASD"; ?>

此代码将输出三个ASD。您应该尝试使用此技术，并尽量减少应用程序的回显输出。你也可以在循环中使用它：

<?php foreach ($menu as $m) { ?>
    <a href="<?php echo $m['url']; ?>" 
       title="<?php echo $m['title']; ?>">
        <?php echo $m['name']; ?>
    </a>
<?php } ?>

我希望你明白这个主意。如果有更多的常量html（如你的例子中所示），那么输出应该更加清晰。

另一件事是将菜单内容存储在一个函数中。这也不是一个好习惯。您应该尝试切换到对象编程样式，并尝试将此菜单数组保存在某种全局/缓存/另一个对象中。

在您的情况下，如果有人需要在同一站点上创建两个菜单，每次他必须调用此函数，并从头开始声明整个菜单变量。虽然它只能从功能中访问。

如果您不需要创建面向对象的应用程序，或者需要更简单的解决方案，最好将$ menu保存在其他位置（更高级别），并将其作为参数传递给例如menu_generator函数。

在这种情况下，$ menu看起来就像我描述的那样，并且你声明了这样的函数：

function generate_menu_html($menu) {
    // your menu generation function base on menu array
}

希望它有所帮助。

最好的问候。

Answer 2

我认为你不应该在一个函数中存储html代码。通常，您将这些存储在单独的文件中，然后将它们包含在内。例如：

require '/menu.php'

然后你就可以添加到每一行

<li><?php if($pid == 1): ?> class='active'<?php endif; ?><a>...</a></li>

Answer 3

您可以尝试这样的事情

$menu = '<ul class="nav navbar-nav">';
$menu .= '<li';
$menu .= ($pid==1)?' class="active"':'';
$menu .= '><a href="?pid=1">HOME ....REST OF THE CODES';

php添加class =＆＃39; active＆＃39;到菜单

3 个答案: