Question

我写了一个具有以下结构的下拉菜单。它应该在点击时下拉，并在点击时关闭。

这是HTML

<ul id="nav" class="nav">
    <li>
        <a id="menu1" class="menu">MENU 1</a> <!-- top menu -->
        <div id="submenu1" class="submenu"> <!-- hidden by default -->
            <a href="#">SUBMENU ITEM 1</a> <!-- submenu item -->
            <a href="#">SUBMENU ITEM 2</a>
        </div>
    </li>

    <li>
        <a id="menu2" class="menu">MENU 2</a>
        <div id="submenu2" class="submenu">
            <a href="#">SUBMENU ITEM 1</a>
            <a href="#">SUBMENU ITEM 2</a>
            <a href="#">SUBMENU ITEM 2</a>
        </div>
    </li>
</ul>

这就是JavaScript（使用jQuery）

$("#menu1").click(function() {
    $("div.submenu").hide(); // hide all menus
    $("#submenu1").toggle(); // open this menu
});
$("#menu2").click(function() {
    $("div.submenu").hide(); // hide all menus
    $("#submenu2").toggle(); // open this menu
});
$("#menu3").click(function() {
    $("div.submenu").hide(); // hide all menus
    $("#submenu3").toggle(); // open this menu
});
$("#menu4").click(function() {
    $("div.submenu").hide(); // hide all menus
    $("#submenu4").toggle(); // open this menu
});

$(document).bind('click', function(e) {
    var $clicked = $(e.target);
    if (! $clicked.parents().hasClass("nav"))
    $("div.submenu").hide();
});

JS的第一部分有很多重复，有没有办法让它更短，更好，更好？

Answer 1

您应该能够将脚本缩小为：

$(".nav .menu").click(function() {
   $("div.submenu").hide(); // hide all menus
   $(this).next().toggle(); // open this menu
});
$(document).click(function(e) {
    if (! $(e.target).parents().hasClass("nav"))
    $("div.submenu").hide();
});

Answer 2

是：

var $submenus = $('.submenu');

$(".menu").click(function(e){
    var $menu = $(this).next('.submenu').toggle();
    $submenus.not('#' + $menu[0].id).hide();
    e.preventDefault();
});

$(document).click(function(e){
    if( !$(e.target).closest('.nav').length ) $submenus.hide();
});

有没有办法简化这个jquery？

2 个答案: