Check each array value for NaN

Question

I have the following array:

 var indexes = {
            'spear' : $('#commands_table th:has(img[src*="unit_spear"])').index(),
            'sword' : $('#commands_table th:has(img[src*="unit_sword"])').index(),
            'axe' : $('#commands_table th:has(img[src*="unit_axe"])').index(),
            'archer' : $('#commands_table th:has(img[src*="unit_archer"])').index(),
            'spy' : $('#commands_table th:has(img[src*="unit_spy"])').index(),
            'light' : $('#commands_table th:has(img[src*="unit_light"])').index(),
            'marcher' : $('#commands_table th:has(img[src*="unit_marcher"])').index(),
            'heavy' : $('#commands_table th:has(img[src*="unit_heavy"])').index(),
            'ram' : $('#commands_table th:has(img[src*="unit_ram"])').index(),
            'cata' : $('#commands_table th:has(img[src*="unit_catapult"])').index(),
            'noble' : $('#commands_table th:has(img[src*="unit_noble"])').index(),
            'knight' : $('#commands_table th:has(img[src*="unit_knight"])').index(),

            };

Sometimes it'll find a certain name, sometimes it won't. What I want to do is:

  var numberspear = $(this).closest('tr').children().eq(indexes[spear]).text();

Wich obviously won't work for a unit if it doens't exist. Now, i could obviously check every index seperatly, but that's not really efficient... Is there a way to check every value of my array and, if it doens't exist, remove it from the array?

Thanks

And for god's sake, please CACHE $("#commands_table th") and use filter()!! — nietonfir
– nietonfir, Commented Jan 26, 2014 at 19:53

KingKongFrog · Accepted Answer · 2014-01-26 19:18:19Z

2

JsFiddle

for (var key in indexes) {
   if (isNaN(indexes[key]))
      delete(indexes[key]);
}

edited Jan 26, 2014 at 19:18

answered Jan 26, 2014 at 19:11

KingKongFrog

14.5k22 gold badges78 silver badges134 bronze badges

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

Comments

Gabriele Petrioli · Accepted Answer · 2014-01-26 19:31:33Z

1

You could automate the filling of the object and just not add the items that are not found..

var items = ['spear', 'sword', 'axe', 'archer',
             'spy', 'light', 'marcher', 'heavy',
             'ram', 'cata', 'noble', 'knight'],
    indexes = {},
    headers = $('#commands_table th');

for (i=var i = 0, len = items.length; i < len; i+++){
    var item = items[i],
        index = headers.filter(':has(img[src*="unit_'+ item +'"])').index();

    if (!isNAN(index)){
        indexes[item] = index;
    }
}

answered Jan 26, 2014 at 19:31

Gabriele Petrioli

197k35 gold badges275 silver badges331 bronze badges

1 Comment

user3117628 Over a year ago

One more question (sorry, I'm not really familliar with JS, Jquery only): var numberofspear = bevelnamen.closest('tr').childeren().eq(index).text(); Could i do this for each index (if it exists) and put in in the same (or a new) array too?

Collectives™ on Stack Overflow

Check each array value for NaN

2 Answers 2

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related