Javascript onclick event not working with variable reference

Question

I am reading some code and trying to replicate the same at my end step by step.

I am attaching an event to a particular button and onclick it should log a statement.

Following is the code that does not work :-

(function(){

var el = function(element){
    if(element.charAt['0'] === '#'){
        return document.querySelector(element);
    }
    return document.querySelectorAll(element);
}

var viewer = el('#viewer');
var clear = el('#clear');

console.log(clear);

 var clearAll = function(){
    console.log('Clearing');
 };

//click event
clear.onclick = clearAll;

})();

Above a function is used to get elements.

The below code works

document.getElementById('clear').onclick = clearAll;

or

document.querySelector('#clear').onclick = clearAll;

I do not understand why the above code does not work. Please Help.

A function which sometimes returns an element and sometimes returns a node list is likely to be confusing. I don't recommend doing that. — Quentin
– Quentin, Commented Mar 7, 2017 at 20:46

Quentin · Accepted Answer · 2017-03-07 20:45:04Z

1

"foo".charAt['0'] is undefined because charAt is a function and doesn't have a 0 property.

You need () (to call the function), not [] (to access properties of the object).

answered Mar 7, 2017 at 20:45

Quentin

949k136 gold badges1.3k silver badges1.4k bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Stacy J Over a year ago

Ohhhhh...yes that worked...stupid mistake...Thanks so much

cnexans · Accepted Answer · 2017-03-07 20:48:55Z

0

You are using charAt as an array while it is a function.

var el = function(element){
    if(element.charAt('0') === '#'){
        return document.querySelector(element);
    }
    return document.querySelectorAll(element);
}

See it working https://jsfiddle.net/

Regards,

answered Mar 7, 2017 at 20:48

cnexans

9841 gold badge7 silver badges21 bronze badges

Collectives™ on Stack Overflow

Javascript onclick event not working with variable reference

2 Answers 2

1 Comment

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related