Question

Just a quick question. I have an NSArray that is generated in my applicationDidFinishLaunching method, but for some reason the array isn’t being used by other methods and it is saying that it is being unused.

-(void)applicationDidFinishLaunching... {
    NSArray* songsArray = [root nodesForXPath:@".//dict/dict/dict" error:nil];

-(id)tableView:(NSTableView *)tableView objectValueForTableColumn... {
    for(NSXMLElement* song in songsArray) {

I have declared the variable in the header file also.

Cheers,
Scott

Answer 1

You have declared songsArray as a local variable, it’s not visible outside applicationDidFinishLaunching.

If you have already declared your variable in header, you need to just assign it:

-(void)applicationDidFinishLaunching... {
    songsArray = [root nodesForXPath:@".//dict/dict/dict" error:nil];

Note on memory management:
If you are not using ARC, you also need to retain your array, otherwise it will be autoreleased at the end of applicationDidFinishLaunching, and your variable will be pointing to a deallocated memory, and your app will crash.

And don’t forget to release it in dealloc (again, if you are not in ARC land).