使用标记href匹配url文件夹以使其处于活动状态

时间:2011-09-27 05:22:01

标签: javascript jquery refactoring

我在某个网址上为我的菜单设置了一个活动状态。我有这样的网址:

  • /products/other-clothing/sporting/adults-anzac-australia-polo
  • /products/other-clothing/sporting/adults-nz-tee
  • /products/bags/backpacks

我的代码从/之后获取文件夹,以便其他服装,运动等。 它工作正常,我只是假设有一种更有效的方法来编写代码。

这是我的代码:

jQuery(".product-nav li a").each(function() {
  // URL url
  var cat = location.pathname.split("/")[2];
  var subcat = location.pathname.split("/")[3];
  var c = "/products/" + cat + "/" + subcat;

  // A tag url
  var acat = this.href.split("/")[4];
  var asubcat = this.href.split("/")[5];
  var e = "/products/" + acat + "/" + asubcat;

  if(e == c) {
    jQuery(this).parent().addClass("active");
    jQuery(this).parent().parent().parent().addClass("active");
  }
});

如果有人能够提供一种更清晰的方式来编写非常棒的代码。我可能不需要"/products/" +

3 个答案:

答案 0 :(得分:3)

请注意以下表达式的输出:

$('<a href="/questions/7564539/match-url-folders-with-a-tag-href-to-make-a-active-state"></a>')[0].href;
/*
 * http://stackoverflow.com/questions/7564539/match-url-folders-with-a-tag-href-to-make-a-active-state
 */
$('<a href="/questions/7564539/match-url-folders-with-a-tag-href-to-make-a-active-state"></a>').eq(0).attr('href');
/*
 * /questions/7564539/match-url-folders-with-a-tag-href-to-make-a-active-state
 */

因此,如果您的<a>代码包含以/开头的网址,那么您可以将.attr('href')location.pathname进行比较。要进行测试,请尝试从此页面在控制台中运行:

$('a').each(function () {
    if ($(this).attr('href') == location.pathname) {
        $(this).css({
            'font-size': '40px',
            'background-color': 'lime'
        });
    }
});

答案 1 :(得分:1)

以下简要介绍一下:

jQuery(".product-nav li a").each(function() {
  // URL url
  var c = location.pathname.split('/').slice(2, 4)
  // A tag url
    , e = this.href.split('/').slice(4, 6)
  ;

  if(e[0] == c[0] && e[1] == c[1]) {
    jQuery(this).parentsUntil(
      'div:not(.subnav)', // go up the tree until the 1st div that isn't .subnav
      '.product-nav li, .subnav' // and only match these parents
    ).addClass('active');
  }
});

.parent().parent().parent()...有一个非常糟糕的代码味道,但如果不查看你的标记就无法改进。您可能应该使用.closest()代替。

答案 2 :(得分:0)

有趣的问题。我试图清理它:

jQuery(function ($) {
  function Category(outer, inner) {
    this.outer = outer
    this.inner = inner
  }

  Category.fromURL = function (url) {
    var parts = url.replace(/^(https?:\/\/.*?)?\//, "").split("/")

    return new Category(parts[1], parts[2])
  }

  Category.prototype.equals = function (other) {
    return this.outer === other.outer
      && this.inner === other.inner
  }

  var category = Subcategory.fromURL(location.href)

  $(".product-nav a").each(function () {
    if (Category.fromURL(this.href).equals(category)) {
      $(this).closest("li.inner").addClass("active")
      $(this).closest("li.outer").addClass("active")
    }
  })
})