Optimization of recursive jQuery images swap function

There's this website I'm working on that needs to swap images every few seconds, like a slideshow or image fade swap in a way, except that you need to target 5 IMG tag elements. I was able to make a weak version of that function. But i'm sure this could be done even better, considering that I still have a few bugs that needs to be fixed. If anyone could land me a hand and steer me in the right direction with this, that would be mucho appreciated.

Bugs:

• Broswer tab switch for a minute or two, and come back and the images

flicker a bit, until the recursive function readjust itself and keep
on going.

• Last IMG element is never swapped. I figured, by commenting a bit of

code, what caused the problem. It's the conditional statement that
determines if the function has ended and must recall itself. Altho
I'm not still sure what needs to be done to fix that.

The function:

• Shuffle a string array containing image path

• jQuery .each() loop through all my IMG elements

• Opacity Fade out current element with TweenMax

• On complete, swap element 'src' path for a new one within the string array

• Opacity Fade in current element with TweenMax

• Increment index var

• Check to see if we reached the end of array. If

true, we re-shuffle our string array and recall the function
(recursion). If false, keep going with the loop, repeat all steps for next element

FYI: I based my recursive function on this SO post

The Shuffle function is basically Fisher-Yates

Here is a JSFiddle demonstrating the current state of the function

The real HTML is more complete than that. Light version for example:

The recursive jQuery/TweenMax function:

```
var globvars = {
imgArr : '',
arrCount: 0,
arrIndex: 0,
totalImageContainer: 5,
currentImageContainer: 1,
currentIMGPath:''
}

$(function () {
globvars.imgArr = ["http://lab1663.net/images/pam_holy_shitsnacks.png", "http://i.ytimg.com/vi/Ftb09o6O7sw/0.jpg", "http://doyoulikelikeme.files.wordpress

-
Functions should be camelCase unless they're constructors. So fadeChange (also, fadeChange is a little vague - what's getting faded? What's being changed?)

-
You probably don't need global variables of any kind. Closures should be fine.

-
You have your quite a few magic numbers in there. A couple aren't bad for animation stuff, but in this case, they're repeated in multiple places (like the 8000ms interval). This can be a maintenance issue.

-
I don't think you need more libraries; jQuery's .fadeOut()/.fadeIn() should be sufficient.

I'd suggest something like this, where you don't loop through all the images in one go, but just do one at a time, wait, then do another, then wait... etc.

$(function () {

    var FADE_INTERVAL = 8000,
        FADE_DURATION = 350;

    var images = $(".img-contnr"),
        imageIndex = 0,
        urlIndex = 0,
        imageUrls = shuffleArray([
          // a bunch of image urls here
        ]);

    function swapImage() {
        // fade out the current image
        images.eq(imageIndex).fadeOut(FADE_DURATION, function () {
            // set next image index
            imageIndex = (imageIndex + 1) % images.length;

            // swap the img src
            this.src = imageUrls[urlIndex];

            // get next url index
            urlIndex = (urlIndex + 1) % imageUrls.length;

            // if we've gone through all the urls, re-shuffle
            if (urlIndex === 0) {
                imageUrls = shuffleArray(imageUrls);
            }

            // fade back in, and set up the next swap
            $(this).fadeIn(FADE_DURATION, delayedSwap);
        });
    }

    function delayedSwap() {
      setTimeout(swapImage, FADE_INTERVAL);
    };

    // call delayedSwap to trigger the first swap
    delayedSwap(); 
 });

jsfiddle

StackExchange Code Review Q#57963, answer score: 3