Question

If I have a property, say an NSArray, that’s going to be initialized only once for each instance of my class, is there anything wrong with this:

(in the interface)

@property(strong, nonatomic)NSArray *bubbleArr;

(in the implementation)

-(NSArray*)bubbleArr
{
    if(!bubbleArr)
    {
        NSMutableArray *tempBubbArr = [[NSMutableArray alloc] init];
        // get filepath for first speech bubble image for page
        NSString *speechBubbleImgPath = [[NSBundle mainBundle] pathForResource:
                                         [NSString stringWithFormat:@"speech_%i_0", pageIndex]
                                                                        ofType:@"png"];

        for(int i = 1; speechBubbleImgPath; i++)
        {
            UIImage *speechBubbleImg = [[UIImage alloc] initWithContentsOfFile:speechBubbleImgPath];
            UIImageView *speechBubbleImgView = [[UIImageView alloc] initWithImage:speechBubbleImg];

            [tempBubbArr addObject:speechBubbleImgView];

            speechBubbleImg = nil;
            speechBubbleImgView = nil;
            speechBubbleImgPath = nil;

            speechBubbleImgPath = [[NSBundle mainBundle] pathForResource:
                                   [NSString stringWithFormat:@"speech_%i_%i", pageIndex, i]
                                                                  ofType:@"png"];
        }

        bubbleArr = [[NSArray alloc] initWithArray:tempBubbArr];

        tempBubbArr = nil;
    }

    return bubbleArr;
}

I’ve never used custom accessor methods, but this seems like a clean way to set it up, so I don’t have to set up each property in my viewDidLoad or elsewhere, and don’t have to worry about it being nil. I don’t recall ever actually come across code that does this. Is this the recommended way to do it? Also, I’ll always want to use self.bubbleArr to make sure this method is called, right?

Answer 1

This is a totally valid way of setting up your property. Apple does this sort of thing very often in their sample code as well as in their project templates. Look, for example, at the core data stack setup in a newly created core data iOS project. As @WendiKidd noted, you have to access your variable through the accessors all the time to make sure this works nicely (which is probably what you should be doing anyway).

In particular, this is a good way of implementing a property of a class that can really only return one thing (which, from your comment, it sounds like is what you’re trying to do). If that’s your goal, here are some guidelines to follow:

1. Declare your property as readonly
2. Declare it in the public header if it should be publicly accessible, or in a class extension in the .m file if it should be “private”
3. If it can/should be backed by a variable, synthesize the ivar and overwrite the getter as you’ve done
4. If it shouldn’t/doesn’t have to be backed by a variable, declare the property as @dynamic in the implementation and overwrite the getter
5. Only access your ivar through the accessor

Declaring the variable as dynamic in the 4th point will signal to anyone that looks at your code that you’ve likely written a custom accessor for that property.