NSMutableArray check if item exists

Question

I've got one NSMutableArray with 100+ characters in which I've populated a table with:

int rowCount = indexPath.row;
Character *character = [self.characterArray objectAtIndex:rowCount];

cell.textLabel.text = character.name;
cell.detailTextLabel.text = character._id;

I've then got a separate NSMutableArray with an _id and neighbour values in, however this array only has around 8 values with random id's in.

Basically I want to check if the character._id exists in the second array, It'll display a checkmark on the table.

I've tried doing:

Neighbour *neighbour = [self.neighbourArray objectAtIndex:rowCount];

if (character._id == neighbour._id && neighbour.neighbour == 1){
         cell.accessoryType = UITableViewCellAccessoryCheckmark;
    }else{
         cell.accessoryType = None;
}

But it crashes the App (because it only has 8 values of 100, I'm guessing)

Is there an easy way around this or a better way to check altogether?

Thanks, any constructive advise would be greatly appreciated, I'm brand new to Xcode.

serrrgi · Accepted Answer · 2013-08-16T15:01:25.300

4

You can use KVC to avoid the enumeration:

BOOL result = [[neighbours valueForKey:@"_id"] containsObject: character._id];

edited Aug 16 '13 at 15:01

answered Aug 16 '13 at 14:55

serrrgi

644
3
7

1

Very elegant solution. – allprog Aug 16 '13 at 15:05
Thanks, that's solved everything. – user2689608 Aug 16 '13 at 19:49

Toseef Khilji · Answer 2 · 2013-08-19T04:58:05.327

0

Currently you are checking Character object at same index of Neighbour, you should check it in whole array,

you can try like this

-(BOOL)check :(Character *) character
{

for(Neighbour *neighbour in  self.neighbourArray )
{
  if(character._id == neighbour._id && neighbour.neighbour == 1)
   return TRUE;
}

  return FALSE;

}

now just call this method in tableviewMethod

int rowCount = indexPath.row;
Character *character = [self.characterArray objectAtIndex:rowCount];

if ([self check:character]){
         cell.accessoryType = UITableViewCellAccessoryCheckmark;
    }else{
         cell.accessoryType = None;
}

edited Aug 19 '13 at 04:58

answered Aug 16 '13 at 14:20

Toseef Khilji

17,192
12
80
121

I gave it a go but I get "Control may reach end of non-void function" error on the first block, I'm guessing because it doesn't give a definite return. – user2689608 Aug 16 '13 at 14:40
yes solved, check updated answer. – Toseef Khilji Aug 19 '13 at 04:58

NSMutableArray check if item exists

2 Answers2