a

Question

I'm trying to learn the best way to code JQuery, since this should affect browser loading time, and in programming speed is everything.
So what I'd like to ask which is a better code in terms of speed, re-usabilty, etc. - And how can I further improve on my code.
In this video the code given was
$("#imgGallery a").click(function(event){
     event.preventDefault();
     var href = $(this).attr("href");
     console.log(herf);
});
wouldn't it be better if we just create a function and call it like this, making the code more readable, easier to maintain, reusable?
var clicked = function(event){
    event.preventDefault();
    var href = $(this).attr("href");
    console.log(href);
}
$("#imageGallery a").click(clicked);

Steven Parker · Accepted Answer

That's a good idea if you need to reuse the function.
But it is very common for event handlers to be unique, and defining them as anonymous functions where the handler is established is generally considered adequately readable.  So there's not much advantage to defining the function separately if it won't be used elsewhere.

Welcome to the Treehouse Community

Looking to learn something new?

Sean Urgel

Sean Urgel

a

Alexander La Bianca

Alexander La Bianca

1 Answer

Steven Parker

Steven Parker

That's a good idea if you need to reuse the function.