How do you get and set a class property across multiple functions in Objective-C?

Update: Unfortunately the help offered below did not solve my problem of sharing a class property across functions. Can anyone else suggest a possible problem? Here's the latest code:

Header .h:

@interface FirstViewController:UIViewController <UITableViewDataSource,开发者_开发百科 UITableViewDelegate, UITabBarControllerDelegate> {
 NSDictionary *sectorDictionary;
 NSInteger sectorCount;
}

@property (nonatomic, retain)  NSDictionary *sectorDictionary;

- (id)initWithData:(NSMutableDictionary*)inData;

Implementation .m:

@synthesize sectorDictionary;

- (id) testFunction:(NSDictionary*)dictionary {
 NSLog(@"Count #1: %d", [dictionary count]);
 return nil;
}

- (id)initWithData:(NSMutableDictionary *)inData {
 self = [self init];
 if (self) {
    [self testFunction:inData];
    // set the retained property
    self.sectorDictionary = inData;
  }

 return self;
}


- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
 NSLog(@"Count #2: %d", [self.sectorDictionaryCopy count]);
 return [self.sectorDictionaryCopy count];
}

Console output:

2010-05-05 20:11:13.126 JSONApp[17378:207] Count #1: 9
2010-05-05 20:11:13.130 JSONApp[17378:207] Count #2: 0

Following up on this question about sharing an object between classes, I now need to figure out how to share that object across various functions in a class.

First, the setup: In my App Delegate I load menu information from JSON into a NSMutableDictionary and message that through to a view controller using a function called initWithData. I need to use this dictionary to populate a new Table View, which has methods like numberOfRowsInSection and cellForRowAtIndexPath.

I'd like to use the dictionary count to return numberOfRowsInSection and info in the dictionary to populate each cell. Unfortunately, my code never gets beyond the init stage and the dictionary is empty so numberOfRowsInSection always returns zero.

I thought I could create a class property, synthesize it and then set it. But it doesn't seem to want to retain the property's value. What am I doing wrong here?

In the header .h:

@interface FirstViewController:UIViewController <UITableViewDataSource, UITableViewDelegate, UITabBarControllerDelegate> {
    NSMutableDictionary *sectorDictionary;
    NSInteger sectorCount;
}

@property (nonatomic, retain)  NSMutableDictionary *sectorDictionary;

- (id)initWithData:(NSMutableDictionary*)data;

@end

in the implementation .m:

@synthesize sectorDictionary;

- (id) testFunction:(NSMutableDictionary*)dictionary {
    NSLog(@"Count #1: %d", [dictionary count]);    
    return nil;
}

- (id)initWithData:(NSMutableDictionary *)data {
    if (!(self=[super init])) {
        return nil;
    }
    [self testFunction:data];

    // this is where I'd like to set a retained property
    self.sectorDictionary = data;

    return nil;
}

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    NSLog(@"Count #2: %d", [self.sectorDictionary count]);
    return [self.sectorDictionary count];
}

Output from NSLog:

2010-05-04 23:00:06.255 JSONApp[15890:207] Count #1: 9
2010-05-04 23:00:06.259 JSONApp[15890:207] Count #2: 0

Any init function should return self if it completes successfully. Change the return nil to return self at the end of initWithData.

-(id) initWithData:(NSMutableDictionary *)inData {
    self = [self init]; // best practice use super for method with same name
    if ( self ) {
        self.sectorDictionary = inData;
    }
    return self;
}

The sectorDictionary member should not be mutable. If it changes, you need to call reloadData on the table. Your setter function should make an immutable copy.

-(void) setSectorDictionary:(NSDictionary *)inData {
    NSDictionary *old = sectorDictionary;
    sectorDicitonary = inData ? [[NSDictionary alloc] intWithDictionary:inData] : nil;
    [self.table reloadData];
    [old release];
}

I believe your problem lies within the line self.sectorDictionary = data.

You need to firstly allocate some memory for the dictionary you are creating, so a line like self.sectorDictionary = [[NSMutableDictionary alloc] init]; is going to be necessary.

After you have initiated the new dictionary you need to populate it with the contents of the dictionary that was passed in.

So...

Try the line:

  self.sectorDictionary = [[NSMutableDictionary alloc] initWithDictionary:data];

instead of:

self.sectorDictionary = data;

instead of

 self.sectorDictionary = data;

try this way...

self.sectorDictionary = [[NSMutableDictionary alloc] init];
self.sectorDictionary = (NSMutableDictionary *) data;