jQuery: this.attr () не функция?

Question

Я не совсем уверен, что я не использую это в правильной области или как, но у меня есть скрипт, который в основном фиксирует щелчок по ссылке и заставляет страницу исчезать перед переходом на связанную страницу.Однако, если ссылка представляет собой JavaScript onclick , сценарий завершится ошибкой.

Вот мой код:

<script type="text/javascript">

    pageObj = {
        init: function(){
            $("body").fadeTo("slow", 1);
        },
        redirectPage: function(redirect){
            window.location = redirect;
        },
        linkLoad: function(location){
            $("body").fadeOut(1000, this.redirectPage(location));
        }
    };

    $(document).ready(function() {
        pageObj.init();

        $("a").click(function(e){
            e.preventDefault();
            if (this.attr('onclick') !== undefined) {
                eval(this.attr('onclick').val());
            } else {
                var location = this.href;
                pageObj.linkLoad(location);
            }
        });
    });

</script>

Как видите, япопытаться проверить, имеет ли ссылка атрибут onclick , а затем вызвать функцию onclick , если она существует.Как мне этого добиться?

Answer 1

Использование: $(this).attr вместо this.attr

Это приводит к контексту jQuery.

Answer 2

Хотя Diodeus и прав, что вам нужно обернуть this в коллекцию jQuery перед использованием attr() (это метод коллекции jQuery, а не HTMLElement), вы также можете пропустить attr() .

$("a").click(function(e){
    var location;
    e.preventDefault();
    if ($.isFunction(this.onclick)) {
        this.onclick.call(this, e);
    } else {
        location = this.href;
        pageObj.linkLoad(location);
    }
});

Обратите внимание, что я использовал это свойство (когда документ HTML загружается, атрибуты обычно предварительно загружаются в свойства, при этом атрибуты on_______ предварительно загружаются в качестве методов. Также обратите внимание, что я использовал this.onclick.call() вместо eval(), устанавливая правильный this для onclick методов и обеспечения доступа к объекту события в качестве аргумента.