JavaScript causing issues

Posted February 11, 2020 834 views

Guys, I’ve been experiencing an issue where my function has been constantly executing without actually.

Here is my JavaScript code

    const testInput    = document.querySelector('[name=test]');

    testInput.addEventListener('click', ExecuteReload());

   function ExecuteReload(){

Please help me out, I’m at a lost, I don’t know what’s going on.

These answers are provided by our Community. If you find them useful, show some love by clicking the heart. If you run into issues leave a comment, or add your own answer to help others.

Submit an Answer
1 answer

Hi @brendRos,

The error is quite common and it’s not actually an error, it’s how JavaScript works.

Turn your attention to your Event Listeners:

    testInput.addEventListener('click', ExecuteReload());

You need to make ExecuteReload() be like ExecuteReload. If you leave the () it will keep calling the function without it actually being called.

One other thing unless this is an example, you don’t need to actually have another function in your Event Listener just for reloading the window.

Here is how I would’ve done it

    testInput.addEventListener('click', location.reload);


  • Thank you! It’s working as it’s supposed to now.

    You mentioned I should just use the location.reload function directly in the Event Listener, is it better to actually have the function there or create another function which is separated?

    • Hi @brendRos,

      In this case there is no point in creating a function for just one line of code however if you had something more sophisticated then yes, you would need to actually create a separate function.