Add counter to javascript function loop

Question

I am quite new to javacript and I am trying to add a numerical counter as a loop to the below function to save me typing it out 8 times!

The Function needs to be called loadPopup1 - loadPopup8 and the #toPopup div needs to be toPopup1 - toPopup8.

Here's my code:

function loadPopup() { 
    if(popupStatus == 0) { // if value is 0, show popup
        closeloading(); // fadeout loading
        $("#toPopup").fadeIn(0500); // fadein popup div
        $("#backgroundPopup").css("opacity", "0.7"); // css opacity, supports IE7, IE8
        $("#backgroundPopup").fadeIn(0001); 
        popupStatus = 1; // and set value to 1
    }   
}

Many thanks for your help!

Pete

mgs · Accepted Answer · 2013-06-26 09:09:55Z

2

Hand over a parameter to the function...

function loadPopup(index) {
   ...
   $("#toPopup" + index).fadeIn(0500);
   ...
}

Then, if you want to act on #1 and #7, call it passing the index...

loadPopup(1);
loadPopup(7);

answered Jun 26, 2013 at 9:09

mgs

5,2122 gold badges19 silver badges18 bronze badges

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

Comments

jcubic · Accepted Answer · 2013-06-26 09:10:27Z

0

this code will create your 8 functions:

for (var i = 1; i <= 8; ++i) {
    (function(i) {
        window['loadPopup' + i] = function loadPopup() {
            if(popupStatus == 0) { // if value is 0, show popup
                closeloading(); // fadeout loading
                $("#toPopup" + i).fadeIn(0500); // fadein popup div
                $("#backgroundPopup").css("opacity", "0.7"); // css opacity, supports IE7, IE8
                $("#backgroundPopup").fadeIn(0001); 
                popupStatus = 1; // and set value to 1
            }
        };
    })(i);
}

answered Jun 26, 2013 at 9:10

jcubic

67.1k58 gold badges252 silver badges466 bronze badges

Comments

Grim · Accepted Answer · 2013-06-26 09:20:28Z

0

function loadPopup(i) {
    if(popupStatus == 0) { // if value is 0, show popup
        closeloading(); // fadeout loading
        $("#toPopup" + i).fadeIn(0500); // fadein popup div
        $("#backgroundPopup").css("opacity", "0.7"); // css opacity, supports IE7, IE8
        $("#backgroundPopup").fadeIn(0001);
        popupStatus = 1; // and set value to 1
    }
}

var wrapper={};
for (var i =0; i < 8; i++){
    wrapper['loadPopup' + i] = new function(){loadPopup(i);};
}

// so you can call
wrapper.loadPopup3();

answered Jun 26, 2013 at 9:20

Grim

2,13312 gold badges63 silver badges138 bronze badges

Collectives™ on Stack Overflow

Add counter to javascript function loop

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related