Pass a function by variable to another function to be bound as a jQuery click event

Question

I'm trying to pass a function via a variable to be assigned as part of a click event, for instance:

function bindClick(func) {
    $('#button').click(function() {
        func();
        return false;
    });
}

function configureClick() {
    bindClick(function () { executeClick(message); });
}

function executeClick(message) {
    alert(message);
}

So configureClick() will run at some point, and then #button.click() needs to call the contents of func.

Whatever's happening, is doing so silently, no errors nor desired behavior.

UPDATE: I'm an idiot! The code above is working. My executeClick had a switch-case block which was being ignored as I was passing the incorrect key. Sorry for the trouble D:

If it doesn't work but there are no errors, make sure you use $(document).ready. — pimvdb
– pimvdb, Commented Jan 3, 2013 at 11:00
sorry, I meant alert('bananas!'), that's just a placeholder for "do other stuff" — DanH
– DanH, Commented Jan 3, 2013 at 11:01
Does the element exist? It works just fine as it stands: jsfiddle.net/NJtn9. — pimvdb
– pimvdb, Commented Jan 3, 2013 at 11:03
Could you perhaps provide a failing demo on jsfiddle.net? Maybe you're calling e.stopImmediatePropagation() or something that prevents the handler from executing. — pimvdb
– pimvdb, Commented Jan 3, 2013 at 11:14

score 1 · Accepted Answer · 2013-01-03 11:19:27Z

1

You do too much passing of callback function. That' creating lots of closures, which is not needed in such a simple case.

function bindClick(func) {
    $('#button').click(function() {
        func();
        return false;
    });
}


function executeClick(message) {
    alert(message);
}

function configureClick(message){
    bindClick(function(){ executeClick(message) });
}

configureClick('Hello');

edited Jan 3, 2013 at 11:19

answered Jan 3, 2013 at 11:05

user1936059

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

1 Comment

DanH Over a year ago

Apologies. I have oversimplied, executeClick() requires a parameter, I'll update my original question.

Collectives™ on Stack Overflow

Pass a function by variable to another function to be bound as a jQuery click event

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related