我有一个叫做'kmenu'的函数

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"'} 但是当我尝试使用函数来存储菜单时，我现在该怎么做。

最佳答案

好的。一开始你可以这样做:

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

在这种情况下，它将打印/返回“MENUSOMETHINGELSE”。这是字符串连接。所以你可以做简单的if(就像你之前说的那样)，然后写:

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

再多说几句: 但这还不是全部。将菜单存储在变量中看起来非常糟糕。也像那样输出 HTML。您应该尝试只在数组中保存重要的值，然后在循环中返回/回显菜单。例如:

$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 循环中，菜单语义在简单数组中。您也可以将 active 存储在该数组中。

如果您愿意，您还可以查看 ob_start 函数，并使用 echo，然后返回 ob_get_contents()，如果这对您来说更容易 (http://php.net/manual/en/ref.outcontrol.php)。

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

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

这段代码会输出三个 ASD。您应该尝试使用此技术，并尽量减少您的应用程序的 echo 输出。您也可以在循环中使用它:

<?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
}

希望对您有所帮助。

最好的问候。

关于php 添加类 ='active' 到菜单，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/28009414/