Простая «потеря $ this» проблемы объема в JS/JQ .. объясните?

Я пытаюсь улучшить свои навыки JS, и я переписал кучу кода как литерал с именами объектов (я думаю.) Но теперь у меня есть «эта» -связанная проблема. Помогите мне понять?

Вот сокращенный код.

var MYAPP = {

    init: function(){
        $(document).on("click",".showLove", MYAPP.showLove);
    },
    showLove: function(){
         var thisId = MYAPP.findId();
         $.post(//// do AJAXy stuff using thisId);
    },
    findId: function(){
        var thisClass = $(this).attr('class');
        var thisIdPos = thisClass.indexOf("id-")+3;
        var thisId = thisClass.substr(thisIdPos, 3);
        return thisId;
    }
}

Поэтому я уверен, что вы, вероятно, видите проблему. В функции findId $ это не определено, и я получаю сообщение об ошибке. Раньше у меня была логика findId в showLove, и все сработало. Я переместил логику findId на собственный метод, потому что он использовался в нескольких разных местах.

So let me just ask this -> Why does $(this) point to the correct element in 'showLove' .. but not in 'findId'? Since 'findId' is called from inside 'showLove', shouldn't it have access to the same variables, including $(this)? Is this my first "self = this" situation?

Я знаю, что это основной вопрос, но если кто-то может помочь мне понять, я бы ... э-э ... «showLove».

0
nl ja de
Спасибо за ссылку.
добавлено автор K.K. Smith, источник
Если у вас есть время, которое может быть полезно. Я просто использовал этот подход как способ обмена именами и организации моего кода. Я просто хотел собрать свой «код спагетти» и сгруппировать его логически. Мне было разумно называть MYCODE.methodGroupingNumber3.specificMethod. Но теперь я понимаю, что я вхожу в дополнительный уровень сложности: OOP. Так что все, что вы хотели бы показать, было бы оценено.
добавлено автор K.K. Smith, источник
Взгляните на эту дискуссию как на свое улучшение, так и на некоторые альтернативы вашему использованию: ejohn.org/ блог/простой класс конкретизации
добавлено автор Mark Schultheiss, источник
ЕСЛИ это поможет, я мог бы опубликовать альтернативу «переработанных», используя ваш пример и эту функцию ... дайте мне знать, если вы найдете это в использовании, если нет, неважно. :)
добавлено автор Mark Schultheiss, источник
проверьте эту скрипку: jsfiddle.net/F7xVn/1 - нажмите «Я хочу», чтобы увидеть это в действии ...
добавлено автор Mark Schultheiss, источник

4 ответы

this in showLove comes from its instantiation in $(document).on("click",".showLove", MYAPP.showLove);

var thisId = MYAPP.findId(this);//pass it to the function

