Question

It might be that the problem is straight forward but I don’t get my head around it.

I have the following iPhone code

for(int x = 0 ; x < [allFriends count] ; x++)   
{
    Friend *f = [[Friend alloc] init];          
    f = [allFriends objectAtIndex:x];

    if([uid isEqualToString:[f uid]])
    {                                           
        [f AddAlbum:album];
        [allFriends replaceObjectAtIndex:x withObject:f];
     }                              
}

It doesn’t matter where I call [f release] the app always crashes. Why? (BTW the loop runs a few thousand times)

Is there a more efficient way to do it?

I think I provided enough code, let me know if not!

Thanks heaps for your help!

Answer 1

I think your code has some serious problems:

1. Why are you allocating a new Friend if you’re going to immediately leak it?
2. Why replace an object in an array with the same object?

I think you can replace your code with this loop:

for (Friend *f in allFriends)
{
    if([uid isEqualToString:[f uid]])
    {
        [f AddAlbum:album];
    }
}