Javascript из OnClick в функцию

Question

Я имею яваскрипт кода для ссылки клика:

document.getElementById('giddy').onclick = function { 
     alert(this.href); 
    }; 

, и я хочу, чтобы отделить функцию часть этого ... У меня есть

document.getElementById('giddy').onclick = poro(this); 
    function poro(yyyy) { 
     alert(yyyy.href); 
    }; 

Но не работает (говорит, что не указано в предупреждении) ... что я делаю неправильно?

Answer 1

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

document.getElementById('giddy').onclick = poro; 
function poro() { 
    alert(this.href); 
}; 

Answer 2

Избавьтесь от (this) и используйте функцию this в функции вместо yyyy.

document.getElementById('giddy').onclick = poro; 
function poro() { 
    alert(this.href); 
}; 

Answer 3

Вы сразу же вызываете функцию poro. По сути, вы сообщаете Javascript, что значение элемента onclick будет равно результату вызова функции poro(this [window]).

Чтобы обойти эту проблему, вы можете обернуть функцию poro(this) внутри пустой функции, например, так:

document.getElementById('giddy').onclick = function(){poro(this)} function poro(yyyy) { alert(yyyy.href); }; 

Вы можете также рассмотреть возможность использования eventListener, так как это позволяет пространство для расширения.

Answer 4

Практически там! Вы должны сделать:

document.getElementById('giddy').onclick = function(){ poro(this); } 
function poro(yyyy) { 
    alert(yyyy.href); 
}; 

Примечания poro(this); завернутых в анонимной функции.

Answer 5

Я бы рекомендовал использовать addEventListener вместо метода onclick.

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

var giddy = document.getElementById('giddy'); 
giddy.addEventListener('click', function(e) { poro(this); }, false); 
function poro(yyyy) { 
    alert(yyyy.href); 
} 

Answer 6

, так как вы используете использование Jquery:

$('#giddy').click(function(){ poro($(this));}); 

или вы можете Используйте bind() функцию

$("#giddy").bind("click", $(this), poro);