Объявление переменной, как вы сказали, является, вероятно, вашим лучшим выбором.
$(this-all)
не имеет смысла во многих ситуациях, потому что событие щелчка не знает конкретного селектора, который вы использовали при привязке этого события, оно просто знает элемент и функцию для выполнения.
Обратите внимание, что есть более аккуратный способ делать то, что вы делаете. Вместо того, чтобы строить селектор по частям, вы можете использовать метод .find()
, чтобы найти предков выбранного элемента:
var $modal = $(modal); // assuming `modal` is a string
var $elems = $modal.find('.modal-close, a.cancel_link, .okBtn'); // hyphens, underscores, and camelCase -- oh my!
$elems.click(...);