Сбой UITableView при выполнении делегата

Question

код ниже:

@interface PreferenceViewController : UIViewController <UITableViewDelegate, UITableViewDataSource> {
    UITableView * preference;
    UISegmentedControl * dimension;
    NSArray * timeslot;
}

@property (nonatomic, retain) IBOutlet UITableView * preference;
@property (nonatomic, retain) IBOutlet UISegmentedControl * dimension;
@property (nonatomic, retain) NSArray * timeslot;

@end

- (void)viewDidLoad
{
    preference.delegate = self;
    preference.dataSource = self;
    timeslot = [NSArray arrayWithObjects:@"7:00-8:00", @"8:00-9:00", @"9:00-10:00", @"10:00-11:00", @"11:00-12:00", @"12:00-13:00", @"13:00-14:00", @"14:00-15:00", @"15:00-16:00", @"16:00-17:00", nil];
    NSLog(@"size of array is %d", [timeslot count]);
    [super viewDidLoad];
    // Do any additional setup after loading the view from its nib.
}

- (void)viewDidUnload
{
    [super viewDidUnload];
    // Release any retained subviews of the main view.
    // e.g. self.myOutlet = nil;
}

- (BOOL)shouldAutorotateToInterfaceOrientation:(UIInterfaceOrientation)interfaceOrientation
{
    // Return YES for supported orientations
    return (interfaceOrientation == UIInterfaceOrientationPortrait);
}

#pragma mark - Table View delegate
- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{

}

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section
{
    if (dimension.selectedSegmentIndex == 0){
        return [timeslot count];
    }else
        return 15;
}

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
    //NSLog(@"TESTING");
    static NSString *CellIdentifier = @"Cell";
    UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
    if (cell == nil) {
        cell = [[[UITableViewCell alloc] initWithFrame:CGRectZero reuseIdentifier:CellIdentifier] autorelease];
    }
    // this is the MAGIC line!

    if (dimension.selectedSegmentIndex == 0)
        cell.text = [timeslot objectAtIndex:indexPath.row];

    return cell;
}

Интересно, что стало причиной аварии? Это происходит в этом делегате numberOfRowsInSection и вызове счетчика временных интервалов.

Answer 1

    timeslot = [NSArray arrayWithObjects:@"7:00-8:00", @"8:00-9:00", @"9:00-10:00", @"10:00-11:00", @"11:00-12:00", @"12:00-13:00", @"13:00-14:00", @"14:00-15:00", @"15:00-16:00", @"16:00-17:00", nil];

Эта строка должна иметь -retain.

timeslot = [[NSArray arrayWithObjects:@"7:00-8:00", @"8:00-9:00", @"9:00-10:00", @"10:00-11:00", @"11:00-12:00", @"12:00-13:00", @"13:00-14:00", @"14:00-15:00", @"15:00-16:00", @"16:00-17:00", nil] retain];

UPD -autorelease не очень хорошая идея.Вы можете только автоматически выпустить / выпустить объекты, которые у вас есть.Когда вы используете -arrayWithObjects, ваш объект принадлежит локальному AutoreleasePool.

Вот как это работает в простых терминах.Каждый раз, когда вы создаете объект без -alloc или -copy методов, он автоматически высвобождается, что в данном случае означает, что он будет иметь счетчик ссылок 1 до конца области сообщения.С технической точки зрения бизнес пула автоматических выпусков асинхронный , поэтому может произойти сбой или нет, но вы должны предположить, что этот объект в данный момент является зомби.Поэтому вы должны использовать -retain, чтобы сохранить этот объект для использования другим методом.если вы использовали явно -retain, вы должны использовать -release явно, когда закончите с этим объектом.Так почему бы не использовать [[NSArray alloc] initWithObjects:...] прямо сейчас.это просто более уместно здесь.Здесь вы также можете использовать сохраненное свойство, но вы не хотите показывать внутреннюю переменную, когда у вас ее тоже нет

Answer 2

В viewDidLoad вы присваиваете вновь созданный (и автоматически выпущенный) NSArray для ivar, а не для свойства, что означает, что он не сохранен для вас. Таким образом, к моменту достижения значения tableView:numberOfRowsInSection: массив будет удален, и вы получите EXC_BAD_ACCESS или похожий сбой.

По этой причине я обычно называю свои ивары _name вместо name, чтобы соответствовать свойству. Это означает, что линия синтеза должна быть более сложной @synthesize name=_name, но она предотвращает случайное присвоение ивару, а не свойству.