CODEWITHSUNDEEP
iphoneobjective-carrayspointers
alex25
alex25

Reputation: 621

Pointers, NSMutableArray, Retain, Loops and Confusion

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!

Upvotes: 2

Views: 155

Answers (3)

Stephen Darlington
Stephen Darlington

Reputation: 52575

Seems like you're doing way more work than you need. Something like this (I think) does the same thing, but without the leaks or crashes:

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

Upvotes: 1

Sixten Otto
Sixten Otto

Reputation: 14816

The object you create in this line (and are presumably trying to release):

Friend *f = [[Friend alloc] init];

is immediately leaked when you then assign f to the object you get out of your array:

f = [allFriends objectAtIndex:x];

So it's really that object you're releasing, meaning that the pointer in the array is no longer valid (it's pointing at a released instance).

Upvotes: 5

Carl Norum
Carl Norum

Reputation: 225202

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];
    }
}

Upvotes: 4

Related Questions