Why is this javascript function running without being called?

Question

$(document).ready(SetupButtonClicks());

function SetupButtonClicks() {
    $(\'#btnJavaPHP\').click(DoPHPStuff());
}

function DoPHPStuff() {
    //stuff
}
<         


        

Answer 1

Change your SetupButtonClicks function:

$('#btnJavaPHP').click(DoHPStuff);

The way you've got it coded, you're telling Javascript to call the function, not to use it as the "click" handler. The parentheses are an operator that causes a function to be called.

Answer 2

Remove the ().

By writing $(document).ready(SetupButtonClicks()), you are calling SetupButtonClicks and passing its return value to ready.
Similarly, by writing $('#btnJavaPHP').click(DoPHPStuff()), you are calling DoPHPStuff (immediately) and passing whatever it returns to click().

You need to pass the functions themselves by writing $(document).ready(SetupButtonClicks) and $('#btnJavaPHP').click(DoPHPStuff).

Answer 3

With the exception of a function declaration, a pair of parentheses following a function's identifier causes the function to execute. Examples:

// function declaration; function not executed
function SetupButtonClicks() {

}

// function executed
SetupButtonClicks();

// function not executed
SetupButtonClicks;

Answer 4

function DoPHPStuff() {
    //stuff
}

function SetupButtonClicks() {
    $('#btnJavaPHP').click(DoPHPStuff);
}

$(document).ready(SetupButtonClicks);