findId: function(thisPassed){ and reference it by that name

Один из способов сделать это, по крайней мере.

1
добавлено

Вы передаете MYAPP.showLove в jQuery и говорите: «Прикрепите эту функцию как обработчик кликов к документу».

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

Поэтому, когда showLove запускается как обработчик кликов, этот больше не является MYAPP . Обычно, когда вы отделяете функцию от объекта , этот становится либо window , либо undefined (ES5). Однако jQuery решает этот должен быть элемент .showLove , который был нажат, используя call() или apply() код>.

Когда вы вызываете MYAPP.findId() в showLove , однако этот по-прежнему установлен в MYAPP ; так как к этому прикреплен findId (вы вызываете метод на MYAPP ).

Чтобы исправить это (каламбур не предназначен), вы можете либо передать значение this в findId (предпочтительно, IMO), либо использовать call() / применить() .

var MYAPP = {

    init: function(){
        $(document).on("click",".showLove", MYAPP.showLove);
    },
    showLove: function(){
         var thisId = MYAPP.findId(this);
         $.post(//// do AJAXy stuff using thisId);
    },
    findId: function(which){
        var thisClass = $(which).attr('class');
        var thisIdPos = thisClass.indexOf("id-")+3;
        var thisId = thisClass.substr(thisIdPos, 3);
        return thisId;
    }
}

или:

var MYAPP = {

    init: function(){
        $(document).on("click",".showLove", MYAPP.showLove);
    },
    showLove: function(){
         var thisId = MYAPP.findId().call(this);
         $.post(//// do AJAXy stuff using thisId);
    },
    findId: function(){
        var thisClass = $(this).attr('class');
        var thisIdPos = thisClass.indexOf("id-")+3;
        var thisId = thisClass.substr(thisIdPos, 3);
        return thisId;
    }
}
1
добавлено
Лучшее объяснение. Муй полезен. Спасибо, что помогли мне понять это. Я буду отмечать вас как решение.
добавлено автор K.K. Smith, источник

Если вы хотите сохранить это, вам нужно сделать

MYAPP.findId.call(this);
1
добавлено

Я думаю, проблема связана с тем, как вы вызываете функцию findId . Всякий раз, когда вы это делаете:

MYAPP.findId()

Вы статически ссылаетесь на функцию, которая будет выполняться без контекста объекта. Если вы хотите сохранить этот контекст, вам нужно вызвать его так:

MYAPP.findId.call(this)

Делая, что ваша переменная this внутри findId будет привязана к любому объекту, который вы передаете как параметр функции вызова (в этом случае текущая эта , следовательно, вы сохраняете эта ссылка)

На стороне: я бы порекомендовал вам взглянуть на функцию привязки underscore.js, это действительно полезный помощник!

1
добавлено
JavaScript Jobs — чат
JavaScript Jobs — чат
8 336 участник(ов)

JavaScript Jobs — чат для поиска работы и людей Правила оформления: https://teletype.in/@telegram-ru/r1WQe5F1m См. также: @mobile_jobs, @devops_jobs, @nodejs_jobs, @react_js, @angular_ru, @js_ru

JavaScript.ru
JavaScript.ru
7 932 участник(ов)

Сообщество сайта JavaScript.ru в Slack.

pro.js
pro.js
4 675 участник(ов)

Про JavaScript и NodeJS Invite: https://t.me/joinchat/Be4rsT5Rsgq30DHutjxXgA Правила: http://telegra.ph/ru-chat-rules-06-19 Вакансии только с ЗП, не чаще раза в неделю.

Верстка сайтов HTML/CSS/JS/PHP
Верстка сайтов HTML/CSS/JS/PHP
3 439 участник(ов)

Правила группы: напишите !rules в чате. Группа Вк: vk.com/web_structure Freelancer: @web_fl Веб Дизайн: @dev_design Маркетолог: @topmarkening Автор: @M_Boroda

JavaScript — русскоговорящее сообщество
JavaScript — русскоговорящее сообщество
3 269 участник(ов)

Рекомендуем сразу отключить уведомления Правила: https://rudevs.network/ByaMH6un7 См. также: @js_noobs_ru, @nodejs_ru, @typescript_ru, @react_js, @electron_ru Вакансии и поиск работы: @javascript_jobs

JavaScript Noobs — сообщество новичков
JavaScript Noobs — сообщество новичков
2 484 участник(ов)

Чат для новичков

javascript_ru
javascript_ru
915 участник(ов)

Сообщество любителей самого популярного языка программирования в мире. Чат основан в 2009 году. Логи: https://goo.gl/9EOeM7 Поддержка бота: @chat_linker (ссылка на репу внутри) Вам будут интересны @frontend_ru и @css_ru

jsChat
jsChat
603 участник(ов)

Чат посвященный программированию на языке javaScript Перед отправкой ссылки на Ваш контент посоветуйтесь с админом Все ссылки удаляются ботом автоматически

JavaScript for Zombies Chat
JavaScript for Zombies Chat
492 участник(ов)

Чат про JavaScript для настоящих zombie! Вход строго по приглашениям! Ссылка для строгих приглашений: https://t.me/joinchat/AAMBHz3Uyr0tuZ7VaB029g

All That JS
All That JS
417 участник(ов)

JS на русском