Home/ Questions/Q 8163835
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-06-06T19:09:08+00:00

Seeking to create an event handler that registers a callback and iterates based on

  • 0

Seeking to create an event handler that registers a callback and iterates based on the event.

An early, working example is something like this:

var elt = document.getElementById("square"),
      clickCount = 0;

  // call a priviledged method with access to prviate vars from 
  // outer scope
  var count = function () {
    clickCount += 1;
    alert("you clicked me " + clickCount + " times!");
  };

  elt.addEventListener("click", count, false);

Now I want to actually write something js I could use. Here is the current construction:

//Create a class object
function CountMe () {

 //privately scoped variables
  var clickCount = 0;

 //methods on 'class' CountMe
  this.countClick = function () {
    //who's context am I in?
    this.addCount();
    alert("you clicked me " + clickCount + " times!");
  };

  this.addCount = function() {
    clickCount += 1 
  };
};

// Create an instance of countMe class
var clickMe = new CountMe();

//Add an event listener for clicks 
document.getElementById("square").addEventListener("click", clickMe.countClick ,false)

The error I receive is Uncaught TypeError: Object #<CountMe> has no method 'addEventListener'
Given an html page like this:

<html>
<body>
 <div style="width:50px; height:50px; background-color:red;" id="square">
 </div>
</body>
</html>

My questions are:

  • How should this be constructed so onclick events function as they do in the first method?
  • In the method countClick what context is the nested this in? The instance of the CountMe class I suspect, just want someone else’s take.
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    How should this be constructed so onclick events function as they do in the first method?

    Uncaught TypeError: Object # has no method ‘addEventListener’

    That’s obcously the reason. You need to add the event listenere to a DOM element, so you could do for example:

    var clickMe = new CountMe();
document.getElementById("square").addEventListener("click", clickMe.countClick, false);

    The variable elt is, as correctly commented, privately scoped and cannot be accessed from outside. That means you just should move the add-process into the constructor (if the element is determinable), or drop it at all.

    In the method countClick what context is the nested this in? The instance of the CountMe class I suspect, just want someone else’s take.

    That depends on the calling. With your code

    clickMe.countClick();

    the instance clickMe would be the context, but when the function is called as an event listener the DOM element will be the context. See MDN’s overview for the this keyword. You can use a closure-scoped variable to hold a reference to the actual instance, as for example described here (there are many questions about that).

    Three possible quick-fixes:

    • domel.addEventListener("click", clickMe.countClick.bind(clickMe), false)
    • domel.addEventListner("click", function(){clickMe.countClick();}, false)
    • var that=this; this.countClick=function(){ that.addCount(); alert(); };
    • 0