Comparing Arrays with nested loops

Question

I am trying to compare values of two arrays against each other. If a match is found - do something - else do this.

I put together a fiddle with my code at http://jsfiddle.net/ZvmHx/1/

If you uncomment the second the alert on line 14 you'll see what is wrong. I can't seem to prevent the second alert from firing.

Thanks!

var getkeywords = ["John","Frank","Sarah"]; 
var captionarray = ["Jim","Joe","Lee","Steve","John","Michelle","Brad"];    

for (k = 0; k < getkeywords.length; k++) {
    for (l = 0; l < captionarray.length; l++) {

    if(getkeywords[k] == captionarray[l]){

        alert('Found > ' + getkeywords[k] + ':filter image');

    }else{

        //alert('not found > ' + getkeywords[k] + ':filter image');

    }
  }
}

freejosh · Accepted Answer · 2013-07-06 12:31:34Z

2

The if/else is being tested for every iteration of your inner loop. I think what you're after is testing if you have a match after the inner loop has run. Something like:

var getkeywords = ["John","Frank","Sarah"]; 
var captionarray = ["Jim","Joe","Lee","Steve","John","Michelle","Brad"];    
var matchFound;

for (k = 0; k < getkeywords.length; k++) {
    matchFound = false;

    for (l = 0; l < captionarray.length; l++) {
        if (getkeywords[k] == captionarray[l]){
            matchFound = true;
            break;
        }
    }

    if(matchFound){

        alert('Found > ' + getkeywords[k] + ':filter image');

    }else{

        alert('not found > ' + getkeywords[k] + ':filter image');

    }
}

answered Jul 6, 2013 at 12:31

freejosh

11.4k4 gold badges36 silver badges47 bronze badges

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

Comments

abhinsit · Accepted Answer · 2013-07-06 12:33:10Z

0

I have created a new fiddle:-

http://jsfiddle.net/WZGyy/

var getkeywords = ["John","Frank","Sarah"]; 
var captionarray = ["Jim","Joe","Lee","Steve","John","Michelle","Brad"];    

imagecode = '';
var found=0;
for (k = 0; k < getkeywords.length; k++) 
{
    for (l = 0; l < captionarray.length; l++) 
    {

        if(getkeywords[k] == captionarray[l])
        {


            found=1;
            break;

        }
    }

    if(found==1)
    {
        alert('Found > ' + getkeywords[k] + ':filter image');
        found=0;  


    }
    else
    {
        alert('not found > ' + getkeywords[k] + ':filter image');

    }
}

Hope that helps..

answered Jul 6, 2013 at 12:33

abhinsit

3,2574 gold badges23 silver badges26 bronze badges

1 Comment

barrylachapelle Over a year ago

Thanks Abhinsit. Super appreciated!

Alex Fitiskin · Accepted Answer · 2013-07-06 12:34:26Z

0

Before alerting the result you must to compare value with all items in second array I updated your jsfiddle — try in out http://jsfiddle.net/ZvmHx/5/

var getkeywords = ["John","Frank","Sarah"]; 
var captionarray = ["Jim","Joe","Lee","Steve","John","Michelle","Brad"];    

imagecode = '';
for (k = 0; k < getkeywords.length; k++) {
    var isExists = false;
    for (l = 0; l < captionarray.length; l++) {
        if (getkeywords[k] == captionarray[l]){                     
            isExists = true;
            break;
        }
    }

    if (isExists) {
        alert('Found > ' + getkeywords[k] + ':filter image');
    } else {
        alert('not found > ' + getkeywords[k] + ':filter image');
    }
}

answered Jul 6, 2013 at 12:34

Alex Fitiskin

7754 silver badges6 bronze badges

Collectives™ on Stack Overflow

Comparing Arrays with nested loops

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related