Wrapping Events in Anonymous Functions - Javascript

Question

I've been reading up on anonymous javascript functions and have a general question about using them with event listeners via jquery.

I have the following event listener that will do some stuff upon the submission of a form.

if(sky === 'blue') {
    $('#my-form').on('submit', function(){
        //do some stuff
        $('#my-div').show();
    })
}

Would it make any difference (i.e., pros & cons, things that could go wrong, etc.) if I were to declare it as-is in my javascript file or should I be wrapping it in an anonymous function like so

 $(function() {
    if(sky === 'blue') {
    //do my stuff
 }

Any input is appreciated, thanks!

Similar topic http://stackoverflow.com/questions/10273185/what-are-the-benefits-to-using-anonymous-functions-instead-of-named-functions-fo — Tasos K., Sep 25 '15 at 17:41
Without using $(function() {}); there are chances to get error pf $ is undefined. So it's better to use this. — Ankur, Sep 25 '15 at 17:41
@AR_Web how? If `$` is undefined, it will error in either case. — Evan Davis, Sep 25 '15 at 17:57
OP, what do __you__ see as the functional difference between those scenarios? — Evan Davis, Sep 25 '15 at 17:58
I'm studying up on this now, can't draw any conclusions with my limited knowledge now — Clay Banks, Sep 25 '15 at 18:06

score 2 · Answer 1 · answered Sep 25 '15 at 17:44

If you're not declaring any vars or functions and you make sure your code will execute after the dom is ready it doesn't make any difference.

BUT the common experience is that maybe you are sure those premises are true now, but someday somebody will break one of them, then you'll propably have a problem (a leak, a conflict, whatever).

As an advice, I would tell you to wrap your code into a Immediately-Invoked Function Expression (IIFE).

Wrapping Events in Anonymous Functions - Javascript

1 Answers1