Why does the code at this JS fiddle not work? I want it to log a blank string, but it's logging a Pointer Event. https://jsfiddle.net/wvom7k2f/
let c = document.getElementById('c');
c.onclick = test;
function test(a='', b=''){
console.log(a);
}
2 Answers
An onclick event handler will put the event as the first parameter. If you want to call test with no parameters you should change c.onclick = test; to the following:
c.onclick = () => test();
7 Comments
Sebastian Simon
@maksymiuk That’s what the default parameters in the question are for.
T.J. Crowder
@maksymiuk - By assigning a function to
onclick that, when called, will call test with no arguments, triggering the default parameter value (which is a blank string).
Josh Cronin
His test function will do that.
maraaaaaaaa
@T.J.Crowder I stand corrected, I somehow misinterpreted the code, and I retract my dislike
T.J. Crowder
@maksymiuk - We've all done that kind of thing. :-) (You can remove the comment, too, if you like.)
|
You're doing c.onclick = test. The onclick event calls its function with the event as the 1st parameter.
Try to do:
function test(e, a='', b='') {
console.log(a);
}
c.onclick = test;
Or, if you don't want to change test(), then you can do:
function test(a='', b='') {
console.log(a);
}
c.onclick = function(e) {
test();
};
3 Comments
user568551
Makes sense. So what would be the proper way to do it if I want to also call that function elsewhere, such as test('foo', 'bar')?
gen_Eric
@user568551 I added a 2nd block of code. You could do
c.onclick = function() { test(); }.gen_Eric
Or check Josh Cronin's answer.
Eventobject itself