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

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

Вот мой код:

<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, если она существует. Как я могу этого добиться?


object jquery attributes
person chaoskreator    schedule 06.02.2012    source источник

Ответы (2)


arrow_upward
8
arrow_downward

Хотя Диодей прав в том, что вам нужно обернуть 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 и обеспечивая доступ к объект события в качестве аргумента.

person eyelidlessness    schedule 06.02.2012
comment
Потрясающий. Работает как шарм. Я понимаю, что call() было бы безопаснее использовать, чем eval(), верно? - person chaoskreator; 06.02.2012

arrow_upward
103
arrow_downward

Используйте: $(this).attr вместо this.attr

Это вынуждает его входить в контекст jQuery.

person Diodeus - James MacFarlane    schedule 06.02.2012
comment
Я бы подумал, что это тоже сработает, однако я получаю $(this).attr("onclick").val is not a function, используя его. - person chaoskreator; 06.02.2012
comment
В этом случае вам не нужен .val(). - person Diodeus - James MacFarlane; 06.02.2012