как решить эту ошибку JQuery?

Question

Я сделал некоторые Jquery, которые вы можете увидеть с этой скрипкой ниже: http://madaxedesign.co.uk/dev/Test/ http://jsfiddle.net/x82mU/1/

Код:

$(document).ready(function() { 
var $root = $('html, body '); 
$('.scroll a').click(function(e) { 
    var href = $.attr(this, 'href'); 
    $root.animate({ 
     scrollTop: $(href).offset().top 
    }, 500, function() { 
     window.location.hash = href; 
    }); 
    return false; 
}); 

// Responsive menu 
$(function() { 
    var pull  = $('#pull'), 
     menu  = $('nav ul'), 
     menuHeight = menu.height() 

    $(pull).on('click', function(e) { 
     e.preventDefault(); 
     menu.slideToggle(); 
    }); 

    $(window).resize(function(){ 
     var w = $(window).width(); 
     if(w > 320 && menu.is(':hidden')) { 
      menu.removeAttr('style'); 
     } 
    }); 
}); 
}); 

Но она тянет через эту ошибку: Uncaught TypeError: Не удается прочитать свойство «топ 'undefined

Это препятствует работе моей следующей части JQuery.

Мне было интересно, может ли кто-нибудь сообщить мне, почему, или дать мне решение?

Большое спасибо

Answer 1

Вы пытаетесь получить селектор из href который не является там во многих пунктах меню.

т.е:

<li><a href="#">Home</a></li> 
    <li><a href="#aboutUs">About us</a></li> 
    <li><a href="#">Portfolio</a></li>   
    <li><a href="#">Contact us</a></li> 

и

$(href).offset().top //here offset() of an empty jquery object is undefined. 

Не вопрос, но вы можете просто сделать this.href вместо $.attr(this, 'href')

Попробуйте это:

$('.scroll a').click(function(e) { 
    var href = $(this).attr("href"), $el = $(href), top=0; //default top to 0 
     if($el.length) //if there is element matching the href 
      top = $el.offset().top; //set the top 
    $root.animate({ 
     scrollTop: top //now scroll 
    }, 500, function() { 
     window.location.hash = href; 
    }); 
    return false; 
}); 

Fiddle

Answer 2

var href = $(this).attr('href'); 

обновление как комментарий

для

scrollTop: $(href).offset().top 

на работу,

href 

вар iable должен быть элементом на странице.

так что если ваша ссылка как

<a href="#an_id_to_a_div">...</a> 

это будет хорошо.

JQuery создание объекта дома

$(dom_element) 

цель HTML-теги, классы, идентификаторы или существующие объекты Dom (окна, документ ..)