NSMutableArray remove duplicate items

Question

 NSLog(@"Ci sono %d categorie",category.count);
 for(int c=0; c<category.count; c++){
 CategoryArray * element =[category objectAtIndex:c];
 NSLog(@"******************************");
 NSLog(@"Titolo %@",element.label);
 NSLog(@"Source %@",element.url);
 NSLog(@"******************************");
 [element release];

This is my code! I need to remove all duplicates array... I use this code but I don't understand why it doesn't work.

uniquearray = [[NSSet setWithArray:category] allObjects];

check [this][1] [1]: http://stackoverflow.com/questions/1025674/the-best-way-to-remove-duplicate-values-from-nsmutablearray-in-objective-c — Narayanan Ramamoorthy, Sep 05 '11 at 12:28

Caleb · Answer 1 · 2011-09-05T13:56:10.360

5

Your code does work. Try this:

NSArray *a = [NSArray arrayWithObjects:@"a", @"b", @"c", @"d", @"b", @"b", @"d", nil];
NSLog(@"a: %@\ndistinctA: %@", a, [[NSSet setWithArray:a] allObjects]);

The output is:

a: (
    a,
    b,
    c,
    d,
    b,
    b,
    d
)
distinctA: (
    c,
    d,
    a,
    b
)

Knowing that the code works for an array that contains duplicate objects, you now have to wonder why objects that you think are duplicates are not considered to be so. The existing answers regarding hash and isEqual point you in the right direction.

edited Sep 05 '11 at 13:56

answered Sep 05 '11 at 13:30

Caleb

120,112
19
171
259

Please note that this method changes original order of items in array. – ivanzoid May 20 '14 at 12:03

rckoenes · Answer 2 · 2011-09-05T13:09:58.740

2

Set can not contain any duplicate records. It uses a hash to tell them apart and will use isEqual if the hash is same (see the comment by Rudy Velthuis).

edited Sep 05 '11 at 13:09

answered Sep 05 '11 at 12:11

rckoenes

68,081
8
130
162

Not just a hash. It uses a hash first, ie. different hash means certainly different. But items with equal hash must be investigated further, using isEqual. – Rudy Velthuis Sep 05 '11 at 12:56

score 2 · Answer 3 · edited May 23 '17 at 12:12

2

The elements inside this CategoryArray must correctly implement isEqual and hash for the NSSet object to be able to figure out which ones are duplicates.

You can see another SO question about overriding isEqual and hash here: Best practices for overriding isEqual: and hash

edited May 23 '17 at 12:12

Community

1
1

answered Sep 05 '11 at 12:13

Maurício Linhares

37,947
14
116
153

Mobile App Dev · Accepted Answer · 2011-09-05T12:34:27.900

you can use following code.

NSMutableArray* filterResults = [[NSMutableArray alloc] init];
BOOL copy;

if (![category count] == 0) {
    for (CategoryArray *a1 in category) {
        copy = YES;
        for (CategoryArray *a2 in filterResults) {
            if ([a1.label isEqualToString:a2.label] && [a1.url isEqualToString:a2.url]) {
                copy = NO;
                break;
            }
        }
        if (copy) {
        [filterResults addObject:a1];
        }
    }
}

And you will get category array without duplicate values.
you can see Edit code. i think this will work.

score 0 · Answer 5 · answered Aug 23 '13 at 12:30

0

If you are worried about the order, then you can do it this way.

NSArray * newArray = [[NSOrderedSet orderedSetWithArray:oldArray] array]; // iOS 5.0 and later

answered Aug 23 '13 at 12:30

keen

2,851
3
31
57

NSMutableArray remove duplicate items

5 Answers5