Trigger function only on "onclick"

Question

Hello I need a function to run when an a tag is clicked. I am using the onclick="" but it seems that the function runs on page load, instead of waiting to run when the a tag is clicked on. How do I make it so the function only runs when the a tag is clicked?

Here is my code.

HTML:

<a class="req_btn" onclick="conversionOne()">text</a>

<script type='text/javascript'>
    (function conversionOne() {
        alert("something"); 
    })();
</script>

Thanks!

Just remove the parentheses -> `function conversionOne() { alert("something"); }` — adeneo, Oct 24 '16 at 18:00

score 6 · Accepted Answer · answered Oct 24 '16 at 18:00

6

You are invoking the function when the script loads using an IIFE. Instead of this:

(function conversionOne() {
    alert("something"); 
})();

Do this:

function conversionOne() {
    alert("something"); 
}

answered Oct 24 '16 at 18:00

Ori Drori

166,183
27
198
186

Awesome, that worked. Thanks! Ill accept your answer once it allows me to. – cup_of Oct 24 '16 at 18:03

score 1 · Answer 2 · answered Oct 24 '16 at 18:01

1

You are using a self-executing function. Declare the function in the global scope without executing it.

function conversionOne() {
    alert("something"); 
}

answered Oct 24 '16 at 18:01

J. Orbe

143
1
5

score 1 · Answer 3 · edited May 23 '17 at 12:24

1

(function conversionOne() {
    alert("something"); 
})();

calling function like this will work onload Change this to

 function conversionOne() {
    alert("something"); 
};

More info

edited May 23 '17 at 12:24

Community

1
1

answered Oct 24 '16 at 18:01

XYZ

4,370
2
13
30

thanks for your answer, but Ori Drori posted this right before you. I will give you an upvote though. Thanks – cup_of Oct 24 '16 at 18:05

score 1 · Answer 4 · answered Oct 24 '16 at 18:04

Doing this

(function(){
    /** code **/
})();

Means the code will be executed imediatelly. in your case you want to create a function so you need this :

function conversionOne() {
    /** code **/
}

or

var conversionOne = function () {
    /** code **/
}

Trigger function only on "onclick"

4 Answers4