jQuery展开和折叠列表

Question

该代码扩展和折叠一个列表，其中列表项可以具有子列表。 任何重构此代码的想法-尤其是切换部分。 这里有必要使用闭包吗？

$(function()
{
    $('li:has(ul)')
    .click(function(event){
        if (this == event.target) 
        {
            var that = this;
            $('li:has(ul)').children().filter(':not(:hidden)').parent().each(function(x){
                if(this != that)
                    toggleList(this);
            });
            toggleList(this);
        }
    })
    .css({cursor:'pointer', 'list-style-image':'url(plus.gif)'})
    .children().hide();

    $('li:not(:has(ul))').css({cursor: 'default', 'list-style-image':'none'});
}); 
function toggleList(L)
{
    $(L).css('list-style-image', (!$(L).children().is(':hidden')) ? 'url(plus.gif)' : 'url(minus.gif)');
    $(L).children().toggle('fast');
}

编辑：

该脚本适用于以下HTML代码段（来源：jQuery in Action）。 实际上，我正在尝试扩展书中给出的脚本。

      <ul>
        <li>Item 1</li>
        <li>Item 2</li>
        <li>
          Item 3
          <ul>
            <li>Item 3.1</li>
            <li>
              Item 3.2
              <ul>
                <li>Item 3.2.1</li>
                <li>Item 3.2.2</li>
                <li>Item 3.2.3</li>
              </ul>
            </li>
            <li>Item 3.3</li>
          </ul>
        </li>
        <li>
          Item 4
          <ul>
            <li>Item 4.1</li>
            <li>
              Item 4.2
              <ul>
                <li>Item 4.2.1</li>
                <li>Item 4.2.2</li>
              </ul>
            </li>
          </ul>
        </li>
        <li>Item 5</li>
      </ul>

Answer 1

您的代码不适用于Safari。 当我单击子列表时，将切换顶部列表。

怎么样：

$(document).ready(function() {
  $('li:has(ul)').click(function(event) {
    $(this).css('list-style-image', $(this).children().is(':hidden') ? 'url(minus.gif)' : 'url(plus.gif)')
    $(this).children().toggle('fast')
    return false
  })
  .css({cursor:'pointer', 'list-style-image':'url(plus.gif)'})
  .children().hide()

  $('li:not(:has(ul))').click(function(event) { return false })
  .css({cursor:'default', 'list-style-image':'none'})
})