Javascript array indexOf returns undefined

Question

When calling my function checkIss(), issFullArray.indexOf(issToCheck) always returns undefined. I've run a .length, output the contents of issFullArray, I can't figure out why it's not working- the array looks fine to me. As you can see below, I've tried explicitly setting issArray as an array and copying the array returned by my getIssList()

function updateIss() {
    var issArray = [];

    var currService = current.u_business_service;
    var currIss = current.u_is_service;
    issArray = getIssList(currService).slice(); //getIssList() returns an arry
    if (checkIss(issArray, currIss) === false) {
        //do stuff
    }
}

function checkIss(issFullArray, issToCheck) {
    if (issFullArray.indexOf(issToCheck) < 0) {
        return false;
    } else {
        return true;
    }
}

IE <=8, for example, doesn't support indexOf() for Arrays. — Tim Down
– Tim Down, Commented Feb 5, 2013 at 12:23
I've tried IE9, Firefox 15 and Chrome 24. Are there actually browsers that wouldn't support indexOf()? Bit silly if you ask me... — user2042970
– user2042970, Commented Feb 5, 2013 at 12:24
make the scope of issArray global and do not pass it to your checkIss() function and see whether it works — asifsid88
– asifsid88, Commented Feb 5, 2013 at 12:27
Thanks for your comments, but it appears it's the software that I'm coding for that doesn't support indexOf. I find it a bit odd that such a simple function isn't supported but heyho, I'll raise a bug and see what happens! — user2042970
– user2042970, Commented Feb 5, 2013 at 12:59

ElPedro · Accepted Answer · 2013-02-05 12:30:08Z

2

Easiest to just loop through the array and compare each value and return true if there is a match otherwise return false. Not much more code and works for all browsers.

function checkIss(issFullArray, issToCheck) {
    for(i=0; i<issFullArray.length; i++) {
        if(issFullArray[i]==issToCheck) {
            return true;
        }
    }
    return false;
}

answered Feb 5, 2013 at 12:30

ElPedro

58612 silver badges19 bronze badges

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

1 Comment

user2042970 Over a year ago

Thanks ElPedro, it looks like I'll ne using this!

Collectives™ on Stack Overflow

Javascript array indexOf returns undefined

1 Answer 1

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related