How To Refresh Page With Javascript For N Number Of Times Through User Input?

i have coded a script that should refresh the page for Number of times the user input the value . here is the code what am i doing wrong in this code?everything is working correct

Solution 1:

I agree with Matt Morgan that your attempt to assign a variable inside a conditional:

if ( reloadCnt <= var x = document.getElementById('a').value; )

probably means you need to brush up on your understanding before proceeding.

But here is a working demo for you. (Not having your index.php, which I assume populates the textbox "a", I've done it client-side and fetched the posted value from the querystring. Once you've got it working, you can always revert to document.getElementById('a').value)

You'll also want to clear your sessionStorage variable when the user next hits the page, otherwise when you try to test it twice in a row, reloadCounter will still be where it was at the end of the last run.

<formmethod="GET"><p>Your link: <inputtype="text"rows="5"style="width:200px; height:50px;"name="link" /></p><p>number of times: <inputtype="text"id="a"name="times" /></p><p><inputtype="submit"rows="5"style="width:200px; height:50px;" / ></p></form><script>functiongetParameterByName(name) {
    var match = RegExp('[?&]' + name + '=([^&]*)').exec(window.location.search);
    return match && decodeURIComponent(match[1].replace(/\+/g, ' '));
}
    var timeinmilliseconds = 600;
    var reloadCnt = window.sessionStorage.getItem( "reloadCounter") ? parseInt(window.sessionStorage.getItem( "reloadCounter")) + 1 : 1;
    var times = getParameterByName('times');
    if (times)
    {
        window.sessionStorage.setItem( "reloadCounter", reloadCnt );
        console.log(reloadCnt + ' / ' + times);
        if ( reloadCnt <  times )
            setTimeout(function(){ window.location.reload(true) }, timeinmilliseconds);
    }
    else
    {
        window.sessionStorage.setItem( "reloadCounter", 0 );
    }
</script>

Solution 2:

You have a some fundamental problems with your JavaScript. It's important that you understand why this is not working if you want to be able to use JS well in the future.

This statement is not doing what you intend:

if ( reloadCnt <= var x = document.getElementById('a').value; )

You are saying "if reloadCnt is less than or equal to some thing" where "some thing" is the result of the statement:

var x = document.getElementById('a').value;

This statement is assigningdocument.getElementById('a').value to the variable x. The return value of such an assignment is always undefined in JavaScript. This means what you've effectively written is the same as:

if ( reloadCnt <= undefined )

This is always going to be false, which is why it's not working for you.

There are two other issues:

1. Your jQuery lookup will return a string, which you should convert to a number before comparing it to another number.

2. You don't need a semicolon inside your conditional.

In the end, this is probably what you want to write:

if (reloadCnt <= Number(document.getElementById('a').value)) { ... }

Solution 3:

Can u try using this instead.

if (reloadCnt <= document.getElementById('a').value;)